Revert of Add unit tests for CdmAdapter

Revert of Add unit tests for CdmAdapter (patchset #6 id:100001 of https://codereview.chromium.org/1428753010/ )

Reason for revert:
valgrind reports leaks with no-debug-symbol callstack

BUG=556346
TBR=jrummell

Original issue's description:
> Add unit tests for CdmAdapter
> 
> BUG=510088
> TEST=new unittests pass
> 
> Committed: https://crrev.com/30996193fe3e69fe9ccd52471900bf5685c54956
> Cr-Commit-Position: refs/heads/master@{#359582}

TBR=xhwang@chromium.org,jrummell@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=510088

[zhaoqin, 2015-11-16 03:37:49]

Created Revert of Add unit tests for CdmAdapter

[commit-bot: I haz the power, 2015-11-16 03:37:56]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1450803002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1450803002/1

[commit-bot: I haz the power, 2015-11-16 03:37:56]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-11-16 03:37:57]

No L-G-T-M from a valid reviewer yet. Only full committers are accepted.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[zhaoqin, 2015-11-16 03:40:30]

Description was changed from

==========
Revert of Add unit tests for CdmAdapter (patchset #6 id:100001 of
https://codereview.chromium.org/1428753010/ )

Reason for revert:
valgrind reports leaks with no-debug-symbol callstack

BUG=556346

Original issue's description:
> Add unit tests for CdmAdapter
> 
> BUG=510088
> TEST=new unittests pass
> 
> Committed: https://crrev.com/30996193fe3e69fe9ccd52471900bf5685c54956
> Cr-Commit-Position: refs/heads/master@{#359582}

TBR=xhwang@chromium.org,jrummell@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=510088
==========

to

==========
Revert of Add unit tests for CdmAdapter (patchset #6 id:100001 of
https://codereview.chromium.org/1428753010/ )

Reason for revert:
valgrind reports leaks with no-debug-symbol callstack

BUG=556346
TBR=jrummell

Original issue's description:
> Add unit tests for CdmAdapter
> 
> BUG=510088
> TEST=new unittests pass
> 
> Committed: https://crrev.com/30996193fe3e69fe9ccd52471900bf5685c54956
> Cr-Commit-Position: refs/heads/master@{#359582}

TBR=xhwang@chromium.org,jrummell@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=510088
==========

[zhaoqin, 2015-11-16 03:40:36]

The CQ bit was checked by zhaoqin@google.com

[commit-bot: I haz the power, 2015-11-16 03:40:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1450803002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1450803002/1

[commit-bot: I haz the power, 2015-11-16 03:40:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-11-16 03:40:42]

No L-G-T-M from a valid reviewer yet. Only full committers are accepted.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[xhwang, 2015-12-05 06:23:48]

On 2015/11/16 03:40:42, commit-bot: I haz the power wrote:
> No L-G-T-M from a valid reviewer yet. Only full committers are accepted.
> Even if an L-G-T-M may have been provided, it was from a non-committer,
> _not_ a full super star committer.
> See http://www.chromium.org/getting-involved/become-a-committer
> Note that this has nothing to do with OWNERS files.

Is this CL still needed?

[chromium-reviews, 2015-12-05 23:25:22]

No, it is not needed any more.
On Dec 5, 2015 1:23 AM, <xhwang@chromium.org> wrote:

> On 2015/11/16 03:40:42, commit-bot: I haz the power wrote:
>
>> No L-G-T-M from a valid reviewer yet. Only full committers are accepted.
>> Even if an L-G-T-M may have been provided, it was from a non-committer,
>> _not_ a full super star committer.
>> See http://www.chromium.org/getting-involved/become-a-committer
>> Note that this has nothing to do with OWNERS files.
>>
>
> Is this CL still needed?
>
> https://chromiumcodereview.appspot.com/1450803002/
>

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.