Migrate components/data_reduction_proxy to Mojo interfaces.

chrome/renderer/page_load_histograms.cc

 // Copyright (c) 2012 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/renderer/page_load_histograms.h"
 
 #include <stddef.h>
 
 #include <memory>
 #include <string>
 
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/metrics/field_trial.h"
 #include "base/metrics/histogram.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_split.h"
 #include "base/strings/string_util.h"
 #include "base/strings/stringprintf.h"
 #include "base/strings/utf_string_conversions.h"
 #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"
 #include "content/public/renderer/render_frame.h"
 #include "content/public/renderer/render_thread.h"
 #include "content/public/renderer/render_view.h"
 #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"
 #include "third_party/WebKit/public/web/WebDocument.h"
 #include "third_party/WebKit/public/web/WebFrame.h"
 #include "third_party/WebKit/public/web/WebPerformance.h"
 #include "third_party/WebKit/public/web/WebView.h"
 #include "url/gurl.h"
 
 #if defined(ENABLE_EXTENSIONS)
@@ skipping 836 matching lines @@
   // for pages visited due to an explicit user navigation.
   if (SearchBouncer::GetInstance()->IsNewTabPage(frame->document().url())) {
     return;
   }
 
   DocumentState* document_state =
       DocumentState::FromDataSource(frame->dataSource());
 
   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/07/08 05:50:53]

How expensive is it to create / tear down a message pipe everytime we query this
state? It definitely seems less efficient than legacy IPC, since we'd just route
one message there.

[leonhsl(Using Gerrit), 2016/07/11 14:36:07]

From my understanding this call will just happen once for each main render
frame, and as I remember that a mojo message pipe should be cheap to create and
destroy, so I made this choice. I referenced some other mojofication works which
are also using mojo message pipes as if it's free ;-)

+        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);
   }
 
   bool came_from_websearch = IsFromGoogleSearchResult(
       frame->document().url(),
       blink::WebStringToGURL(frame->document().referrer()));
   int websearch_chrome_joint_experiment_id = kNoExperiment;
   bool is_preview = false;
@@ skipping 61 matching lines @@
   Time start = document_state->start_load_time();
   Time finish = document_state->finish_load_time();
   // TODO(mbelshe): should we log more stats?
   VLOG(1) << "PLT: " << (finish - start).InMilliseconds() << "ms "
           << url.spec();
 }
 
 void PageLoadHistograms::OnDestruct() {
   delete this;
 }