Migrate components/data_reduction_proxy to Mojo interfaces.

chrome/common/chrome_content_client.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/common/chrome_content_client.h"
 
 #include <stdint.h>
 
 #include <memory>
 #include <tuple>
 
 #include "base/command_line.h"
 #include "base/debug/crash_logging.h"
 #include "base/files/file_util.h"
 #include "base/json/json_reader.h"
+#include "base/memory/ptr_util.h"

[Anand Mistry (off Chromium), 2016/06/24 07:12:28]

Is this needed? I'm guessing the messages file brought this in transitively and
now you need it. Is that right?

[leonhsl(Using Gerrit), 2016/06/24 09:59:21]

Actually it's not my codes need it ;-)
https://codereview.chromium.org/2049783002 started to use base::WrapUnique but
did not include this header file explicitly, it implicitly uses via the message
file I delete here in this CL. To let all trybots happy I have to add this line
here. I did a rebase on ps#12, which will show the usage of base::WrapUnique
clearly.

Although, I've created a separate CL https://codereview.chromium.org/2095473003/
to add this explicitly. 

 #include "base/memory/scoped_vector.h"
 #include "base/native_library.h"
 #include "base/path_service.h"
 #include "base/strings/string16.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 "build/build_config.h"
 #include "chrome/common/child_process_logging.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/crash_keys.h"
 #include "chrome/common/pepper_flash.h"
 #include "chrome/common/secure_origin_whitelist.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/grit/common_resources.h"
-#include "components/data_reduction_proxy/content/common/data_reduction_proxy_messages.h"
 #include "components/dom_distiller/core/url_constants.h"
 #include "components/version_info/version_info.h"
 #include "content/public/common/cdm_info.h"
 #include "content/public/common/content_constants.h"
 #include "content/public/common/content_switches.h"
 #include "content/public/common/url_constants.h"
 #include "content/public/common/user_agent.h"
 #include "extensions/common/constants.h"
 #include "gpu/config/gpu_info.h"
 #include "net/http/http_util.h"
@@ skipping 549 matching lines @@
   referrer_schemes->push_back(
       {chrome::kAndroidAppScheme, url::SCHEME_WITHOUT_PORT});
 #endif
 
   savable_schemes->push_back(extensions::kExtensionScheme);
   savable_schemes->push_back(extensions::kExtensionResourceScheme);
   savable_schemes->push_back(chrome::kChromeSearchScheme);
   savable_schemes->push_back(dom_distiller::kDomDistillerScheme);
 }
 
-bool ChromeContentClient::CanSendWhileSwappedOut(const IPC::Message* message) {

[Anand Mistry (off Chromium), 2016/06/24 07:12:28]

I honestly don't understand what's supposed to happen with swapped-out frames.
Someone that understand the effect of this should really take a look.

[leonhsl(Using Gerrit), 2016/06/24 09:59:21]

From my understanding, under swapped-out status RenderWidget::Send() can send
only some explicitly announced IPCs, here is just the annunciation for
DataReductionProxyViewHostMsg_IsDataReductionProxy. After switched to Mojo, the
restriction itself does not exist anymore, so I think we need to do nothing for
this.
Anyway, I'll try to find someone familiar with this to take a look.

[leonhsl(Using Gerrit), 2016/06/24 15:33:47]

Hi, tbansal@, megjablon@, I found that you have touched these codes before,
would you please help to take a look? Thanks.

[tbansal1, 2016/06/24 15:57:08]

leonhsi@, you are right. By default the IPCs are blocked for swapped out frames.
Here, we are trying to record page load metrics (e.g., page load time) if the
page used the data reduction proxy.

So, the renderer queries the browser if the page used the data reduction proxy.
This code punches a hole in the IPC mechanism to allow this message to go
through even after the swap out.

I am not familiar with Mojo, but if the IPC restriction (messages are blocked
after renderer is swapped out) is no longer there, then this code can be
removed.

-  return message->type() ==
-         DataReductionProxyViewHostMsg_IsDataReductionProxy::ID;
-}
-
 std::string ChromeContentClient::GetProduct() const {
   return ::GetProduct();
 }
 
 std::string ChromeContentClient::GetUserAgent() const {
   return ::GetUserAgent();
 }
 
 base::string16 ChromeContentClient::GetLocalizedString(int message_id) const {
   return l10n_util::GetStringUTF16(message_id);
@@ skipping 82 matching lines @@
 
 base::StringPiece ChromeContentClient::GetOriginTrialPublicKey() {
   return origin_trial_key_manager_.GetPublicKey();
 }
 
 #if defined(OS_ANDROID)
 media::MediaClientAndroid* ChromeContentClient::GetMediaClientAndroid() {
   return new ChromeMediaClientAndroid();
 }
 #endif  // OS_ANDROID