Chromium Code Reviews
DescriptionReland of wptserve: Enable WPTServe by default. (patchset #1 id:1 of https://codereview.chromium.org/2537413002/ )
Reason for revert:
The problem was resolved.
Original issue's description:
> Revert of wptserve: Enable WPTServe by default. (patchset #1 id:1 of https://codereview.chromium.org/2497193002/ )
>
> Reason for revert:
> I suspect that the startup/shutdown logic in wptserve is not right, and we're seeing intermittent failures where we're hanging trying to start wptserve.
>
> See crbug.com/669194 and https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Mac10.11/builds/13447/steps/webkit_tests/logs/stdio for an example.
>
> I'm speculatively reverting this for now, to see if things become happier.
>
> Original issue's description:
> > Reland of wptserve: Enable WPTServe by default.
>
> TBR=qyearsley@chromium.org,jsbell@chromium.org,tkent@chromium.org
> NOTRY=true
> NOPRESUBMIT=true
> BUG=618366
>
> Committed: https://crrev.com/b5ce7e0a17e32d6d145124938a90220e9f9990c5
> Cr-Commit-Position: refs/heads/master@{#435129}
TBR=qyearsley@chromium.org,jsbell@chromium.org,dpranke@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=618366
Review-Url: https://codereview.chromium.org/2633083002
Cr-Commit-Position: refs/heads/master@{#443953}
Committed: https://chromium.googlesource.com/chromium/src/+/01c97b5106a69c02931e5aee55e460e1e7eb398a
Patch Set 1 #
Messages
Total messages: 12 (7 generated)
|
|||||||||||||||||||