Comment #6 on issue 21029 by [email protected]: Plugin name in crashed
plug-in info bar has .plugin appended. Ugly.
http://code.google.com/p/chromium/issues/detail?id=21029
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=25854
------------------------------------------------------------------------
r25854 | [email protected] | 2009-09-10 00:49:03 -0700 (Thu, 10 Sep 2009) |
13 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/branches/195/src/base/string_util.cc?r1=25854&r2=25853
M
http://src.chromium.org/viewvc/chrome/branches/195/src/base/string_util.h?r1=25854&r2=25853
M
http://src.chromium.org/viewvc/chrome/branches/195/src/base/string_util_unittest.cc?r1=25854&r2=25853
M
http://src.chromium.org/viewvc/chrome/branches/195/src/chrome/browser/tab_contents/tab_contents.cc?r1=25854&r2=25853
Merge 25487 - Strip .plugin off of Mac plugin names when showing the crash
info bar.
** This fixes the build break. Merging r25834 introduced a dependency on
EndsWith.
** This change only adds code, and we know that the only caller is the code
in r25834.
BUG=21029
TEST=Kill a plugin process; the plugin crash info bar shouldn't
have ".plugin" in the plugin name.
Review URL: http://codereview.chromium.org/197018
[email protected], [email protected]
Review URL: http://codereview.chromium.org/192065
------------------------------------------------------------------------
--
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
-~----------~----~----~----~------~----~------~--~---