(In reply to Karl Tomlinson (needinfo?:karlt) from comment #41)
> >+WakeLockTopic::UninhibitScreensaver()
> >+{
> >+  if (!mShouldInhibit) {
> >+    // Screensaver isn't inhibited. Nothing to do here.
> >+    return NS_OK;
> 
> Could this instead assert that this path is not reached, as in
> InhibitScreensaver()?

No. There are two states "unlocked" and "locked-background" that we map
to Uninhibit, that can both be reached directly from "locked-
foreground".

Addressing other comments, new patch coming.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/434476

Title:
  screensaver starts while playing HTML5 videos

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/434476/+subscriptions

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to