Make WebUI objects not derive from WebUI. WebUI objects own the controller. This is the ownership...

chrome/browser/ui/webui/certificate_viewer_ui.cc

 // Copyright (c) 2011 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/browser/ui/webui/certificate_viewer_ui.h"
 
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/tab_contents/tab_contents_wrapper.h"
 #include "chrome/browser/ui/webui/chrome_web_ui_data_source.h"
 #include "chrome/common/url_constants.h"
+#include "content/browser/webui/web_ui.h"
 #include "content/public/browser/web_contents.h"
 #include "grit/browser_resources.h"
 #include "grit/generated_resources.h"
 
-using content::WebContents;
-
-CertificateViewerUI::CertificateViewerUI(WebContents* contents)
+CertificateViewerUI::CertificateViewerUI(WebUI* web_ui)
 #if defined(USE_AURA)
-    : ConstrainedHtmlUI(contents) {
+    : ConstrainedHtmlUI(web_ui) {
 #else
-    : HtmlDialogUI(contents) {
+    : HtmlDialogUI(web_ui) {
 #endif
 
   // Set up the chrome://view-cert source.
   ChromeWebUIDataSource* html_source =
       new ChromeWebUIDataSource(chrome::kChromeUICertificateViewerHost);
 
   // Localized strings.
   html_source->AddLocalizedString("general", IDS_CERT_INFO_GENERAL_TAB_LABEL);
   html_source->AddLocalizedString("details", IDS_CERT_INFO_DETAILS_TAB_LABEL);
   html_source->AddLocalizedString("close", IDS_CLOSE);
@@ skipping 24 matching lines @@
       IDS_CERT_DETAILS_CERTIFICATE_FIELD_VALUE_LABEL);
   html_source->set_json_path("strings.js");
 
   // Add required resources.
   html_source->add_resource_path("certificate_viewer.js",
       IDR_CERTIFICATE_VIEWER_JS);
   html_source->add_resource_path("certificate_viewer.css",
       IDR_CERTIFICATE_VIEWER_CSS);
   html_source->set_default_resource(IDR_CERTIFICATE_VIEWER_HTML);
 
-  Profile* profile = Profile::FromBrowserContext(contents->GetBrowserContext());
+  Profile* profile = Profile::FromBrowserContext(
+      web_ui->web_contents()->GetBrowserContext());
   profile->GetChromeURLDataManager()->AddDataSource(html_source);
 }
 
 CertificateViewerUI::~CertificateViewerUI() {
 }