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

Unified Diff: chrome/browser/history/history_unittest.cc

Issue 10669038: base: Remove dereference structure operator (i.e ->) from ScopedVector. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 8 years, 6 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 | « chrome/browser/history/history_backend.cc ('k') | chrome/browser/importer/importer_list.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/history/history_unittest.cc
diff --git a/chrome/browser/history/history_unittest.cc b/chrome/browser/history/history_unittest.cc
index f51bc2a14a1dd37da06e8ada8babcf4d6480d503..a83c595eefac41ca2de2248ead7b8102edbe3e68 100644
--- a/chrome/browser/history/history_unittest.cc
+++ b/chrome/browser/history/history_unittest.cc
@@ -127,7 +127,7 @@ class HistoryTest : public testing::Test {
void OnSegmentUsageAvailable(CancelableRequestProvider::Handle handle,
std::vector<PageUsageData*>* data) {
- page_usage_data_->swap(*data);
+ page_usage_data_.swap(*data);
MessageLoop::current()->Quit();
}
@@ -723,7 +723,7 @@ TEST_F(HistoryTest, Segments) {
// Wait for processing.
MessageLoop::current()->Run();
- ASSERT_EQ(1U, page_usage_data_->size());
+ ASSERT_EQ(1U, page_usage_data_.size());
EXPECT_TRUE(page_usage_data_[0]->GetURL() == existing_url);
EXPECT_DOUBLE_EQ(3.0, page_usage_data_[0]->GetScore());
@@ -743,7 +743,7 @@ TEST_F(HistoryTest, Segments) {
MessageLoop::current()->Run();
// Make sure we still have one segment.
- ASSERT_EQ(1U, page_usage_data_->size());
+ ASSERT_EQ(1U, page_usage_data_.size());
EXPECT_TRUE(page_usage_data_[0]->GetURL() == existing_url);
// Add a page linked from existing_url.
@@ -761,7 +761,7 @@ TEST_F(HistoryTest, Segments) {
MessageLoop::current()->Run();
// Make sure we still have one segment.
- ASSERT_EQ(1U, page_usage_data_->size());
+ ASSERT_EQ(1U, page_usage_data_.size());
EXPECT_TRUE(page_usage_data_[0]->GetURL() == existing_url);
// However, the score should have increased.
« no previous file with comments | « chrome/browser/history/history_backend.cc ('k') | chrome/browser/importer/importer_list.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698