Move networking_private to src/extensions

chrome/browser/extensions/chrome_extensions_browser_client.cc

 // Copyright 2013 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.
 
 #include "chrome/browser/extensions/chrome_extensions_browser_client.h"
 
 #include "base/command_line.h"
 #include "base/version.h"
 #include "chrome/browser/app_mode/app_mode_utils.h"
 #include "chrome/browser/browser_process.h"
@@ skipping 21 matching lines @@
 #include "chrome/common/chrome_version_info.h"
 #include "chrome/common/extensions/features/feature_channel.h"
 #include "chrome/common/pref_names.h"
 #include "extensions/browser/api/generated_api_registration.h"
 #include "extensions/browser/extension_function_registry.h"
 #include "extensions/browser/extension_prefs.h"
 #include "extensions/browser/pref_names.h"
 #include "extensions/browser/url_request_util.h"
 
 #if defined(OS_CHROMEOS)
+#include "chrome/browser/chromeos/profiles/profile_helper.h"
 #include "chrome/browser/extensions/updater/extension_cache_impl.h"
 #include "chromeos/chromeos_switches.h"
 #else
 #include "extensions/browser/updater/null_extension_cache.h"
 #endif
 
 namespace extensions {
 
 ChromeExtensionsBrowserClient::ChromeExtensionsBrowserClient() {
   process_manager_delegate_.reset(new ChromeProcessManagerDelegate);
@@ skipping 40 matching lines @@
 content::BrowserContext* ChromeExtensionsBrowserClient::GetOffTheRecordContext(
     content::BrowserContext* context) {
   return static_cast<Profile*>(context)->GetOffTheRecordProfile();
 }
 
 content::BrowserContext* ChromeExtensionsBrowserClient::GetOriginalContext(
     content::BrowserContext* context) {
   return static_cast<Profile*>(context)->GetOriginalProfile();
 }
 
+std::string ChromeExtensionsBrowserClient::GetUserIdHashFromContext(
+    content::BrowserContext* context) {
+#if defined(OS_CHROMEOS)
+  return chromeos::ProfileHelper::GetUserIdHashFromProfile(
+      static_cast<Profile*>(context));
+#else
+  return context->GetPath().BaseName().value();

[Daniel Erat, 2015/01/23 23:24:29]

is this approach already used for non-chrome-os elsewhere?

[stevenjb, 2015/01/23 23:57:57]

Currently "GetUserIdHash" is a chromeos specific concept. We could wrap the
entire method inside #if defined(OS_CHROMEOS).

However, if you look at ProfileHelper::GetUserIdHashFromProfile(), this is the
"legacy" behavior, and seems sufficient for hypothetical non-chromeos usage.

That said, if making this call chromeos specific seems better, I am OK with
that.

[Daniel Erat, 2015/01/24 02:14:03]

i don't have strong opinions about this / don't know if there will be issues due
to the non-chrome-os implementation. i'm fine with this if james is. :-)

+#endif
+}
+
 bool ChromeExtensionsBrowserClient::IsGuestSession(
     content::BrowserContext* context) const {
   return static_cast<Profile*>(context)->IsGuestSession();
 }
 
 bool ChromeExtensionsBrowserClient::IsExtensionIncognitoEnabled(
     const std::string& extension_id,
     content::BrowserContext* context) const {
   return IsGuestSession(context)
       || util::IsIncognitoEnabled(extension_id, context);
@@ skipping 179 matching lines @@
   base::Version browser_version = base::Version(version_info.Version());
   Version browser_min_version(min_version);
   if (browser_version.IsValid() && browser_min_version.IsValid() &&
       browser_min_version.CompareTo(browser_version) > 0) {
     return false;
   }
   return true;
 }
 
 }  // namespace extensions