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

Issue 134643003: Add libwebm in third_party. (Closed)

Created:
6 years, 11 months ago by Sergey Ulanov
Modified:
6 years, 11 months ago
Reviewers:
Tom Finegan, Wez
CC:
chromium-reviews
Visibility:
Public.

Description

Add libwebm in third_party. libwebm will be used by chromoting client to generate WebM stream for MediaSource API. BUG=321825 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=245139 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=245368

Patch Set 1 : #

Total comments: 6

Patch Set 2 : #

Patch Set 3 : #

Patch Set 4 : fix checklicenses issue #

Patch Set 5 : merge #

Unified diffs Side-by-side diffs Delta from patch set Stats (+59 lines, -9 lines) Patch
M .gitignore View 1 2 3 4 1 chunk +1 line, -0 lines 0 comments Download
M DEPS View 1 2 3 4 1 chunk +4 lines, -0 lines 0 comments Download
A third_party/libwebm/OWNERS View 1 chunk +2 lines, -0 lines 0 comments Download
A third_party/libwebm/README.chromium View 1 2 1 chunk +13 lines, -0 lines 0 comments Download
A third_party/libwebm/libwebm.gyp View 1 1 chunk +18 lines, -0 lines 0 comments Download
A third_party/libwebm/libwebm.gypi View 1 2 3 4 1 chunk +13 lines, -0 lines 0 comments Download
A + third_party/libwebm/libwebm_nacl.gyp View 1 2 3 4 1 chunk +8 lines, -9 lines 0 comments Download

Messages

Total messages: 14 (0 generated)
Sergey Ulanov
6 years, 11 months ago (2014-01-14 02:26:40 UTC) #1
Tom Finegan
The URL change is the only thing that's really important. I believe git.chromium.org is eventually ...
6 years, 11 months ago (2014-01-14 02:55:29 UTC) #2
Sergey Ulanov
https://codereview.chromium.org/134643003/diff/40001/DEPS File DEPS (right): https://codereview.chromium.org/134643003/diff/40001/DEPS#newcode298 DEPS:298: "http://git.chromium.org/webm/libwebm.git@5efd6e3c1df766c08294ad19168e71522ee0d808", On 2014/01/14 02:55:29, Tom Finegan wrote: > https://chromium.googlesource.com/webm/libwebm.git ...
6 years, 11 months ago (2014-01-14 18:19:06 UTC) #3
Tom Finegan
> Done. http://www.webmproject.org/code/ still refers to http://git.chromium.org - does > it need to be updated? ...
6 years, 11 months ago (2014-01-14 18:39:22 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sergeyu@chromium.org/134643003/120001
6 years, 11 months ago (2014-01-14 19:30:00 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sergeyu@chromium.org/134643003/120001
6 years, 11 months ago (2014-01-14 23:21:14 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sergeyu@chromium.org/134643003/120001
6 years, 11 months ago (2014-01-15 02:44:43 UTC) #7
commit-bot: I haz the power
Retried try job too often on android_dbg for step(s) slave_steps http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_dbg&number=139424
6 years, 11 months ago (2014-01-15 12:55:35 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sergeyu@chromium.org/134643003/550001
6 years, 11 months ago (2014-01-16 03:14:53 UTC) #9
commit-bot: I haz the power
Change committed as 245139
6 years, 11 months ago (2014-01-16 05:28:56 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sergeyu@chromium.org/134643003/800001
6 years, 11 months ago (2014-01-16 20:14:11 UTC) #11
commit-bot: I haz the power
Failed to apply patch for third_party/libwebm/libwebm_nacl.gyp: While running svn copy third_party/modp_b64/modp_b64_untrusted.gyp third_party/libwebm/libwebm_nacl.gyp --config-dir /b/commit-queue/subversion_config --non-interactive ...
6 years, 11 months ago (2014-01-16 20:14:15 UTC) #12
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sergeyu@chromium.org/134643003/870001
6 years, 11 months ago (2014-01-16 20:18:38 UTC) #13
commit-bot: I haz the power
6 years, 11 months ago (2014-01-16 23:11:23 UTC) #14
Message was sent while issue was closed.
Change committed as 245368

Powered by Google App Engine
This is Rietveld 408576698