Flush form cache when next document is loaded to avoid Document leak.

Flush form cache when next document is loaded to avoid Document leak.

This should fix a document leak from r270279.

BUG=392121
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=282524

[kouhei (in TOK), 2014-07-09 01:10:40]

Would you take a look?

[kouhei (in TOK), 2014-07-09 02:31:01]

Looking at bot failures. I think their baseline is wrong (testing that there are
2 docs when loading a tab w/o iframes), but checking git log just in case.

[please use gerrit instead, 2014-07-09 02:41:36]

On 2014/07/09 02:31:01, kouhei wrote:
> Looking at bot failures. I think their baseline is wrong (testing that there
are
> 2 docs when loading a tab w/o iframes), but checking git log just in case.

You can pass -b 〈revision〉 to the try command to specify the baseline.

[kouhei (in TOK), 2014-07-09 06:28:33]

I think we can rebaseline the DOM count, we fixed a leak in this CL so the
expected document count should be 1.
The dbg failures are to be fixed in https://codereview.chromium.org/380603002/

On 2014/07/09 02:41:36, Rouslan Solomakhin wrote:
> On 2014/07/09 02:31:01, kouhei wrote:
> > Looking at bot failures. I think their baseline is wrong (testing that there
> are
> > 2 docs when loading a tab w/o iframes), but checking git log just in case.
> 
> You can pass -b 〈revision〉 to the try command to specify the baseline.

Thanks for the tip!

[Ilya Sherman, 2014-07-09 06:40:09]

Autofill changes LGTM.  Thanks!

[kouhei (in TOK), 2014-07-09 06:41:45]

Thanks!

nduca: Would you take a look at telemetry rebaseline?

[nduca, 2014-07-10 18:01:43]

telemetry lgtm

[kouhei (in TOK), 2014-07-11 00:20:56]

The CQ bit was checked by kouhei@chromium.org

[commit-bot: I haz the power, 2014-07-11 00:22:08]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/kouhei@chromium.org/380533003/20001

[commit-bot: I haz the power, 2014-07-11 03:18:31]

Change committed as 282524