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

chrome/browser/ui/webui/sync_internals_ui.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_WEBUI_SYNC_INTERNALS_UI_H_
 #define CHROME_BROWSER_UI_WEBUI_SYNC_INTERNALS_UI_H_
 #pragma once
 
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/memory/weak_ptr.h"
 #include "chrome/browser/sync/js/js_event_handler.h"
 #include "chrome/browser/sync/js/js_reply_handler.h"
-#include "content/browser/webui/web_ui.h"
 #include "content/public/browser/web_ui_controller.h"
 
 namespace browser_sync {
 class JsController;
 }  // namespace browser_sync
 
 // The implementation for the chrome://sync-internals page.
-class SyncInternalsUI : public WebUI, public content::WebUIController,
+class SyncInternalsUI : public content::WebUIController,
                         public browser_sync::JsEventHandler,
                         public browser_sync::JsReplyHandler {
  public:
-  explicit SyncInternalsUI(content::WebContents* contents);
+  explicit SyncInternalsUI(WebUI* web_ui);
   virtual ~SyncInternalsUI();
 
   // WebUIController implementation.
   //
   // The following messages are processed:
   //
   // getAboutInfo():
   //   Immediately fires a onGetAboutInfoFinished() event with a
   //   dictionary of sync-related stats and info.
   //
@@ skipping 17 matching lines @@
       const browser_sync::JsArgList& args) OVERRIDE;
 
  private:
   base::WeakPtrFactory<SyncInternalsUI> weak_ptr_factory_;
   base::WeakPtr<browser_sync::JsController> js_controller_;
 
   DISALLOW_COPY_AND_ASSIGN(SyncInternalsUI);
 };
 
 #endif  // CHROME_BROWSER_UI_WEBUI_SYNC_INTERNALS_UI_H_