diff options
author | droger@chromium.org <droger@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-04-01 11:44:03 +0000 |
---|---|---|
committer | droger@chromium.org <droger@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-04-01 11:44:03 +0000 |
commit | c1b4622fe8c4dcd7114d4717323a85a26869cb40 (patch) | |
tree | d81fe2221b24ce45efebade660c858d873df6f05 /chrome/browser/content_settings | |
parent | 3177c8fd4a3b00a9ee79b4c9d09d88397e0f4227 (diff) | |
download | chromium_src-c1b4622fe8c4dcd7114d4717323a85a26869cb40.zip chromium_src-c1b4622fe8c4dcd7114d4717323a85a26869cb40.tar.gz chromium_src-c1b4622fe8c4dcd7114d4717323a85a26869cb40.tar.bz2 |
Split InfoBarService core code into InfoBarManager
InfoBarService is now responsible for associating an InfoBarManager to a Tab
and handling the interactions with the rest of chrome (notifications, navigation
events).
InfoBarManager has the core logic for infobar management. It owns the list of infobars.
Eventually InfoBarManager will no longer rely on content/, but for now it still
has a reference to a WebContents instance. This reference was kept for now to
minimize the size of the change, but will be removed in a future CL.
BUG=354379
TBR=jochen
Review URL: https://codereview.chromium.org/211273007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@260847 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/content_settings')
-rw-r--r-- | chrome/browser/content_settings/permission_queue_controller.cc | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/chrome/browser/content_settings/permission_queue_controller.cc b/chrome/browser/content_settings/permission_queue_controller.cc index 3b30691..97828d9 100644 --- a/chrome/browser/content_settings/permission_queue_controller.cc +++ b/chrome/browser/content_settings/permission_queue_controller.cc @@ -9,6 +9,7 @@ #include "chrome/browser/content_settings/host_content_settings_map.h" #include "chrome/browser/geolocation/geolocation_infobar_delegate.h" #include "chrome/browser/infobars/infobar.h" +#include "chrome/browser/infobars/infobar_manager.h" #include "chrome/browser/infobars/infobar_service.h" #include "chrome/browser/media/midi_permission_infobar_delegate.h" #include "chrome/browser/profiles/profile.h" @@ -174,7 +175,7 @@ void PermissionQueueController::CancelInfoBarRequest( i != pending_infobar_requests_.end(); ++i) { if (i->id().Equals(id)) { if (i->has_infobar()) - GetInfoBarService(id)->RemoveInfoBar(i->infobar()); + GetInfoBarService(id)->infobar_manager()->RemoveInfoBar(i->infobar()); else pending_infobar_requests_.erase(i); return; @@ -208,7 +209,7 @@ void PermissionQueueController::CancelInfoBarRequests(int group_id) { for (PendingInfobarRequests::iterator i = infobar_requests_to_cancel.begin(); i != infobar_requests_to_cancel.end(); ++i) { - GetInfoBarService(i->id())->RemoveInfoBar(i->infobar()); + GetInfoBarService(i->id())->infobar_manager()->RemoveInfoBar(i->infobar()); } } @@ -254,7 +255,7 @@ void PermissionQueueController::OnPermissionSet( // Remove all infobars for the same |requesting_frame| and |embedder|. for (PendingInfobarRequests::iterator i = infobars_to_remove.begin(); i != infobars_to_remove.end(); ++i) - GetInfoBarService(i->id())->RemoveInfoBar(i->infobar()); + GetInfoBarService(i->id())->infobar_manager()->RemoveInfoBar(i->infobar()); // Send out the permission notifications. for (PendingInfobarRequests::iterator i = requests_to_notify.begin(); |