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

Unified Diff: components/cronet/android/test/javatests/src/org/chromium/net/TestNetworkQualityRttListener.java

Issue 2849093003: Demonstrate race in NQE test
Patch Set: Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/cronet/android/test/javatests/src/org/chromium/net/TestNetworkQualityRttListener.java
diff --git a/components/cronet/android/test/javatests/src/org/chromium/net/TestNetworkQualityRttListener.java b/components/cronet/android/test/javatests/src/org/chromium/net/TestNetworkQualityRttListener.java
index afeec1f4f53651b45986b50ca842f4fbe79115b2..8d2cda799607bff8ef89f4954ffd82ad84782fb5 100644
--- a/components/cronet/android/test/javatests/src/org/chromium/net/TestNetworkQualityRttListener.java
+++ b/components/cronet/android/test/javatests/src/org/chromium/net/TestNetworkQualityRttListener.java
@@ -6,6 +6,7 @@ package org.chromium.net;
import static junit.framework.Assert.assertEquals;
+import android.util.Log;
import android.util.SparseIntArray;
import java.util.concurrent.Executor;
@@ -26,6 +27,11 @@ class TestNetworkQualityRttListener extends NetworkQualityRttListener {
@Override
public void onRttObservation(int rttMs, long when, int source) {
+ try {
+ Thread.sleep(1000);
+ } catch (Exception e) {
+ Log.e("PJPJ", "uhoh " + e);
+ }
synchronized (mLock) {
mRttObservationCount++;
mRttObservationCountBySource.put(source, mRttObservationCountBySource.get(source) + 1);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698