Enable and fix tests after page serializer merge

Enable and fix tests after page serializer merge

Enable and fix two SavePageBrowserTests and two BrowserEncodingTest that
where removed in <https://codereview.chromium.org/1218743006/>; in
preparation of the page serializer merger in
<https://codereview.chromium.org/1177733003/>;.

Patch from gustav.tiger@sonymobile.com

R=jochen@chromium.org, rdsmith@chromium.org
BUG=328354
Committed: https://chromium.googlesource.com/chromium/src/+/bbf281076da0c593ec9d586eb78b2498581d070e

[Tiger (Sony Mobile), 2015-07-08 09:37:34]

PTAL

[Randy Smith (Not in Mondays), 2015-07-08 13:17:54]

LGTM.

[Tiger (Sony Mobile), 2015-07-08 21:02:43]

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

[commit-bot: I haz the power, 2015-07-08 21:04:03]

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

[commit-bot: I haz the power, 2015-07-08 21:09:11]

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

[commit-bot: I haz the power, 2015-07-08 21:09:12]

Try jobs failed on following builders:
  linux_chromium_compile_dbg_32_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)
  win_chromium_compile_dbg_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)
  win_chromium_x64_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

(exceeded global retry quota)

[Tiger (Sony Mobile), 2015-07-09 09:38:16]

Does this need to be committed manually? I can download the patch and build it
fine on my machine. I'll retry the CQ here but if it doesn't work, does anyone
have any ideas?

[Tiger (Sony Mobile), 2015-07-09 09:38:25]

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

[commit-bot: I haz the power, 2015-07-09 09:38:51]

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

[commit-bot: I haz the power, 2015-07-09 09:40:48]

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

[commit-bot: I haz the power, 2015-07-09 09:40:48]

Try jobs failed on following builders:
  linux_chromium_compile_dbg_32_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

(exceeded global retry quota)

[Tiger (Sony Mobile), 2015-07-09 12:38:12]

gustav.tiger@sonymobile.com changed reviewers:
+ jochen@chromium.org

[Tiger (Sony Mobile), 2015-07-09 12:39:10]

jochen: can you take a look at this as well?

[Randy Smith (Not in Mondays), 2015-07-09 13:10:22]

Usually a "patch failed" error on the bots means that there's some merge problem
between your checkout and the top of tree, and you fix it by merging the
checkout to ToT and uploading a new patch set.  Have you tried that?

(Jochen's not going to be available today, according to internal information
flows.)

[jochen (gone - plz use gerrit), 2015-07-10 08:56:24]

lgtm

[Mattias Nissler (ping if slow), 2015-07-10 10:05:31]

Committed patchset #1 (id:1) manually as
bbf281076da0c593ec9d586eb78b2498581d070e (presubmit successful).

[Alexander Alekseev, 2015-07-27 20:45:23]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1255223003/ by alemate@chromium.org.

The reason for reverting is: BrowserEncodingTest.TestEncodingAutoDetect is dead
since this CL, so cros_trunk bot is constantly red.

See
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=...
.
.

[tkent, 2015-08-17 03:13:23]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1301503003/ by tkent@chromium.org.

The reason for reverting is: Needs to revert Blink 198503, which this depends
on.
.