New issue
Advanced search Search tips

Issue 919853 link

Starred by 1 user
Status: Fixed
Owner:
laisminchillo@chromium.org
Closed: Jan 17
Cc:
tobiasjs@chromium.org
tedc...@chromium.org
boliu@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Enable multiple instances of TestWebServer

Project Member Reported by laisminchillo@google.com, Jan 8 
Enable multiple instances of TestWebServer: https://cs.chromium.org/chromium/src/net/test/android/javatests/src/org/chromium/net/test/util/TestWebServer.java

Only one instance and one secure instance are allowed. For  http://crbug.com/902215  we need at least 2 instances.
Project Member

Comment 1 by bugdroid1@chromium.org, Jan 15 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/45af9b01f5fce4f02ef413d7a795a472aa52d2c0

commit 45af9b01f5fce4f02ef413d7a795a472aa52d2c0
Author: Laís Minchillo <laisminchillo@chromium.org>
Date: Tue Jan 15 10:31:10 2019

Enable multiple TestWebServer instances

Bug:  919853 
Change-Id: I1205ae353e614ce58b72a26ffe66698e277f4ecd
Reviewed-on: https://chromium-review.googlesource.com/c/1400693
Reviewed-by: Paul Jensen <pauljensen@chromium.org>
Reviewed-by: Tobias Sargeant <tobiasjs@chromium.org>
Commit-Queue: Laís Minchillo <laisminchillo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#622800}
[modify] https://crrev.com/45af9b01f5fce4f02ef413d7a795a472aa52d2c0/net/test/android/javatests/src/org/chromium/net/test/util/TestWebServer.java
[modify] https://crrev.com/45af9b01f5fce4f02ef413d7a795a472aa52d2c0/net/test/android/javatests/src/org/chromium/net/test/util/WebServer.java

Comment 2 by laisminchillo@chromium.org, Jan 17 (5 days ago)

Status: Fixed (was: Started)

Comment 3 by laisminchillo@chromium.org, Jan 17 (5 days ago) 

To manually verify this just run tests in AwProxyControllerTest
►

Sign in to add a comment