Chromium Code Reviews| Index: components/ntp_tiles/most_visited_sites.cc |
| diff --git a/components/ntp_tiles/most_visited_sites.cc b/components/ntp_tiles/most_visited_sites.cc |
| index ca75d4c77cd7a01006ebc850b30225b43ca05d70..64592cb00febd1252fe4c1e26dc6f2978595d6a4 100644 |
| --- a/components/ntp_tiles/most_visited_sites.cc |
| +++ b/components/ntp_tiles/most_visited_sites.cc |
| @@ -420,8 +420,12 @@ void MostVisitedSites::OnPopularSitesDownloaded(bool success) { |
| return; |
| } |
| - // Re-build the tile list. Once done, this will notify the observer. |
| - BuildCurrentTiles(); |
| + // Only build the tiles if there haven't been any. Otherwise, the UI should |
| + // not flicker. |
| + if (current_tiles_.empty()) { |
|
sfiera
2017/02/09 12:39:06
Can this actually happen? For a new user, we shoul
fhorschig
2017/02/09 15:30:18
Removed completely.
|
| + // Re-build the tile list. Once done, this will notify the observer. |
| + BuildCurrentTiles(); |
| + } |
| } |
| void MostVisitedSites::OnIconMadeAvailable(const GURL& site_url, |