Merge page serializers [6/12]

Move css resources test

Move WebPageNewSerializeTest.CSSResources (currently as FAILS_*) to
PageSerializerTest and fixing it, removing FAILS_ prefix.

--

This review is part of set aiming to merge the two existing page
serializers, WebPageSerializerImpl and PageSerializer into the latter
one.

In addition to this it moves all the old tests from
WebPageNewSerializerTest and WebPageSerializerTest to the newer
PageSerializerTest structure and splits out a test for MHTML into the
MHTMLTest file.

Note: This is a rebase and split of
https://codereview.chromium.org/68613003 from Opera which was reverted
due to failing tests. In addition to the rebase some smaller fixes was
done to accommodate to changes in the code from when it was first
landed.

BUG=328354
R=hajimehoshi@chromium.org

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=196875

[Tiger (Sony Mobile), 2015-06-10 08:37:58]

PTAL

[hajimehoshi, 2015-06-10 08:51:12]

On 2015/06/10 08:37:58, Tiger (Sony Mobile) wrote:
> PTAL

CSS and image files seem to be just copied. Should we remove old ones?

[Tiger (Sony Mobile), 2015-06-10 08:54:11]

On 2015/06/10 08:51:12, hajimehoshi wrote:
> On 2015/06/10 08:37:58, Tiger (Sony Mobile) wrote:
> > PTAL
> 
> CSS and image files seem to be just copied. Should we remove old ones?

Yes correct. They are still used in the MHTML test in that file, so they will be
removed when that test is step 7 (see
https://codereview.chromium.org/1172993002/).

[hajimehoshi, 2015-06-10 08:57:04]

On 2015/06/10 08:54:11, Tiger (Sony Mobile) wrote:
> On 2015/06/10 08:51:12, hajimehoshi wrote:
> > On 2015/06/10 08:37:58, Tiger (Sony Mobile) wrote:
> > > PTAL
> > 
> > CSS and image files seem to be just copied. Should we remove old ones?
> 
> Yes correct. They are still used in the MHTML test in that file, so they will
be
> removed when that test is step 7 (see
> https://codereview.chromium.org/1172993002/).

lgtm

[philipj_slow, 2015-06-10 11:35:36]

philipj@opera.com changed reviewers:
+ philipj@opera.com

[philipj_slow, 2015-06-10 11:35:36]

The CQ bit was checked by philipj@opera.com

[philipj_slow, 2015-06-10 11:35:36]

lgtm

[commit-bot: I haz the power, 2015-06-10 11:35:54]

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

[philipj_slow, 2015-06-10 11:36:40]

The CQ bit was unchecked by philipj@opera.com

[philipj_slow, 2015-06-10 11:36:51]

Sorry, CQ'd too early.

[Tiger (Sony Mobile), 2015-06-10 14:01:38]

The CQ bit was checked by gustav.tiger@sonymobile.com

[commit-bot: I haz the power, 2015-06-10 14:02:01]

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

[commit-bot: I haz the power, 2015-06-10 15:09:32]

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