[Extensions] Make extension message bubble factory platform-abstract

chrome/browser/extensions/proxy_overridden_bubble_controller.cc

Index: chrome/browser/extensions/proxy_overridden_bubble_controller.cc
diff --git a/chrome/browser/extensions/proxy_overridden_bubble_controller.cc b/chrome/browser/extensions/proxy_overridden_bubble_controller.cc
index a3cde4cacb2f36c9ebb0fe95f232d9cd66182080..d25204d58bdf097f6c06fcd19cd6a3ac57e658dd 100644
--- a/chrome/browser/extensions/proxy_overridden_bubble_controller.cc
+++ b/chrome/browser/extensions/proxy_overridden_bubble_controller.cc
@@ -51,6 +51,7 @@ class ProxyOverriddenBubbleDelegate
   base::string16 GetActionButtonLabel() const override;
   base::string16 GetDismissButtonLabel() const override;
   bool ShouldShowExtensionList() const override;
+  bool ShouldHighlightExtensions() const override;
   void RestrictToSingleExtension(const std::string& extension_id) override;
   void LogExtensionCount(size_t count) override;
   void LogAction(
@@ -161,6 +162,10 @@ bool ProxyOverriddenBubbleDelegate::ShouldShowExtensionList() const {
   return false;
 }
 
+bool ProxyOverriddenBubbleDelegate::ShouldHighlightExtensions() const {
+  return true;

[Finnur, 2015/04/10 11:24:21]

The NTP override bubble delegate returns |false| from this function, but this
one |true|. Why the difference?

[Devlin, 2015/04/10 15:55:19]

Just to keep with current behavior.  Right now, we don't highlight the
extensions for the NTP override bubble, but do for the proxy override behavior. 
I figured it best to try and have no behavior changes as a result of this patch,
just so it's more obvious if it breaks stuff. :)

+}
+
 void ProxyOverriddenBubbleDelegate::RestrictToSingleExtension(
     const std::string& extension_id) {
   extension_id_ = extension_id;