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

Unified Diff: mobile/android/base/BrowserApp.java

Issue 6222010732511232: Noissue - Prefix our IDs/strings with abb_ and group them (Closed)
Patch Set: Created May 8, 2015, 4:14 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | mobile/android/base/locales/en-US/android_strings.dtd » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mobile/android/base/BrowserApp.java
===================================================================
--- a/mobile/android/base/BrowserApp.java
+++ b/mobile/android/base/BrowserApp.java
@@ -288,7 +288,7 @@
updateHomePagerForTab(tab);
if (mMenu != null)
org.adblockplus.browser.BrowserAppUtils.updateBlockAdsMenuItem(
- mMenu.findItem(R.id.block_ads));
+ mMenu.findItem(R.id.abb_block_ads));
}
mHideDynamicToolbarOnActionModeEnd = false;
@@ -2589,7 +2589,7 @@
}
org.adblockplus.browser.BrowserAppUtils.updateBlockAdsMenuItem(
- mMenu.findItem(R.id.block_ads));
+ mMenu.findItem(R.id.abb_block_ads));
return true;
}
@@ -2955,7 +2955,7 @@
return true;
}
- if (itemId == R.id.block_ads) {
+ if (itemId == R.id.abb_block_ads) {
org.adblockplus.browser.BrowserAppUtils.updateCurrentTabWhitelisting(item);
return true;
}
« no previous file with comments | « no previous file | mobile/android/base/locales/en-US/android_strings.dtd » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld