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
-~----------~----~----~----~------~----~------~--~---

Reply via email to