Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(51)

Issue 2948293002: [Fork M59] android: Warmup after library load (Closed)

Created:
3 years, 6 months ago by boliu
Modified:
3 years, 6 months ago
Reviewers:
Ted C
CC:
chromium-reviews, jam, darin-cc_chromium.org, agrieve+watch_chromium.org
Target Ref:
refs/branch-heads/3071
Project:
chromium
Visibility:
Public.

Description

[Fork M59] android: Warmup after library load This is partial revert of r462158 in an attempt to workaround an android bug that consistently kill the warmup service without notifying us. It looks like if warmup is delayed until after native library is loaded, then this corner case doesn't happen anymore, so revert that part. BUG=735626 R=tedchoc@chromium.org Review-Url: https://codereview.chromium.org/2948293002 . Cr-Commit-Position: refs/branch-heads/3071@{#819} Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641} Committed: https://chromium.googlesource.com/chromium/src/+/aefe4fdabf53cb59ca93dadefc6a63f8af2c1706

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -7 lines) Patch
M chrome/android/java/src/org/chromium/chrome/browser/init/AsyncInitTaskRunner.java View 3 chunks +6 lines, -6 lines 0 comments Download
M content/public/android/java/src/org/chromium/content/browser/ChildProcessLauncher.java View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 6 (3 generated)
boliu
stamp pls?
3 years, 6 months ago (2017-06-22 19:50:48 UTC) #3
Ted C
lgtm
3 years, 6 months ago (2017-06-22 19:56:51 UTC) #4
boliu
3 years, 6 months ago (2017-06-22 20:06:47 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
aefe4fdabf53cb59ca93dadefc6a63f8af2c1706 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698