Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(569)

Unified Diff: content/browser/renderer_host/render_widget_host_unittest.cc

Issue 396483003: Separate ResizeHelper from RenderWidgetHelper (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix Linux build again Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/renderer_host/render_widget_host_unittest.cc
diff --git a/content/browser/renderer_host/render_widget_host_unittest.cc b/content/browser/renderer_host/render_widget_host_unittest.cc
index 29f1b658eddb194ee713aee108facf4cbb1bec00..9790c4661e14ca6789775f5a568a592d88aace52 100644
--- a/content/browser/renderer_host/render_widget_host_unittest.cc
+++ b/content/browser/renderer_host/render_widget_host_unittest.cc
@@ -202,15 +202,11 @@ class RenderWidgetHostProcess : public MockRenderProcessHost {
public:
explicit RenderWidgetHostProcess(BrowserContext* browser_context)
: MockRenderProcessHost(browser_context),
- update_msg_should_reply_(false),
update_msg_reply_flags_(0) {
}
virtual ~RenderWidgetHostProcess() {
}
- void set_update_msg_should_reply(bool reply) {
- update_msg_should_reply_ = reply;
- }
void set_update_msg_reply_flags(int flags) {
update_msg_reply_flags_ = flags;
}
@@ -221,14 +217,6 @@ class RenderWidgetHostProcess : public MockRenderProcessHost {
virtual bool HasConnection() const OVERRIDE { return true; }
protected:
- virtual bool WaitForBackingStoreMsg(int render_widget_id,
- const base::TimeDelta& max_delay,
- IPC::Message* msg) OVERRIDE;
-
- // Set to true when WaitForBackingStoreMsg should return a successful update
- // message reply. False implies timeout.
- bool update_msg_should_reply_;
-
// Indicates the flags that should be sent with a repaint request. This
// only has an effect when update_msg_should_reply_ is true.
int update_msg_reply_flags_;
@@ -244,22 +232,6 @@ void RenderWidgetHostProcess::InitUpdateRectParams(
params->flags = update_msg_reply_flags_;
}
-bool RenderWidgetHostProcess::WaitForBackingStoreMsg(
- int render_widget_id,
- const base::TimeDelta& max_delay,
- IPC::Message* msg) {
- if (!update_msg_should_reply_)
- return false;
-
- // Construct a fake update reply.
- ViewHostMsg_UpdateRect_Params params;
- InitUpdateRectParams(&params);
-
- ViewHostMsg_UpdateRect message(render_widget_id, params);
- *msg = message;
- return true;
-}
-
// TestView --------------------------------------------------------------------
// This test view allows us to specify the size, and keep track of acked

Powered by Google App Engine
This is Rietveld 408576698