Improve error messaging of webRequest API in case of conflicts

chrome/browser/extensions/extension_warning_set.cc

Index: chrome/browser/extensions/extension_warning_set.cc
diff --git a/chrome/browser/extensions/extension_warning_set.cc b/chrome/browser/extensions/extension_warning_set.cc
index 2d73cd684c4b3ea58c9a276ba879757399b98583..fb18f91af01f1e8b48f910e7147318420574d408 100644
--- a/chrome/browser/extensions/extension_warning_set.cc
+++ b/chrome/browser/extensions/extension_warning_set.cc
@@ -4,192 +4,204 @@
 
 #include "chrome/browser/extensions/extension_warning_set.h"
 
+#include "base/utf_string_conversions.h"
 #include "chrome/browser/browser_process.h"
-#include "chrome/browser/extensions/extension_global_error_badge.h"
-#include "chrome/browser/extensions/extension_service.h"
-#include "chrome/browser/profiles/profile.h"
-#include "chrome/browser/profiles/profile_manager.h"
-#include "chrome/browser/ui/global_error/global_error_service.h"
-#include "chrome/browser/ui/global_error/global_error_service_factory.h"
 #include "chrome/common/chrome_notification_types.h"
+#include "chrome/common/extensions/extension.h"
+#include "chrome/common/extensions/extension_set.h"
 #include "content/public/browser/browser_thread.h"
-#include "content/public/browser/notification_service.h"
 #include "grit/chromium_strings.h"
 #include "grit/generated_resources.h"
+#include "net/base/escape.h"
 #include "ui/base/l10n/l10n_util.h"
 
 using content::BrowserThread;
 
