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

Unified Diff: chrome/browser/extensions/api/developer_private/extension_info_generator.cc

Issue 2898203004: [Extensions] Remove "show button" UI from chrome://extensions (Closed)
Patch Set: Created 3 years, 7 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/api/developer_private/extension_info_generator.cc
diff --git a/chrome/browser/extensions/api/developer_private/extension_info_generator.cc b/chrome/browser/extensions/api/developer_private/extension_info_generator.cc
index a9f4ffc6de9746b9b0b4946a5c2e2117d743acef..e0dcc70f24d6b296d5302dc71e297578e49e0dea 100644
--- a/chrome/browser/extensions/api/developer_private/extension_info_generator.cc
+++ b/chrome/browser/extensions/api/developer_private/extension_info_generator.cc
@@ -311,12 +311,6 @@ void ExtensionInfoGenerator::CreateExtensionInfoHelper(
std::unique_ptr<developer::ExtensionInfo> info(
new developer::ExtensionInfo());
- // Don't consider the button hidden with the redesign, because "hidden"
- // buttons are now just hidden in the wrench menu.
- info->action_button_hidden =
- !extension_action_api_->GetBrowserActionVisibility(extension.id()) &&
karandeepb 2017/05/24 21:02:22 I think the Get/SetBrowserActionVisibility and ass
Devlin 2017/05/26 21:06:32 Correct. :)
- !FeatureSwitch::extension_action_redesign()->IsEnabled();
-
// Blacklist text.
int blacklist_text = -1;
switch (extension_prefs_->GetExtensionBlacklistState(extension.id())) {

Powered by Google App Engine
This is Rietveld 408576698