chrome://plugins Mojo-ification part 2/2, populating the page.

chrome/browser/ui/webui/plugins_ui.h

Index: chrome/browser/ui/webui/plugins_ui.h
diff --git a/chrome/browser/ui/webui/plugins/plugins_ui.h b/chrome/browser/ui/webui/plugins_ui.h
similarity index 57%
copy from chrome/browser/ui/webui/plugins/plugins_ui.h
copy to chrome/browser/ui/webui/plugins_ui.h
index d187d6dd2de80d4998d58521040a50ace8118053..0782bebfbba6eb944a9a9aa96526d0c1457fc371 100644
--- a/chrome/browser/ui/webui/plugins/plugins_ui.h
+++ b/chrome/browser/ui/webui/plugins_ui.h
@@ -2,11 +2,12 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef CHROME_BROWSER_UI_WEBUI_PLUGINS_PLUGINS_UI_H_
-#define CHROME_BROWSER_UI_WEBUI_PLUGINS_PLUGINS_UI_H_
+#ifndef CHROME_BROWSER_UI_WEBUI_PLUGINS_UI_H_
+#define CHROME_BROWSER_UI_WEBUI_PLUGINS_UI_H_
 
 #include "base/macros.h"
-#include "content/public/browser/web_ui_controller.h"
+#include "chrome/browser/ui/webui/mojo_web_ui_controller.h"
+#include "chrome/browser/ui/webui/plugins.mojom.h"
 #include "ui/base/layout.h"
 
 namespace base {
@@ -17,7 +18,7 @@ namespace user_prefs {
 class PrefRegistrySyncable;
 }
 
-class PluginsUI : public content::WebUIController {
+class PluginsUI : public MojoWebUIController<PluginsHandlerMojo> {

[sky, 2016/01/28 16:45:27]

I assume this is from earlier work and you don't intend to actually commit this,
right?

[dpapad, 2016/01/28 18:32:54]

I was following the example from site_engagement and omnibox, see
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/ui/...
and
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/ui/....
Isn't it necessary?

[Dan Beam, 2016/01/28 19:20:34]

those actually have .cc files to implement them

this just looks like a botched file move (note the path doesn't have "/plugins/"
in it like "chrome/browser/ui/webui/plugins/plugins_ui.h" does)

[dpapad, 2016/01/28 20:24:01]

Ah, I see now. Yes, this was a stale file in this CL from some previous
intermediate step. Removed.

  public:
   explicit PluginsUI(content::WebUI* web_ui);
 
@@ -26,7 +27,11 @@ class PluginsUI : public content::WebUIController {
   static void RegisterProfilePrefs(user_prefs::PrefRegistrySyncable* registry);
 
  private:
+  // MojoWebUIController overrides:
+  void BindUIHandler(
+      mojo::InterfaceRequest<PluginsHandlerMojo> request) override;
+
   DISALLOW_COPY_AND_ASSIGN(PluginsUI);
 };
 
-#endif  // CHROME_BROWSER_UI_WEBUI_PLUGINS_PLUGINS_UI_H_
+#endif  // CHROME_BROWSER_UI_WEBUI_PLUGINS_UI_H_