Differentiate new permissions from old in extension permission update bubble

chrome/browser/extensions/extension_disabled_ui.cc

Index: chrome/browser/extensions/extension_disabled_ui.cc
diff --git a/chrome/browser/extensions/extension_disabled_ui.cc b/chrome/browser/extensions/extension_disabled_ui.cc
index d019e8e5bacab378a611f54e2ab42f9e05fd069e..c976bcdd0242fcaf1bd8b2b11ed784d3a21f1ce0 100644
--- a/chrome/browser/extensions/extension_disabled_ui.cc
+++ b/chrome/browser/extensions/extension_disabled_ui.cc
@@ -215,8 +215,20 @@ base::string16 ExtensionDisabledGlobalError::GetBubbleViewTitle() {
 std::vector<base::string16>
 ExtensionDisabledGlobalError::GetBubbleViewMessages() {
   std::vector<base::string16> messages;
+
+  std::unique_ptr<const extensions::PermissionSet> granted =

[Devlin, 2017/04/19 19:18:33]

no need for extensions:: prefix (this whole file)

[catmullings, 2017/04/20 02:01:59]

Done.

+      service_->extension_prefs()->GetGrantedPermissions(extension_->id());
+
+  const extensions::PermissionSet& active =
+      extension_->permissions_data()->active_permissions();
+
+  std::unique_ptr<const extensions::PermissionSet> requested_permissions =
+      extensions::PermissionSet::CreateDifference(active, *granted);
+
   PermissionMessages permission_warnings =
-      extension_->permissions_data()->GetPermissionMessages();
+      extension_->permissions_data()->GetPermissionMessages(
+          *requested_permissions);
+
   if (is_remote_install_) {
     if (!permission_warnings.empty())
       messages.push_back(
@@ -224,12 +236,9 @@ ExtensionDisabledGlobalError::GetBubbleViewMessages() {
   } else {
     // TODO(treib): If NeedCustodianApprovalForPermissionIncrease, add an extra
     // message for supervised users. crbug.com/461261
-    messages.push_back(l10n_util::GetStringFUTF16(
-        extension_->is_app() ? IDS_APP_DISABLED_ERROR_LABEL
-                             : IDS_EXTENSION_DISABLED_ERROR_LABEL,
-        base::UTF8ToUTF16(extension_->name())));
     messages.push_back(l10n_util::GetStringUTF16(
-        IDS_EXTENSION_PROMPT_WILL_NOW_HAVE_ACCESS_TO));
+        extension_->is_app() ? IDS_APP_DISABLED_ERROR_LABEL
+                             : IDS_EXTENSION_DISABLED_ERROR_LABEL));
   }
   for (const PermissionMessage& msg : permission_warnings) {
     messages.push_back(l10n_util::GetStringFUTF16(IDS_EXTENSION_PERMISSION_LINE,
@@ -251,7 +260,8 @@ base::string16 ExtensionDisabledGlobalError::GetBubbleViewAcceptButtonLabel() {
             ? IDS_EXTENSION_PROMPT_REMOTE_INSTALL_BUTTON_APP
             : IDS_EXTENSION_PROMPT_REMOTE_INSTALL_BUTTON_EXTENSION);
   }
-  return l10n_util::GetStringUTF16(IDS_EXTENSION_PROMPT_RE_ENABLE_BUTTON);
+  return l10n_util::GetStringUTF16(
+      IDS_EXTENSION_PROMPT_PERMISSIONS_ACCEPT_BUTTON);
 }
 
 base::string16 ExtensionDisabledGlobalError::GetBubbleViewCancelButtonLabel() {
@@ -262,7 +272,7 @@ base::string16 ExtensionDisabledGlobalError::GetBubbleViewCancelButtonLabel() {
     // is not shown in the dialog.
     return base::string16();
   }
-  return l10n_util::GetStringUTF16(IDS_EXTENSIONS_UNINSTALL);

[Devlin, 2017/04/19 19:18:33]

I'm a little worried to re-open this to bikeshedding, but what about actually
keeping the uninstall string instead of remove?  I remember you brought up the
good point that "remove" made it unclear if it were the extension or the
permissions; "uninstall" might address that, while still being short?  Maybe
worth chatting with Shimi about quickly?

I'd be fine either way; just thought I'd mention it.

[catmullings, 2017/04/20 02:01:59]

Sure, will look into it.

+  return l10n_util::GetStringUTF16(IDS_EXTENSION_PROMPT_UNINSTALL_BUTTON);
 }
 
 void ExtensionDisabledGlobalError::OnBubbleViewDidClose(Browser* browser) {