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

Side by Side Diff: src/org/adblockplus/android/Starter.java

Issue 8837074: ABP/Android Post-update starter (Closed)
Patch Set: ABP/Android Post-update starter Created Nov. 15, 2012, 11:31 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « AndroidManifest.xml ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 package org.adblockplus.android; 1 package org.adblockplus.android;
2 2
3 import android.content.BroadcastReceiver; 3 import android.content.BroadcastReceiver;
4 import android.content.Context; 4 import android.content.Context;
5 import android.content.Intent; 5 import android.content.Intent;
6 import android.content.SharedPreferences; 6 import android.content.SharedPreferences;
7 import android.preference.PreferenceManager; 7 import android.preference.PreferenceManager;
8 8
9 public class BootBroadcastReceiver extends BroadcastReceiver 9 public class Starter extends BroadcastReceiver
10 { 10 {
11
11 @Override 12 @Override
12 public void onReceive(Context context, Intent intent) 13 public void onReceive(Context context, Intent intent)
13 { 14 {
14 SharedPreferences prefs = PreferenceManager.getDefaultSharedPreferences(cont ext); 15 SharedPreferences prefs = PreferenceManager.getDefaultSharedPreferences(cont ext);
15 boolean enabled = prefs.getBoolean(context.getString(R.string.pref_enabled), false); 16 boolean enabled = prefs.getBoolean(context.getString(R.string.pref_enabled), false);
16 boolean startAtBoot = prefs.getBoolean(context.getString(R.string.pref_start atboot), context.getResources().getBoolean(R.bool.def_startatboot)); 17 if (Intent.ACTION_PACKAGE_REPLACED.equals(intent.getAction()))
Felix Dahlke 2012/11/16 13:54:18 intent.getAction() is used twice, I'd assign it to
Andrey Novikov 2012/11/16 13:56:08 Yes, I wanted to, but forgot. :) Fixed.
17 if (enabled && startAtBoot) 18 {
19 enabled &= "org.adblockplus.android".equals(intent.getData().getSchemeSpec ificPart());
20 }
21 if (Intent.ACTION_BOOT_COMPLETED.equals(intent.getAction()))
22 {
23 boolean startAtBoot = prefs.getBoolean(context.getString(R.string.pref_sta rtatboot), context.getResources().getBoolean(R.bool.def_startatboot));
24 enabled &= startAtBoot;
25 }
26 if (enabled)
18 context.startService(new Intent(context, ProxyService.class)); 27 context.startService(new Intent(context, ProxyService.class));
19 } 28 }
29
20 } 30 }
OLDNEW
« no previous file with comments | « AndroidManifest.xml ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld