Implement a useful context menu for the blocked plug-in frame:...

chrome/renderer/blocked_plugin.h

 // Copyright (c) 2010 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.
 
 #ifndef CHROME_RENDERER_BLOCKED_PLUGIN_H_
 #define CHROME_RENDERER_BLOCKED_PLUGIN_H_
 #pragma once
 
 #include "chrome/common/notification_observer.h"
 #include "chrome/common/notification_registrar.h"
@@ skipping 11 matching lines @@
  public:
   BlockedPlugin(RenderView* render_view,
                 WebKit::WebFrame* frame,
                 const PluginGroup& info,
                 const WebKit::WebPluginParams& params,
                 const WebPreferences& settings,
                 int template_id,
                 const string16& message);
 
   WebViewPlugin* plugin() { return plugin_; }
+  void menuOptionSelected(unsigned id);

[brettw, 2010/12/07 06:23:56]

This should be capitalized. I'd probably also put a blank line abofe this since
it's not at all related to plugin().

[Chris Evans, 2010/12/07 15:36:14]

Done.

 
   // WebViewPlugin::Delegate methods:
   virtual void BindWebFrame(WebKit::WebFrame* frame);
   virtual void WillDestroyPlugin();
+  virtual void ShowContextMenu(const WebKit::WebMouseEvent&);
 
   // NotificationObserver methods:
   virtual void Observe(NotificationType type,
                        const NotificationSource& source,
                        const NotificationDetails& details);
 
  private:
   virtual ~BlockedPlugin();
 
   // Javascript callbacks:
   // Load the blocked plugin by calling LoadPlugin() below.
   // Takes no arguments, and returns nothing.
   void Load(const CppArgumentList& args, CppVariant* result);
 
   // Load the blocked plugin.
   void LoadPlugin();
 
+  // Hide the blocked plugin.
+  void HidePlugin();
+
   RenderView* render_view_;
   WebKit::WebFrame* frame_;
   WebKit::WebPluginParams plugin_params_;
   WebViewPlugin* plugin_;
+  string16 name_;

[brettw, 2010/12/07 06:23:56]

Can you provide a comment for this? It's very generic.

[Chris Evans, 2010/12/07 15:36:14]

Done.

 
   NotificationRegistrar registrar_;
 };
 
 #endif  // CHROME_RENDERER_BLOCKED_PLUGIN_H_