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

Unified Diff: chrome/browser/renderer_host/render_view_host.cc

Issue 251103: Move MediaPlayerAction to WebMediaPlayerAction. Switch to an enum and a... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 11 years, 2 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: chrome/browser/renderer_host/render_view_host.cc
===================================================================
--- chrome/browser/renderer_host/render_view_host.cc (revision 28235)
+++ chrome/browser/renderer_host/render_view_host.cc (working copy)
@@ -55,6 +55,7 @@
using WebKit::WebDragOperationsMask;
using WebKit::WebFindOptions;
using WebKit::WebInputEvent;
+using WebKit::WebMediaPlayerAction;
using WebKit::WebTextDirection;
namespace {
@@ -1339,11 +1340,10 @@
delegate_->ShowModalHTMLDialog(url, width, height, json_arguments, reply_msg);
}
-void RenderViewHost::MediaPlayerActionAt(int x,
- int y,
- const MediaPlayerAction& action) {
+void RenderViewHost::MediaPlayerActionAt(const gfx::Point& location,
+ const WebMediaPlayerAction& action) {
// TODO(ajwong): Which thread should run this? Does it matter?
- Send(new ViewMsg_MediaPlayerActionAt(routing_id(), x, y, action));
+ Send(new ViewMsg_MediaPlayerActionAt(routing_id(), location, action));
}
void RenderViewHost::OnMsgPasswordFormsSeen(
« no previous file with comments | « chrome/browser/renderer_host/render_view_host.h ('k') | chrome/browser/tab_contents/render_view_context_menu.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698