Browser Plugin: Reload and Stop operations

content/browser/browser_plugin/browser_plugin_guest.cc

Index: content/browser/browser_plugin/browser_plugin_guest.cc
diff --git a/content/browser/browser_plugin/browser_plugin_guest.cc b/content/browser/browser_plugin/browser_plugin_guest.cc
index ada9dfbaa0f2c1dbacfe1307fe8682ac238148c0..6a56f819b57fb3517caba64a906b8fe3d3e11617 100644
--- a/content/browser/browser_plugin/browser_plugin_guest.cc
+++ b/content/browser/browser_plugin/browser_plugin_guest.cc
@@ -225,6 +225,14 @@ void BrowserPluginGuest::HandleInputEventAck(RenderViewHost* render_view_host,
   SendMessageToEmbedder(reply_message);
 }
 
+void BrowserPluginGuest::Stop() {
+  web_contents()->Stop();
+}
+
+void BrowserPluginGuest::Reload() {
+  web_contents()->GetController().Reload(false);

[Charlie Reis, 2012/09/19 22:17:09]

The NavigationController documentation says most callers should pass in true for
|check_for_repost|.  Wouldn't we want that to happen here, or are you trying to
avoid Chrome's repost warning page from showing?  (I suppose we could offer a
separate API for registering a callback if a repost is about to happen?)

Either way, please add a comment saying which decision is made here and why.

[Fady Samuel, 2012/09/19 23:29:20]

Done.

+}
+
 void BrowserPluginGuest::SetFocus(bool focused) {
   RenderViewHost* render_view_host = web_contents()->GetRenderViewHost();
   render_view_host->Send(