New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 612957 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 2
Type: Bug



Sign in to add a comment

Add web shell redirect integration tests

Project Member Reported by baxley@chromium.org, May 18 2016

Issue description

add redirect tests for:
301 redirect
302 redirect
multiple redirects
 
Cc: eugene...@chromium.org
Menglu, would you mind if I take this bug?
Cc: -eugene...@chromium.org
Owner: eugene...@chromium.org
No, take them all :D
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 28 2016

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

commit 56e1f38bb93dac4fd3ef4b7645b28486546ba658
Author: eugenebut <eugenebut@chromium.org>
Date: Thu Jul 28 22:43:10 2016

[ios] Added web::test::SetUpHttpServer which takes ResponseProvider.

This function will be used by EG tests and reused by other response
provider util functions.

BUG= 612957 

Review-Url: https://codereview.chromium.org/2193543002
Cr-Commit-Position: refs/heads/master@{#408504}

[modify] https://crrev.com/56e1f38bb93dac4fd3ef4b7645b28486546ba658/ios/web/public/test/http_server_util.h
[modify] https://crrev.com/56e1f38bb93dac4fd3ef4b7645b28486546ba658/ios/web/public/test/http_server_util.mm

Status: Fixed (was: Assigned)

Sign in to add a comment