Comment #10 on issue 1978 by [email protected]: Replace 'Search Google
for "some text"' with 'Go to "some text"' in context menu when "some text"
is a link.
http://code.google.com/p/chromium/issues/detail?id=1978
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=30745
------------------------------------------------------------------------
r30745 | [email protected] | 2009-11-02 13:40:11 -0800 (Mon, 02 Nov
2009) | 7 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/tab_contents/render_view_context_menu.cc?r1=30745&r2=30744
Fix a problem in the implementation of search-vs.-go-in-context-menus. We
were eliding the text before passing it to the classifier, resulting in
wrong destination URLs for elided strings.
This also eliminates a couple string conversions and shortens the
EscapeAmpersand() algorithm.
BUG=1978
TEST=Highlight a long string in a page. Right-click it. Verify the string
is elided in the context menu. Click the search entry. Verify the string
you searched for is not elided.
Review URL: http://codereview.chromium.org/343081
------------------------------------------------------------------------
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---
