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

Unified Diff: content/browser/renderer_host/render_widget_host_view_mac_editcommand_helper_unittest.mm

Issue 10377158: Move keyboard related methods from RenderViewHostDelegate to a new RenderWidgetHostDelegate (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: remove crbug link Created 8 years, 7 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_view_mac_editcommand_helper_unittest.mm
diff --git a/content/browser/renderer_host/render_widget_host_view_mac_editcommand_helper_unittest.mm b/content/browser/renderer_host/render_widget_host_view_mac_editcommand_helper_unittest.mm
index a0d8628c50e9f9bfc477016086a68c90c8acccf7..f4066a2c177d76c3daaecd0c0d5d31eb81576b38 100644
--- a/content/browser/renderer_host/render_widget_host_view_mac_editcommand_helper_unittest.mm
+++ b/content/browser/renderer_host/render_widget_host_view_mac_editcommand_helper_unittest.mm
@@ -7,6 +7,7 @@
#import <Cocoa/Cocoa.h>
#include "base/message_loop.h"
+#include "content/browser/renderer_host/render_widget_host_delegate.h"
#include "content/browser/renderer_host/render_widget_host_impl.h"
#include "content/common/view_messages.h"
#include "content/test/mock_render_process_host.h"
@@ -71,12 +72,20 @@ namespace {
}
} // namespace
+class MockRenderWidgetHostDelegate : public content::RenderWidgetHostDelegate {
+ public:
+ MockRenderWidgetHostDelegate() {}
+ virtual ~MockRenderWidgetHostDelegate() {}
+};
+
// Create a RenderWidget for which we can filter messages.
class RenderWidgetHostEditCommandCounter : public RenderWidgetHostImpl {
public:
- RenderWidgetHostEditCommandCounter(content::RenderProcessHost* process,
- int routing_id)
- : RenderWidgetHostImpl(process, routing_id),
+ RenderWidgetHostEditCommandCounter(
+ content::RenderWidgetHostDelegate* delegate,
+ content::RenderProcessHost* process,
+ int routing_id)
+ : RenderWidgetHostImpl(delegate, process, routing_id),
edit_command_message_count_(0) {
}
@@ -102,7 +111,8 @@ TEST_F(RenderWidgetHostViewMacEditCommandHelperTest,
MessageLoopForUI message_loop;
TestBrowserContext browser_context;
MockRenderProcessHost mock_process(&browser_context);
- RenderWidgetHostEditCommandCounter render_widget(&mock_process, 0);
+ MockRenderWidgetHostDelegate delegate;
+ RenderWidgetHostEditCommandCounter render_widget(&delegate, &mock_process, 0);
// RenderWidgetHostViewMac self destructs (RenderWidgetHostViewMacCocoa
// takes ownership) so no need to delete it ourselves.

Powered by Google App Engine
This is Rietveld 408576698