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

Issue 650364 link

Starred by 6 users
Status: Fixed
Owner:
█ roc...@chromium.org
please use my google.com address
Closed: Sep 2016
Cc:
█ ben@chromium.org
nyerramilli@chromium.org
█ ranjitkan@chromium.org
sa...@chromium.org
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug-Regression



Sign in to add a comment

Visited link highlighting is broken

Project Member Reported by roc...@chromium.org, Sep 26 2016 
This interface can't be bound due to capability filtering. Needs an entry in the browser's "web" capability class.

Comment 1 by roc...@chromium.org, Sep 26 2016

Summary: VisistedLinkNotificationSink unreachable by the browser (was: VisistedLinkNotificationSink unreachable by renderers)
Oops - other way around. This is browser to renderer.

Comment 2 by roc...@chromium.org, Sep 26 2016

Summary: Visited link highlighting is broken (was: VisistedLinkNotificationSink unreachable by the browser)
(updating bug to have a useful user POV description)

Visited link highlighting is completely broken.

The root cause is that browser->renderer IPCs never reach the renderer to inform it about visited links.

Comment 4 by roc...@chromium.org, Sep 26 2016

Cc: nyerramilli@chromium.org ben@chromium.org ranjitkan@chromium.org
 Issue 649615  has been merged into this issue.

Comment 5 by roc...@chromium.org, Sep 26 2016 

Thanks for the pointer!
Project Member

Comment 6 by bugdroid1@chromium.org, Sep 27 2016 

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

commit 7ebc48c59be5ca12c0ebec989c46aeb4e6c7612b
Author: rockot <rockot@chromium.org>
Date: Tue Sep 27 01:33:49 2016

Add VisitedLinkNotificationSink to renderer's browser capability

BUG= 650364 
TBR=ben@chromium.org

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

[modify] https://crrev.com/7ebc48c59be5ca12c0ebec989c46aeb4e6c7612b/content/public/app/mojo/content_renderer_manifest.json

Comment 7 by roc...@chromium.org, Sep 27 2016

Labels: -Type-Bug Type-Bug-Regression
Status: Fixed (was: Assigned)

Comment 8 by dtapu...@chromium.org, Sep 28 2016 

 Issue 651013  has been merged into this issue.
►

Sign in to add a comment