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

Side by Side Diff: chrome/browser/ui/omnibox/location_bar.cc

Issue 183883033: Move bookmarks_ui manifest key from chrome_settings_overrides to chrome_ui_overrides (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 9 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
« no previous file with comments | « chrome/browser/extensions/extension_keybinding_apitest.cc ('k') | chrome/chrome_common.gypi » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 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/omnibox/location_bar.h" 5 #include "chrome/browser/ui/omnibox/location_bar.h"
6 6
7 #include "chrome/browser/extensions/extension_service.h" 7 #include "chrome/browser/extensions/extension_service.h"
8 #include "chrome/browser/profiles/profile.h" 8 #include "chrome/browser/profiles/profile.h"
9 #include "chrome/common/extensions/manifest_handlers/settings_overrides_handler. h" 9 #include "chrome/common/extensions/manifest_handlers/settings_overrides_handler. h"
10 #include "extensions/browser/extension_system.h" 10 #include "chrome/common/extensions/manifest_handlers/ui_overrides_handler.h"
11 #include "extensions/browser/extension_registry.h"
11 #include "extensions/common/extension_set.h" 12 #include "extensions/common/extension_set.h"
12 #include "extensions/common/feature_switch.h" 13 #include "extensions/common/feature_switch.h"
13 #include "extensions/common/permissions/permissions_data.h" 14 #include "extensions/common/permissions/permissions_data.h"
14 15
15 16
16 LocationBar::LocationBar(Profile* profile) : profile_(profile) { 17 LocationBar::LocationBar(Profile* profile) : profile_(profile) {
17 } 18 }
18 19
19 LocationBar::~LocationBar() { 20 LocationBar::~LocationBar() {
20 } 21 }
21 22
22 bool LocationBar::IsBookmarkStarHiddenByExtension() const { 23 bool LocationBar::IsBookmarkStarHiddenByExtension() const {
23 const ExtensionService* extension_service = 24 const extensions::ExtensionSet& extension_set =
24 extensions::ExtensionSystem::Get(profile_)->extension_service(); 25 extensions::ExtensionRegistry::Get(profile_)->enabled_extensions();
25 // Extension service may be NULL during unit test execution. 26 for (extensions::ExtensionSet::const_iterator i = extension_set.begin();
26 if (!extension_service) 27 i != extension_set.end(); ++i) {
27 return false; 28 using extensions::UIOverrides;
28
29 const extensions::ExtensionSet* extension_set =
30 extension_service->extensions();
31 for (extensions::ExtensionSet::const_iterator i = extension_set->begin();
32 i != extension_set->end(); ++i) {
33 using extensions::SettingsOverrides; 29 using extensions::SettingsOverrides;
34 const SettingsOverrides* settings_overrides = 30 const UIOverrides* ui_overrides = UIOverrides::Get(i->get());
31 const SettingsOverrides* settings_overrides =
Peter Kasting 2014/03/06 19:11:26 Nit: Fix indenting
Mike Wittman 2014/03/06 19:48:24 Done.
35 SettingsOverrides::Get(i->get()); 32 SettingsOverrides::Get(i->get());
36 if (settings_overrides && 33 if (((settings_overrides &&
37 SettingsOverrides::RemovesBookmarkButton(*settings_overrides) && 34 SettingsOverrides::RemovesBookmarkButton(*settings_overrides)) ||
35 (ui_overrides &&
36 UIOverrides::RemovesBookmarkButton(*ui_overrides))) &&
38 (extensions::PermissionsData::HasAPIPermission( 37 (extensions::PermissionsData::HasAPIPermission(
39 *i, 38 *i,
40 extensions::APIPermission::kBookmarkManagerPrivate) || 39 extensions::APIPermission::kBookmarkManagerPrivate) ||
41 extensions::FeatureSwitch::enable_override_bookmarks_ui()-> 40 extensions::FeatureSwitch::enable_override_bookmarks_ui()->
42 IsEnabled())) 41 IsEnabled()))
43 return true; 42 return true;
44 } 43 }
45 44
46 return false; 45 return false;
47 } 46 }
OLDNEW
« no previous file with comments | « chrome/browser/extensions/extension_keybinding_apitest.cc ('k') | chrome/chrome_common.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698