For "Copy link address" context menu, copy the URL with Unicode host name ins...

chrome/browser/tab_contents/render_view_context_menu.cc

 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/tab_contents/render_view_context_menu.h"
 
 #include "app/l10n_util.h"
 #include "base/clipboard.h"
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/scoped_clipboard_writer.h"
 #include "chrome/app/chrome_dll_resource.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/download/download_manager.h"
 #include "chrome/browser/profile.h"
 #include "chrome/browser/search_engines/template_url_model.h"
 #include "chrome/browser/spellchecker.h"
 #include "chrome/browser/tab_contents/navigation_entry.h"
 #include "chrome/browser/tab_contents/tab_contents.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/platform_util.h"
 #include "chrome/common/pref_service.h"
 #include "chrome/common/url_constants.h"
 #include "grit/generated_resources.h"
+#include "net/base/net_util.h"
 
 #if defined(OS_WIN)
 #include "chrome/browser/views/options/fonts_languages_window_view.h"
 #include "chrome/browser/views/page_info_window.h"
 #endif
 
 RenderViewContextMenu::RenderViewContextMenu(
     TabContents* tab_contents,
     const ContextMenuParams& params)
     : params_(params),
@@ skipping 165 matching lines @@
   AppendSeparator();
   AppendMenuItem(IDS_CONTENT_CONTEXT_SELECTALL);
 }
 
 // Menu delegate functions -----------------------------------------------------
 
 bool RenderViewContextMenu::IsItemCommandEnabled(int id) const {
   // Allow Spell Check language items on sub menu for text area context menu.
   if ((id >= IDC_SPELLCHECK_LANGUAGES_FIRST) &&
       (id < IDC_SPELLCHECK_LANGUAGES_LAST)) {
-    return source_tab_contents_->profile()->GetPrefs()->GetBoolean(
-        prefs::kEnableSpellCheck);
+    return profile_->GetPrefs()->GetBoolean(prefs::kEnableSpellCheck);
   }
 
   switch (id) {
     case IDS_CONTENT_CONTEXT_BACK:
       return source_tab_contents_->controller().CanGoBack();
 
     case IDS_CONTENT_CONTEXT_FORWARD:
       return source_tab_contents_->controller().CanGoForward();
 
     case IDS_CONTENT_CONTEXT_VIEWPAGESOURCE:
@@ skipping 53 matching lines @@
     case IDS_CONTENT_CONTEXT_PASTE:
       return !!(params_.edit_flags & ContextNode::CAN_PASTE);
 
     case IDS_CONTENT_CONTEXT_DELETE:
       return !!(params_.edit_flags & ContextNode::CAN_DELETE);
 
     case IDS_CONTENT_CONTEXT_SELECTALL:
       return !!(params_.edit_flags & ContextNode::CAN_SELECT_ALL);
 
     case IDS_CONTENT_CONTEXT_OPENLINKOFFTHERECORD:
-      return !source_tab_contents_->profile()->IsOffTheRecord() &&
-             params_.link_url.is_valid();
+      return !profile_->IsOffTheRecord() && params_.link_url.is_valid();
 
     case IDS_CONTENT_CONTEXT_OPENFRAMEOFFTHERECORD:
-      return !source_tab_contents_->profile()->IsOffTheRecord() &&
-             params_.frame_url.is_valid();
+      return !profile_->IsOffTheRecord() && params_.frame_url.is_valid();
 
     case IDS_CONTENT_CONTEXT_ADD_TO_DICTIONARY:
       return !params_.misspelled_word.empty();
 
     case IDS_CONTENT_CONTEXT_RELOAD:
     case IDS_CONTENT_CONTEXT_COPYIMAGE:
     case IDS_CONTENT_CONTEXT_PRINT:
     case IDS_CONTENT_CONTEXT_SEARCHWEBFOR:
     case IDC_SPELLCHECK_SUGGESTION_0:
     case IDC_SPELLCHECK_SUGGESTION_1:
     case IDC_SPELLCHECK_SUGGESTION_2:
     case IDC_SPELLCHECK_SUGGESTION_3:
     case IDC_SPELLCHECK_SUGGESTION_4:
     case IDC_SPELLCHECK_MENU:
     case IDS_CONTENT_CONTEXT_LANGUAGE_SETTINGS:
     case IDS_CONTENT_CONTEXT_VIEWFRAMEINFO:
       return true;
 
     case IDC_CHECK_SPELLING_OF_THIS_FIELD:
-      return source_tab_contents_->profile()->GetPrefs()->GetBoolean(
-          prefs::kEnableSpellCheck);
+      return profile_->GetPrefs()->GetBoolean(prefs::kEnableSpellCheck);
 
     case IDS_CONTENT_CONTEXT_SAVEFRAMEAS:
     case IDS_CONTENT_CONTEXT_PRINTFRAME:
     case IDS_CONTENT_CONTEXT_ADDSEARCHENGINE:  // Not implemented.
     default:
       return false;
   }
 }
 
 bool RenderViewContextMenu::ItemIsChecked(int id) const {
   // Check box for 'Check the Spelling of this field'.
   if (id == IDC_CHECK_SPELLING_OF_THIS_FIELD) {
-    PrefService* prefs = source_tab_contents_->profile()->GetPrefs();
     return (params_.spellcheck_enabled &&
-            prefs->GetBoolean(prefs::kEnableSpellCheck));
+            profile_->GetPrefs()->GetBoolean(prefs::kEnableSpellCheck));
   }
 
   // Don't bother getting the display language vector if this isn't a spellcheck
   // language.
   if ((id < IDC_SPELLCHECK_LANGUAGES_FIRST) ||
       (id >= IDC_SPELLCHECK_LANGUAGES_LAST))
     return false;
 
   SpellChecker::Languages languages;
