NTP5: Implementing new specification for favicons of Most Visited Pages.

chrome/browser/resources/ntp_search/thumbnail_page.css

Index: chrome/browser/resources/ntp_search/thumbnail_page.css
diff --git a/chrome/browser/resources/ntp_search/thumbnail_page.css b/chrome/browser/resources/ntp_search/thumbnail_page.css
index 1835d3d9611b5eaf399060e781d27955db47ea0c..55c98324b3c918d60f1d755eebb68a6c2dd522a1 100644
--- a/chrome/browser/resources/ntp_search/thumbnail_page.css
+++ b/chrome/browser/resources/ntp_search/thumbnail_page.css
@@ -50,7 +50,7 @@
 
 .thumbnail .title,
 .thumbnail-banner {
-  bottom: -18px;
+  bottom: -26px;
   color: #777;
   font-size: 0.9167em;
   overflow: hidden;
@@ -75,7 +75,7 @@
 }
 
 .thumbnail-banner {
-  bottom: 13px;
+  bottom: 25px;
   display: block;
   font-size: 1.14em;
   margin: 0 7px;
@@ -84,9 +84,10 @@
 
 .thumbnail-favicon {
   height: 16px;
+  left: 46px;

[Evan Stade, 2012/11/20 20:05:00]

RTL?

[pedro (no code reviews), 2012/11/20 20:14:00]

This is being centered, so RTL is not an issue here.

[Evan Stade, 2012/11/20 20:15:14]

this is a fragile way to center something.

[pedro (no code reviews), 2012/11/20 21:38:45]

The favicon should be on top of the thumbnail, and half of it should "bleed off"
the thumbnail, which means we must use absolute position. Because of that, we
can't use "margin: 0 auto" to center the favicon, that is, we must position the
favicon manually using top and left properties.

   margin: 0 auto;
-  position: relative;
-  top: 19px;
+  position: absolute;
+  top: 58px;
   width: 16px;
 }