-// This class is used to represent warnings if extensions misbehave.
-class ExtensionWarning {
- public:
-  // Default constructor for storing ExtensionServiceWarning in STL containers
-  // do not use.
-  ExtensionWarning();
-
-  // Constructs a warning of type |type| for extension |extension_id|. This
-  // could be for example the fact that an extension conflicted with others.
-  ExtensionWarning(ExtensionWarningSet::WarningType type,
-                   const std::string& extension_id);
-
-  ~ExtensionWarning();
-
-  // Returns the specific warning type.
-  ExtensionWarningSet::WarningType warning_type() const { return type_; }
-
-  // Returns the id of the extension for which this warning is valid.
-  const std::string& extension_id() const { return extension_id_; }
+namespace {
+// Prefix for message parameters indicating that the parameter needs to
+// be translated from an extension id to the extension name.
+const char kTranslate[] = "TO_TRANSLATE:";
+const size_t kMaxNumberOfParameters = 4;
+}
 
- private:
-  ExtensionWarningSet::WarningType type_;
-  std::string extension_id_;
+namespace extensions {
 
-  // Allow implicit copy and assign operator.
-};
-
-ExtensionWarning::ExtensionWarning() : type_(ExtensionWarningSet::kInvalid) {
-}
+//
+// ExtensionWarning
+//
 
 ExtensionWarning::ExtensionWarning(
-    ExtensionWarningSet::WarningType type,
-    const std::string& extension_id)
-    : type_(type), extension_id_(extension_id) {
+    WarningType type,
+    const std::string& extension_id,
+    int message_id,
+    const std::vector<std::string>& message_parameters)
+    : type_(type),
+      extension_id_(extension_id),
+      message_id_(message_id),
+      message_parameters_(message_parameters) {
   // These are invalid here because they do not have corresponding warning
   // messages in the UI.
-  CHECK(type != ExtensionWarningSet::kInvalid);
-  CHECK(type != ExtensionWarningSet::kMaxWarningType);
-}
-
-ExtensionWarning::~ExtensionWarning() {
+  CHECK_NE(type, kInvalid);
+  CHECK_NE(type, kMaxWarningType);
+  CHECK_LE(message_parameters.size(), kMaxNumberOfParameters);
 }
 
-bool operator<(const ExtensionWarning& a, const ExtensionWarning& b) {
-  if (a.warning_type() == b.warning_type())
-    return a.extension_id() < b.extension_id();
-  return a.warning_type() < b.warning_type();
-}
+ExtensionWarning::ExtensionWarning(const ExtensionWarning& other)
+  : type_(other.type_),
+    extension_id_(other.extension_id_),
+    message_id_(other.message_id_),
+    message_parameters_(other.message_parameters_) {}
 
-// Static
-string16 ExtensionWarningSet::GetLocalizedWarning(
-    ExtensionWarningSet::WarningType warning_type) {
-  switch (warning_type) {
-    case kInvalid:
-    case kMaxWarningType:
-      NOTREACHED();
-      return string16();
-    case kNetworkDelay:
-      return l10n_util::GetStringFUTF16(
-          IDS_EXTENSION_WARNINGS_NETWORK_DELAY,
-          l10n_util::GetStringUTF16(IDS_PRODUCT_NAME));
-    case kNetworkConflict:
-      return l10n_util::GetStringUTF16(IDS_EXTENSION_WARNINGS_NETWORK_CONFLICT);
-    case kRepeatedCacheFlushes:
-      return l10n_util::GetStringFUTF16(
-          IDS_EXTENSION_WARNINGS_NETWORK_DELAY,
-          l10n_util::GetStringUTF16(IDS_PRODUCT_NAME));
-  }
-  NOTREACHED();  // Switch statement has no default branch.
-  return string16();
+ExtensionWarning::~ExtensionWarning() {
 }
 
-ExtensionWarningSet::ExtensionWarningSet(Profile* profile) : profile_(profile) {
+ExtensionWarning& ExtensionWarning::operator=(const ExtensionWarning& other) {
+  type_ = other.type_;
+  extension_id_ = other.extension_id_;
+  message_id_ = other.message_id_;
+  message_parameters_ = other.message_parameters_;
+  return *this;
 }
 
-ExtensionWarningSet::~ExtensionWarningSet() {
+// static
+ExtensionWarning ExtensionWarning::CreateNetworkDelayWarning(
+    const std::string& extension_id) {
+  std::vector<std::string> message_parameters;
+  message_parameters.push_back(l10n_util::GetStringUTF8(IDS_PRODUCT_NAME));
+  return ExtensionWarning(
+      kNetworkDelay,
+      extension_id,
+      IDS_EXTENSION_WARNINGS_NETWORK_DELAY,
+      message_parameters);
 }
 
-void ExtensionWarningSet::SetWarning(ExtensionWarningSet::WarningType type,
-                                     const std::string& extension_id) {
-  ExtensionWarning warning(type, extension_id);
-  bool inserted = warnings_.insert(warning).second;
-  if (inserted) {
-    NotifyWarningsChanged();
-    UpdateWarningBadge();
-  }
+// static
+ExtensionWarning ExtensionWarning::CreateNetworkConflictWarning(
+    const std::string& extension_id) {
+  std::vector<std::string> message_parameters;
+  return ExtensionWarning(
+      kNetworkConflict,
+      extension_id,
+      IDS_EXTENSION_WARNINGS_NETWORK_CONFLICT,
+      message_parameters);
 }
 
-void ExtensionWarningSet::ClearWarnings(
-    const std::set<ExtensionWarningSet::WarningType>& types) {
-  bool deleted_anything = false;
-  for (iterator i = warnings_.begin(); i != warnings_.end();) {
-    if (types.find(i->warning_type()) != types.end()) {
-      deleted_anything = true;
-      warnings_.erase(i++);
-    } else {
-      ++i;
-    }
-  }
-
-  if (deleted_anything) {
-    NotifyWarningsChanged();
-    UpdateWarningBadge();
-  }
+// static
+ExtensionWarning ExtensionWarning::CreateRedirectConflictWarning(
+    const std::string& extension_id,
+    const std::string& winning_extension_id,
+    const GURL& attempted_redirect_url,
+    const GURL& winning_redirect_url) {
+  std::vector<std::string> message_parameters;
+  message_parameters.push_back(attempted_redirect_url.spec());
+  message_parameters.push_back(kTranslate + winning_extension_id);
+  message_parameters.push_back(winning_redirect_url.spec());
+  return ExtensionWarning(
+      kRedirectConflict,
+      extension_id,
+      IDS_EXTENSION_WARNINGS_REDIRECT_CONFLICT,
+      message_parameters);
 }
 
-void ExtensionWarningSet::GetWarningsAffectingExtension(
+// static
+ExtensionWarning ExtensionWarning::CreateRequestHeaderConflictWarning(
     const std::string& extension_id,
-    std::set<ExtensionWarningSet::WarningType>* result) const {
-  result->clear();
-  for (const_iterator i = warnings_.begin(); i != warnings_.end(); ++i) {
-    if (i->extension_id() == extension_id)
-      result->insert(i->warning_type());
-  }
+    const std::string& winning_extension_id,
+    const std::string& conflicting_header) {
+  std::vector<std::string> message_parameters;
+  message_parameters.push_back(conflicting_header);
+  message_parameters.push_back(kTranslate + winning_extension_id);
+  return ExtensionWarning(
+      kNetworkConflict,
+      extension_id,
+      IDS_EXTENSION_WARNINGS_REQUEST_HEADER_CONFLICT,
+      message_parameters);
 }
 
 // static
-void ExtensionWarningSet::NotifyWarningsOnUI(
-    void* profile_id,
-    std::set<std::string> extension_ids,
-    WarningType warning_type) {
-  CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
-  Profile* profile = reinterpret_cast<Profile*>(profile_id);
-  if (!profile ||
-      !g_browser_process->profile_manager() ||
-      !g_browser_process->profile_manager()->IsValidProfile(profile)) {
-    return;
-  }
-
-  ExtensionWarningSet* warnings =
-      profile->GetExtensionService()->extension_warnings();
-
-  for (std::set<std::string>::const_iterator i = extension_ids.begin();
-       i != extension_ids.end(); ++i) {
-    warnings->SetWarning(warning_type, *i);
-  }
+ExtensionWarning ExtensionWarning::CreateResponseHeaderConflictWarning(
+    const std::string& extension_id,
+    const std::string& winning_extension_id,
+    const std::string& conflicting_header) {
+  std::vector<std::string> message_parameters;
+  message_parameters.push_back(conflicting_header);
+  message_parameters.push_back(kTranslate + winning_extension_id);
+  return ExtensionWarning(
+      kNetworkConflict,
+      extension_id,
+      IDS_EXTENSION_WARNINGS_RESPONSE_HEADER_CONFLICT,
+      message_parameters);
 }
 
-void ExtensionWarningSet::SuppressBadgeForCurrentWarnings() {
-  badge_suppressions_.insert(warnings_.begin(), warnings_.end());
-  UpdateWarningBadge();
+// static
+ExtensionWarning ExtensionWarning::CreateCredentialsConflictWarning(
+    const std::string& extension_id,
+    const std::string& winning_extension_id) {
+  std::vector<std::string> message_parameters;
+  message_parameters.push_back(kTranslate + winning_extension_id);
+  return ExtensionWarning(
+      kNetworkConflict,
+      extension_id,
+      IDS_EXTENSION_WARNINGS_CREDENTIALS_CONFLICT,
+      message_parameters);
 }
 
-void ExtensionWarningSet::NotifyWarningsChanged() {
-  content::NotificationService::current()->Notify(
-      chrome::NOTIFICATION_EXTENSION_WARNING_CHANGED,
-      content::Source<Profile>(profile_),
-      content::NotificationService::NoDetails());
+// static
+ExtensionWarning ExtensionWarning::CreateRepeatedCacheFlushesWarning(
+    const std::string& extension_id) {
+  std::vector<std::string> message_parameters;
+  message_parameters.push_back(l10n_util::GetStringUTF8(IDS_PRODUCT_NAME));
+  return ExtensionWarning(
+      kRepeatedCacheFlushes,
+      extension_id,
+      IDS_EXTENSION_WARNINGS_NETWORK_DELAY,
+      message_parameters);
 }
 
-void ExtensionWarningSet::UpdateWarningBadge() {
-  // We need a badge if a warning exists that has not been suppressed.
-  bool need_warning_badge = false;
-  for (const_iterator i = warnings_.begin(); i != warnings_.end(); ++i) {
-    if (badge_suppressions_.find(*i) == badge_suppressions_.end()) {
-      need_warning_badge = true;
-      break;
+std::string ExtensionWarning::GetLocalizedMessage(
+    const ExtensionSet* extensions) const {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+
+  // These parameters may be unsafe (URLs and Extension names) and need
+  // to be HTML-escaped before being embedded in the UI. Also extension IDs
+  // are translated to full extension names.
+  std::vector<string16> final_parameters;
+  for (size_t i = 0; i < message_parameters_.size(); ++i) {
+    std::string message = message_parameters_[i];
+    if (StartsWithASCII(message, kTranslate, true)) {
+      std::string extension_id = message.substr(sizeof(kTranslate) - 1);
+      const extensions::Extension* extension =
+          extensions->GetByID(extension_id);
+      message = extension ? extension->name() : extension_id;
     }
+    final_parameters.push_back(UTF8ToUTF16(net::EscapeForHTML(message)));
   }
 
-  GlobalErrorService* service =
-      GlobalErrorServiceFactory::GetForProfile(profile_);
-  GlobalError* error = service->GetGlobalErrorByMenuItemCommandID(
-      ExtensionGlobalErrorBadge::GetMenuItemCommandID());
-
-  // Activate or hide the warning badge in case the current state is incorrect.
-  if (error && !need_warning_badge) {
-    service->RemoveGlobalError(error);
-    delete error;
-  } else if (!error && need_warning_badge) {
-    service->AddGlobalError(new ExtensionGlobalErrorBadge);
+  COMPILE_ASSERT(kMaxNumberOfParameters == 4u, YouNeedToAddMoreCaseStatements);
+  switch (final_parameters.size()) {
+    case 0:
+      return l10n_util::GetStringUTF8(message_id_);
+    case 1:
+      return l10n_util::GetStringFUTF8(message_id_, final_parameters[0]);
+    case 2:
+      return l10n_util::GetStringFUTF8(message_id_, final_parameters[0],
+          final_parameters[1]);
+    case 3:
+      return l10n_util::GetStringFUTF8(message_id_, final_parameters[0],
+          final_parameters[1], final_parameters[2]);
+    case 4:
+      return l10n_util::GetStringFUTF8(message_id_, final_parameters[0],
+          final_parameters[1], final_parameters[2], final_parameters[3]);
+    default:
+      NOTREACHED();
+      return std::string();
   }
 }
+
+bool operator<(const ExtensionWarning& a, const ExtensionWarning& b) {
+  if (a.extension_id() != b.extension_id())
+    return a.extension_id() < b.extension_id();
+  return a.warning_type() < b.warning_type();
+}
+
+}  // namespace extensions