Bug 1552328 - Add missing unregisterListener in GeckoViewProgress. r=droeh
☠☠ backed out by 55e45fc9d529 ☠ ☠
authorAgi Sferro <agi@sferro.dev>
Fri, 17 May 2019 04:11:29 +0000
changeset 536176 aee7bbc03bff773a8ce895d0bd58174f2f8aafa1
parent 536175 2f69f765ef513a61ad39b4fc2b294135cf514e98
child 536177 877ae9359a2774f9ccee1f68935b4e7194a5711e
push id2082
push userffxbld-merge
push dateMon, 01 Jul 2019 08:34:18 +0000
treeherdermozilla-release@2fb19d0466d2 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersdroeh
bugs1552328
milestone68.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1552328 - Add missing unregisterListener in GeckoViewProgress. r=droeh Differential Revision: https://phabricator.services.mozilla.com/D31538
mobile/android/modules/geckoview/GeckoViewProgress.jsm
--- a/mobile/android/modules/geckoview/GeckoViewProgress.jsm
+++ b/mobile/android/modules/geckoview/GeckoViewProgress.jsm
@@ -164,16 +164,17 @@ class GeckoViewProgress extends GeckoVie
     debug `onDisable`;
 
     if (this.progressFilter) {
       this.progressFilter.removeProgressListener(this);
       this.browser.removeProgressListener(this.progressFilter);
     }
 
     Services.obs.removeObserver(this, "oop-frameloader-crashed");
+    this.unregisterListener("GeckoView:FlushSessionState");
   }
 
   onEvent(aEvent, aData, aCallback) {
     debug `onEvent: event=${aEvent}, data=${aData}`;
 
     switch (aEvent) {
       case "GeckoView:FlushSessionState":
         this.messageManager.sendAsyncMessage("GeckoView:FlushSessionState");