Comment #10 on issue 8345 by [email protected]: Browser crash
on "Remove/Remove All' of Password's Exceptions !!
http://code.google.com/p/chromium/issues/detail?id=8345
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=11553
------------------------------------------------------------------------
r11553 | [email protected] | 2009-03-12 11:14:42 -0700 (Thu, 12 Mar
2009) | 10 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/branches/169/src/chrome/browser/views/password_manager_exceptions_view.cc?r1=11553&r2=11552
M
http://src.chromium.org/viewvc/chrome/branches/169/src/chrome/browser/views/password_manager_exceptions_view.h?r1=11553&r2=11552
M
http://src.chromium.org/viewvc/chrome/branches/169/src/chrome/browser/views/password_manager_view.cc?r1=11553&r2=11552
M
http://src.chromium.org/viewvc/chrome/branches/169/src/chrome/browser/views/password_manager_view.h?r1=11553&r2=11552
Merge 11279 - Added a PasswordManagerTableModelObserver to listen for row
count change events.
PasswordManagerView and PasswordManagerExceptionsView are listening to the
event.
Move the |instance_| variable from static global to the respective class to
avoid future misuse of the variable.
Review URL: http://codereview.chromium.org/39313
BUG=8345
[email protected]
Review URL: http://codereview.chromium.org/43147
------------------------------------------------------------------------
--
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
-~----------~----~----~----~------~----~------~--~---
