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

Unified Diff: chrome/content/ui/firstRun.js

Issue 5450822905233408: Issue 417 - Reinitializing removed filter lists (Core/Firefox) (Closed)
Patch Set: Moved check to Platform code Created May 26, 2014, 5:41 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 | « chrome/content/ui/firstRun.html ('k') | chrome/locale/en-US/firstRun.properties » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/content/ui/firstRun.js
===================================================================
--- a/chrome/content/ui/firstRun.js
+++ b/chrome/content/ui/firstRun.js
@@ -56,11 +56,21 @@
setLinks("acceptableAdsExplanation", Utils.getDocLink("acceptable_ads_criteria"), openFilters);
setLinks("share-headline", Utils.getDocLink("contribute"));
- // Show warning if data corruption was detected
- if (typeof backgroundPage != "undefined" && backgroundPage.seenDataCorruption)
+ if (typeof backgroundPage != "undefined")
{
- E("dataCorruptionWarning").removeAttribute("hidden");
- setLinks("dataCorruptionWarning", Utils.getDocLink("knownIssuesChrome_filterstorage"));
+ // Show warning if data corruption was detected
+ if (backgroundPage.seenDataCorruption)
+ {
+ E("dataCorruptionWarning").removeAttribute("hidden");
+ setLinks("dataCorruptionWarning", Utils.getDocLink("knownIssuesChrome_filterstorage"));
+ }
+
+ // Show warning if filterlists settings were reinitialized
+ if (backgroundPage.filterlistsReinitialized)
+ {
+ E("filterlistsReinitializedWarning").removeAttribute("hidden");
+ setLinks("filterlistsReinitializedWarning", openFilters);
+ }
}
// Show warning if Safari version isn't supported
« no previous file with comments | « chrome/content/ui/firstRun.html ('k') | chrome/locale/en-US/firstRun.properties » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld