Get rid of some more dependencies on chrome\common from plugin, in preparatio...

chrome/plugin/webplugin_proxy.cc

Index: chrome/plugin/webplugin_proxy.cc
===================================================================
--- chrome/plugin/webplugin_proxy.cc	(revision 78225)
+++ chrome/plugin/webplugin_proxy.cc	(working copy)
@@ -10,12 +10,11 @@
 #include "base/scoped_handle.h"
 #include "base/shared_memory.h"
 #include "build/build_config.h"
-#include "chrome/common/child_process_logging.h"
-#include "chrome/common/url_constants.h"
 #include "chrome/plugin/npobject_proxy.h"
 #include "chrome/plugin/npobject_util.h"
 #include "chrome/plugin/plugin_channel.h"
 #include "chrome/plugin/plugin_thread.h"
+#include "content/common/content_client.h"
 #include "content/common/plugin_messages.h"
 #include "skia/ext/platform_device.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebBindings.h"
@@ -290,9 +289,9 @@
         webkit::npapi::WebPluginDelegateImpl::
             PLUGIN_QUIRK_BLOCK_NONSTANDARD_GETURL_REQUESTS) {
       GURL request_url(url);
-      if (!request_url.SchemeIs(chrome::kHttpScheme) &&
-          !request_url.SchemeIs(chrome::kHttpsScheme) &&
-          !request_url.SchemeIs(chrome::kFtpScheme)) {
+      if (!request_url.SchemeIs("http") &&
+          !request_url.SchemeIs("https") &&
+          !request_url.SchemeIs("ftp")) {

[Avi (use Gerrit), 2011/03/15 19:50:28]

:(

Can we please move the constants too?

[jam, 2011/03/15 20:30:06]

I searched the code to see what modules below chrome use, and all (i.e. even
net) hardcoded them.

         return;
       }
     }
@@ -628,7 +627,7 @@
 #endif
 
 void WebPluginProxy::OnPaint(const gfx::Rect& damaged_rect) {
-  child_process_logging::SetActiveURL(page_url_);
+  content::GetContentClient()->SetActiveURL(page_url_);
 
   Paint(damaged_rect);
   Send(new PluginHostMsg_InvalidateRect(route_id_, damaged_rect));