Updates:
        Status: Assigned
        Owner: [email protected]
        Labels: -OS-All OS-Win

Comment #3 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

Looks like this is related to the exceptions dialog. The design here is  
just wrong.

PasswordManagerExceptionsView has-a PasswordManagerExceptionsTableModel  
which isa-
PasswordManagerTableModel... however PasswordManagerTableModel expects to  
have a
instance_ member which is a PasswordManagerView... not a
PasswordManagerExceptionsView!

Also, there's a static global |instance_| in  
password_manager_exceptions_view - this
should be static to the class, not the file. The naming is wrong per  
c-style guide.

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