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

Side by Side Diff: chrome/browser/ui/views/browser_actions_container.cc

Issue 25366003: Moved some functions off ExtensionService into a new file extension_util. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Windows compile Created 7 years, 2 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/browser_actions_container.h" 5 #include "chrome/browser/ui/views/browser_actions_container.h"
6 6
7 #include "base/compiler_specific.h" 7 #include "base/compiler_specific.h"
8 #include "base/prefs/pref_service.h" 8 #include "base/prefs/pref_service.h"
9 #include "base/stl_util.h" 9 #include "base/stl_util.h"
10 #include "chrome/browser/extensions/extension_service.h" 10 #include "chrome/browser/extensions/extension_service.h"
11 #include "chrome/browser/extensions/extension_system.h" 11 #include "chrome/browser/extensions/extension_system.h"
12 #include "chrome/browser/extensions/extension_util.h"
12 #include "chrome/browser/extensions/tab_helper.h" 13 #include "chrome/browser/extensions/tab_helper.h"
13 #include "chrome/browser/profiles/profile.h" 14 #include "chrome/browser/profiles/profile.h"
14 #include "chrome/browser/sessions/session_tab_helper.h" 15 #include "chrome/browser/sessions/session_tab_helper.h"
15 #include "chrome/browser/ui/browser.h" 16 #include "chrome/browser/ui/browser.h"
16 #include "chrome/browser/ui/tabs/tab_strip_model.h" 17 #include "chrome/browser/ui/tabs/tab_strip_model.h"
17 #include "chrome/browser/ui/view_ids.h" 18 #include "chrome/browser/ui/view_ids.h"
18 #include "chrome/browser/ui/views/browser_action_view.h" 19 #include "chrome/browser/ui/views/browser_action_view.h"
19 #include "chrome/browser/ui/views/extensions/browser_action_drag_data.h" 20 #include "chrome/browser/ui/views/extensions/browser_action_drag_data.h"
20 #include "chrome/browser/ui/views/extensions/extension_keybinding_registry_views .h" 21 #include "chrome/browser/ui/views/extensions/extension_keybinding_registry_views .h"
21 #include "chrome/browser/ui/views/extensions/extension_popup.h" 22 #include "chrome/browser/ui/views/extensions/extension_popup.h"
(...skipping 779 matching lines...) Expand 10 before | Expand all | Expand 10 after
801 animation_target_size_ = target_size; 802 animation_target_size_ = target_size;
802 AnimationEnded(resize_animation_.get()); 803 AnimationEnded(resize_animation_.get());
803 } 804 }
804 } 805 }
805 806
806 bool BrowserActionsContainer::ShouldDisplayBrowserAction( 807 bool BrowserActionsContainer::ShouldDisplayBrowserAction(
807 const Extension* extension) { 808 const Extension* extension) {
808 // Only display incognito-enabled extensions while in incognito mode. 809 // Only display incognito-enabled extensions while in incognito mode.
809 return 810 return
810 (!profile_->IsOffTheRecord() || 811 (!profile_->IsOffTheRecord() ||
811 extensions::ExtensionSystem::Get(profile_)->extension_service()-> 812 extension_util::IsIncognitoEnabled(
812 IsIncognitoEnabled(extension->id())); 813 extension->id(),
814 extensions::ExtensionSystem::Get(profile_)->extension_service()));
813 } 815 }
814 816
815 void BrowserActionsContainer::ShowPopup( 817 void BrowserActionsContainer::ShowPopup(
816 BrowserActionButton* button, 818 BrowserActionButton* button,
817 ExtensionPopup::ShowAction show_action) { 819 ExtensionPopup::ShowAction show_action) {
818 const Extension* extension = button->extension(); 820 const Extension* extension = button->extension();
819 GURL popup_url; 821 GURL popup_url;
820 if (model_->ExecuteBrowserAction(extension, browser_, &popup_url) != 822 if (model_->ExecuteBrowserAction(extension, browser_, &popup_url) !=
821 ExtensionToolbarModel::ACTION_SHOW_POPUP) { 823 ExtensionToolbarModel::ACTION_SHOW_POPUP) {
822 return; 824 return;
(...skipping 17 matching lines...) Expand all
840 views::BubbleBorder::TOP_LEFT : views::BubbleBorder::TOP_RIGHT; 842 views::BubbleBorder::TOP_LEFT : views::BubbleBorder::TOP_RIGHT;
841 popup_ = ExtensionPopup::ShowPopup(popup_url, 843 popup_ = ExtensionPopup::ShowPopup(popup_url,
842 browser_, 844 browser_,
843 reference_view, 845 reference_view,
844 arrow, 846 arrow,
845 show_action); 847 show_action);
846 popup_->GetWidget()->AddObserver(this); 848 popup_->GetWidget()->AddObserver(this);
847 popup_button_ = button; 849 popup_button_ = button;
848 popup_button_->SetButtonPushed(); 850 popup_button_->SetButtonPushed();
849 } 851 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698