Migrate components/data_reduction_proxy to Mojo interfaces.

chrome/renderer/page_load_histograms.cc

Index: chrome/renderer/page_load_histograms.cc
diff --git a/chrome/renderer/page_load_histograms.cc b/chrome/renderer/page_load_histograms.cc
index df435a6e45aa053366f392323963fe1c90988281..63d9cbc977216b7726a5a4b83779eec6d5049891 100644
--- a/chrome/renderer/page_load_histograms.cc
+++ b/chrome/renderer/page_load_histograms.cc
@@ -22,7 +22,7 @@
 #include "base/time/time.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/renderer/searchbox/search_bouncer.h"
-#include "components/data_reduction_proxy/content/common/data_reduction_proxy_messages.h"
+#include "components/data_reduction_proxy/content/common/data_reduction_proxy.mojom.h"
 #include "components/data_reduction_proxy/core/common/data_reduction_proxy_params.h"
 #include "content/public/common/content_constants.h"
 #include "content/public/renderer/document_state.h"
@@ -32,6 +32,7 @@
 #include "extensions/common/url_pattern.h"
 #include "net/base/url_util.h"
 #include "net/http/http_response_headers.h"
+#include "services/shell/public/cpp/interface_provider.h"
 #include "third_party/WebKit/public/platform/URLConversion.h"
 #include "third_party/WebKit/public/platform/WebURLRequest.h"
 #include "third_party/WebKit/public/platform/WebURLResponse.h"
@@ -836,9 +837,12 @@ void PageLoadHistograms::Dump(WebFrame* frame) {
 
   bool data_reduction_proxy_was_used = false;
   if (!document_state->proxy_server().IsEmpty()) {
-    bool handled =
-        Send(new DataReductionProxyViewHostMsg_IsDataReductionProxy(
-            document_state->proxy_server(), &data_reduction_proxy_was_used));
+    data_reduction_proxy::mojom::DataReductionProxyHostPtr
+        data_reduction_proxy_host;
+    content::RenderThread::Get()->GetRemoteInterfaces()->GetInterface(
+        mojo::GetProxy(&data_reduction_proxy_host));
+    bool handled = data_reduction_proxy_host->IsDataReductionProxy(

[dcheng, 2016/08/26 19:16:20]

What happens to this request if the other side is never bound? Wouldn't it hang?

[leonhsl(Using Gerrit), 2016/08/29 06:12:06]

Normally the other side will either bind or reject(close) the request. Closing
request means breaking the message pipe connection, which will wake up the
waiting sync call to return with no any value set to
|ata_reduction_proxy_was_used|.
Will hang here ONLY IF the other side holds the request deliberately but does
not bind it, which would never happen for this CL.

+        document_state->proxy_server(), &data_reduction_proxy_was_used);
     // If the IPC call is not handled, then |data_reduction_proxy_was_used|
     // should remain |false|.
     DCHECK(handled || !data_reduction_proxy_was_used);