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

Issue 2628433003: Capture from Element: use a smaller webm file for a LayoutTest (Closed)

Created:
3 years, 11 months ago by mcasas
Modified:
3 years, 11 months ago
CC:
chromium-reviews, blink-reviews, emircan+watch+capturefromdom_chromium.org, mcasas+watch+capturefromdom_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Capture from Element: use a smaller webm file for a LayoutTest This CL only affects the Layout test fast/mediacapturefromelement/HTMLMediaElementCapture-capture.html that is slow on Linux (see bug). This CL changes the file used for the third test case (video and audio) from using test.webm (~215KB) to test-av-384k-44100Hz-1ch-320x240-30fps-10kfr.webm (~75KB) to speed up the test. BUG=659457 Review-Url: https://codereview.chromium.org/2628433003 Cr-Commit-Position: refs/heads/master@{#442482} Committed: https://chromium.googlesource.com/chromium/src/+/d9c4c30beab57d80195824f84bb30d7d651a48de

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -4 lines) Patch
M third_party/WebKit/LayoutTests/fast/mediacapturefromelement/HTMLMediaElementCapture-capture.html View 1 chunk +5 lines, -4 lines 0 comments Download

Messages

Total messages: 13 (9 generated)
mcasas
reillyg@ PTAL
3 years, 11 months ago (2017-01-10 00:24:13 UTC) #3
Reilly Grant (use Gerrit)
lgtm
3 years, 11 months ago (2017-01-10 00:40:06 UTC) #4
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/2628433003/1
3 years, 11 months ago (2017-01-10 04:23:48 UTC) #10
commit-bot: I haz the power
3 years, 11 months ago (2017-01-10 04:52:58 UTC) #13
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/d9c4c30beab57d80195824f84bb3...

Powered by Google App Engine
This is Rietveld 408576698