[Sync] Fix namespaces for the browser_sync component.

chrome/browser/ui/sync/one_click_signin_sync_starter.cc

Index: chrome/browser/ui/sync/one_click_signin_sync_starter.cc
diff --git a/chrome/browser/ui/sync/one_click_signin_sync_starter.cc b/chrome/browser/ui/sync/one_click_signin_sync_starter.cc
index 861f1b95cbee5e1769429f4c6520ac1a89f32a7d..e50f234ce1755e9f62d7ffaaca72276f995e0d06 100644
--- a/chrome/browser/ui/sync/one_click_signin_sync_starter.cc
+++ b/chrome/browser/ui/sync/one_click_signin_sync_starter.cc
@@ -44,6 +44,8 @@
 #include "net/url_request/url_request_context_getter.h"
 #include "ui/base/l10n/l10n_util.h"
 
+using browser_sync::ProfileSyncService;

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

I can't seem to figure out why you sometimes go with using, and sometimes not. I
suppose it doesn't really matter.

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

General rule of thumb was if the file had > ~5 uses, or used subclasses like
PSS::SyncStatusSummary, ::InitParams, etc. I don't think I was super consistent
though.

+
 namespace {
 
 // UMA histogram for tracking what users do when presented with the signin