[sync] Scope BrowserListRouterHelper to browsers with a matching profile

chrome/browser/sync/sessions/sync_sessions_web_contents_router.h

Index: chrome/browser/sync/sessions/sync_sessions_web_contents_router.h
diff --git a/chrome/browser/sync/sessions/sync_sessions_web_contents_router.h b/chrome/browser/sync/sessions/sync_sessions_web_contents_router.h
index fbb057be6b9f005e24c9dadb5441c32807318470..d2554c9ae4f37ce82a1a45d4dc04e13649a3162d 100644
--- a/chrome/browser/sync/sessions/sync_sessions_web_contents_router.h
+++ b/chrome/browser/sync/sessions/sync_sessions_web_contents_router.h
@@ -62,6 +62,7 @@ class SyncSessionsWebContentsRouter : public LocalSessionEventRouter,
   LocalSessionEventHandler* handler_ = nullptr;
 
 #if !defined(OS_ANDROID)
+  friend class BrowserListRouterHelperTest;

[skym, 2017/05/15 20:58:01]

I'd prefer to not use friending for tests, when possible. It seems like the
first class way to do this would to make a MockLocalSessionEventHandler class,
right? Then inject that with SyncSessionsWebContentsRouter::StartRoutingTo. Then
any time a tab is modified, you should get events, right?

[Patrick Noland, 2017/05/15 21:56:51]

Done.

   std::unique_ptr<BrowserListRouterHelper> browser_list_helper_;
 #endif  // !defined(OS_ANDROID)