Migrate chrome_net_benchmarking_message_filter to mojo

chrome/renderer/net_benchmarking_extension.cc

 // Copyright (c) 2013 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/net_benchmarking_extension.h"
 
-#include "chrome/common/benchmarking_messages.h"
+#include "chrome/common/chrome_net_benchmarking.mojom.h"
 #include "content/public/renderer/render_thread.h"
+#include "services/service_manager/public/cpp/interface_provider.h"
 #include "third_party/WebKit/public/web/WebCache.h"
 #include "v8/include/v8.h"
 
 using blink::WebCache;
 
 const char kNetBenchmarkingExtensionName[] = "v8/NetBenchmarking";
 
 namespace extensions_v8 {
 
 class NetBenchmarkingWrapper : public v8::Extension {
@@ skipping 37 matching lines @@
       return v8::FunctionTemplate::New(isolate, ClearPredictorCache);
     } else if (name->Equals(
                    v8::String::NewFromUtf8(isolate, "CloseConnections"))) {
       return v8::FunctionTemplate::New(isolate, CloseConnections);
     }
 
     return v8::Local<v8::FunctionTemplate>();
   }
 
   static void ClearCache(const v8::FunctionCallbackInfo<v8::Value>& args) {
-    int rv;
-    content::RenderThread::Get()->Send(new ChromeViewHostMsg_ClearCache(&rv));
-    WebCache::clear();
+    content::RenderThread* thread = content::RenderThread::Get();
+    chrome::mojom::ChromeNetBenchmarkingPtr chrome_net_benchmarking;
+    thread->GetRemoteInterfaces()->GetInterface(&chrome_net_benchmarking);

[Sam McNally, 2016/12/05 06:57:10]

I think I'd prefer a single shared connection instead. Something like

  static chrome::mojom::NetBenchmarking& GetNetBenchmarking() {
    CR_DEFINE_STATIC_LOCAL(chrome::mojom::NetBenchmarkingPtr, net_benchmarking,
                           ConnectToBrowser());
    return *net_benchmarking;
  }

  static chrome::mojom::NetBenchmarkingPtr ConnectToBrowser() {
    chrome::mojom::NetBenchmarkingPtr net_benchmarking;
    content::RenderThread::Get()->GetRemoteInterfaces()->GetInterface(
        &net_benchmarking);
    return net_benchmarking;
  }

should do the right thing.

[dvallet, 2016/12/07 02:21:03]

Done, not sure if correctly though, PTAL

+    
+    chrome::mojom::ChromeNetBenchmarking::ClearCacheCallback callback;
+    chrome_net_benchmarking->ClearCache(callback);
   }
 
   static void ClearHostResolverCache(
       const v8::FunctionCallbackInfo<v8::Value>& args) {
-    content::RenderThread::Get()->Send(
-        new ChromeViewHostMsg_ClearHostResolverCache());
+    content::RenderThread* thread = content::RenderThread::Get();
+    chrome::mojom::ChromeNetBenchmarkingPtr chrome_net_benchmarking;
+    
+    thread->GetRemoteInterfaces()->GetInterface(&chrome_net_benchmarking);
+    chrome_net_benchmarking->ClearHostResolverCache();
   }
 
   static void ClearPredictorCache(
       const v8::FunctionCallbackInfo<v8::Value>& args) {
-    content::RenderThread::Get()->Send(
-        new ChromeViewHostMsg_ClearPredictorCache());
+    content::RenderThread* thread = content::RenderThread::Get();
+    chrome::mojom::ChromeNetBenchmarkingPtr chrome_net_benchmarking;
+    
+    thread->GetRemoteInterfaces()->GetInterface(&chrome_net_benchmarking);
+    chrome_net_benchmarking->ClearPredictorCache();
   }
 
   static void CloseConnections(
       const v8::FunctionCallbackInfo<v8::Value>& args) {
-    content::RenderThread::Get()->Send(
-        new ChromeViewHostMsg_CloseCurrentConnections());
+    content::RenderThread* thread = content::RenderThread::Get();
+    chrome::mojom::ChromeNetBenchmarkingPtr chrome_net_benchmarking;
+    
+    thread->GetRemoteInterfaces()->GetInterface(&chrome_net_benchmarking);
+    chrome_net_benchmarking->CloseCurrentConnections();
   }
 };
 
 v8::Extension* NetBenchmarkingExtension::Get() {
   return new NetBenchmarkingWrapper();
 }
 
 }  // namespace extensions_v8