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

Issue 787211 link

Starred by 1 user
Status: Untriaged
Owner: ----
Cc:
█ juliatut...@chromium.org
dcheng@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Remove URL parameter from ReportingServiceProxy methods

Project Member Reported by dcheng@chromium.org, Nov 21 2017 
In all cases, it looks like document->Url() is passed as the parameter. We should just get the URL from GetLastCommittedUrl() and avoid the need to pass this over IPC from a less to more trusted process.

Comment 1 by juliatut...@chromium.org, Nov 21 2017 

This might not always be the case; in the future, it could be a subresource. That said, if you gesture vaguely in the direction of where I can grab that URL, I can rewrite the current ones to use it.
Project Member

Comment 2 by sheriffbot@chromium.org, Nov 21

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
►

Sign in to add a comment