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

Side by Side Diff: chrome/browser/tab_contents/render_view_context_menu.cc

Issue 5716003: View source after POST command isn't what you expected. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Reviewers comments addressed. Created 10 years 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 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 <algorithm> 5 #include <algorithm>
6 #include <set> 6 #include <set>
7 7
8 #include "chrome/browser/tab_contents/render_view_context_menu.h" 8 #include "chrome/browser/tab_contents/render_view_context_menu.h"
9 9
10 #include "app/l10n_util.h" 10 #include "app/l10n_util.h"
(...skipping 1196 matching lines...) Expand 10 before | Expand all | Expand 10 after
1207 // Prevent the modal "Resubmit form post" dialog from appearing in the 1207 // Prevent the modal "Resubmit form post" dialog from appearing in the
1208 // context of an external context menu. 1208 // context of an external context menu.
1209 source_tab_contents_->controller().Reload(!external_); 1209 source_tab_contents_->controller().Reload(!external_);
1210 break; 1210 break;
1211 1211
1212 case IDC_PRINT: 1212 case IDC_PRINT:
1213 source_tab_contents_->PrintPreview(); 1213 source_tab_contents_->PrintPreview();
1214 break; 1214 break;
1215 1215
1216 case IDC_VIEW_SOURCE: 1216 case IDC_VIEW_SOURCE:
1217 OpenURL(GURL(chrome::kViewSourceScheme + std::string(":") + 1217 source_tab_contents_->ViewSource();
1218 params_.page_url.spec()), NEW_FOREGROUND_TAB, PageTransition::LINK);
1219 break; 1218 break;
1220 1219
1221 case IDC_CONTENT_CONTEXT_INSPECTELEMENT: 1220 case IDC_CONTENT_CONTEXT_INSPECTELEMENT:
1222 Inspect(params_.x, params_.y); 1221 Inspect(params_.x, params_.y);
1223 break; 1222 break;
1224 1223
1225 case IDC_CONTENT_CONTEXT_VIEWPAGEINFO: { 1224 case IDC_CONTENT_CONTEXT_VIEWPAGEINFO: {
1226 NavigationEntry* nav_entry = 1225 NavigationEntry* nav_entry =
1227 source_tab_contents_->controller().GetActiveEntry(); 1226 source_tab_contents_->controller().GetActiveEntry();
1228 source_tab_contents_->ShowPageInfo(nav_entry->url(), nav_entry->ssl(), 1227 source_tab_contents_->ShowPageInfo(nav_entry->url(), nav_entry->ssl(),
(...skipping 231 matching lines...) Expand 10 before | Expand all | Expand 10 after
1460 profile_->GetPrefs()->GetString(prefs::kAcceptLanguages), 1459 profile_->GetPrefs()->GetString(prefs::kAcceptLanguages),
1461 g_browser_process->clipboard()); 1460 g_browser_process->clipboard());
1462 } 1461 }
1463 1462
1464 void RenderViewContextMenu::MediaPlayerActionAt( 1463 void RenderViewContextMenu::MediaPlayerActionAt(
1465 const gfx::Point& location, 1464 const gfx::Point& location,
1466 const WebMediaPlayerAction& action) { 1465 const WebMediaPlayerAction& action) {
1467 source_tab_contents_->render_view_host()->MediaPlayerActionAt( 1466 source_tab_contents_->render_view_host()->MediaPlayerActionAt(
1468 location, action); 1467 location, action);
1469 } 1468 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698