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

Issue 2505933002: Add virtual/mojo-loading/http/tests (Closed)

Created:
4 years, 1 month ago by yhirano
Modified:
4 years ago
Reviewers:
tzik
CC:
chromium-reviews, qsr+mojo_chromium.org, viettrungluu+watch_chromium.org, yzshen+watch_chromium.org, abarth-chromium, Aaron Boodman, blink-reviews, darin (slow to review)
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Add virtual/mojo-loading/http/tests Test mojo-loading feature with http/tests layout tests. BUG=669357 Committed: https://crrev.com/3d083693b1e5c9dfd99b42f09a3b47d67e753386 Cr-Commit-Position: refs/heads/master@{#435577}

Patch Set 1 #

Patch Set 2 : fix #

Patch Set 3 : rebase #

Patch Set 4 : fix #

Patch Set 5 : SlowTests, LeakExpectations #

Patch Set 6 : reparent #

Patch Set 7 : RandomOrderExpectations #

Patch Set 8 : rebase #

Patch Set 9 : fix #

Patch Set 10 : fix #

Patch Set 11 : fix #

Patch Set 12 : fix #

Patch Set 13 : fix #

Patch Set 14 : fix #

Patch Set 15 : rebase #

Patch Set 16 : fix #

Patch Set 17 : imported #

Patch Set 18 : fix #

Unified diffs Side-by-side diffs Delta from patch set Stats (+164 lines, -1 line) Patch
M third_party/WebKit/LayoutTests/LeakExpectations View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 5 chunks +11 lines, -0 lines 0 comments Download
M third_party/WebKit/LayoutTests/NeverFixTests View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 4 chunks +9 lines, -0 lines 0 comments Download
M third_party/WebKit/LayoutTests/RandomOrderExpectations View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 1 chunk +37 lines, -0 lines 0 comments Download
M third_party/WebKit/LayoutTests/SlowTests View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 7 chunks +47 lines, -0 lines 0 comments Download
M third_party/WebKit/LayoutTests/TestExpectations View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 25 chunks +60 lines, -0 lines 0 comments Download
M third_party/WebKit/LayoutTests/VirtualTestSuites View 1 2 3 4 5 6 7 1 chunk +1 line, -1 line 0 comments Download
A + third_party/WebKit/LayoutTests/virtual/mojo-loading/http/tests/README.txt View 0 chunks +-1 lines, --1 lines 0 comments Download
D third_party/WebKit/LayoutTests/virtual/mojo-loading/http/tests/xmlhttprequest/README.txt View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 71 (66 generated)
yhirano
I think we can land this after your change https://codereview.chromium.org/2503813002/ lands. PTAL.
4 years ago (2016-11-29 10:26:55 UTC) #37
tzik
lgtm
4 years ago (2016-11-30 06:02:06 UTC) #50
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2505933002/330001
4 years ago (2016-12-01 06:38:16 UTC) #66
commit-bot: I haz the power
Committed patchset #18 (id:330001)
4 years ago (2016-12-01 07:55:19 UTC) #69
commit-bot: I haz the power
4 years ago (2016-12-01 07:58:36 UTC) #71
Message was sent while issue was closed.
Patchset 18 (id:??) landed as
https://crrev.com/3d083693b1e5c9dfd99b42f09a3b47d67e753386
Cr-Commit-Position: refs/heads/master@{#435577}

Powered by Google App Engine
This is Rietveld 408576698