Perform initial GC to improve inspector/search test stability.

Perform initial GC to improve inspector/search test stability.

With Oilpan, inspector tests enumerating resources are susceptible to
"GC lag", including resources that have yet to be GCed and swept out.

To ensure a stable baseline for the tests, perform an initial,
flushing GC.

R=haraken
BUG=513393
Committed: https://crrev.com/1bedea1cfe11f4c8c2a6b7ea2a60794d943c853c
git-svn-id: svn://svn.chromium.org/blink/trunk@199665 bbb929c8-8fbe-4397-9dbb-9b2b20218538

[sof, 2015-07-29 13:41:39]

sigbjornf@opera.com changed reviewers:
+ lushnikov@chromium.org, oilpan-reviews@chromium.org, pfeldman@chromium.org

[sof, 2015-07-29 13:41:39]

please take a look.

[haraken, 2015-07-29 13:43:23]

haraken@chromium.org changed reviewers:
+ haraken@chromium.org

[haraken, 2015-07-29 13:43:23]

LGTM

https://codereview.chromium.org/1254293008/diff/1/LayoutTests/http/tests/insp...
File LayoutTests/http/tests/inspector/search/search-test.js (right):

https://codereview.chromium.org/1254293008/diff/1/LayoutTests/http/tests/insp...
LayoutTests/http/tests/inspector/search/search-test.js:4:
GCController.collect();

Shall we use GCController.collectAll()?

[sof, 2015-07-29 13:43:57]

https://codereview.chromium.org/1254293008/diff/1/LayoutTests/http/tests/insp...
File LayoutTests/http/tests/inspector/search/search-test.js (right):

https://codereview.chromium.org/1254293008/diff/1/LayoutTests/http/tests/insp...
LayoutTests/http/tests/inspector/search/search-test.js:4:
GCController.collect();
On 2015/07/29 13:43:23, haraken wrote:
> 
> Shall we use GCController.collectAll()?

I've repeatedly tested, and didn't see a need for it.

[haraken, 2015-07-29 13:52:02]

On 2015/07/29 13:43:57, sof wrote:
>
https://codereview.chromium.org/1254293008/diff/1/LayoutTests/http/tests/insp...
> File LayoutTests/http/tests/inspector/search/search-test.js (right):
> 
>
https://codereview.chromium.org/1254293008/diff/1/LayoutTests/http/tests/insp...
> LayoutTests/http/tests/inspector/search/search-test.js:4:
> GCController.collect();
> On 2015/07/29 13:43:23, haraken wrote:
> > 
> > Shall we use GCController.collectAll()?
> 
> I've repeatedly tested, and didn't see a need for it.

OK, but collectAll() would be safer :)

[sof, 2015-07-29 14:44:43]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2015-07-29 14:44:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1254293008/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1254293008/1

[commit-bot: I haz the power, 2015-07-29 14:47:59]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=199665

[commit-bot: I haz the power, 2015-09-23 11:53:46]

Patchset 1 (id:??) landed as
https://crrev.com/1bedea1cfe11f4c8c2a6b7ea2a60794d943c853c