Mozilla Firefox

  1. Bug #434476
  2. Comment #99

Comment 99 for bug 434476

Revision history for this message
In , Edwin-d (edwin-d) wrote :

(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.