-  return SpellChecker::GetSpellCheckLanguages(
-      source_tab_contents_->profile(), &languages) ==
+  return SpellChecker::GetSpellCheckLanguages(profile_, &languages) ==
       (id - IDC_SPELLCHECK_LANGUAGES_FIRST);
 }
 
 void RenderViewContextMenu::ExecuteItemCommand(int id) {
   // Check to see if one of the spell check language ids have been clicked.
   if (id >= IDC_SPELLCHECK_LANGUAGES_FIRST &&
       id < IDC_SPELLCHECK_LANGUAGES_LAST) {
     const size_t language_number = id - IDC_SPELLCHECK_LANGUAGES_FIRST;
     SpellChecker::Languages languages;
-    SpellChecker::GetSpellCheckLanguages(
-        source_tab_contents_->profile(), &languages);
+    SpellChecker::GetSpellCheckLanguages(profile_, &languages);
     if (language_number < languages.size()) {
       StringPrefMember dictionary_language;
       dictionary_language.Init(prefs::kSpellCheckDictionary,
-          source_tab_contents_->profile()->GetPrefs(), NULL);
+          profile_->GetPrefs(), NULL);
       dictionary_language.SetValue(ASCIIToWide(languages[language_number]));
     }
 
     return;
   }
 
   switch (id) {
     case IDS_CONTENT_CONTEXT_OPENLINKNEWTAB:
       OpenURL(params_.link_url, NEW_BACKGROUND_TAB, PageTransition::LINK);
       break;
 
     case IDS_CONTENT_CONTEXT_OPENLINKNEWWINDOW:
       OpenURL(params_.link_url, NEW_WINDOW, PageTransition::LINK);
       break;
 
     case IDS_CONTENT_CONTEXT_OPENLINKOFFTHERECORD:
       OpenURL(params_.link_url, OFF_THE_RECORD, PageTransition::LINK);
       break;
 
     // TODO(paulg): Prompt the user for file name when saving links and images.
     case IDS_CONTENT_CONTEXT_SAVEIMAGEAS:
     case IDS_CONTENT_CONTEXT_SAVELINKAS: {
       const GURL& referrer =
           params_.frame_url.is_empty() ? params_.page_url : params_.frame_url;
       const GURL& url =
           (id == IDS_CONTENT_CONTEXT_SAVELINKAS ? params_.link_url :
                                                   params_.image_url);
-      DownloadManager* dlm =
-          source_tab_contents_->profile()->GetDownloadManager();
+      DownloadManager* dlm = profile_->GetDownloadManager();
       dlm->DownloadUrl(url, referrer, params_.frame_charset,
                        source_tab_contents_);
       break;
     }
 
     case IDS_CONTENT_CONTEXT_COPYLINKLOCATION:
       WriteURLToClipboard(params_.unfiltered_link_url);
       break;
 
     case IDS_CONTENT_CONTEXT_COPYIMAGELOCATION:
@@ skipping 35 matching lines @@
 
     case IDS_CONTENT_CONTEXT_INSPECTELEMENT:
       Inspect(params_.x, params_.y);
       break;
 
     case IDS_CONTENT_CONTEXT_VIEWPAGEINFO: {
 #if defined(OS_WIN)
       NavigationEntry* nav_entry =
           source_tab_contents_->controller().GetActiveEntry();
       PageInfoWindow::CreatePageInfo(
-          source_tab_contents_->profile(),
+          profile_,
           nav_entry,
           source_tab_contents_->GetContentNativeView(),
           PageInfoWindow::SECURITY);
 #else
      // TODO(port): port PageInfoWindow.
      NOTIMPLEMENTED() << "IDS_CONTENT_CONTEXT_VIEWPAGEINFO";
 #endif
       break;
     }
 
@@ skipping 32 matching lines @@
         SSLManager::DeserializeSecurityInfo(params_.security_info,
                                             &cert_id,
                                             &cert_status,
                                             &security_bits);
         ssl.set_cert_id(cert_id);
         ssl.set_cert_status(cert_status);
         ssl.set_security_bits(security_bits);
       }
 #if defined(OS_WIN)
       PageInfoWindow::CreateFrameInfo(
-          source_tab_contents_->profile(),
+          profile_,
           params_.frame_url,
           ssl,
           source_tab_contents_->GetContentNativeView(),
           PageInfoWindow::SECURITY);
 #else
       // TODO(port): port PageInfoWindow.
       NOTIMPLEMENTED() << "IDS_CONTENT_CONTEXT_VIEWFRAMEINFO";
 #endif
       break;
     }
