New issue
Advanced search Search tips

Issue 601811 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Links in Reader Mode do not navigate correctly

Project Member Reported by mdjones@chromium.org, Apr 8 2016

Issue description

Links from sites like technologyreview.com do not navigate correctly in Reader Mode. On these pages the InterceptNavigationDelegate is not triggered and the navigation appears to come from a different source.

Sample page:
https://www.technologyreview.com/s/601156/the-scientific-swap-meet-behind-the-gene-editing-boom/#/set/id/601223/
 
Cc: mdjones@chromium.org
 Issue 614080  has been merged into this issue.
Labels: -Restrict-View-Google
Project Member

Comment 3 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 4 by wychen@chromium.org, Jun 23 2016

Labels: Merge-Request-52

Comment 5 by dimu@google.com, Jun 23 2016

Labels: -Merge-Request-52 Merge-Review-52 Hotlist-Merge-Review
[Automated comment] DEPS changes referenced in bugdroid comments, needs manual review.
Labels: -Hotlist-Merge-review -Merge-Review-52
Status: Fixed (was: Assigned)

Sign in to add a comment