Skip to content

Commit

Permalink
webview: temporarily remove use of new timestamp API.
Browse files Browse the repository at this point in the history
The API needs to change slightly; temporarily remove the call to it.
This was only called on S preview builds so this doesn't affect other
Android versions.

Bug: 1208089
Change-Id: Ifb8894aa01552fd0ed7cc70a82abfb7fc180b960
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2889009
Auto-Submit: Richard Coles <[email protected]>
Commit-Queue: Nate Fischer <[email protected]>
Reviewed-by: Nate Fischer <[email protected]>
Cr-Commit-Position: refs/heads/master@{#881745}
  • Loading branch information
tornewuff authored and Chromium LUCI CQ committed May 11, 2021
1 parent 523eec1 commit 5cc5e4d
Showing 1 changed file with 2 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -412,13 +412,15 @@ private void initialize(WebViewDelegate webViewDelegate) {
"Android.WebView.Startup.CreationTime.Stage1.FactoryInit",
SystemClock.uptimeMillis() - startTime);

/* TODO(torne): re-enable this once the API change is sorted out
if (BuildInfo.isAtLeastS()) {
// TODO: Use the framework constants as indices in timestamps array.
startTime = mWebViewDelegate.getTimestamps()[0];
RecordHistogram.recordTimesHistogram(
"Android.WebView.Startup.CreationTime.TotalFactoryInitTime",
SystemClock.uptimeMillis() - startTime);
}
*/
}

/* package */ static void checkStorageIsNotDeviceProtected(Context context) {
Expand Down

0 comments on commit 5cc5e4d

Please sign in to comment.