Chromium Code Reviews| Index: chrome/browser/renderer_context_menu/render_view_context_menu.cc |
| diff --git a/chrome/browser/renderer_context_menu/render_view_context_menu.cc b/chrome/browser/renderer_context_menu/render_view_context_menu.cc |
| index e7b57ee9ec29b050e78c5c5b519e5f7d9eddf515..6f836cdea6d9bb11b4e96432c2f1f6834995f6fa 100644 |
| --- a/chrome/browser/renderer_context_menu/render_view_context_menu.cc |
| +++ b/chrome/browser/renderer_context_menu/render_view_context_menu.cc |
| @@ -1146,6 +1146,11 @@ bool RenderViewContextMenu::IsCommandIdEnabled(int id) const { |
| case IDC_VIEW_SOURCE: |
| case IDC_CONTENT_CONTEXT_VIEWFRAMESOURCE: |
| + // If we're viewing in a MimeHandlerViewGuest source should not be shown. |
|
AKV
2015/09/05 11:15:32
Please protect it using ENABLE_EXTENSIONS flag.
Deepak
2015/09/05 11:52:14
Done.
|
| + if (extensions::MimeHandlerViewGuest::FromWebContents( |
| + source_web_contents_)) { |
|
AKV
2015/09/05 11:15:32
Please indent to 4 spaces instead of 8
Deepak
2015/09/05 11:52:14
Done.
|
| + return false; |
|
AKV
2015/09/05 11:15:32
nit: single line body block doesn't need braces.
Deepak
2015/09/05 11:52:14
Done.
|
| + } |
| return embedder_web_contents_->GetController().CanViewSource(); |
| case IDC_CONTENT_CONTEXT_INSPECTELEMENT: |