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

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

Issue 274040: More style nits. (Closed)
Patch Set: Created 11 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 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/tab_contents/render_view_context_menu.h" 5 #include "chrome/browser/tab_contents/render_view_context_menu.h"
6 6
7 #include "app/clipboard/clipboard.h" 7 #include "app/clipboard/clipboard.h"
8 #include "app/clipboard/scoped_clipboard_writer.h" 8 #include "app/clipboard/scoped_clipboard_writer.h"
9 #include "app/l10n_util.h" 9 #include "app/l10n_util.h"
10 #include "base/command_line.h" 10 #include "base/command_line.h"
(...skipping 188 matching lines...) Expand 10 before | Expand all | Expand 10 after
199 void RenderViewContextMenu::AppendFrameItems() { 199 void RenderViewContextMenu::AppendFrameItems() {
200 AppendMenuItem(IDS_CONTENT_CONTEXT_BACK); 200 AppendMenuItem(IDS_CONTENT_CONTEXT_BACK);
201 AppendMenuItem(IDS_CONTENT_CONTEXT_FORWARD); 201 AppendMenuItem(IDS_CONTENT_CONTEXT_FORWARD);
202 AppendSeparator(); 202 AppendSeparator();
203 AppendMenuItem(IDS_CONTENT_CONTEXT_OPENFRAMENEWTAB); 203 AppendMenuItem(IDS_CONTENT_CONTEXT_OPENFRAMENEWTAB);
204 AppendMenuItem(IDS_CONTENT_CONTEXT_OPENFRAMENEWWINDOW); 204 AppendMenuItem(IDS_CONTENT_CONTEXT_OPENFRAMENEWWINDOW);
205 AppendMenuItem(IDS_CONTENT_CONTEXT_OPENFRAMEOFFTHERECORD); 205 AppendMenuItem(IDS_CONTENT_CONTEXT_OPENFRAMEOFFTHERECORD);
206 AppendSeparator(); 206 AppendSeparator();
207 // These two menu items have yet to be implemented. 207 // These two menu items have yet to be implemented.
208 // http://code.google.com/p/chromium/issues/detail?id=11827 208 // http://code.google.com/p/chromium/issues/detail?id=11827
209 //AppendMenuItem(IDS_CONTENT_CONTEXT_SAVEFRAMEAS); 209 // AppendMenuItem(IDS_CONTENT_CONTEXT_SAVEFRAMEAS);
210 //AppendMenuItem(IDS_CONTENT_CONTEXT_PRINTFRAME); 210 // AppendMenuItem(IDS_CONTENT_CONTEXT_PRINTFRAME);
211 AppendMenuItem(IDS_CONTENT_CONTEXT_VIEWFRAMESOURCE); 211 AppendMenuItem(IDS_CONTENT_CONTEXT_VIEWFRAMESOURCE);
212 AppendMenuItem(IDS_CONTENT_CONTEXT_VIEWFRAMEINFO); 212 AppendMenuItem(IDS_CONTENT_CONTEXT_VIEWFRAMEINFO);
213 } 213 }
214 214
215 void RenderViewContextMenu::AppendCopyItem() { 215 void RenderViewContextMenu::AppendCopyItem() {
216 AppendMenuItem(IDS_CONTENT_CONTEXT_COPY); 216 AppendMenuItem(IDS_CONTENT_CONTEXT_COPY);
217 } 217 }
218 218
219 void RenderViewContextMenu::AppendSearchProvider() { 219 void RenderViewContextMenu::AppendSearchProvider() {
220 DCHECK(profile_); 220 DCHECK(profile_);
(...skipping 595 matching lines...) Expand 10 before | Expand all | Expand 10 after
816 profile_->GetPrefs()->GetString(prefs::kAcceptLanguages), 816 profile_->GetPrefs()->GetString(prefs::kAcceptLanguages),
817 g_browser_process->clipboard()); 817 g_browser_process->clipboard());
818 } 818 }
819 819
820 void RenderViewContextMenu::MediaPlayerActionAt( 820 void RenderViewContextMenu::MediaPlayerActionAt(
821 const gfx::Point& location, 821 const gfx::Point& location,
822 const WebMediaPlayerAction& action) { 822 const WebMediaPlayerAction& action) {
823 source_tab_contents_->render_view_host()->MediaPlayerActionAt( 823 source_tab_contents_->render_view_host()->MediaPlayerActionAt(
824 location, action); 824 location, action);
825 } 825 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698