Move networking_private to src/extensions

extensions/shell/browser/shell_extensions_browser_client.cc

Index: extensions/shell/browser/shell_extensions_browser_client.cc
diff --git a/extensions/shell/browser/shell_extensions_browser_client.cc b/extensions/shell/browser/shell_extensions_browser_client.cc
index 013714c8e45bd9a472e739302cd57d44c549845c..8a9f7f04349337d88cf4b16922fde7db24159b5c 100644
--- a/extensions/shell/browser/shell_extensions_browser_client.cc
+++ b/extensions/shell/browser/shell_extensions_browser_client.cc
@@ -4,6 +4,7 @@
 
 #include "extensions/shell/browser/shell_extensions_browser_client.h"
 
+#include "content/public/browser/browser_context.h"
 #include "content/public/browser/browser_thread.h"
 #include "extensions/browser/api/extensions_api_client.h"
 #include "extensions/browser/api/generated_api_registration.h"
@@ -71,6 +72,11 @@ BrowserContext* ShellExtensionsBrowserClient::GetOriginalContext(
   return context;
 }
 
+std::string ShellExtensionsBrowserClient::GetUserIdHashFromContext(
+    content::BrowserContext* context) {
+  return context->GetPath().BaseName().value();

[James Cook, 2015/01/24 03:59:24]

app_shell on Brillo doesn't have a real logged-in user, so not really a
cryptohome, so this may not be very hash-y. It might just end up as "user". Is
that OK, or would you prefer returning a known fixed string?

[stevenjb, 2015/01/26 20:36:19]

Returning an empty string. If that ends up causing problems down the road we can
fix it for real at that point.

[James Cook, 2015/01/26 21:20:55]

I'm nervous that this might cause a hard-to-interpret app_shell bug sometime
down the road. I'm having a hard time figuring out what this ID is used for.

Do you know if this function is called just by starting up chrome/app_shell? If
it won't spam, maybe it should also have a NOTIMPLEMENTED()?

[stevenjb, 2015/01/27 01:39:37]

This is only called from the networkingPrivate API to ensure that
networkingPrivate is called for the correct (i.e. primary) user.

I'll change this to call chromeos::LoginState::Get()->primary_user_hash() which
is what networkingPrivate expects, with a TODO to change this if we ever support
multi profile in app_shell.

+}
+
 bool ShellExtensionsBrowserClient::IsGuestSession(
     BrowserContext* context) const {
   return false;