Remove dependency of chrome in WebRequestPermissions.

chrome/browser/extensions/api/chrome_extensions_api_client.cc

Index: chrome/browser/extensions/api/chrome_extensions_api_client.cc
diff --git a/chrome/browser/extensions/api/chrome_extensions_api_client.cc b/chrome/browser/extensions/api/chrome_extensions_api_client.cc
index c15c1df7aab1490e6ff4690c7f0c3369f742dc75..4325b1bdde4b093eb692bafe18afd7fa6f590497 100644
--- a/chrome/browser/extensions/api/chrome_extensions_api_client.cc
+++ b/chrome/browser/extensions/api/chrome_extensions_api_client.cc
@@ -10,6 +10,7 @@
 #include "chrome/browser/guest_view/extension_options/extension_options_guest.h"
 #include "chrome/browser/guest_view/web_view/chrome_web_view_guest_delegate.h"
 #include "chrome/browser/guest_view/web_view/chrome_web_view_permission_helper_delegate.h"
+#include "chrome/common/extensions/extension_constants.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/browser_thread.h"
 #include "device/hid/hid_service.h"
@@ -93,4 +94,17 @@ void ChromeExtensionsAPIClient::RegisterGuestViewTypes() {
   ExtensionOptionsGuest::Register();
 }
 
+bool ChromeExtensionsAPIClient::MustNotBeModifiedURL(const GURL& url) {

[Ken Rockot(use gerrit already), 2014/08/29 07:00:22]

Please wait on this. I need to dig up a patch I have in a local branch which
also refactored these extension_urls. The end result is that you won't need this
method at all on the API client, because the extension_urls functions are going
to move to //extensions.

So I would say, for this CL, just leave this code in its original place with the
dependency remaining on chrome. I will clean it up ASAP with the patch I already
have in progress.

[Xi Han, 2014/08/29 14:39:24]

Cool, I will revert these changes and wait for your cl landed:)

+   GURL::Replacements replacements;
+  replacements.ClearQuery();
+  replacements.ClearRef();
+  GURL url_without_query = url.ReplaceComponents(replacements);
+  return extension_urls::IsWebstoreUpdateUrl(url_without_query) ||
+      extension_urls::IsBlacklistUpdateUrl(url);
+}
+
+const std::string ChromeExtensionsAPIClient::WebStoreAppId() {
+  return extension_misc::kWebStoreAppId;

[Ken Rockot(use gerrit already), 2014/08/29 07:00:22]

I think it's OK to just move this constant to //extensions/common/constants.h/cc
for now. No need to add the client interface!

[Xi Han, 2014/08/29 14:39:24]

Done.

+}
+
 }  // namespace extensions