Infobar system refactor.

chrome/browser/content_settings/permission_queue_controller_unittest.cc

 // Copyright 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/browser/content_settings/permission_queue_controller.h"
 
 #include "base/synchronization/waitable_event.h"
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/content_settings/permission_request_id.h"
-#include "chrome/browser/infobars/infobar.h"
 #include "chrome/browser/infobars/infobar_service.h"
 #include "chrome/common/content_settings_types.h"
 #include "chrome/test/base/chrome_render_view_host_test_harness.h"
 #include "chrome/test/base/testing_profile.h"
-#include "content/public/browser/notification_details.h"
-#include "content/public/browser/notification_service.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/test/mock_render_process_host.h"
 #include "content/public/test/test_browser_thread.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 
 // PermissionQueueControllerTests ---------------------------------------------
 
-class PermissionQueueControllerTests : public ChromeRenderViewHostTestHarness,
-                                       public content::NotificationObserver {
+class PermissionQueueControllerTests : public ChromeRenderViewHostTestHarness {
  protected:
-  PermissionQueueControllerTests() {
-    registrar_.Add(this,
-                   chrome::NOTIFICATION_TAB_CONTENTS_INFOBAR_REMOVED,
-                   content::NotificationService::AllSources());
-  }
+  PermissionQueueControllerTests() {}
   virtual ~PermissionQueueControllerTests() {}
 
   PermissionRequestID RequestID(int bridge_id) {
     return PermissionRequestID(
         web_contents()->GetRenderProcessHost()->GetID(),
         web_contents()->GetRenderViewHost()->GetRoutingID(),
         bridge_id);
   }
 
  private:
   // ChromeRenderViewHostTestHarness:
   virtual void SetUp() OVERRIDE {
     ChromeRenderViewHostTestHarness::SetUp();
     InfoBarService::CreateForWebContents(web_contents());
   }
 
-  // content::NotificationObserver:
-  virtual void Observe(int type,
-                       const content::NotificationSource& source,
-                       const content::NotificationDetails& details) OVERRIDE {
-    DCHECK_EQ(chrome::NOTIFICATION_TAB_CONTENTS_INFOBAR_REMOVED, type);
-    // Delete the removed infobar. In normal Chrome code, this would be handled
-    // by the InfoBarContainer. It's safe to do this even if the queue
-    // controller Observe() function has not yet been called; see comments in
-    // PermissionQueueController::Observe().
-    // TODO(pkasting): This will no longer be necessary once the InfoBarService
-    // truly owns infobars.
-    delete content::Details<InfoBar::RemovedDetails>(details)->first;
-  }
-
-  content::NotificationRegistrar registrar_;
-
   DISALLOW_COPY_AND_ASSIGN(PermissionQueueControllerTests);
 };
 
 
 // ObservationCountingQueueController -----------------------------------------
 
 class ObservationCountingQueueController : public PermissionQueueController {
  public:
   explicit ObservationCountingQueueController(Profile* profile);
   virtual ~ObservationCountingQueueController();
@@ skipping 43 matching lines @@
   // test checks that in the case where the controller should remain registered
   // for notifications, it gets notified exactly once."
   ObservationCountingQueueController queue_controller(profile());
   GURL url("http://www.example.com/geolocation");
   base::Callback<void(bool)> callback;
   queue_controller.CreateInfoBarRequest(RequestID(0), url, url, callback);
   queue_controller.CreateInfoBarRequest(RequestID(1), url, url, callback);
   queue_controller.CancelInfoBarRequest(RequestID(0));
   EXPECT_EQ(1, queue_controller.call_count());
 };