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

Side by Side Diff: chrome/browser/history/android/android_provider_backend_unittest.cc

Issue 1020913002: Cleanup ChromeHistoryClientFactory (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix android compilation of unit tests Created 5 years, 9 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
« no previous file with comments | « no previous file | chrome/browser/history/chrome_history_client_factory.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 "chrome/browser/history/android/android_provider_backend.h" 5 #include "chrome/browser/history/android/android_provider_backend.h"
6 6
7 #include <vector> 7 #include <vector>
8 8
9 #include "base/files/file_path.h" 9 #include "base/files/file_path.h"
10 #include "base/files/file_util.h" 10 #include "base/files/file_util.h"
(...skipping 236 matching lines...) Expand 10 before | Expand all | Expand 10 after
247 base::ScopedTempDir temp_dir_; 247 base::ScopedTempDir temp_dir_;
248 base::FilePath android_cache_db_name_; 248 base::FilePath android_cache_db_name_;
249 base::FilePath history_db_name_; 249 base::FilePath history_db_name_;
250 base::FilePath thumbnail_db_name_; 250 base::FilePath thumbnail_db_name_;
251 251
252 TestingProfileManager profile_manager_; 252 TestingProfileManager profile_manager_;
253 BookmarkModel* bookmark_model_; 253 BookmarkModel* bookmark_model_;
254 base::MessageLoopForUI message_loop_; 254 base::MessageLoopForUI message_loop_;
255 content::TestBrowserThread ui_thread_; 255 content::TestBrowserThread ui_thread_;
256 content::TestBrowserThread file_thread_; 256 content::TestBrowserThread file_thread_;
257 ChromeHistoryClient* history_client_; 257 history::HistoryClient* history_client_;
258 258
259 DISALLOW_COPY_AND_ASSIGN(AndroidProviderBackendTest); 259 DISALLOW_COPY_AND_ASSIGN(AndroidProviderBackendTest);
260 }; 260 };
261 261
262 TEST_F(AndroidProviderBackendTest, UpdateTables) { 262 TEST_F(AndroidProviderBackendTest, UpdateTables) {
263 GURL url1("http://www.cnn.com"); 263 GURL url1("http://www.cnn.com");
264 URLID url_id1 = 0; 264 URLID url_id1 = 0;
265 std::vector<VisitInfo> visits1; 265 std::vector<VisitInfo> visits1;
266 Time last_visited1 = Time::Now() - TimeDelta::FromDays(1); 266 Time last_visited1 = Time::Now() - TimeDelta::FromDays(1);
267 Time created1 = last_visited1 - TimeDelta::FromDays(20); 267 Time created1 = last_visited1 - TimeDelta::FromDays(20);
(...skipping 1880 matching lines...) Expand 10 before | Expand all | Expand 10 after
2148 update_args, &update_count)); 2148 update_args, &update_count));
2149 // Verify notifications. 2149 // Verify notifications.
2150 EXPECT_FALSE(notifier_.deleted_details()); 2150 EXPECT_FALSE(notifier_.deleted_details());
2151 ASSERT_TRUE(notifier_.modified_details()); 2151 ASSERT_TRUE(notifier_.modified_details());
2152 ASSERT_EQ(1u, notifier_.modified_details()->size()); 2152 ASSERT_EQ(1u, notifier_.modified_details()->size());
2153 // No favicon will be updated as thumbnail database is missing. 2153 // No favicon will be updated as thumbnail database is missing.
2154 EXPECT_FALSE(notifier_.favicon_changed()); 2154 EXPECT_FALSE(notifier_.favicon_changed());
2155 } 2155 }
2156 2156
2157 } // namespace history 2157 } // namespace history
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/history/chrome_history_client_factory.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698