Commit e44d3738 authored by Torne (Richard Coles)'s avatar Torne (Richard Coles) Committed by Commit Bot

webview: don't use UI thread pool unnecessarily.

Just use the general pool at best-effort priority for logging command
line and field trials during startup, since it doesn't need to be on the
UI thread.

Change-Id: I73dfb14ae67bcc953fa7c90b5840354d5b4a84b3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2302946Reviewed-by: default avatarNate Fischer <ntfschr@chromium.org>
Commit-Queue: Richard Coles <torne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#789538}
parent bef2d181
...@@ -51,7 +51,7 @@ import org.chromium.base.library_loader.LibraryLoader; ...@@ -51,7 +51,7 @@ import org.chromium.base.library_loader.LibraryLoader;
import org.chromium.base.metrics.RecordHistogram; import org.chromium.base.metrics.RecordHistogram;
import org.chromium.base.metrics.ScopedSysTraceEvent; import org.chromium.base.metrics.ScopedSysTraceEvent;
import org.chromium.base.task.PostTask; import org.chromium.base.task.PostTask;
import org.chromium.content_public.browser.UiThreadTaskTraits; import org.chromium.base.task.TaskTraits;
import org.chromium.net.NetworkChangeNotifier; import org.chromium.net.NetworkChangeNotifier;
import org.chromium.ui.base.ResourceBundle; import org.chromium.ui.base.ResourceBundle;
...@@ -457,7 +457,7 @@ public class WebViewChromiumAwInit { ...@@ -457,7 +457,7 @@ public class WebViewChromiumAwInit {
// Log extra information, for debugging purposes. Do the work asynchronously to avoid blocking // Log extra information, for debugging purposes. Do the work asynchronously to avoid blocking
// startup. // startup.
private static void logCommandLineAndActiveTrials() { private static void logCommandLineAndActiveTrials() {
PostTask.postTask(UiThreadTaskTraits.BEST_EFFORT, () -> { PostTask.postTask(TaskTraits.BEST_EFFORT, () -> {
// TODO(ntfschr): CommandLine can change at any time. For simplicity, only log it // TODO(ntfschr): CommandLine can change at any time. For simplicity, only log it
// once during startup. // once during startup.
AwContentsStatics.logCommandLineForDebugging(); AwContentsStatics.logCommandLineForDebugging();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment