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

Unified Diff: chrome_frame/chrome_frame_plugin.h

Issue 6458004: Remove includes of message headers in headers. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 10 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
« no previous file with comments | « chrome_frame/chrome_frame_delegate.cc ('k') | chrome_frame/chrome_frame_plugin.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome_frame/chrome_frame_plugin.h
===================================================================
--- chrome_frame/chrome_frame_plugin.h (revision 74235)
+++ chrome_frame/chrome_frame_plugin.h (working copy)
@@ -20,6 +20,12 @@
#define IDC_ABOUT_CHROME_FRAME 40018
+// Helper so that this file doesn't include the messages header.
+void ChromeFramePluginGetParamsCoordinates(
+ const MiniContextMenuParams& params,
+ int* x,
+ int* y);
+
// A class to implement common functionality for all types of
// plugins: NPAPI. ActiveX and ActiveDoc
template <typename T>
@@ -141,8 +147,9 @@
SetFocus(GetWindow());
ignore_setfocus_ = false;
UINT flags = align_flags | TPM_LEFTBUTTON | TPM_RETURNCMD | TPM_RECURSE;
- UINT selected = TrackPopupMenuEx(copy, flags, params.screen_x,
- params.screen_y, GetWindow(), NULL);
+ int x, y;
+ ChromeFramePluginGetParamsCoordinates(params, &x, &y);
+ UINT selected = TrackPopupMenuEx(copy, flags, x, y, GetWindow(), NULL);
// Menu is over now give focus back to chrome
GiveFocusToChrome(false);
if (IsValid() && selected != 0 &&
« no previous file with comments | « chrome_frame/chrome_frame_delegate.cc ('k') | chrome_frame/chrome_frame_plugin.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698