Create profile_sync_test_util in components

components/browser_sync/browser/profile_sync_test_util.h

+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef COMPONENTS_BROWSER_SYNC_BROWSER_PROFILE_SYNC_TEST_UTIL_H_
+#define COMPONENTS_BROWSER_SYNC_BROWSER_PROFILE_SYNC_TEST_UTIL_H_
+
+#include "base/time/time.h"
+#include "components/sync_driver/sync_service_observer.h"
+#include "testing/gmock/include/gmock/gmock.h"
+
+namespace base {
+class Time;
+class TimeDelta;
+}
+

[droger, 2016/01/18 16:58:44]

Should this be in the browser_sync namespace?

[vabr (Chromium), 2016/01/18 17:04:24]

I was thinking about that, but the component does not seem to have a unified
style yet (most of the code has no namespace, some is in sync_driver, etc.).

I suggest going with a sync OWNER suggestion -- stanisc@, what's your take on
this?

[stanisc, 2016/01/19 19:04:00]

The code in components/browser_sync has been moved around recently and the
namespace hasn't been changed everywhere yet. The namespace should be
browser_sync going forward.

[vabr (Chromium), 2016/01/20 16:43:41]

Thanks!
Done.

+// An empty syncer::NetworkTimeUpdateCallback. Used in various tests to
+// instantiate ProfileSyncService.
+void EmptyNetworkTimeUpdate(const base::Time&,
+                            const base::TimeDelta&,
+                            const base::TimeDelta&);
+
+class SyncServiceObserverMock : public sync_driver::SyncServiceObserver {
+ public:
+  SyncServiceObserverMock();
+  virtual ~SyncServiceObserverMock();
+
+  MOCK_METHOD0(OnStateChanged, void());
+};
+
+#endif  // COMPONENTS_BROWSER_SYNC_BROWSER_PROFILE_SYNC_TEST_UTIL_H_