Add DEPS for new third party library in third_party/openmax_dl.

Add DEPS for new third party library in third_party/openmax_dl.


BUG=


Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=190022

[Raymond Toy (Google), 2013-02-22 18:43:39]

+ ajm

I think r3504 is the correct svn version; it's the latest.

[ajm, 2013-02-22 19:04:38]

lgtm

https://codereview.chromium.org/12316073/diff/1/DEPS
File DEPS (right):

https://codereview.chromium.org/12316073/diff/1/DEPS#newcode43
DEPS:43: "openmax_dl_revision": "3504",
This is only used once, so you don't _need_ to add it here. Not sure if there's
a consistent rule.

[ajm, 2013-02-22 19:04:48]

On 2013/02/22 18:43:39, rtoy wrote:
> + ajm
> 
> I think r3504 is the correct svn version; it's the latest.

Yep, that's right.

[Raymond Toy (Google), 2013-02-25 17:58:06]

On 2013/02/22 19:04:38, ajm wrote:
> lgtm
> 
> https://codereview.chromium.org/12316073/diff/1/DEPS
> File DEPS (right):
> 
> https://codereview.chromium.org/12316073/diff/1/DEPS#newcode43
> DEPS:43: "openmax_dl_revision": "3504",
> This is only used once, so you don't _need_ to add it here. Not sure if
there's
> a consistent rule.

For simplicity, I've uploaded a new patch that doesn't use openmax_dl_revision
and just adds the appropriate revision to the URL, as you suggest.

[szager, 2013-02-26 18:32:30]

lgtm

[Raymond Toy (Google), 2013-02-26 18:37:02]

On 2013/02/26 18:32:30, szager wrote:
> lgtm

Thanks, everyone, for reviewing.

[commit-bot: I haz the power, 2013-02-26 18:38:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rtoy@google.com/12316073/4001

[commit-bot: I haz the power, 2013-02-26 19:46:45]

Retried try job too often on android_dbg for step(s) slave_steps
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...

[Jorge Lucangeli Obes, 2013-02-27 17:47:41]

On 2013/02/26 19:46:45, I haz the power (commit-bot) wrote:
> Retried try job too often on android_dbg for step(s) slave_steps
>
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...

Please don't CQ this until the security review is done. Thanks!

[Raymond Toy (Google), 2013-03-22 17:00:29]

Updated openmax_dl revision.  Security review completed for that revision.

[Jorge Lucangeli Obes, 2013-03-22 17:09:14]

On 2013/03/22 17:00:29, rtoy wrote:
> Updated openmax_dl revision.  Security review completed for that revision.

For completeness, security review was at
https://codereview.chromium.org/12317152/

lgtm

[commit-bot: I haz the power, 2013-03-22 18:11:03]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rtoy@google.com/12316073/14001

[commit-bot: I haz the power, 2013-03-23 14:42:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rtoy@google.com/12316073/14001

[commit-bot: I haz the power, 2013-03-23 16:27:21]

Change committed as 190022