@@ skipping 20 matching lines @@
 
     case IDS_CONTENT_CONTEXT_DELETE:
       source_tab_contents_->render_view_host()->Delete();
       break;
 
     case IDS_CONTENT_CONTEXT_SELECTALL:
       source_tab_contents_->render_view_host()->SelectAll();
       break;
 
     case IDS_CONTENT_CONTEXT_SEARCHWEBFOR: {
-      const TemplateURL* const default_provider = source_tab_contents_->
-          profile()->GetTemplateURLModel()->GetDefaultSearchProvider();
+      const TemplateURL* const default_provider =
+          profile_->GetTemplateURLModel()->GetDefaultSearchProvider();
       DCHECK(default_provider);  // The context menu should not contain this
                                  // item when there is no provider.
       const TemplateURLRef* const search_url = default_provider->url();
       DCHECK(search_url->SupportsReplacement());
       OpenURL(GURL(search_url->ReplaceSearchTerms(*default_provider,
           params_.selection_text, TemplateURLRef::NO_SUGGESTIONS_AVAILABLE,
           std::wstring())), NEW_FOREGROUND_TAB, PageTransition::GENERATED);
       break;
     }
 
@@ skipping 12 matching lines @@
     case IDS_CONTENT_CONTEXT_ADD_TO_DICTIONARY:
       source_tab_contents_->render_view_host()->AddToDictionary(
           params_.misspelled_word);
       break;
 
     case IDS_CONTENT_CONTEXT_LANGUAGE_SETTINGS: {
 #if defined(OS_WIN)
       // TODO(yusukes): This should be moved to some shared place of commands
       // for the options stuff so that we don't have to do all this work here.
       FontsLanguagesWindowView* window_ = new FontsLanguagesWindowView(
-          source_tab_contents_->profile());
+          profile_);
       views::Window::CreateChromeWindow(
           platform_util::GetTopLevel(
               source_tab_contents_->GetContentNativeView()),
           gfx::Rect(), window_)->Show();
       window_->SelectLanguagesTab();
 #else
       // TODO(port): need views::Window
       NOTIMPLEMENTED() << "IDS_CONTENT_CONTEXT_LANGUAGE_SETTINGS";
 #endif
       break;
@@ skipping 29 matching lines @@
 
   // Don't inspect about:network, about:memory, etc.
   // However, we do want to inspect about:blank, which is often
   // used by ordinary web pages.
   if (active_entry->display_url().SchemeIs(chrome::kAboutScheme) &&
       !LowerCaseEqualsASCII(active_entry->display_url().path(), "blank"))
     return false;
 
   // Don't enable the web inspector if JavaScript is disabled
   if (id == IDS_CONTENT_CONTEXT_INSPECTELEMENT) {
-    PrefService* prefs = source_tab_contents_->profile()->GetPrefs();
-    if (!prefs->GetBoolean(prefs::kWebKitJavascriptEnabled) ||
+    if (!profile_->GetPrefs()->GetBoolean(prefs::kWebKitJavascriptEnabled) ||
         command_line.HasSwitch(switches::kDisableJavaScript))
       return false;
   }
 
   return true;
 }
 
 // Controller functions --------------------------------------------------------
 
 void RenderViewContextMenu::OpenURL(
@@ skipping 16 matching lines @@
   Clipboard* clipboard = g_browser_process->clipboard();
 
   if (!clipboard)
     return;
 
   ScopedClipboardWriter scw(clipboard);
   scw.WriteText(text);
 }
 
 void RenderViewContextMenu::WriteURLToClipboard(const GURL& url) {
-  if (url.SchemeIs(chrome::kMailToScheme))
+  if (url.SchemeIs(chrome::kMailToScheme)) {
     WriteTextToClipboard(UTF8ToUTF16(url.path()));
-  else
-    WriteTextToClipboard(UTF8ToUTF16(url.spec()));
+  } else {
+    std::wstring languages =
+        profile_->GetPrefs()->GetString(prefs::kAcceptLanguages);
+    // Unescaping path and query is not a good idea because other
+    // applications may not enocode non-ASCII characters in UTF-8.
+    // So the 4th parameter of net::FormatUrl() should be false.
+    // See crbug.com/2820.
+    WriteTextToClipboard(WideToUTF16(
+        net::FormatUrl(url, languages, false, false, NULL, NULL)));
+  }
 }