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

Unified Diff: chrome/browser/webdata/keyword_table_unittest.cc

Issue 9965143: Revert 130431 - Move the URL string from TemplateURLRef onto the owning TemplateURL. This will mak… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/webdata/keyword_table.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/webdata/keyword_table_unittest.cc
===================================================================
--- chrome/browser/webdata/keyword_table_unittest.cc (revision 130565)
+++ chrome/browser/webdata/keyword_table_unittest.cc (working copy)
@@ -100,8 +100,11 @@
EXPECT_EQ(keyword.created_by_policy(), restored_keyword->created_by_policy());
EXPECT_EQ(keyword.usage_count(), restored_keyword->usage_count());
EXPECT_EQ(keyword.prepopulate_id(), restored_keyword->prepopulate_id());
- EXPECT_EQ(keyword.url(), restored_keyword->url());
- EXPECT_EQ(keyword.instant_url(), restored_keyword->instant_url());
+ ASSERT_TRUE(restored_keyword->url());
+ EXPECT_EQ(keyword.url()->url(), restored_keyword->url()->url());
+ ASSERT_TRUE(restored_keyword->instant_url());
+ EXPECT_EQ(keyword.instant_url()->url(),
+ restored_keyword->instant_url()->url());
EXPECT_EQ(keyword.favicon_url(), restored_keyword->favicon_url());
EXPECT_TRUE(keyword_table->RemoveKeyword(restored_keyword->id()));
@@ -178,10 +181,13 @@
EXPECT_EQ(keyword.short_name(), backup_url->short_name());
EXPECT_EQ(keyword.keyword(), backup_url->keyword());
EXPECT_EQ(keyword.favicon_url(), backup_url->favicon_url());
- EXPECT_EQ(keyword.url(), backup_url->url());
+ ASSERT_TRUE(backup_url->url());
+ EXPECT_EQ(keyword.url()->url(), backup_url->url()->url());
EXPECT_EQ(keyword.safe_for_autoreplace(), backup_url->safe_for_autoreplace());
EXPECT_EQ(keyword.show_in_default_list(), backup_url->show_in_default_list());
- EXPECT_EQ(keyword.suggestions_url(), backup_url->suggestions_url());
+ ASSERT_TRUE(backup_url->suggestions_url());
+ EXPECT_EQ(keyword.suggestions_url()->url(),
+ backup_url->suggestions_url()->url());
EXPECT_FALSE(keyword_table->DidDefaultSearchProviderChange());
// Change the actual setting.
@@ -195,10 +201,13 @@
EXPECT_EQ(keyword.short_name(), backup_url->short_name());
EXPECT_EQ(keyword.keyword(), backup_url->keyword());
EXPECT_EQ(keyword.favicon_url(), backup_url->favicon_url());
- EXPECT_EQ(keyword.url(), backup_url->url());
+ ASSERT_TRUE(backup_url->url());
+ EXPECT_EQ(keyword.url()->url(), backup_url->url()->url());
EXPECT_EQ(keyword.safe_for_autoreplace(), backup_url->safe_for_autoreplace());
EXPECT_EQ(keyword.show_in_default_list(), backup_url->show_in_default_list());
- EXPECT_EQ(keyword.suggestions_url(), backup_url->suggestions_url());
+ ASSERT_TRUE(backup_url->suggestions_url());
+ EXPECT_EQ(keyword.suggestions_url()->url(),
+ backup_url->suggestions_url()->url());
EXPECT_TRUE(keyword_table->DidDefaultSearchProviderChange());
// Change the backup.
@@ -234,10 +243,13 @@
EXPECT_EQ(keyword.short_name(), backup_url->short_name());
EXPECT_EQ(keyword.keyword(), backup_url->keyword());
EXPECT_EQ(keyword.favicon_url(), backup_url->favicon_url());
- EXPECT_EQ(keyword.url(), backup_url->url());
+ ASSERT_TRUE(backup_url->url());
+ EXPECT_EQ(keyword.url()->url(), backup_url->url()->url());
EXPECT_EQ(keyword.safe_for_autoreplace(), backup_url->safe_for_autoreplace());
EXPECT_EQ(keyword.show_in_default_list(), backup_url->show_in_default_list());
- EXPECT_EQ(keyword.suggestions_url(), backup_url->suggestions_url());
+ ASSERT_TRUE(backup_url->suggestions_url());
+ EXPECT_EQ(keyword.suggestions_url()->url(),
+ backup_url->suggestions_url()->url());
EXPECT_TRUE(keyword_table->DidDefaultSearchProviderChange());
// Change keywords backup.
@@ -375,9 +387,13 @@
EXPECT_EQ(keyword.input_encodings(), restored_keyword->input_encodings());
EXPECT_EQ(keyword.id(), restored_keyword->id());
EXPECT_EQ(keyword.prepopulate_id(), restored_keyword->prepopulate_id());
- EXPECT_EQ(keyword.suggestions_url(), restored_keyword->suggestions_url());
+ ASSERT_TRUE(restored_keyword->suggestions_url());
+ EXPECT_EQ(keyword.suggestions_url()->url(),
+ restored_keyword->suggestions_url()->url());
+ ASSERT_TRUE(restored_keyword->instant_url());
EXPECT_EQ(keyword.favicon_url(), restored_keyword->favicon_url());
- EXPECT_EQ(keyword.instant_url(), restored_keyword->instant_url());
+ EXPECT_EQ(keyword.instant_url()->url(),
+ restored_keyword->instant_url()->url());
STLDeleteElements(&keywords);
}
« no previous file with comments | « chrome/browser/webdata/keyword_table.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698