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

Unified Diff: chrome/browser/translate/translate_manager_browsertest.cc

Issue 130773004: Start moving context menu code to RenderFrame. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 6 years, 11 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/translate/translate_manager_browsertest.cc
===================================================================
--- chrome/browser/translate/translate_manager_browsertest.cc (revision 247964)
+++ chrome/browser/translate/translate_manager_browsertest.cc (working copy)
@@ -402,7 +402,7 @@
params.writing_direction_right_to_left = 0;
#endif // OS_MACOSX
params.edit_flags = blink::WebContextMenuData::CanTranslate;
- return new TestRenderViewContextMenu(web_contents, params);
+ return new TestRenderViewContextMenu(web_contents->GetMainFrame(), params);
}
bool IsItemPresent(int id) {
@@ -416,9 +416,9 @@
ui::Accelerator* accelerator) OVERRIDE { return false; }
private:
- TestRenderViewContextMenu(content::WebContents* web_contents,
+ TestRenderViewContextMenu(content::RenderFrameHost* render_frame_host,
const content::ContextMenuParams& params)
- : RenderViewContextMenu(web_contents, params) {
+ : RenderViewContextMenu(render_frame_host, params) {
}
DISALLOW_COPY_AND_ASSIGN(TestRenderViewContextMenu);

Powered by Google App Engine
This is Rietveld 408576698