[autofill] When hiding popups in the renderer also hide those in the browser.

[autofill] When hiding popups in the renderer also hide those in the browser.

BUG=157661
R=isherman@chromium.org

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=166147

[Ilya Sherman, 2012-10-29 23:14:19]

LGTM, thanks :)

https://chromiumcodereview.appspot.com/11273096/diff/1/chrome/renderer/autofi...
File chrome/renderer/autofill/autofill_agent.cc (right):

https://chromiumcodereview.appspot.com/11273096/diff/1/chrome/renderer/autofi...
chrome/renderer/autofill/autofill_agent.cc:739: }
nit: I think this should also call HideHostPopups(), rather than requiring the
caller to call both in sequence.

[Dan Beam, 2012-10-30 00:44:15]

https://chromiumcodereview.appspot.com/11273096/diff/1/chrome/renderer/autofi...
File chrome/renderer/autofill/autofill_agent.cc (right):

https://chromiumcodereview.appspot.com/11273096/diff/1/chrome/renderer/autofi...
chrome/renderer/autofill/autofill_agent.cc:739: }
On 2012/10/29 23:14:19, Ilya Sherman wrote:
> nit: I think this should also call HideHostPopups(), rather than requiring the
> caller to call both in sequence.

Done.

https://chromiumcodereview.appspot.com/11273096/diff/3001/chrome/renderer/aut...
File chrome/renderer/autofill/autofill_agent.cc (left):

https://chromiumcodereview.appspot.com/11273096/diff/3001/chrome/renderer/aut...
chrome/renderer/autofill/autofill_agent.cc:497: 
tell me if there's any problem with this ever-so-slight logical change

[Ilya Sherman, 2012-10-30 00:47:43]

LGTM

[commit-bot: I haz the power, 2012-11-06 01:51:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dbeam@chromium.org/11273096/7001

[commit-bot: I haz the power, 2012-11-06 04:40:11]

Change committed as 166147