Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3525)

Unified Diff: chrome/browser/resources/ntp4/tile_page.js

Issue 8190008: ntp4: hack around skia gradient brokenness. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/ntp4/tile_page.js
diff --git a/chrome/browser/resources/ntp4/tile_page.js b/chrome/browser/resources/ntp4/tile_page.js
index edd3eb0cc616841155ebf757895036ff1ba46830..590a45f1bcae14d22a3f84290a991320ddbf190e 100644
--- a/chrome/browser/resources/ntp4/tile_page.js
+++ b/chrome/browser/resources/ntp4/tile_page.js
@@ -723,7 +723,14 @@ cr.define('ntp4', function() {
}
var leftMargin = this.layoutValues_.leftMargin;
- var fadeDistance = Math.min(leftMargin, 20);
+ // The fade distance is the space between tiles.
+ var fadeDistance = (this.gridValues_.tileSpacingFraction *
+ this.layoutValues_.tileWidth);
+ fadeDistance = Math.min(leftMargin, fadeDistance);
+ // On Skia we don't use any fade because it works very poorly. See
+ // http://crbug.com/99373
+ if (!cr.isMac)
Dan Beam 2011/10/07 20:05:42 I still think you should mention in the above comm
+ fadeDistance = 1;
var gradient =
'-webkit-linear-gradient(left,' +
'transparent, ' +
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698