Set page height for printing tests.

Set page height for printing tests.

Without this we'd fail to properly re-lay out the document when switching to
print layout, because LayoutView::pageLogicalHeight() would be 0, which
typically makes everyone believe that we're not paginated.

Added a layout tree dump test for this (found no other way of testing this).
Also have to rebaseline two existing tests, because the layout tree dump
changes. The pixel results are unaffected, though.

BUG=544786
R=leviw@chromium.org

Committed: https://crrev.com/4ae57a7213e21f6fd32b81d619986a212470033a
Cr-Commit-Position: refs/heads/master@{#355526}

[mstensho (USE GERRIT), 2015-10-21 15:04:43]

Looks like I need to rebaseline more tests, but in the meantime, Levi, are you
ready for some printing / testing craziness? :)

[leviw_travelin_and_unemployed, 2015-10-21 15:41:07]

Yeah, looks like there's at least one more test to rebase, but the change makes
sense.

LGTM.

[mstensho (USE GERRIT), 2015-10-21 18:36:44]

The CQ bit was checked by mstensho@opera.com

[mstensho (USE GERRIT), 2015-10-21 18:36:44]

The patchset sent to the CQ was uploaded after l-g-t-m from leviw@chromium.org
Link to the patchset: https://codereview.chromium.org/1413573004/#ps20001
(title: "Rebaseline ALL THE THINGS!")

[commit-bot: I haz the power, 2015-10-21 18:36:57]

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

[commit-bot: I haz the power, 2015-10-21 19:31:19]

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

[commit-bot: I haz the power, 2015-10-21 19:31:20]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[mstensho (USE GERRIT), 2015-10-21 19:35:09]

Thanks for reviewing, Levi.

Looks like I have another problem here. See e.g.
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...

webkit_lint complains because TestExpectations already contained Windows
10-specific entries for the tests that I now want to rebaseline.

Any idea what to do?

[leviw_travelin_and_unemployed, 2015-10-21 21:29:50]

On 2015/10/21 at 19:35:09, mstensho wrote:
> Thanks for reviewing, Levi.
> 
> Looks like I have another problem here. See e.g.
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...
> 
> webkit_lint complains because TestExpectations already contained Windows
10-specific entries for the tests that I now want to rebaseline.
> 
> Any idea what to do?

You have a couple options. The easiest is probably just to land with
"NeedsManualRebaseline" and do a follow-up with the new results. You may need to
comment the Win10 line out when you do it.

If you're not familiar, webkit-patch has a rebaseline mode that will grab the
results from the bots. You could also download them directly from the trybots,
but that's a pain in the ass so I wouldn't bother.

[mstensho (USE GERRIT), 2015-10-22 10:49:41]

The CQ bit was checked by mstensho@opera.com

[mstensho (USE GERRIT), 2015-10-22 10:49:41]

The patchset sent to the CQ was uploaded after l-g-t-m from leviw@chromium.org
Link to the patchset: https://codereview.chromium.org/1413573004/#ps80001
(title: "Back out patch set 4 and mark tests with NeedsManualRebaseline.")

[commit-bot: I haz the power, 2015-10-22 10:49:54]

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

[commit-bot: I haz the power, 2015-10-22 11:40:35]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2015-10-22 11:41:44]

Patchset 5 (id:??) landed as
https://crrev.com/4ae57a7213e21f6fd32b81d619986a212470033a
Cr-Commit-Position: refs/heads/master@{#355526}

[mstensho (USE GERRIT), 2015-10-22 19:35:08]

On 2015/10/21 21:29:50, leviw wrote:
> On 2015/10/21 at 19:35:09, mstensho wrote:
> > Looks like I have another problem here. See e.g.
>
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...
> > 
> > webkit_lint complains because TestExpectations already contained Windows
> 10-specific entries for the tests that I now want to rebaseline.
> > 
> > Any idea what to do?
> 
> You have a couple options. The easiest is probably just to land with
> "NeedsManualRebaseline" and do a follow-up with the new results. You may need
to
> comment the Win10 line out when you do it.
> 
> If you're not familiar, webkit-patch has a rebaseline mode that will grab the
> results from the bots. You could also download them directly from the trybots,
> but that's a pain in the ass so I wouldn't bother.

Thanks a lot for the info. This was fun. :) No, I didn't know about webkit-patch
and its ability to rebaseline, but it was a most enjoyable experience. OK, I'm
exaggerating, but thanks!