media: Add MockCdmClient

media: Add MockCdmClient

This can be shared among multiple CDM unittests and deserve a place in
mock_filters.h

Also, by having a mock class, we can use StrictMock to turn uninterested
mock calls into test failure, which wasn't possible with standalone
MOCK_METHODs.

BUG=657069
TEST=This only refactored test infrastructure.

Committed: https://crrev.com/bc829a7d9755e01bf12444e6aa9c37cc074884b6
Cr-Commit-Position: refs/heads/master@{#436801}

[xhwang, 2016-12-06 20:39:09]

xhwang@chromium.org changed reviewers:
+ jrummell@chromium.org

[xhwang, 2016-12-06 20:39:10]

PTAL

[xhwang, 2016-12-06 20:51:49]

https://codereview.chromium.org/2554163002/diff/1/media/cdm/aes_decryptor_uni...
File media/cdm/aes_decryptor_unittest.cc (right):

https://codereview.chromium.org/2554163002/diff/1/media/cdm/aes_decryptor_uni...
media/cdm/aes_decryptor_unittest.cc:46: MATCHER(NotEmpty, "") {
IsEmpty/NotEmpty is consistent with IsNull/NotNull, and saves us 2 chars :)

[jrummell, 2016-12-06 20:53:53]

lgtm

[xhwang, 2016-12-06 21:00:33]

The CQ bit was checked by xhwang@chromium.org

[commit-bot: I haz the power, 2016-12-06 21:01:13]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-06 21:04:25]

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

[commit-bot: I haz the power, 2016-12-06 21:04:25]

Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[xhwang, 2016-12-06 22:38:43]

The CQ bit was checked by xhwang@chromium.org

[commit-bot: I haz the power, 2016-12-06 22:39:24]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-07 00:57:06]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481063923949510, "parent_rev":
"c721fb01816689de345c5d36963bb2070e2e9928", "commit_rev":
"abdb3beff81a7677ced7e4659e4fc7b0e476e5a1"}

[commit-bot: I haz the power, 2016-12-07 00:57:33]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-07 00:59:34]

Description was changed from

==========
media: Add MockCdmClient

This can be shared among multiple CDM unittests and deserve a place in
mock_filters.h

Also, by having a mock class, we can use StrictMock to turn uninterested
mock calls into test failure, which wasn't possible with standalone
MOCK_METHODs.

BUG=657069
TEST=This only refactored test infrastructure.
==========

to

==========
media: Add MockCdmClient

This can be shared among multiple CDM unittests and deserve a place in
mock_filters.h

Also, by having a mock class, we can use StrictMock to turn uninterested
mock calls into test failure, which wasn't possible with standalone
MOCK_METHODs.

BUG=657069
TEST=This only refactored test infrastructure.

Committed: https://crrev.com/bc829a7d9755e01bf12444e6aa9c37cc074884b6
Cr-Commit-Position: refs/heads/master@{#436801}
==========

[commit-bot: I haz the power, 2016-12-07 00:59:34]

Patchset 1 (id:??) landed as
https://crrev.com/bc829a7d9755e01bf12444e6aa9c37cc074884b6
Cr-Commit-Position: refs/heads/master@{#436801}