Move part of browsing data counter text util method to components

chrome/browser/browsing_data/browsing_data_counter_utils_unittest.cc

Index: chrome/browser/browsing_data/browsing_data_counter_utils_unittest.cc
diff --git a/chrome/browser/browsing_data/browsing_data_counter_utils_unittest.cc b/chrome/browser/browsing_data/browsing_data_counter_utils_unittest.cc
index c00812865fe05159ddc9e5a7e3e4f90ebbc67cca..bc5316043dd30aeef33d63523b90ba727c3f21e2 100644
--- a/chrome/browser/browsing_data/browsing_data_counter_utils_unittest.cc
+++ b/chrome/browser/browsing_data/browsing_data_counter_utils_unittest.cc
@@ -77,7 +77,7 @@ TEST_F(BrowsingDataCounterUtilsTest, AutofillCounterResult) {
         test_case.num_suggestions
     ));
 
-    base::string16 output = GetCounterTextFromResult(&result);
+    base::string16 output = GetChromeCounterTextFromResult(&result);

[msramek, 2016/08/09 16:28:32]

The autofill-related functionality now lives in GetCounterTextFromResult(). So
this should be changed back and instead, this entire test should be moved to
//components.

[ioanap, 2016/08/10 09:34:37]

Done.

[msramek, 2016/08/10 09:59:50]

Just for the record - for maximum pedantness, this whole file should be only
compiled if ENABLE_EXTENSIONS are defined, since that is the only functionality
left. But I think that would make things very confusing as
browsing_data_counter_utils.cc is compiled always, so let's just leave it as it
is.

[ioanap, 2016/08/10 11:12:32]

Acknowledged. Thank you!

     EXPECT_EQ(output, base::ASCIIToUTF16(test_case.expected_output));
   }
 }
@@ -120,7 +120,7 @@ TEST_F(BrowsingDataCounterUtilsTest, HostedAppsCounterResult) {
         apps.size(),
         examples);
 
-    base::string16 output = GetCounterTextFromResult(&result);
+    base::string16 output = GetChromeCounterTextFromResult(&result);
     EXPECT_EQ(output, base::ASCIIToUTF16(test_case.expected_output));
   }
 }