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

Side by Side Diff: chrome/browser/renderer_host/render_view_host.cc

Issue 149604: Begin implementation of the context menu for Video and Audio tags. (Closed)
Patch Set: Fix small style issues. Created 11 years, 5 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2009 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/renderer_host/render_view_host.h" 5 #include "chrome/browser/renderer_host/render_view_host.h"
6 6
7 #include <string> 7 #include <string>
8 #include <vector> 8 #include <vector>
9 9
10 #include "app/resource_bundle.h" 10 #include "app/resource_bundle.h"
(...skipping 1100 matching lines...) Expand 10 before | Expand all | Expand 10 after
1111 // Validate the URLs in |params|. If the renderer can't request the URLs 1111 // Validate the URLs in |params|. If the renderer can't request the URLs
1112 // directly, don't show them in the context menu. 1112 // directly, don't show them in the context menu.
1113 ContextMenuParams validated_params(params); 1113 ContextMenuParams validated_params(params);
1114 const int renderer_id = process()->pid(); 1114 const int renderer_id = process()->pid();
1115 ChildProcessSecurityPolicy* policy = 1115 ChildProcessSecurityPolicy* policy =
1116 ChildProcessSecurityPolicy::GetInstance(); 1116 ChildProcessSecurityPolicy::GetInstance();
1117 1117
1118 // We don't validate |unfiltered_link_url| so that this field can be used 1118 // We don't validate |unfiltered_link_url| so that this field can be used
1119 // when users want to copy the original link URL. 1119 // when users want to copy the original link URL.
1120 FilterURL(policy, renderer_id, &validated_params.link_url); 1120 FilterURL(policy, renderer_id, &validated_params.link_url);
1121 FilterURL(policy, renderer_id, &validated_params.image_url); 1121 FilterURL(policy, renderer_id, &validated_params.src_url);
1122 FilterURL(policy, renderer_id, &validated_params.page_url); 1122 FilterURL(policy, renderer_id, &validated_params.page_url);
1123 FilterURL(policy, renderer_id, &validated_params.frame_url); 1123 FilterURL(policy, renderer_id, &validated_params.frame_url);
1124 1124
1125 view->ShowContextMenu(validated_params); 1125 view->ShowContextMenu(validated_params);
1126 } 1126 }
1127 1127
1128 void RenderViewHost::OnMsgOpenURL(const GURL& url, 1128 void RenderViewHost::OnMsgOpenURL(const GURL& url,
1129 const GURL& referrer, 1129 const GURL& referrer,
1130 WindowOpenDisposition disposition) { 1130 WindowOpenDisposition disposition) {
1131 GURL validated_url(url); 1131 GURL validated_url(url);
(...skipping 443 matching lines...) Expand 10 before | Expand all | Expand 10 after
1575 1575
1576 void RenderViewHost::SignalModalDialogEvent() { 1576 void RenderViewHost::SignalModalDialogEvent() {
1577 if (modal_dialog_count_++ == 0) 1577 if (modal_dialog_count_++ == 0)
1578 modal_dialog_event_->Signal(); 1578 modal_dialog_event_->Signal();
1579 } 1579 }
1580 1580
1581 void RenderViewHost::ResetModalDialogEvent() { 1581 void RenderViewHost::ResetModalDialogEvent() {
1582 if (--modal_dialog_count_ == 0) 1582 if (--modal_dialog_count_ == 0)
1583 modal_dialog_event_->Reset(); 1583 modal_dialog_event_->Reset();
1584 } 1584 }
OLDNEW
« no previous file with comments | « chrome/app/generated_resources.grd ('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