Only skip certain hidden elements at MHTML serialization

Only skip certain hidden elements at MHTML serialization
1) Elements with hidden attributes
2) Hidden form elements

We orginally tried to act more aggresively by skipping all possible
hidden elements. But this does break certain web sites that uses
nth CSS selectors.

BUG=697970
TEST=tests updated
TBR=svaldez@chromium.org

Review-Url: https://codereview.chromium.org/2819723003
Cr-Commit-Position: refs/heads/master@{#464856}
Committed: https://chromium.googlesource.com/chromium/src/+/2ddf990431d6b7977a94e5da69f18a1a3d220a52

[jianli, 2017-04-14 01:01:09]

Description was changed from

==========
Only skip certain hidden elements
1) Elements with hidden attributes
2) Hidden form elements

We orginally tried to act more aggresively by skipping all possible
hidden elements. But this does break certain web sites that uses
nth CSS selectors.

BUG=697970
TEST=tests updated
==========

to

==========
Only skip certain hidden elements at MHTML serialization
1) Elements with hidden attributes
2) Hidden form elements

We orginally tried to act more aggresively by skipping all possible
hidden elements. But this does break certain web sites that uses
nth CSS selectors.

BUG=697970
TEST=tests updated
==========

[jianli, 2017-04-14 01:01:25]

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

[commit-bot: I haz the power, 2017-04-14 01:01:48]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jianli, 2017-04-14 01:01:51]

jianli@chromium.org changed reviewers:
+ tkent@chromium.org

[tkent, 2017-04-14 01:06:30]

lgtm

[commit-bot: I haz the power, 2017-04-14 02:03:48]

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

[commit-bot: I haz the power, 2017-04-14 02:03:49]

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

[jianli, 2017-04-14 21:03:24]

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

[commit-bot: I haz the power, 2017-04-14 21:03:53]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-14 21:52:38]

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

[commit-bot: I haz the power, 2017-04-14 21:52:40]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[jianli, 2017-04-14 21:53:54]

The CQ bit was checked by jianli@chromium.org

[jianli, 2017-04-14 21:53:54]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/2819723003/#ps20001
(title: "Fix test")

[commit-bot: I haz the power, 2017-04-14 21:54:20]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jianli, 2017-04-14 22:35:52]

The CQ bit was unchecked by jianli@chromium.org

[jianli, 2017-04-14 23:56:29]

The CQ bit was checked by jianli@chromium.org

[jianli, 2017-04-14 23:56:30]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/2819723003/#ps40001
(title: "Fix one more test")

[jianli, 2017-04-14 23:56:50]

The CQ bit was unchecked by jianli@chromium.org

[jianli, 2017-04-14 23:57:40]

Description was changed from

==========
Only skip certain hidden elements at MHTML serialization
1) Elements with hidden attributes
2) Hidden form elements

We orginally tried to act more aggresively by skipping all possible
hidden elements. But this does break certain web sites that uses
nth CSS selectors.

BUG=697970
TEST=tests updated
==========

to

==========
Only skip certain hidden elements at MHTML serialization
1) Elements with hidden attributes
2) Hidden form elements

We orginally tried to act more aggresively by skipping all possible
hidden elements. But this does break certain web sites that uses
nth CSS selectors.

BUG=697970
TEST=tests updated
TBR=svaldez@chromium.org
==========

[jianli, 2017-04-14 23:58:37]

jianli@chromium.org changed reviewers:
+ svaldez@chromium.org

[jianli, 2017-04-14 23:58:37]

TBR svaldez for trivial test change

[jianli, 2017-04-14 23:58:43]

The CQ bit was checked by jianli@chromium.org

[commit-bot: I haz the power, 2017-04-14 23:59:07]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-15 03:11:38]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1492214323296350,
"parent_rev": "fb546e9c25224ccf4a99b9f1c3f588f57d8cef92", "commit_rev":
"2ddf990431d6b7977a94e5da69f18a1a3d220a52"}

[commit-bot: I haz the power, 2017-04-15 03:13:12]

Description was changed from

==========
Only skip certain hidden elements at MHTML serialization
1) Elements with hidden attributes
2) Hidden form elements

We orginally tried to act more aggresively by skipping all possible
hidden elements. But this does break certain web sites that uses
nth CSS selectors.

BUG=697970
TEST=tests updated
TBR=svaldez@chromium.org
==========

to

==========
Only skip certain hidden elements at MHTML serialization
1) Elements with hidden attributes
2) Hidden form elements

We orginally tried to act more aggresively by skipping all possible
hidden elements. But this does break certain web sites that uses
nth CSS selectors.

BUG=697970
TEST=tests updated
TBR=svaldez@chromium.org

Review-Url: https://codereview.chromium.org/2819723003
Cr-Commit-Position: refs/heads/master@{#464856}
Committed:
https://chromium.googlesource.com/chromium/src/+/2ddf990431d6b7977a94e5da69f1...
==========

[commit-bot: I haz the power, 2017-04-15 03:13:14]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/2ddf990431d6b7977a94e5da69f1...