Bug 1039881 - Use an empty directory tiles data source pref before uplift. r=adw, a=sledru
☠☠ backed out by 5bb486b0f89a ☠ ☠
authorEd Lee <edilee>
Fri, 18 Jul 2014 00:07:00 -0400
changeset 209168 7da62c51fa9742e1f9590b5a0363199c04271d48
parent 209167 238174823b7c77b998984e3b73c536c5c39488ad
child 209169 589fd10b25751e0a6e5eda5b719994a88ee947b2
push id494
push userraliiev@mozilla.com
push dateMon, 25 Aug 2014 18:42:16 +0000
treeherdermozilla-release@a3cc3e46b571 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersadw, sledru
bugs1039881
milestone32.0a2
Bug 1039881 - Use an empty directory tiles data source pref before uplift. r=adw, a=sledru Update tests to use a dummy pref, so it changes no matter what's the default.
browser/app/profile/firefox.js
browser/base/content/test/general/browser_tabopen_reflows.js
browser/base/content/test/newtab/head.js
--- a/browser/app/profile/firefox.js
+++ b/browser/app/profile/firefox.js
@@ -1466,17 +1466,17 @@ pref("browser.newtab.preload", true);
 pref("browser.newtabpage.enabled", true);
 
 // number of rows of newtab grid
 pref("browser.newtabpage.rows", 3);
 
 // number of columns of newtab grid
 pref("browser.newtabpage.columns", 3);
 
-pref("browser.newtabpage.directory.source", "chrome://global/content/directoryLinks.json");
+pref("browser.newtabpage.directory.source", "data:application/json,{}");
 
 // Enable the DOM fullscreen API.
 pref("full-screen-api.enabled", true);
 
 // True if the fullscreen API requires approval upon a domain entering fullscreen.
 // Domains that have already had fullscreen permission granted won't re-request
 // approval.
 pref("full-screen-api.approval-required", true);
--- a/browser/base/content/test/general/browser_tabopen_reflows.js
+++ b/browser/base/content/test/general/browser_tabopen_reflows.js
@@ -102,18 +102,18 @@ function test() {
       // Remove reflow observer and clean up.
       docShell.removeWeakReflowObserver(observer);
       gBrowser.removeCurrentTab();
       finish();
     });
   });
 
   Services.prefs.setBoolPref(PREF_PRELOAD, false);
-  // set directory source to empty links
-  Services.prefs.setCharPref(PREF_NEWTAB_DIRECTORYSOURCE, "data:application/json,{}");
+  // set directory source to dummy/empty links
+  Services.prefs.setCharPref(PREF_NEWTAB_DIRECTORYSOURCE, 'data:application/json,{"test":1}');
 }
 
 let observer = {
   reflow: function (start, end) {
     // Gather information about the current code path.
     let path = (new Error().stack).split("\n").slice(1).map(line => {
       return line.replace(/:\d+:\d+$/, "");
     }).join("|");
--- a/browser/base/content/test/newtab/head.js
+++ b/browser/base/content/test/newtab/head.js
@@ -84,17 +84,18 @@ function watchLinksChangeOnce() {
  * Provide the default test function to start our test runner.
  */
 function test() {
   waitForExplicitFinish();
   // start TestRunner.run() after directory links is downloaded and written to disk
   watchLinksChangeOnce().then(() => {
     TestRunner.run();
   });
-  Services.prefs.setCharPref(PREF_NEWTAB_DIRECTORYSOURCE, "data:application/json,{}");
+  // set directory source to dummy/empty links
+  Services.prefs.setCharPref(PREF_NEWTAB_DIRECTORYSOURCE, 'data:application/json,{"test":1}');
 }
 
 /**
  * The test runner that controls the execution flow of our tests.
  */
 let TestRunner = {
   /**
    * Starts the test runner.