Bug 1055261 - Reduce new tab performance impact by only attempting to show 5 columns (from 8) [r=adw a=sylvestre]
authorEd Lee <edilee@mozilla.com>
Mon, 18 Aug 2014 20:36:04 -0700
changeset 217679 f5d8b8bf02ffbf2b7890eec4bf991b7b30c0dc8b
parent 217678 7095ec4aa45e4e2ed272b047cb4a936dc3b68112
child 217680 7f6e4da5d10323cca8fd0b0e7d895248dd063cfb
push id515
push userraliiev@mozilla.com
push dateMon, 06 Oct 2014 12:51:51 +0000
treeherdermozilla-release@267c7a481bef [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersadw, sylvestre
bugs1055261
milestone33.0a2
Bug 1055261 - Reduce new tab performance impact by only attempting to show 5 columns (from 8) [r=adw a=sylvestre]
browser/app/profile/firefox.js
browser/base/content/test/newtab/browser_newtab_bug752841.js
--- a/browser/app/profile/firefox.js
+++ b/browser/app/profile/firefox.js
@@ -1513,17 +1513,17 @@ pref("browser.newtabpage.enabled", true)
 
 // Toggles the enhancement of history content of 'about:newtab'
 pref("browser.newtabpage.enhanced", false);
 
 // number of rows of newtab grid
 pref("browser.newtabpage.rows", 3);
 
 // number of columns of newtab grid
-pref("browser.newtabpage.columns", 8);
+pref("browser.newtabpage.columns", 5);
 
 // directory tiles download URL
 pref("browser.newtabpage.directory.source", "chrome://global/content/directoryLinks.json");
 
 // endpoint to send newtab click reports
 pref("browser.newtabpage.directory.reportClickEndPoint", "https://tiles.up.mozillalabs.com/ping/click");
 
 // Enable the DOM fullscreen API.
--- a/browser/base/content/test/newtab/browser_newtab_bug752841.js
+++ b/browser/base/content/test/newtab/browser_newtab_bug752841.js
@@ -12,18 +12,18 @@ function runTests() {
     {row: 0, column: -1},
     {row: 2, column: 4},
     {row: 2, column: 5},
   ];
 
   // Expected length of grid
   let expectedValues = [1, 1, 1, 1, 8, 10];
 
-   // Values before setting new pref values (24 is the default value -> 8 x 3)
-  let previousValues = [24, 1, 1, 1, 1, 8];
+   // Values before setting new pref values (15 is the default value -> 5 x 3)
+  let previousValues = [15, 1, 1, 1, 1, 8];
 
   let existingTab, existingTabGridLength, newTab, newTabGridLength;
   yield addNewTabPageTab();
   existingTab = gBrowser.selectedTab;
 
   for (let i = 0; i < expectedValues.length; i++) {
     gBrowser.selectedTab = existingTab;
     existingTabGridLength = getGrid().cells.length;