New issue
Advanced search Search tips

Issue 593476 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 621512
Owner: ----
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

Move chrome.csi and chrome.loadTimes to blink

Project Member Reported by mea...@chromium.org, Mar 9 2016

Issue description

Context: https://codereview.chromium.org/1750893003/

Haven't had a chance to look at this, filing the bug so as not to forget about it.
 

Comment 1 by jochen@chromium.org, Mar 10 2016

Cc: haraken@chromium.org esprehn@chromium.org

Comment 2 by mea...@chromium.org, Mar 10 2016

Given that this is a chrome.* API, does it still make sense to move it to blink? Or does blink already have support for such cases? (I don't know much about bindings so sorry if this is a naive question)

Comment 3 by jochen@chromium.org, Mar 10 2016

chrome referes the the UI concept, not to the directory or similar.

there's also eg chromeclient.cpp in blink

Comment 4 by mea...@chromium.org, Mar 10 2016

Is that actually the case here? chrome.csi looks like a pre Navigation Timing API which doesn't refer to anything in the browser UI.

There is also bug 113048 to remove at least chrome.csi. Given that performance.timing gives much more  detailed results, should we just do that?

Comment 5 by jochen@chromium.org, Mar 10 2016

if usage is low enough, removing is always an option
Correct, chrome.loadTimes() is a pre-navtiming API. I'd like us to remove both of these. I opened a bug to track removal here: https://bugs.chromium.org/p/chromium/issues/detail?id=621512
Mergedinto: 621512
Status: Duplicate (was: Untriaged)
Duping this to the bug tracking removing these APIs.

Sign in to add a comment