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

Issue 2806513002: Use long timeout for the svg/interfaces.html test (Closed)

Created:
3 years, 8 months ago by fs
Modified:
3 years, 8 months ago
Reviewers:
Stephen Chennney
CC:
blink-reviews, blink-reviews-w3ctests_chromium.org, chromium-reviews
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Use long timeout for the svg/interfaces.html test Lots of interfaces, so can take a while to run. Use the 'long' timeout option. BUG=709030 Review-Url: https://codereview.chromium.org/2806513002 Cr-Commit-Position: refs/heads/master@{#462655} Committed: https://chromium.googlesource.com/chromium/src/+/4325582595a4b8b0c30e6390dfb48bdf66c36b60

Patch Set 1 #

Patch Set 2 : Remove entry from SlowTests #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -3 lines) Patch
M third_party/WebKit/LayoutTests/SlowTests View 1 1 chunk +0 lines, -3 lines 0 comments Download
M third_party/WebKit/LayoutTests/external/wpt/svg/interfaces.html View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 16 (9 generated)
fs
3 years, 8 months ago (2017-04-06 17:04:11 UTC) #3
Stephen Chennney
Thanks for jumping on it. I didn't know the meta option existed. lgtm. Once the ...
3 years, 8 months ago (2017-04-06 17:09:43 UTC) #5
fs
On 2017/04/06 at 17:09:43, schenney wrote: > Thanks for jumping on it. I didn't know ...
3 years, 8 months ago (2017-04-06 18:19:24 UTC) #8
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/2806513002/20001
3 years, 8 months ago (2017-04-06 19:12:13 UTC) #11
commit-bot: I haz the power
Committed patchset #2 (id:20001) as https://chromium.googlesource.com/chromium/src/+/4325582595a4b8b0c30e6390dfb48bdf66c36b60
3 years, 8 months ago (2017-04-06 22:36:57 UTC) #14
jeffcarp
On 2017/04/06 at 22:36:57, commit-bot wrote: > Committed patchset #2 (id:20001) as https://chromium.googlesource.com/chromium/src/+/4325582595a4b8b0c30e6390dfb48bdf66c36b60 A WPT ...
3 years, 8 months ago (2017-04-06 23:29:20 UTC) #15
jeffcarp
3 years, 8 months ago (2017-04-10 19:16:13 UTC) #16
Message was sent while issue was closed.
On 2017/04/06 at 23:29:20, jeffcarp wrote:
> On 2017/04/06 at 22:36:57, commit-bot wrote:
> > Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/4325582595a4b8b0c30e6390dfb4...
> 
> A WPT PR on GitHub was created for this CL but a Travis CI infra failure means
export will be delayed:
> https://github.com/w3c/web-platform-tests/pull/5414

Update: the PR landed.

Powered by Google App Engine
This is Rietveld 408576698