Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(43)

Side by Side Diff: chrome/browser/ui/views/extensions/extension_action_platform_delegate_views.cc

Issue 1169823005: Add sidebar views implementation using ExtensionSystem SidebarManager Base URL: https://chromium.googlesource.com/chromium/src.git@patch-a
Patch Set: Rebased patch set to 1168383002/#ps220001 Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/views/extensions/extension_action_platform_delegate_ views.h" 5 #include "chrome/browser/ui/views/extensions/extension_action_platform_delegate_ views.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "chrome/browser/extensions/extension_action.h" 8 #include "chrome/browser/extensions/extension_action.h"
9 #include "chrome/browser/extensions/extension_view_host.h" 9 #include "chrome/browser/extensions/extension_view_host.h"
10 #include "chrome/browser/profiles/profile.h" 10 #include "chrome/browser/profiles/profile.h"
(...skipping 76 matching lines...) Expand 10 before | Expand all | Expand 10 after
87 popup_show_action); 87 popup_show_action);
88 } 88 }
89 89
90 void ExtensionActionPlatformDelegateViews::CloseOverflowMenu() { 90 void ExtensionActionPlatformDelegateViews::CloseOverflowMenu() {
91 ToolbarView* toolbar = 91 ToolbarView* toolbar =
92 BrowserView::GetBrowserViewForBrowser(controller_->browser())->toolbar(); 92 BrowserView::GetBrowserViewForBrowser(controller_->browser())->toolbar();
93 if (toolbar->IsWrenchMenuShowing()) 93 if (toolbar->IsWrenchMenuShowing())
94 toolbar->CloseAppMenu(); 94 toolbar->CloseAppMenu();
95 } 95 }
96 96
97 void ExtensionActionPlatformDelegateViews::ShowSidebar(
98 content::WebContents* sidebar_contents) {
99 BrowserView::GetBrowserViewForBrowser(controller_->browser())
100 ->ShowSidebar(sidebar_contents);
101 }
102
103 void ExtensionActionPlatformDelegateViews::HideSidebar() {
104 BrowserView::GetBrowserViewForBrowser(controller_->browser())->HideSidebar();
105 }
106
97 void ExtensionActionPlatformDelegateViews::Observe( 107 void ExtensionActionPlatformDelegateViews::Observe(
98 int type, 108 int type,
99 const content::NotificationSource& source, 109 const content::NotificationSource& source,
100 const content::NotificationDetails& details) { 110 const content::NotificationDetails& details) {
101 DCHECK(type == extensions::NOTIFICATION_EXTENSION_COMMAND_ADDED || 111 DCHECK(type == extensions::NOTIFICATION_EXTENSION_COMMAND_ADDED ||
102 type == extensions::NOTIFICATION_EXTENSION_COMMAND_REMOVED); 112 type == extensions::NOTIFICATION_EXTENSION_COMMAND_REMOVED);
103 extensions::ExtensionCommandRemovedDetails* payload = 113 extensions::ExtensionCommandRemovedDetails* payload =
104 content::Details<extensions::ExtensionCommandRemovedDetails>(details) 114 content::Details<extensions::ExtensionCommandRemovedDetails>(details)
105 .ptr(); 115 .ptr();
106 if (controller_->extension()->id() == payload->extension_id && 116 if (controller_->extension()->id() == payload->extension_id &&
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
159 focus_manager->UnregisterAccelerator(*action_keybinding_, this); 169 focus_manager->UnregisterAccelerator(*action_keybinding_, this);
160 action_keybinding_.reset(); 170 action_keybinding_.reset();
161 } 171 }
162 } 172 }
163 173
164 ToolbarActionViewDelegateViews* 174 ToolbarActionViewDelegateViews*
165 ExtensionActionPlatformDelegateViews::GetDelegateViews() const { 175 ExtensionActionPlatformDelegateViews::GetDelegateViews() const {
166 return static_cast<ToolbarActionViewDelegateViews*>( 176 return static_cast<ToolbarActionViewDelegateViews*>(
167 controller_->view_delegate()); 177 controller_->view_delegate());
168 } 178 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698