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

Unified Diff: content/browser/tab_contents/tab_contents.cc

Issue 8588039: Remove "open in new tab" items from context menu if the process doesn't (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 9 years, 1 month 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/tab_contents/tab_contents.cc
===================================================================
--- content/browser/tab_contents/tab_contents.cc (revision 112335)
+++ content/browser/tab_contents/tab_contents.cc (working copy)
@@ -921,13 +921,13 @@
const GURL& opener_url,
const GURL& url) {
bool is_error_page = (url.spec() == chrome::kUnreachableWebDataURL);
+ RenderViewHost* rvh = render_manager_.pending_render_view_host() ?
+ render_manager_.pending_render_view_host() : render_view_host();
+
GURL validated_url(url);
- render_view_host()->FilterURL(ChildProcessSecurityPolicy::GetInstance(),
- GetRenderProcessHost()->GetID(), &validated_url);
+ RenderViewHost::FilterURL(ChildProcessSecurityPolicy::GetInstance(),
+ rvh->process()->GetID(), &validated_url);
- RenderViewHost* rvh =
- render_manager_.pending_render_view_host() ?
- render_manager_.pending_render_view_host() : render_view_host();
// Notify observers about the start of the provisional load.
FOR_EACH_OBSERVER(TabContentsObserver, observers_,
DidStartProvisionalLoadForFrame(frame_id, is_main_frame,
« chrome/browser/browser_process_impl.cc ('K') | « content/browser/child_process_security_policy.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698