Make test_runner::LayoutDumpFlags an internal detail of components/test_runner.

Make test_runner::LayoutDumpFlags an internal detail of components/test_runner.

To simplify public API surface of components/test_runner we can hide
LayoutDumpFlags behind WebTestRunner interface.  For example - instead
of returning the whole LayoutDumpFlags struct, we can only expose
IsRecursiveLayoutDumpRequested method - this is consistent with how
other similar methods are exposed via WebTestRunner (i.e. we already
have ShouldDumpAsAudio and/or ShouldGeneratePixelResults).

After doing this components/test_runner/layout_dump_flags.h and
components/test_runner/layout_dump.h headers no longer needs to be
included outside of components/test_runner.

BUG=595089
Committed: https://crrev.com/ef264fc5601239984a67e3a39580b6a7855142b8
Cr-Commit-Position: refs/heads/master@{#381819}

[Łukasz Anforowicz, 2016-03-16 00:16:39]

lukasza@chromium.org changed reviewers:
+ jochen@chromium.org

[Łukasz Anforowicz, 2016-03-16 00:16:39]

jochen@, can you take a look please?

[jochen (gone - plz use gerrit), 2016-03-17 19:51:43]

lgtm

[Łukasz Anforowicz, 2016-03-17 20:58:32]

The CQ bit was checked by lukasza@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-03-17 20:59:07]

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

[commit-bot: I haz the power, 2016-03-17 22:04:30]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-17 22:04:31]

Dry run: This issue passed the CQ dry run.

[Łukasz Anforowicz, 2016-03-17 22:41:55]

The CQ bit was checked by lukasza@chromium.org

[commit-bot: I haz the power, 2016-03-17 22:42:27]

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

[commit-bot: I haz the power, 2016-03-17 22:49:28]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-17 22:51:33]

Description was changed from

==========
Make test_runner::LayoutDumpFlags an internal detail of components/test_runner.

To simplify public API surface of components/test_runner we can hide
LayoutDumpFlags behind WebTestRunner interface.  For example - instead
of returning the whole LayoutDumpFlags struct, we can only expose
IsRecursiveLayoutDumpRequested method - this is consistent with how
other similar methods are exposed via WebTestRunner (i.e. we already
have ShouldDumpAsAudio and/or ShouldGeneratePixelResults).

After doing this components/test_runner/layout_dump_flags.h and
components/test_runner/layout_dump.h headers no longer needs to be
included outside of components/test_runner.

BUG=595089
==========

to

==========
Make test_runner::LayoutDumpFlags an internal detail of components/test_runner.

To simplify public API surface of components/test_runner we can hide
LayoutDumpFlags behind WebTestRunner interface.  For example - instead
of returning the whole LayoutDumpFlags struct, we can only expose
IsRecursiveLayoutDumpRequested method - this is consistent with how
other similar methods are exposed via WebTestRunner (i.e. we already
have ShouldDumpAsAudio and/or ShouldGeneratePixelResults).

After doing this components/test_runner/layout_dump_flags.h and
components/test_runner/layout_dump.h headers no longer needs to be
included outside of components/test_runner.

BUG=595089

Committed: https://crrev.com/ef264fc5601239984a67e3a39580b6a7855142b8
Cr-Commit-Position: refs/heads/master@{#381819}
==========

[commit-bot: I haz the power, 2016-03-17 22:51:34]

Patchset 1 (id:??) landed as
https://crrev.com/ef264fc5601239984a67e3a39580b6a7855142b8
Cr-Commit-Position: refs/heads/master@{#381819}