Move browsing_data counter sync tests to separate file

components/browsing_data/core/counters/browsing_data_counter.cc

Index: components/browsing_data/core/counters/browsing_data_counter.cc
diff --git a/components/browsing_data/core/counters/browsing_data_counter.cc b/components/browsing_data/core/counters/browsing_data_counter.cc
index fecc32c8740d37d4abdbb5f678774b751febc065..e31ed2be2e9f900c2fc82cb57893a061e2e6c9e8 100644
--- a/components/browsing_data/core/counters/browsing_data_counter.cc
+++ b/components/browsing_data/core/counters/browsing_data_counter.cc
@@ -167,4 +167,17 @@ BrowsingDataCounter::ResultInt BrowsingDataCounter::FinishedResult::Value()
   return value_;
 }
 
+// BrowsingDataCounter::SyncResult -----------------------------------------
+
+BrowsingDataCounter::SyncResult::SyncResult(const BrowsingDataCounter* source,
+                                            ResultInt value,
+                                            bool sync_enabled)
+    : FinishedResult(source, value), sync_enabled_(sync_enabled) {}
+
+BrowsingDataCounter::SyncResult::~SyncResult() {}
+
+bool BrowsingDataCounter::SyncResult::IsSyncEnabled() const {

[msramek, 2017/05/11 15:49:14]

style nit: Simple non-virtual getters and setters matching the format

set_x(X x) { x_ = x; };

and

get_x() const { return x_ };

are typically underscore_case and in the .h file.

[dullweber, 2017/05/12 13:37:31]

done. Is Value() using the wrong style then?

[msramek, 2017/05/15 17:56:53]

Hmm, apparently so. It was probably virtual at one point.

+  return sync_enabled_;
+}
+
 }  // namespace browsing_data