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

Side by Side Diff: chrome/browser/sync/test/integration/two_client_bookmarks_sync_test.cc

Issue 24175004: Remove dependency on ui::ScaleFactor from ui/gfx (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix new usage of scale in FastShowPickler Created 7 years, 3 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/rand_util.h" 5 #include "base/rand_util.h"
6 #include "chrome/browser/profiles/profile.h" 6 #include "chrome/browser/profiles/profile.h"
7 #include "chrome/browser/sync/profile_sync_service_harness.h" 7 #include "chrome/browser/sync/profile_sync_service_harness.h"
8 #include "chrome/browser/sync/test/integration/bookmarks_helper.h" 8 #include "chrome/browser/sync/test/integration/bookmarks_helper.h"
9 #include "chrome/browser/sync/test/integration/passwords_helper.h" 9 #include "chrome/browser/sync/test/integration/passwords_helper.h"
10 #include "chrome/browser/sync/test/integration/sync_test.h" 10 #include "chrome/browser/sync/test/integration/sync_test.h"
(...skipping 147 matching lines...) Expand 10 before | Expand all | Expand 10 after
158 // In particular, the synced 16x16 favicon bitmap should overwrite 16x16 158 // In particular, the synced 16x16 favicon bitmap should overwrite 16x16
159 // favicon bitmaps on all clients. (Though non-16x16 favicon bitmaps 159 // favicon bitmaps on all clients. (Though non-16x16 favicon bitmaps
160 // are unchanged). 160 // are unchanged).
161 IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, SC_SetFaviconHiDPI) { 161 IN_PROC_BROWSER_TEST_F(TwoClientBookmarksSyncTest, SC_SetFaviconHiDPI) {
162 // Set the supported scale factors to include 2x such that CreateFavicon() 162 // Set the supported scale factors to include 2x such that CreateFavicon()
163 // creates a favicon with hidpi representations and that methods in the 163 // creates a favicon with hidpi representations and that methods in the
164 // FaviconService request hidpi favicons. 164 // FaviconService request hidpi favicons.
165 std::vector<ui::ScaleFactor> supported_scale_factors; 165 std::vector<ui::ScaleFactor> supported_scale_factors;
166 supported_scale_factors.push_back(ui::SCALE_FACTOR_100P); 166 supported_scale_factors.push_back(ui::SCALE_FACTOR_100P);
167 supported_scale_factors.push_back(ui::SCALE_FACTOR_200P); 167 supported_scale_factors.push_back(ui::SCALE_FACTOR_200P);
168 ui::test::SetSupportedScaleFactors(supported_scale_factors); 168 ui::SetSupportedScaleFactors(supported_scale_factors);
169 169
170 const GURL page_url(kGenericURL); 170 const GURL page_url(kGenericURL);
171 const GURL icon_url1("http://www.google.com/favicon1.ico"); 171 const GURL icon_url1("http://www.google.com/favicon1.ico");
172 const GURL icon_url2("http://www.google.com/favicon2.ico"); 172 const GURL icon_url2("http://www.google.com/favicon2.ico");
173 173
174 ASSERT_TRUE(SetupSync()) << "SetupSync() failed."; 174 ASSERT_TRUE(SetupSync()) << "SetupSync() failed.";
175 ASSERT_TRUE(AllModelsMatchVerifier()); 175 ASSERT_TRUE(AllModelsMatchVerifier());
176 176
177 const BookmarkNode* bookmark0 = AddURL(0, kGenericURLTitle, page_url); 177 const BookmarkNode* bookmark0 = AddURL(0, kGenericURLTitle, page_url);
178 ASSERT_TRUE(bookmark0 != NULL); 178 ASSERT_TRUE(bookmark0 != NULL);
(...skipping 1829 matching lines...) Expand 10 before | Expand all | Expand 10 after
2008 2008
2009 // Remove all 2009 // Remove all
2010 RemoveAll(0); 2010 RemoveAll(0);
2011 2011
2012 ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1))); 2012 ASSERT_TRUE(GetClient(0)->AwaitMutualSyncCycleCompletion(GetClient(1)));
2013 // Verify other node has no children now. 2013 // Verify other node has no children now.
2014 EXPECT_EQ(0, GetOtherNode(0)->child_count()); 2014 EXPECT_EQ(0, GetOtherNode(0)->child_count());
2015 EXPECT_EQ(0, GetBookmarkBarNode(0)->child_count()); 2015 EXPECT_EQ(0, GetBookmarkBarNode(0)->child_count());
2016 ASSERT_TRUE(AllModelsMatch()); 2016 ASSERT_TRUE(AllModelsMatch());
2017 } 2017 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698