Add field to monitor last visited time for each search engine

components/search_engines/default_search_manager_unittest.cc

Index: components/search_engines/default_search_manager_unittest.cc
diff --git a/components/search_engines/default_search_manager_unittest.cc b/components/search_engines/default_search_manager_unittest.cc
index f2a58e6c904f2ef9d760821df29c3521f9b917e6..2d3d706ac72e3fc9fb899bcfbd05e938885a3d6b 100644
--- a/components/search_engines/default_search_manager_unittest.cc
+++ b/components/search_engines/default_search_manager_unittest.cc
@@ -29,8 +29,8 @@ namespace {
 const char kDefaultSearchProviderData[] =
     "default_search_provider_data.template_url_data";
 
-// Checks that the two TemplateURLs are similar. Does not check the id, the
-// date_created or the last_modified time.  Neither pointer should be NULL.
+// Checks that the two TemplateURLs are similar. Does not check the id and

[Peter Kasting, 2016/12/01 07:38:24]

Nit: and -> or

[ltian, 2016/12/01 10:02:58]

Done.

+// any time-related fields.  Neither pointer should be NULL.

[Peter Kasting, 2016/12/01 07:38:24]

Nit: While here: NULL -> null

[ltian, 2016/12/01 10:02:58]

Done.

 void ExpectSimilar(const TemplateURLData* expected,
                    const TemplateURLData* actual) {
   ASSERT_TRUE(expected != NULL);
@@ -111,6 +111,7 @@ std::unique_ptr<TemplateURLData> GenerateDummyTemplateURLData(
       "UTF-8;UTF-16", ";", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL);
   data->date_created = base::Time();
   data->last_modified = base::Time();
+  data->last_visited = base::Time();
   return data;
 }
 
@@ -152,6 +153,7 @@ TEST_F(DefaultSearchManagerTest, ReadAndWritePref) {
       "UTF-8;UTF-16", ";", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL);
   data.date_created = base::Time();
   data.last_modified = base::Time();
+  data.last_modified = base::Time();
 
   manager.SetUserSelectedDefaultSearchEngine(data);
   TemplateURLData* read_data = manager.GetDefaultSearchEngine(NULL);