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

Unified Diff: lib/survey.js

Issue 8433028: added hook function to appIntegration to handle function-overwrites from other extensions (Closed)
Patch Set: applied code review changes Created Sept. 28, 2012, 8:01 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
« lib/appIntegration.js ('K') | « lib/rules.js ('k') | lib/typoFixer.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/survey.js
===================================================================
--- a/lib/survey.js
+++ b/lib/survey.js
@@ -7,11 +7,15 @@
let {Prefs} = require("prefs");
let surveyLang = null;
-let surveyUrl = "http://urlfixer.org"; //"http://adblockplus.org/usersurvey/index.php?sid=68316";
+let surveyUrl = null; //"http://urlfixer.org/usersurvey/...";
exports.incrementCorrectionsCounter = incrementCorrectionsCounter;
function incrementCorrectionsCounter()
{
+ // Only if survey exists
+ if (!surveyUrl)
+ return;
+
// Only users with 5 URL corrections
if (++Prefs.corrections_count == 5)
{
« lib/appIntegration.js ('K') | « lib/rules.js ('k') | lib/typoFixer.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld