Closed
Bug 708152
Opened 13 years ago
Closed 13 years ago
Convert uses of nsIScreen_MOZILLA_2_0_BRANCH to the new brightness API
Categories
(Core :: Widget, defect)
Core
Widget
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: emorley, Unassigned)
Details
Bug 702256 added a new brightness API, which makes nsIScreen_MOZILLA_2_0_BRANCH redundant.
The interface only appears to be used once in the tree (http://mxr.mozilla.org/mozilla-central/source/mobile/xul/chrome/content/common-ui.js#1179) to ensure that the screen stays on whilst watching videos fullscreen.
Presume we want to convert that instance to the new API and then depreciate and/or remove nsIScreen_MOZILLA_2_0_BRANCH (not sure what the addons situation is). Once that is done, bug 672166 will be redundant.
http://mxr.mozilla.org/mozilla-central/search?string=nsIScreen_MOZILLA_2_0_BRANCH
Comment 1•13 years ago
|
||
Now that I've thought about this more, maybe these should stay as separate APIs.
The mozBrightness API is for devices where we control the screen's brightness. There's no auto-shutoff to speak of.
But on devices where the screen automatically dims or shuts off, mozBrightness doesn't make much sense, and this API is more suitable.
Reporter | ||
Comment 2•13 years ago
|
||
True; happy to wontfix if you like?
Comment 3•13 years ago
|
||
Yes, I think so. There's some work around power APIs right now, so we may yet get rid of this one.
Reporter | ||
Updated•13 years ago
|
Status: NEW → RESOLVED
Closed: 13 years ago
Keywords: addon-compat,
dev-doc-needed
Resolution: --- → WONTFIX
Whiteboard: [mentor=jlebar]
You need to log in
before you can comment on or make changes to this bug.
Description
•