Extensions: Switch to new permission message system, part III

chrome/browser/extensions/extension_install_prompt.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/extensions/extension_install_prompt.h"
 
 #include <map>
 
 #include "base/command_line.h"
 #include "base/message_loop/message_loop.h"
@@ skipping 19 matching lines @@
 #include "extensions/browser/install/extension_install_ui.h"
 #include "extensions/common/constants.h"
 #include "extensions/common/extension.h"
 #include "extensions/common/extension_icon_set.h"
 #include "extensions/common/extension_resource.h"
 #include "extensions/common/feature_switch.h"
 #include "extensions/common/manifest.h"
 #include "extensions/common/manifest_constants.h"
 #include "extensions/common/manifest_handlers/icons_handler.h"
 #include "extensions/common/manifest_handlers/permissions_parser.h"
-#include "extensions/common/permissions/permission_message_provider.h"
 #include "extensions/common/permissions/permission_set.h"
 #include "extensions/common/permissions/permissions_data.h"
 #include "extensions/common/url_pattern.h"
 #include "grit/theme_resources.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/base/ui_base_types.h"
 #include "ui/gfx/image/image.h"
 
 using extensions::BundleInstaller;
 using extensions::Extension;
 using extensions::Manifest;
+using extensions::PermissionMessageString;
+using extensions::PermissionMessageStrings;
 using extensions::PermissionSet;
 
 namespace {
 
 bool AllowWebstoreData(ExtensionInstallPrompt::PromptType type) {
   return type == ExtensionInstallPrompt::INLINE_INSTALL_PROMPT ||
          type == ExtensionInstallPrompt::EXTERNAL_INSTALL_PROMPT ||
          type == ExtensionInstallPrompt::REPAIR_PROMPT;
 }
 
@@ skipping 179 matching lines @@
       average_rating_(0.0),
       rating_count_(0),
       show_user_count_(false),
       has_webstore_data_(false) {
 }
 
 ExtensionInstallPrompt::Prompt::~Prompt() {
 }
 
 void ExtensionInstallPrompt::Prompt::SetPermissions(
-    const std::vector<base::string16>& permissions,
-    PermissionsType permissions_type) {
-  GetPermissionsForType(permissions_type).permissions = permissions;
-}
-
-void ExtensionInstallPrompt::Prompt::SetPermissionsDetails(
-    const std::vector<base::string16>& details,
+    const PermissionMessageStrings& permissions,
     PermissionsType permissions_type) {
   InstallPromptPermissions& install_permissions =
       GetPermissionsForType(permissions_type);
 
-  // Add a dash to the front of each permission detail.
-  for (const auto& details_entry : details) {
-    if (!details_entry.empty()) {
-      std::vector<base::string16> detail_lines;
-      base::SplitString(details_entry, base::char16('\n'), &detail_lines);
+  install_permissions.permissions.clear();
+  install_permissions.details.clear();
+  install_permissions.is_showing_details.clear();
 
+  for (const PermissionMessageString& str : permissions) {
+    install_permissions.permissions.push_back(str.message);
+    // Add a dash to the front of each permission detail.
+    base::string16 details;
+    if (!str.submessages.empty()) {
       std::vector<base::string16> detail_lines_with_bullets;
-      for (const auto& detail_line : detail_lines)
+      for (const auto& detail_line : str.submessages) {
         detail_lines_with_bullets.push_back(base::ASCIIToUTF16("- ") +
                                             detail_line);
+      }
 
-      install_permissions.details.push_back(
-          JoinString(detail_lines_with_bullets, '\n'));
-    } else {
-      install_permissions.details.push_back(details_entry);
+      details = JoinString(detail_lines_with_bullets, '\n');
     }
+    install_permissions.details.push_back(details);
+    install_permissions.is_showing_details.push_back(false);
   }
-
-  install_permissions.is_showing_details.clear();
-  install_permissions.is_showing_details.insert(
-      install_permissions.is_showing_details.begin(), details.size(), false);
 }
 
 void ExtensionInstallPrompt::Prompt::SetIsShowingDetails(
     DetailsType type,
     size_t index,
     bool is_showing_details) {
   switch (type) {
     case PERMISSIONS_DETAILS:
       prompt_permissions_.is_showing_details[index] = is_showing_details;
       break;
@@ skipping 627 matching lines @@
   }
 
   if (permissions_to_display.get() &&
       (!extension_ ||
        !extensions::PermissionsData::ShouldSkipPermissionWarnings(
            extension_->id()))) {
     Manifest::Type type =
         extension_ ? extension_->GetType() : Manifest::TYPE_UNKNOWN;
     const extensions::PermissionMessageProvider* message_provider =
         extensions::PermissionMessageProvider::Get();
-    prompt_->SetPermissions(message_provider->GetLegacyWarningMessages(
+
+    prompt_->SetPermissions(message_provider->GetPermissionMessageStrings(
                                 permissions_to_display.get(), type),
                             REGULAR_PERMISSIONS);
-    prompt_->SetPermissionsDetails(
-        message_provider->GetLegacyWarningMessagesDetails(
-            permissions_to_display.get(), type),
-        REGULAR_PERMISSIONS);
 
     scoped_refptr<const extensions::PermissionSet> withheld =
         extension_->permissions_data()->withheld_permissions();
     if (!withheld->IsEmpty()) {
       prompt_->SetPermissions(
-          message_provider->GetLegacyWarningMessages(withheld.get(), type),
-          PermissionsType::WITHHELD_PERMISSIONS);
-      prompt_->SetPermissionsDetails(
-          message_provider->GetLegacyWarningMessagesDetails(withheld.get(),
-                                                            type),
+          message_provider->GetPermissionMessageStrings(withheld.get(), type),
           PermissionsType::WITHHELD_PERMISSIONS);
     }
   }
 
   switch (prompt_->type()) {
     case PERMISSIONS_PROMPT:
     case RE_ENABLE_PROMPT:
     case INLINE_INSTALL_PROMPT:
     case EXTERNAL_INSTALL_PROMPT:
     case INSTALL_PROMPT:
@@ skipping 24 matching lines @@
   if (show_params_->WasParentDestroyed()) {
     delegate_->InstallUIAbort(false);
     return;
   }
 
   if (show_dialog_callback_.is_null())
     GetDefaultShowDialogCallback().Run(show_params_.get(), delegate_, prompt_);
   else
     show_dialog_callback_.Run(show_params_.get(), delegate_, prompt_);
 }