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

Unified Diff: components/renderer_context_menu/render_view_context_menu_base.cc

Issue 2383573002: [DO NOT COMMIT] Remove chrome::NOTIFICATION_RETARGETING
Patch Set: Rebase on ToT Created 4 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: components/renderer_context_menu/render_view_context_menu_base.cc
diff --git a/components/renderer_context_menu/render_view_context_menu_base.cc b/components/renderer_context_menu/render_view_context_menu_base.cc
index 63d8df500f6438c23031d9b5a00e28f22a499e14..548b70a9294789dfed81f69ea8770b61d725f857 100644
--- a/components/renderer_context_menu/render_view_context_menu_base.cc
+++ b/components/renderer_context_menu/render_view_context_menu_base.cc
@@ -162,7 +162,9 @@ RenderViewContextMenuBase::RenderViewContextMenuBase(
menu_model_(this),
render_frame_id_(render_frame_host->GetRoutingID()),
render_process_id_(render_frame_host->GetProcess()->GetID()),
- command_executed_(false) {}
+ command_executed_(false) {
+ LOG(ERROR) << "ContextMenuBase: rfh:" << render_frame_host;
+}
RenderViewContextMenuBase::~RenderViewContextMenuBase() {
}
@@ -395,7 +397,7 @@ void RenderViewContextMenuBase::OpenURLWithExtraHeaders(
if (!new_contents)
return;
- NotifyURLOpened(url, new_contents);
+ // NotifyURLOpened(url, new_contents);
}
bool RenderViewContextMenuBase::IsCustomItemChecked(int id) const {

Powered by Google App Engine
This is Rietveld 408576698