New issue
Advanced search Search tips

Issue 705964 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 2
Type: Bug

Blocked on:
issue 705987



Sign in to add a comment

Appstore app links in reading list are in unread section.

Project Member Reported by pmadalla@chromium.org, Mar 28 2017

Issue description

App Version: 59.0.3054.0 canary
iOS Version: 9.3.5,iOS 10
Device:  iPhone,iPad

Steps to reproduce:
 1. Launch chrome
 2. Perform any iPhone app search (Say : templerun2)
 3. Long tap itunes link.
 4. Tap on Read later.
 5. Tap on Menu > Read later.
 6. Tap on the itunes link in unread section.
 7. Relaunch chrome tap on Menu > Read later.
   
Observed results:
At step 6 - User is redirected to app store
At step 7 - Link in reading list is in unread section.

Expected results:
Links in reading links should be in read section.

Number of times you were able to reproduce: 5/5
Bug reproducible after clean install: Yes
Bug reproducible after clearing cache and cookies: Yes
Bug reproducible on Chrome Mobile on Android: NA
Bug reproducible on Dolphin/Safari/Firefox: NA
Bug reproducible on current stable build (App Version, iOS Version): Yes in M57
Bug reproducible on the current beta channel build (App Version, iOS Version): Yes in M58

Link to Video:
https://drive.google.com/a/google.com/file/d/0B--UpU2GW2EpZ0hoM2s4RzFfOFE/view?usp=sharing

 
Owner: olivierrobin@chromium.org
Status: Started (was: Untriaged)
Blockedon: 705987
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 31 2017

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

commit 664989aae268019ac64667f276a74dd89a5185fb
Author: olivierrobin <olivierrobin@chromium.org>
Date: Fri Mar 31 15:26:03 2017

Reading List: Mark entry read when opening external app.

When opening an external app, the navigation is cancelled.
Report the opening of External app to the ReadingListWebStateObserver
and mark the entry read.

BUG= 705964 

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

[modify] https://crrev.com/664989aae268019ac64667f276a74dd89a5185fb/ios/chrome/browser/tabs/tab.mm
[modify] https://crrev.com/664989aae268019ac64667f276a74dd89a5185fb/ios/web/public/web_state/ui/crw_web_delegate.h
[modify] https://crrev.com/664989aae268019ac64667f276a74dd89a5185fb/ios/web/web_state/ui/crw_web_controller.mm

Status: Fixed (was: Started)
Status: Verified (was: Fixed)
Verified on chrome canary version 59.0.3062.0 on iPhone 6 plus with iOS 10.3 and iPad Air with iOS 10.2.1, following steps mentioned in comment #0.  Links in the reading list is marked as read.  Looks good.

Sign in to add a comment