Add a link (to view Permissions) to the Extension details on chrome://extensions.

chrome/browser/ui/webui/extensions/extension_settings_handler.cc

Index: chrome/browser/ui/webui/extensions/extension_settings_handler.cc
===================================================================
--- chrome/browser/ui/webui/extensions/extension_settings_handler.cc	(revision 179321)
+++ chrome/browser/ui/webui/extensions/extension_settings_handler.cc	(working copy)
@@ -300,6 +300,8 @@
       l10n_util::GetStringUTF16(IDS_EXTENSIONS_RELOAD_UNPACKED));
   source->AddString("extensionSettingsOptions",
       l10n_util::GetStringUTF16(IDS_EXTENSIONS_OPTIONS_LINK));
+  source->AddString("extensionSettingsPermissions",
+      l10n_util::GetStringUTF16(IDS_EXTENSIONS_PERMISSIONS_LINK));
   source->AddString("extensionSettingsActivity",
       l10n_util::GetStringUTF16(IDS_EXTENSIONS_ACTIVITY_LINK));
   source->AddString("extensionSettingsVisitWebsite",
@@ -394,6 +396,9 @@
   web_ui()->RegisterMessageCallback("extensionSettingsOptions",
       base::Bind(&ExtensionSettingsHandler::HandleOptionsMessage,
                  base::Unretained(this)));
+  web_ui()->RegisterMessageCallback("extensionSettingsPermissions",
+      base::Bind(&ExtensionSettingsHandler::HandlePermissionsMessage,
+                 base::Unretained(this)));
   web_ui()->RegisterMessageCallback("extensionSettingsShowButton",
       base::Bind(&ExtensionSettingsHandler::HandleShowButtonMessage,
                  base::Unretained(this)));
@@ -501,6 +506,13 @@
   MaybeUpdateAfterNotification();
 }
 
+void ExtensionSettingsHandler::InstallUIProceed() {
+  CHECK(false);

[James Hawkins, 2013/01/29 18:27:46]

Why is this here?

[Finnur, 2013/01/30 18:27:32]

Because this should never happen. Changed it to a NOTREACHED.

On 2013/01/29 18:27:46, James Hawkins wrote:

+}
+
+void ExtensionSettingsHandler::InstallUIAbort(bool user_initiated) {
+}
+
 void ExtensionSettingsHandler::ReloadUnpackedExtensions() {
   const ExtensionSet* extensions = extension_service_->extensions();
   std::vector<const Extension*> unpacked_extensions;
@@ -794,6 +806,18 @@
           chrome::FindBrowserWithWebContents(web_ui()->GetWebContents()));
 }
 
+void ExtensionSettingsHandler::HandlePermissionsMessage(const ListValue* args) {
+  std::string extension_id = UTF16ToUTF8(ExtractStringValue(args));
+  CHECK(!extension_id.empty());

[James Hawkins, 2013/01/29 18:27:46]

s/CHECK/DCHECK/

[Finnur, 2013/01/30 18:27:32]

No, this is consistent with how we handle untrusted input in the rest of the
file (it is littered with CHECKs).

On 2013/01/29 18:27:46, James Hawkins wrote:

+  const Extension* extension =
+      extension_service_->GetExtensionById(extension_id, true);
+  if (!extension)
+    return;
+
+  prompt_.reset(new ExtensionInstallPrompt(web_contents()));
+  prompt_->ReviewPermissions(this, extension);
+}
+
 void ExtensionSettingsHandler::HandleShowButtonMessage(const ListValue* args) {
   const Extension* extension = GetActiveExtension(args);
   if (!extension)