Closed Bug 1492736 Opened 6 years ago Closed 6 years ago

[remote-dbg-next] Prevent locking device when connected to remote debugging

Categories

(DevTools :: about:debugging, enhancement, P3)

enhancement

Tracking

(firefox65 fixed)

RESOLVED FIXED
Firefox 65
Tracking Status
firefox65 --- fixed

People

(Reporter: jdescottes, Assigned: m_kato)

References

(Blocks 2 open bugs)

Details

Attachments

(2 files)

Feature requested for web compat support. When a Device is being remote debugged, it should not lock. This feature is currently available in Chrome's remote debug solution.
I believe Makoto is looking into this.
Past years, we have implemented WeakLock API for B2G (and also Android backend). We may be able to use that API for this.
Ah, bug 1369194 removed WebIDL...
Assignee: nobody → m_kato
DevTools wants to use wake lock to keep screen. But since MozWakeLock is removed by bug 1369194, there is no way to unlock wake lock from script. So this issue adds nsIWakeLock to unlock wake lock.
Use wake lock to keep screen when tab is attached from debugger. Depends on D7158
Julian, I have a question for this. Chrome/Android seems to keep wake lock when devtool in desktop selects same tab in remote device. Example, when remote device has A and B tab, and foreground in remote device is A tab, devtool only keeps wake lock when devtool selects foreground's A tab. If devtool selects B tab (B isn't foreground in remote device), wake lock is unlocked. Does Firefox's devtools use same behaviour? Or, when device is connected (via wifi or USB) to devtool, should wake lock be kept even if selected tab in devtool isn't foreground in remote device?
Flags: needinfo?(jdescottes)
Flags: needinfo?(jdescottes)
> Or, when device is connected (via wifi or USB) to devtool, should wake lock be kept even if selected tab in devtool isn't foreground in remote device? Chrome's behavior seems more like a side effect. Is there value for users that I am missing that tab-centric wake-lock adds?
(In reply to :Harald Kirschner :digitarald from comment #8) > > Or, when device is connected (via wifi or USB) to devtool, should wake lock be kept even if selected tab in devtool isn't foreground in remote device? > > Chrome's behavior seems more like a side effect. Is there value for users > that I am missing that tab-centric wake-lock adds? I don't know. I think that Chrome approach is that they connect to device automatically (remote device tab of Chrome shows "connected" state when adb connection is found), so they acquire wake lock when selecting tab. Our devtool connects to device manually, so I think that it is better to acquire wake lock when connected to device.
Pushed by m_kato@ga2.so-net.ne.jp: https://hg.mozilla.org/integration/mozilla-inbound/rev/77ccb4525731 Part 1. Add nsIWakeLock to manage wake lock from devtools. r=baku https://hg.mozilla.org/integration/mozilla-inbound/rev/e358e36fc048 Part 2. Acquire wake lock when device actor is attached. r=ochameau
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: