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

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

Issue 91002: Always enable copy-link-address in context menu. (Closed) Base URL: http://src.chromium.org/svn/trunk/src/
Patch Set: '' Created 11 years, 8 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 | Annotate | Revision Log
« no previous file with comments | « chrome/browser/renderer_host/render_view_host.cc ('k') | chrome/common/render_messages.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2006-2008 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 "base/command_line.h" 7 #include "base/command_line.h"
8 #include "base/logging.h" 8 #include "base/logging.h"
9 #include "base/scoped_clipboard_writer.h" 9 #include "base/scoped_clipboard_writer.h"
10 #include "chrome/app/chrome_dll_resource.h" 10 #include "chrome/app/chrome_dll_resource.h"
(...skipping 202 matching lines...) Expand 10 before | Expand all | Expand 10 after
213 case IDS_CONTENT_CONTEXT_FORWARD: 213 case IDS_CONTENT_CONTEXT_FORWARD:
214 return source_web_contents_->controller().CanGoForward(); 214 return source_web_contents_->controller().CanGoForward();
215 215
216 case IDS_CONTENT_CONTEXT_VIEWPAGESOURCE: 216 case IDS_CONTENT_CONTEXT_VIEWPAGESOURCE:
217 case IDS_CONTENT_CONTEXT_VIEWFRAMESOURCE: 217 case IDS_CONTENT_CONTEXT_VIEWFRAMESOURCE:
218 case IDS_CONTENT_CONTEXT_INSPECTELEMENT: 218 case IDS_CONTENT_CONTEXT_INSPECTELEMENT:
219 return IsDevCommandEnabled(id); 219 return IsDevCommandEnabled(id);
220 220
221 case IDS_CONTENT_CONTEXT_OPENLINKNEWTAB: 221 case IDS_CONTENT_CONTEXT_OPENLINKNEWTAB:
222 case IDS_CONTENT_CONTEXT_OPENLINKNEWWINDOW: 222 case IDS_CONTENT_CONTEXT_OPENLINKNEWWINDOW:
223 return params_.link_url.is_valid();
224
223 case IDS_CONTENT_CONTEXT_COPYLINKLOCATION: 225 case IDS_CONTENT_CONTEXT_COPYLINKLOCATION:
224 return params_.link_url.is_valid(); 226 return params_.unfiltered_link_url.is_valid();
225 227
226 case IDS_CONTENT_CONTEXT_SAVELINKAS: 228 case IDS_CONTENT_CONTEXT_SAVELINKAS:
227 return params_.link_url.is_valid() && 229 return params_.link_url.is_valid() &&
228 URLRequest::IsHandledURL(params_.link_url); 230 URLRequest::IsHandledURL(params_.link_url);
229 231
230 case IDS_CONTENT_CONTEXT_SAVEIMAGEAS: 232 case IDS_CONTENT_CONTEXT_SAVEIMAGEAS:
231 return params_.image_url.is_valid() && 233 return params_.image_url.is_valid() &&
232 URLRequest::IsHandledURL(params_.image_url); 234 URLRequest::IsHandledURL(params_.image_url);
233 235
234 case IDS_CONTENT_CONTEXT_OPENIMAGENEWTAB: 236 case IDS_CONTENT_CONTEXT_OPENIMAGENEWTAB:
(...skipping 135 matching lines...) Expand 10 before | Expand all | Expand 10 after
370 const GURL& url = 372 const GURL& url =
371 (id == IDS_CONTENT_CONTEXT_SAVELINKAS ? params_.link_url : 373 (id == IDS_CONTENT_CONTEXT_SAVELINKAS ? params_.link_url :
372 params_.image_url); 374 params_.image_url);
373 DownloadManager* dlm = 375 DownloadManager* dlm =
374 source_web_contents_->profile()->GetDownloadManager(); 376 source_web_contents_->profile()->GetDownloadManager();
375 dlm->DownloadUrl(url, referrer, source_web_contents_); 377 dlm->DownloadUrl(url, referrer, source_web_contents_);
376 break; 378 break;
377 } 379 }
378 380
379 case IDS_CONTENT_CONTEXT_COPYLINKLOCATION: 381 case IDS_CONTENT_CONTEXT_COPYLINKLOCATION:
380 WriteURLToClipboard(params_.link_url); 382 WriteURLToClipboard(params_.unfiltered_link_url);
381 break; 383 break;
382 384
383 case IDS_CONTENT_CONTEXT_COPYIMAGELOCATION: 385 case IDS_CONTENT_CONTEXT_COPYIMAGELOCATION:
384 WriteURLToClipboard(params_.image_url); 386 WriteURLToClipboard(params_.image_url);
385 break; 387 break;
386 388
387 case IDS_CONTENT_CONTEXT_COPYIMAGE: 389 case IDS_CONTENT_CONTEXT_COPYIMAGE:
388 CopyImageAt(params_.x, params_.y); 390 CopyImageAt(params_.x, params_.y);
389 break; 391 break;
390 392
(...skipping 257 matching lines...) Expand 10 before | Expand all | Expand 10 after
648 ScopedClipboardWriter scw(clipboard); 650 ScopedClipboardWriter scw(clipboard);
649 scw.WriteText(text); 651 scw.WriteText(text);
650 } 652 }
651 653
652 void RenderViewContextMenu::WriteURLToClipboard(const GURL& url) { 654 void RenderViewContextMenu::WriteURLToClipboard(const GURL& url) {
653 if (url.SchemeIs(chrome::kMailToScheme)) 655 if (url.SchemeIs(chrome::kMailToScheme))
654 WriteTextToClipboard(UTF8ToUTF16(url.path())); 656 WriteTextToClipboard(UTF8ToUTF16(url.path()));
655 else 657 else
656 WriteTextToClipboard(UTF8ToUTF16(url.spec())); 658 WriteTextToClipboard(UTF8ToUTF16(url.spec()));
657 } 659 }
OLDNEW
« no previous file with comments | « chrome/browser/renderer_host/render_view_host.cc ('k') | chrome/common/render_messages.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698