On 2012/11/08 06:11:26, Ilya Sherman wrote:
> nit: It looks like the other tests also include "EXPECT_CALL(delegate_,
> FillPasswordForm(_));" -- perhaps include that here as well?

Agreed -- it should make the unit tests look more consistent.
I'll need to extend the unit test a bit then because the expected outcome of
FillPasswordForm() depends on the password store's content. Should be fairly
simple though.