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

Issue 2551953002: Add tests for FLAC and OPUS decoding (Closed)

Created:
4 years ago by Raymond Toy
Modified:
4 years ago
Reviewers:
hongchan
CC:
chromium-reviews, blink-reviews
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Add tests for FLAC and OPUS decoding These tests aren't run on the trybots so I ran them locally on linux, mac, and win. They passed. BUG=669167 TESTS=flac-decode.html, opus-decode.html Committed: https://crrev.com/7d6828326a25a95110cf006e1fe7cc24d6a615d2 Cr-Commit-Position: refs/heads/master@{#436459}

Patch Set 1 #

Messages

Total messages: 11 (6 generated)
Raymond Toy
PTAL
4 years ago (2016-12-05 21:18:32 UTC) #3
hongchan
lgtm
4 years ago (2016-12-05 22:15:00 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/2551953002/1
4 years ago (2016-12-05 22:34:36 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years ago (2016-12-05 23:47:55 UTC) #9
commit-bot: I haz the power
4 years ago (2016-12-05 23:49:37 UTC) #11
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/7d6828326a25a95110cf006e1fe7cc24d6a615d2
Cr-Commit-Position: refs/heads/master@{#436459}

Powered by Google App Engine
This is Rietveld 408576698