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

Delta Between Two Patch Sets: mobile/android/base/GeckoApplication.java

Issue 29350065: Issue 2853 - Settings changes are sometimes not saved if the user quits the app (Closed)
Left Patch Set: Created Aug. 23, 2016, 12:39 p.m.
Right Patch Set: Renaming 'uncompleted' to 'pending' Created Jan. 31, 2017, 8 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « adblockplus/Api.jsm ('k') | mobile/android/thirdparty/org/adblockplus/browser/AddOnBridge.java » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
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
155 org.adblockplus.browser.AddOnBridge.init(context); 158 org.adblockplus.browser.AddOnBridge.init(context);
156 159
157 super.onCreate(); 160 super.onCreate();
158 } 161 }
159 162
160 public boolean isApplicationInBackground() { 163 public boolean isApplicationInBackground() {
161 return mInBackground; 164 return mInBackground;
162 } 165 }
163 166
164 public LightweightTheme getLightweightTheme() { 167 public LightweightTheme getLightweightTheme() {
165 return mLightweightTheme; 168 return mLightweightTheme;
166 } 169 }
167 170
168 public void prepareLightweightTheme() { 171 public void prepareLightweightTheme() {
169 mLightweightTheme = new LightweightTheme(this); 172 mLightweightTheme = new LightweightTheme(this);
170 } 173 }
171 } 174 }
LEFTRIGHT

Powered by Google App Engine
This is Rietveld