New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 896839 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Dec 4
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

History entries duplicated for custom tab navigations

Project Member Reported by zea@google.com, Oct 18

Issue description

Version: Chrome Dev 71.0.3574.0
OS: Android

Repro steps:
Open a link in a custom tab (i.e. a news article from the AGSA Feed)
Open Chrome from the home screen
Go to the history page

Expected:
page visited appears once in history page

Actual:
page visited is present as two duplicate entries.

Looks like the entries can be individually deleted just fine. Also, when I look at another syncing device, the navigation only appears once, so it appears that history sync is working correctly on other devices.

I tried turning off the local network and opening the history page, and the duplicate entry still appeared. As such, this seems like a local history issue, but +mastiz FYI in case.


 
Thanks for filing the bug! Does the issue repro with sync disabled?
Good question, looks like it doesn't repro with sync disabled actually. So perhaps this is Sync related, although strange that having the device offline when opening history didn't make a difference?
Components: Services>Sync
Adding sync component, as in fact this does seem sync related.
Cc: -mastiz@chromium.org
Labels: -Pri-3 Sync-Triaged Pri-2
Owner: mastiz@chromium.org
Status: Assigned (was: Untriaged)
Cc: mastiz@chromium.org
Owner: ----
Status: Available (was: Assigned)
Sorry for the long delay!

I could repro this, with two entries in history with slightly different URLs (URL fragment stripped in one of the two):

Entry 1 (longer, received from Google Servers): https://www-foxnews-com.cdn.ampproject.org/v/s/www.foxnews.com/entertainment/cnn-panel-trashes-trump-after-u-s-capitol-visit-to-honor-george-h-w-bush.amp?amp_js_v=a2&amp_gsa=1#referrer=https%3A%2F%2Fwww.google.com&amp_tf=From%20%251%24s&ampshare=https%3A%2F%2Fwww.foxnews.com%2Fentertainment%2Fcnn-panel-trashes-trump-after-u-s-capitol-visit-to-honor-george-h-w-bush

Entry 2 (truncated, provided by local history): https://www-foxnews-com.cdn.ampproject.org/v/s/www.foxnews.com/entertainment/cnn-panel-trashes-trump-after-u-s-capitol-visit-to-honor-george-h-w-bush.amp?amp_js_v=a2&amp_gsa=1#referrer=https%3A%2F%2Fwww.google.com&amp_tf=From%20%251%24s

The second entry is missing the URL fragment.

I don't see any redirects being relevant here: not observed in Footprints and the URLs actually don't redirect to each other.

BrowsingHistoryService seems to implement deduping based on exact URL matches, which in this case is too strict.

I don't really know why the two URLs differ, but it doesn't seem a sync issue, so unassigning for now.
Owner: sky@chromium.org
Status: Assigned (was: Available)
I skimmed through the AGSA code and it looks like the app appends those fragments (including ampshare and referrer). I can't confirm this 100%, but googlers can codesearch for REFERRER_FRAGMENT_PARAM.

On chromium's side, the only alternative would be to implement more aggressive deduping in BrowsingHistoryService, which is questionable.

On to sky@ as owner of history: has this deduping logic in BrowsingHistoryService been discussed from the product's PoV?
Status: WontFix (was: Assigned)
I'm not aware of any discussion of trying to find similar visits and prune them. Any heuristics we try would likely cause problems for certain use cases. IMO if the page is triggering visits, we should show them as is and not attempt to decide what is relevant and what isn't. I'm going to close this out.

Sign in to add a comment