Fix more partially hidden articles

Fix more partially hidden articles

When traversing the DOM tree, invisible elements are ignored. In order
to correctly handle partially hidden articles on sites having the
"continue reading" button, if the class name contains "continue", it
would still be processed.

BUG=687071
R=mdjones@chromium.org

Committed: 6bd2ebbcce6ecfb349c812adcf15ef047aef54a4

[wychen, 2017-02-01 05:02:22]

wychen@chromium.org changed reviewers:
+ mdjones@chromium.org

[wychen, 2017-02-01 05:02:23]

PTAL

[mdjones, 2017-02-01 21:44:54]

lgtm % update description to include something about white listing "continue"

[wychen, 2017-02-02 09:49:19]

Description was changed from

==========
Fix more partially hidden article

BUG=687071
==========

to

==========
Fix more partially hidden article

When traversing the DOM tree, invisible elements are ignored. In order to
correctly handle partially hidden articles on sites having the "continue
reading"
button, if the class name contains "continue", it would still be processed.

BUG=687071
==========

[wychen, 2017-02-02 09:52:58]

Description was changed from

==========
Fix more partially hidden article

When traversing the DOM tree, invisible elements are ignored. In order to
correctly handle partially hidden articles on sites having the "continue
reading"
button, if the class name contains "continue", it would still be processed.

BUG=687071
==========

to

==========
Fix more partially hidden article

When traversing the DOM tree, invisible elements are ignored. In order
to correctly handle partially hidden articles on sites having the
"continue reading" button, if the class name contains "continue", it
would still be processed.

BUG=687071
==========

[wychen, 2017-02-02 16:38:54]

Changed the logic to not be blocked by
https://bugs.chromium.org/p/chromium/issues/detail?id=645690.

[mdjones, 2017-02-02 17:10:53]

still lgtm

[wychen, 2017-02-02 17:34:42]

Description was changed from

==========
Fix more partially hidden article

When traversing the DOM tree, invisible elements are ignored. In order
to correctly handle partially hidden articles on sites having the
"continue reading" button, if the class name contains "continue", it
would still be processed.

BUG=687071
==========

to

==========
Fix more partially hidden articles

When traversing the DOM tree, invisible elements are ignored. In order
to correctly handle partially hidden articles on sites having the
"continue reading" button, if the class name contains "continue", it
would still be processed.

BUG=687071
==========

[wychen, 2017-02-02 18:01:17]

Description was changed from

==========
Fix more partially hidden articles

When traversing the DOM tree, invisible elements are ignored. In order
to correctly handle partially hidden articles on sites having the
"continue reading" button, if the class name contains "continue", it
would still be processed.

BUG=687071
==========

to

==========
Fix more partially hidden articles

When traversing the DOM tree, invisible elements are ignored. In order
to correctly handle partially hidden articles on sites having the
"continue reading" button, if the class name contains "continue", it
would still be processed.

BUG=687071
R=mdjones@chromium.org

Committed: 6bd2ebbcce6ecfb349c812adcf15ef047aef54a4
==========

[wychen, 2017-02-02 18:01:19]

Committed patchset #3 (id:40001) manually as
6bd2ebbcce6ecfb349c812adcf15ef047aef54a4 (presubmit successful).