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

Unified Diff: background.js

Issue 8787016: Remove "disable inline text-ads" feature (Closed)
Patch Set: Created Nov. 6, 2012, 7:19 a.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 | include.textads.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: background.js
===================================================================
--- a/background.js
+++ b/background.js
@@ -72,7 +72,6 @@
defaultOptionValue("shouldShowIcon", "true");
defaultOptionValue("shouldShowBlockElementMenu", "true");
- defaultOptionValue("disableInlineTextAds", "false");
// If user had older version installed, get rid of old option
if ("specialCaseYouTube" in localStorage)
@@ -562,7 +561,7 @@
// The page action popup asks us this.
if(sender.tab)
{
- sendResponse({enabled: !isWhitelisted(sender.tab.url), specialCaseYouTube: localStorage["specialCaseYouTube"] == "true", disableInlineTextAds: localStorage["disableInlineTextAds"] == "true"});
+ sendResponse({enabled: !isWhitelisted(sender.tab.url), specialCaseYouTube: localStorage["specialCaseYouTube"] == "true"});
Wladimir Palant 2012/11/06 08:06:11 Please get rid of specialCaseYouTube as well here
Felix Dahlke 2012/11/06 08:16:38 Done.
return;
}
break;
« no previous file with comments | « no previous file | include.textads.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld