Issue metadata
Sign in to add a comment
|
Move chrome.csi and chrome.loadTimes to blink |
||||||||||||||||||||||||
Issue descriptionContext: https://codereview.chromium.org/1750893003/ Haven't had a chance to look at this, filing the bug so as not to forget about it.
,
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)
,
Mar 10 2016
chrome referes the the UI concept, not to the directory or similar. there's also eg chromeclient.cpp in blink
,
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?
,
Mar 10 2016
if usage is low enough, removing is always an option
,
Jun 20 2016
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
,
Jun 20 2016
Duping this to the bug tracking removing these APIs. |
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by jochen@chromium.org
, Mar 10 2016