EME: Fix nits for encrypted media layout tests
During recent reviews there have been several nits reported. This
change fixes a few of them.
BUG=442566
TEST=updated tests pass
Committed: https://crrev.com/5a02d174417d296c8e7a8b3a8cebd27cbbd4df71
Cr-Commit-Position: refs/heads/master@{#414267}
4 years, 4 months ago
(2016-08-24 20:42:07 UTC)
#2
PTAL.
xhwang
Please update the CL title to be more specific, e.g. this is nits for encrypted ...
4 years, 4 months ago
(2016-08-24 23:43:05 UTC)
#3
Please update the CL title to be more specific, e.g. this is nits for encrypted
media layout tests. "Update tests" is a bit too generic.
Otherwise LGTM. Thanks!
jrummell
Description was changed from ========== EME: Update tests During recent reviews there have been several ...
4 years, 4 months ago
(2016-08-25 00:17:00 UTC)
#4
Description was changed from
==========
EME: Update tests
During recent reviews there have been several nits reported. This
change fixes a few of them.
BUG=442566
TEST=updated tests pass
==========
to
==========
EME: Fix nits for encrypted media layout tests
During recent reviews there have been several nits reported. This
change fixes a few of them.
BUG=442566
TEST=updated tests pass
==========
jrummell
The CQ bit was checked by jrummell@chromium.org
4 years, 4 months ago
(2016-08-25 00:17:12 UTC)
#5
Description was changed from ========== EME: Fix nits for encrypted media layout tests During recent ...
4 years, 4 months ago
(2016-08-25 02:26:29 UTC)
#7
Message was sent while issue was closed.
Description was changed from
==========
EME: Fix nits for encrypted media layout tests
During recent reviews there have been several nits reported. This
change fixes a few of them.
BUG=442566
TEST=updated tests pass
==========
to
==========
EME: Fix nits for encrypted media layout tests
During recent reviews there have been several nits reported. This
change fixes a few of them.
BUG=442566
TEST=updated tests pass
==========
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 4 months ago
(2016-08-25 02:26:30 UTC)
#8
Message was sent while issue was closed.
Committed patchset #1 (id:1)
commit-bot: I haz the power
Description was changed from ========== EME: Fix nits for encrypted media layout tests During recent ...
4 years, 4 months ago
(2016-08-25 02:28:16 UTC)
#9
Message was sent while issue was closed.
Description was changed from
==========
EME: Fix nits for encrypted media layout tests
During recent reviews there have been several nits reported. This
change fixes a few of them.
BUG=442566
TEST=updated tests pass
==========
to
==========
EME: Fix nits for encrypted media layout tests
During recent reviews there have been several nits reported. This
change fixes a few of them.
BUG=442566
TEST=updated tests pass
Committed: https://crrev.com/5a02d174417d296c8e7a8b3a8cebd27cbbd4df71
Cr-Commit-Position: refs/heads/master@{#414267}
==========
commit-bot: I haz the power
Patchset 1 (id:??) landed as https://crrev.com/5a02d174417d296c8e7a8b3a8cebd27cbbd4df71 Cr-Commit-Position: refs/heads/master@{#414267}
4 years, 4 months ago
(2016-08-25 02:28:17 UTC)
#10
Issue 2169713002: EME: Fix nits for encrypted media layout tests
(Closed)
Created 4 years, 5 months ago by jrummell
Modified 4 years, 4 months ago
Reviewers: xhwang
Base URL: https://chromium.googlesource.com/chromium/src.git@master
Comments: 0