Commit 4d071315 authored by Sam Maier's avatar Sam Maier Committed by Commit Bot

Android: cronet perf tests explicitly run on SERIAL_EXECUTOR

We would like to get rid of all usages of execute() since it is unclear
whether it goes to SERIAL_EXECUTOR or THREAD_POOL_EXECUTOR (many assume
THREAD_POOL_EXECUTOR, but it actually goes to SERIAL_EXECUTOR). This is
just cleaning up this test usage.

Bug: 869907
Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester
Change-Id: Id783526b0daead7f413f55bb443e9d9792142667
Reviewed-on: https://chromium-review.googlesource.com/1181540
Commit-Queue: Sam Maier <smaier@chromium.org>
Commit-Queue: Paul Jensen <pauljensen@chromium.org>
Reviewed-by: default avatarPaul Jensen <pauljensen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#584487}
parent 3f0b1151
......@@ -622,6 +622,6 @@ public class CronetPerfTestActivity extends Activity {
PathUtils.setPrivateDataDirectorySuffix(PRIVATE_DATA_DIRECTORY_SUFFIX);
mConfig = getIntent().getData();
// Execute benchmarks on another thread to avoid networking on main thread.
new BenchmarkTask().execute();
new BenchmarkTask().executeOnExecutor(AsyncTask.SERIAL_EXECUTOR);
}
}
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