Browser Plugin: Reload and Stop operations

content/browser/browser_plugin/browser_plugin_guest.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 "content/browser/browser_plugin/browser_plugin_guest.h"
 
 #include <algorithm>
 
 #include "content/browser/browser_plugin/browser_plugin_guest_helper.h"
 #include "content/browser/browser_plugin/browser_plugin_host_factory.h"
@@ skipping 207 matching lines @@
       static_cast<RenderViewHostImpl*>(render_view_host);
   guest_rvh->StopHangMonitorTimeout();
   DCHECK(pending_input_event_reply_.get());
   IPC::Message* reply_message = pending_input_event_reply_.release();
   BrowserPluginHostMsg_HandleInputEvent::WriteReplyParams(reply_message,
                                                           handled,
                                                           cursor_);
   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(
       new ViewMsg_SetFocus(render_view_host->GetRoutingID(), focused));
 }
 
 void BrowserPluginGuest::ShowWidget(RenderViewHost* render_view_host,
                                     int route_id,
                                     const gfx::Rect& initial_pos) {
   gfx::Rect screen_pos(initial_pos);
@@ skipping 33 matching lines @@
     pending_updates_.Remove(iter.GetCurrentKey());
     iter.Advance();
   }
 }
 
 void BrowserPluginGuest::SendMessageToEmbedder(IPC::Message* msg) {
   embedder_render_process_host()->Send(msg);
 }
 
 }  // namespace content