[Sync] Fix namespaces for the browser_sync component.

chrome/browser/sync/test/integration/p2p_invalidation_forwarder.h

Index: chrome/browser/sync/test/integration/p2p_invalidation_forwarder.h
diff --git a/chrome/browser/sync/test/integration/p2p_invalidation_forwarder.h b/chrome/browser/sync/test/integration/p2p_invalidation_forwarder.h
index 52e93bcaad0283a10eca71d87afc10c025aeab9f..5d507ce15ab306078842627590fd38b9d929b33b 100644
--- a/chrome/browser/sync/test/integration/p2p_invalidation_forwarder.h
+++ b/chrome/browser/sync/test/integration/p2p_invalidation_forwarder.h
@@ -9,11 +9,13 @@
 #include "base/macros.h"
 #include "components/sync/driver/sync_service_observer.h"
 
+namespace browser_sync {
 class ProfileSyncService;
+}

[skym, 2016/09/22 17:25:58]

Shouldn't these have  // namespace foo upon close? Does lint catch this? Or does
lint miss the integration folder for some reason?

[maxbogue, 2016/09/22 19:41:14]

Apparently it doesn't catch it. I don't even know if the style guide demands it
for forward declarations like this, but I'll add them.

 
 namespace invalidation {
 class P2PInvalidationService;
-};
+}
 
 // This class links the ProfileSyncService to a P2PInvalidationService.
 //
@@ -25,7 +27,7 @@ class P2PInvalidationService;
 class P2PInvalidationForwarder : public sync_driver::SyncServiceObserver {
  public:
   P2PInvalidationForwarder(
-      ProfileSyncService* sync_service,
+      browser_sync::ProfileSyncService* sync_service,
       invalidation::P2PInvalidationService* invalidation_service);
   ~P2PInvalidationForwarder() override;
 
@@ -34,7 +36,7 @@ class P2PInvalidationForwarder : public sync_driver::SyncServiceObserver {
   void OnSyncCycleCompleted() override;
 
  private:
-  ProfileSyncService* sync_service_;
+  browser_sync::ProfileSyncService* sync_service_;
   invalidation::P2PInvalidationService* invalidation_service_;
 
   DISALLOW_COPY_AND_ASSIGN(P2PInvalidationForwarder);