Updated Plugin bubble model to add "learn more" link and to appear with a sliding yellow thing

chrome/browser/ui/browser_content_setting_bubble_model_delegate.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/ui/browser_content_setting_bubble_model_delegate.h"
 
 #include "chrome/browser/google/google_util.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_dialogs.h"
 #include "chrome/browser/ui/browser_tabstrip.h"
@@ skipping 30 matching lines @@
           content::PAGE_TRANSITION_LINK);
       return;
     case CONTENT_SETTINGS_TYPE_PROTOCOL_HANDLERS:
       chrome::ShowSettingsSubPage(browser_, chrome::kHandlerSettingsSubPage);
       return;
     default:
       chrome::ShowContentSettings(browser_, type);
       return;
   }
 }
+
+void BrowserContentSettingBubbleModelDelegate::ShowLearnMorePage(
+    ContentSettingsType type) {
+  switch (type) {

[felt, 2014/06/11 02:37:30]

Does this make sense as a switch, if there's only one case you handle? Should it
be an if-statement?

[radhikabhar, 2014/06/11 17:31:46]

Done.

+    case CONTENT_SETTINGS_TYPE_PLUGINS:
+      chrome::AddSelectedTabWithURL(
+          browser_,
+          google_util::AppendGoogleLocaleParam(
+              GURL(chrome::kBlockedPluginLearnMoreURL)),
+          content::PAGE_TRANSITION_LINK);
+      return;
+    default:
+      return;
+  }
+}