Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Side by Side Diff: mobile/android/base/GeckoApplication.java

Issue 29350065: Issue 2853 - Settings changes are sometimes not saved if the user quits the app (Closed)
Patch Set: Instead of pooling, adding observer for save operations performed by the extension Created Dec. 21, 2016, 2:22 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* This Source Code Form is subject to the terms of the Mozilla Public 1 /* This Source Code Form is subject to the terms of the Mozilla Public
2 * License, v. 2.0. If a copy of the MPL was not distributed with this file, 2 * License, v. 2.0. If a copy of the MPL was not distributed with this file,
3 * You can obtain one at http://mozilla.org/MPL/2.0/. */ 3 * You can obtain one at http://mozilla.org/MPL/2.0/. */
4 4
5 package org.mozilla.gecko; 5 package org.mozilla.gecko;
6 6
7 import org.mozilla.gecko.db.BrowserContract; 7 import org.mozilla.gecko.db.BrowserContract;
8 import org.mozilla.gecko.db.BrowserDB; 8 import org.mozilla.gecko.db.BrowserDB;
9 import org.mozilla.gecko.db.LocalBrowserDB; 9 import org.mozilla.gecko.db.LocalBrowserDB;
10 import org.mozilla.gecko.home.HomePanelsManager; 10 import org.mozilla.gecko.home.HomePanelsManager;
(...skipping 134 matching lines...) Expand 10 before | Expand all | Expand 10 after
145 GeckoProfile.setBrowserDBFactory(new BrowserDB.Factory() { 145 GeckoProfile.setBrowserDBFactory(new BrowserDB.Factory() {
146 @Override 146 @Override
147 public BrowserDB get(String profileName, File profileDir) { 147 public BrowserDB get(String profileName, File profileDir) {
148 // Note that we don't use the profile directory -- we 148 // Note that we don't use the profile directory -- we
149 // send operations to the ContentProvider, which does 149 // send operations to the ContentProvider, which does
150 // its own thing. 150 // its own thing.
151 return new LocalBrowserDB(profileName); 151 return new LocalBrowserDB(profileName);
152 } 152 }
153 }); 153 });
154 154
155 // Added AddOnBridge initialisation here for resending requests that for some reason,
156 // couldn't be saved by the extension
157 // See https://issues.adblockplus.org/ticket/2853
158 org.adblockplus.browser.AddOnBridge.init(context);
159
155 super.onCreate(); 160 super.onCreate();
156 } 161 }
157 162
158 public boolean isApplicationInBackground() { 163 public boolean isApplicationInBackground() {
159 return mInBackground; 164 return mInBackground;
160 } 165 }
161 166
162 public LightweightTheme getLightweightTheme() { 167 public LightweightTheme getLightweightTheme() {
163 return mLightweightTheme; 168 return mLightweightTheme;
164 } 169 }
165 170
166 public void prepareLightweightTheme() { 171 public void prepareLightweightTheme() {
167 mLightweightTheme = new LightweightTheme(this); 172 mLightweightTheme = new LightweightTheme(this);
168 } 173 }
169 } 174 }
OLDNEW

Powered by Google App Engine
This is Rietveld