New issue
Advanced search Search tips

Issue 597366 link

Starred by 1 user
Status: Assigned
Owner:
wychen@chromium.org
Cc:
mdjones@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

DOM distiller gets extra images

Project Member Reported by wychen@chromium.org, Mar 23 2016 
Distill this page:
http://altwall.net/texts.php?show=bullet&number=2642&skip=2

At the end of the distilled content, there's one extra image, which is a avatar in the comment section.

Comment 1 by wychen@chromium.org, Apr 11 2016 

Another example:
http://www.people.com/article/julianne-hough-brooks-laich-celebrate-engagment-party

Extra image at the end, after "Around the web".
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 23 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/96665328d45790abbbcc5136fd94b1b3c4954c9c

commit 96665328d45790abbbcc5136fd94b1b3c4954c9c
Author: wychen <wychen@chromium.org>
Date: Thu Jun 23 01:24:03 2016

Roll DOM Distiller JavaScript distribution package

Diff since last roll:
https://github.com/chromium/dom-distiller/compare/0adf24afe4...54d05ba208

Picked up changes:
54d05ba Discard hidden articles when using fast path
f6d2dc1 Support extraction of lazily-loaded images
11fdddc Strip "target" attribute from anchor elements
65c0b6d Fix for LeadImage getting images after last relevant content
0455a46 Support deprecated <object> API of Youtube embeds
8637690 Fix some warnings in Eclipse

BUG=478142,481111, 544962 ,597366, 601811 , 616954 

Review-Url: https://codereview.chromium.org/2087233004
Cr-Commit-Position: refs/heads/master@{#401500}

[modify] https://crrev.com/96665328d45790abbbcc5136fd94b1b3c4954c9c/DEPS
[modify] https://crrev.com/96665328d45790abbbcc5136fd94b1b3c4954c9c/third_party/dom_distiller_js/README.chromium

Comment 3 by benhenry@chromium.org, Aug 1

Status: Assigned (was: Untriaged)
►

Sign in to add a comment