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) {
+  if (type == CONTENT_SETTINGS_TYPE_PLUGINS) {

[felt, 2014/06/12 01:44:33]

cleaner to gate with an if-statement at the very beginning, like:

if (type != CONTENT_SETTINGS_TYPE_PLUGINS)
  return;
chrome::AddSelectedTabWithURL(...

[radhikabhar, 2014/06/12 16:21:45]

Done.

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