Chromium Code Reviews
Description[M53] service worker: Refactor OnNoControllees
OnNoControllees doesn't need to unset the flags.
* If |is_uninstalling_| is set, the call to Clear() would unset it.
* If |should_activate_when_ready_| is set, the call to ActivateWaitingVersion() would
unset it. We also change Clear() to unset |should_activate_when_ready_| which
makes sense for any callsite of Clear().
Minor simplification in anticipation of changes for https://crbug.com/616331
Refactoring only; no behavior change.
BUG=616331
Review-Url: https://codereview.chromium.org/2115393002
Cr-Commit-Position: refs/heads/master@{#403650}
(cherry picked from commit 6c74874b9c6969a2afc6f195c96a1a3b15cda890)
Committed: https://chromium.googlesource.com/chromium/src/+/09a50f12f80cb14568479cf33b10d55cdb6a54a2
Patch Set 1 #
Messages
Total messages: 2 (1 generated)
|
|||||||||||||||||||