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

chrome/browser/download/download_browsertest.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 "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/file_path.h"
 #include "base/file_util.h"
 #include "base/memory/ref_counted.h"
 #include "base/path_service.h"
@@ skipping 576 matching lines @@
   // is present in the UI (currently only on chromeos).
   void CheckDownloadUI(Browser* browser, bool expected_non_cros,
       bool expected_cros, const FilePath& filename) {
 #if defined(OS_CHROMEOS) && !defined(USE_AURA)
     Browser* popup = ActiveDownloadsUI::GetPopup();
     EXPECT_EQ(expected_cros, popup != NULL);
     if (!popup || filename.empty())
       return;
 
     ActiveDownloadsUI* downloads_ui = static_cast<ActiveDownloadsUI*>(
-        popup->GetSelectedWebContents()->GetWebUI());
+        popup->GetSelectedWebContents()->GetWebUI()->GetController());
 
     ASSERT_TRUE(downloads_ui);
     const ActiveDownloadsUI::DownloadList& downloads =
         downloads_ui->GetDownloads();
     EXPECT_EQ(downloads.size(), 1U);
 
     FilePath full_path(DestinationFile(browser, filename));
     bool exists = false;
     for (size_t i = 0; i < downloads.size(); ++i) {
       if (downloads[i]->GetFullPath() == full_path) {
@@ skipping 1122 matching lines @@
 
   // Check state.
   EXPECT_EQ(1, browser()->tab_count());
   ASSERT_TRUE(CheckDownloadFullPaths(browser(),
                                      target_file_full_path,
                                      OriginFile(file)));
 
   // Temporary downloads won't be visible.
   CheckDownloadUI(browser(), false, false, file);
 }