Extract media/capture/capture_unittests from media_unittests

Extract media/capture/capture_unittests from media_unittests

This CL:
- extracts capture_unittests out of media_unittests.
- adds it to all testing/buildbot/*.json where appropriate,
  i.e. where there is media_unittests.
- adds the necessary .isolate and _apk.isolate.

BUG=618718
TEST= No new code added, all bots running capture_unittests
and being happy with it.¯

TBR=miu@chromium.org

Committed: https://crrev.com/13b4123d553a41899acebba9020b99ace0f2472d
Cr-Commit-Position: refs/heads/master@{#401692}

[mcasas, 2016-06-20 22:46:44]

Description was changed from

==========
[WIP] Extract capture_unittests from media/


BUG=618718
==========

to

==========
[WIP] Extract capture_unittests from media/


[WIP]

Needs changing
build/scripts/slave/recipe_modules/isolate/resources/compare_build_artifacts.py

BUG=618718
==========

[mcasas, 2016-06-20 22:47:04]

Patchset #1 (id:1) has been deleted

[mcasas, 2016-06-20 23:00:31]

Description was changed from

==========
[WIP] Extract capture_unittests from media/


[WIP]

Needs changing
build/scripts/slave/recipe_modules/isolate/resources/compare_build_artifacts.py

BUG=618718
==========

to

==========
[WIP] Extract capture_unittests from media/


[WIP]

Needs changing
build/scripts/slave/recipe_modules/isolate/resources/compare_build_artifacts.py

Also needs adding capture_unittests to
testing/buildbot/*.json where appropriate

BUG=618718
==========

[mcasas, 2016-06-21 22:25:48]

Description was changed from

==========
[WIP] Extract capture_unittests from media/


[WIP]

Needs changing
build/scripts/slave/recipe_modules/isolate/resources/compare_build_artifacts.py

Also needs adding capture_unittests to
testing/buildbot/*.json where appropriate

BUG=618718
==========

to

==========
[WIP] Extract capture_unittests from media/


[WIP]

This CL:
- extracts capture_unittests out of media_unittests.
- adds it to all testing/buildbot/*.json where appropriate,
i.e. where there is media_unittests. 
- adds the necessary .isolate and _apk.isolate.

Needs changing
build/scripts/slave/recipe_modules/isolate/resources/compare_build_artifacts.py

BUG=618718
==========

[mcasas, 2016-06-21 22:25:59]

Patchset #1 (id:20001) has been deleted

[mcasas, 2016-06-21 22:26:09]

Patchset #1 (id:40001) has been deleted

[mcasas, 2016-06-21 23:42:46]

Description was changed from

==========
[WIP] Extract capture_unittests from media/


[WIP]

This CL:
- extracts capture_unittests out of media_unittests.
- adds it to all testing/buildbot/*.json where appropriate,
i.e. where there is media_unittests. 
- adds the necessary .isolate and _apk.isolate.

Needs changing
build/scripts/slave/recipe_modules/isolate/resources/compare_build_artifacts.py

BUG=618718
==========

to

==========
Extract media/capture/capture_unittests from media_unittests

This CL:
- extracts capture_unittests out of media_unittests.
- adds it to all testing/buildbot/*.json where appropriate,
  i.e. where there is media_unittests. 
- adds the necessary .isolate and _apk.isolate.

I'm uncertain if this needs changing the file:
build/scripts/slave/recipe_modules/isolate/resources/compare_build_artifacts.py

BUG=618718
TEST= No new code added, all bots running capture_unittests
and being happy with it.
==========

[mcasas, 2016-06-21 23:54:53]

mcasas@chromium.org changed reviewers:
+ maruel@chromium.org, miu@chromium.org, xhwang@chromium.org

[mcasas, 2016-06-21 23:54:54]

xhwang@ PTAL at media/*

miu@ PTAL at the changes in media/capture/*

maruel@ PTAL at testing/builbot/* and the .isolates

[M-A Ruel, 2016-06-22 00:02:33]

Description was changed from

==========
Extract media/capture/capture_unittests from media_unittests

This CL:
- extracts capture_unittests out of media_unittests.
- adds it to all testing/buildbot/*.json where appropriate,
  i.e. where there is media_unittests. 
- adds the necessary .isolate and _apk.isolate.

I'm uncertain if this needs changing the file:
build/scripts/slave/recipe_modules/isolate/resources/compare_build_artifacts.py

BUG=618718
TEST= No new code added, all bots running capture_unittests
and being happy with it.
==========

to

==========
Extract media/capture/capture_unittests from media_unittests

This CL:
- extracts capture_unittests out of media_unittests.
- adds it to all testing/buildbot/*.json where appropriate,
  i.e. where there is media_unittests. 
- adds the necessary .isolate and _apk.isolate.


BUG=618718
TEST= No new code added, all bots running capture_unittests
and being happy with it.
==========

[M-A Ruel, 2016-06-22 00:04:14]

lgtm

Removed the following from the CL description:
> I'm uncertain if this needs changing the file:
>
build/scripts/slave/recipe_modules/isolate/resources/compare_build_artifacts.py

questions are not part of commit messages. The answer is that you don't need to
update it so you're fine.

[xhwang, 2016-06-22 04:59:20]

Thanks for closing the loop. LGTM!

[mcasas, 2016-06-22 15:50:31]

mcasas@chromium.org changed reviewers:
+ brettw@chromium.orgbrettw

[mcasas, 2016-06-22 15:50:31]

brettw@ RS the two top level build files plz

[mcasas, 2016-06-22 15:54:38]

mcasas@chromium.org changed reviewers:
+ brettw@chromium.org
- brettw@chromium.orgbrettw

[mcasas, 2016-06-22 15:54:39]

brettw@ RS the top level files please.

[brettw, 2016-06-22 18:22:59]

rs lgtm

[mcasas, 2016-06-22 18:32:13]

On 2016/06/22 18:22:59, brettw wrote:
> rs lgtm

miu@ ping

[mcasas, 2016-06-23 13:30:50]

Description was changed from

==========
Extract media/capture/capture_unittests from media_unittests

This CL:
- extracts capture_unittests out of media_unittests.
- adds it to all testing/buildbot/*.json where appropriate,
  i.e. where there is media_unittests. 
- adds the necessary .isolate and _apk.isolate.


BUG=618718
TEST= No new code added, all bots running capture_unittests
and being happy with it.
==========

to

==========
Extract media/capture/capture_unittests from media_unittests

This CL:
- extracts capture_unittests out of media_unittests.
- adds it to all testing/buildbot/*.json where appropriate,
  i.e. where there is media_unittests. 
- adds the necessary .isolate and _apk.isolate.


BUG=618718
TEST= No new code added, all bots running capture_unittests
and being happy with it.¯

TBR=miu@chromium.org
==========

[mcasas, 2016-06-23 13:30:56]

The CQ bit was checked by mcasas@chromium.org

[commit-bot: I haz the power, 2016-06-23 13:31:07]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2087683002/80001

[commit-bot: I haz the power, 2016-06-23 13:32:50]

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

[commit-bot: I haz the power, 2016-06-23 13:32:52]

Try jobs failed on following builders:
  ios-device on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-gn/bui...)
  ios-simulator-gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-gn/...)

[mcasas, 2016-06-23 15:41:58]

The CQ bit was checked by mcasas@chromium.org

[mcasas, 2016-06-23 15:41:59]

The patchset sent to the CQ was uploaded after l-g-t-m from brettw@chromium.org,
xhwang@chromium.org, maruel@chromium.org
Link to the patchset: https://codereview.chromium.org/2087683002/#ps100001
(title: "Rebased (testing/buildbot/chromium.mac.json)")

[commit-bot: I haz the power, 2016-06-23 15:42:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2087683002/100001

[commit-bot: I haz the power, 2016-06-23 16:52:15]

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

[commit-bot: I haz the power, 2016-06-23 16:52:17]

Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[mcasas, 2016-06-23 18:52:09]

The CQ bit was checked by mcasas@chromium.org

[mcasas, 2016-06-23 18:52:10]

The patchset sent to the CQ was uploaded after l-g-t-m from brettw@chromium.org,
xhwang@chromium.org, maruel@chromium.org
Link to the patchset: https://codereview.chromium.org/2087683002/#ps120001
(title: "media/base/run_all_unittests.cc shouldn't know about
RegisterCaptureJni")

[commit-bot: I haz the power, 2016-06-23 18:52:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2087683002/120001

[commit-bot: I haz the power, 2016-06-23 19:52:23]

Description was changed from

==========
Extract media/capture/capture_unittests from media_unittests

This CL:
- extracts capture_unittests out of media_unittests.
- adds it to all testing/buildbot/*.json where appropriate,
  i.e. where there is media_unittests. 
- adds the necessary .isolate and _apk.isolate.


BUG=618718
TEST= No new code added, all bots running capture_unittests
and being happy with it.¯

TBR=miu@chromium.org
==========

to

==========
Extract media/capture/capture_unittests from media_unittests

This CL:
- extracts capture_unittests out of media_unittests.
- adds it to all testing/buildbot/*.json where appropriate,
  i.e. where there is media_unittests. 
- adds the necessary .isolate and _apk.isolate.


BUG=618718
TEST= No new code added, all bots running capture_unittests
and being happy with it.¯

TBR=miu@chromium.org
==========

[commit-bot: I haz the power, 2016-06-23 19:52:26]

Committed patchset #4 (id:120001)

[commit-bot: I haz the power, 2016-06-23 19:55:46]

Description was changed from

==========
Extract media/capture/capture_unittests from media_unittests

This CL:
- extracts capture_unittests out of media_unittests.
- adds it to all testing/buildbot/*.json where appropriate,
  i.e. where there is media_unittests. 
- adds the necessary .isolate and _apk.isolate.


BUG=618718
TEST= No new code added, all bots running capture_unittests
and being happy with it.¯

TBR=miu@chromium.org
==========

to

==========
Extract media/capture/capture_unittests from media_unittests

This CL:
- extracts capture_unittests out of media_unittests.
- adds it to all testing/buildbot/*.json where appropriate,
  i.e. where there is media_unittests.
- adds the necessary .isolate and _apk.isolate.

BUG=618718
TEST= No new code added, all bots running capture_unittests
and being happy with it.¯

TBR=miu@chromium.org

Committed: https://crrev.com/13b4123d553a41899acebba9020b99ace0f2472d
Cr-Commit-Position: refs/heads/master@{#401692}
==========

[commit-bot: I haz the power, 2016-06-23 19:55:47]

Patchset 4 (id:??) landed as
https://crrev.com/13b4123d553a41899acebba9020b99ace0f2472d
Cr-Commit-Position: refs/heads/master@{#401692}

[mcasas, 2016-06-23 21:15:53]

A revert of this CL (patchset #4 id:120001) has been created in
https://codereview.chromium.org/2093563004/ by mcasas@chromium.org.

The reason for reverting is: Broke Win x64 bot 
https://build.chromium.org/p/chromium/builders/Win%20x64/builds/1849

FAILED: audio_unittests.exe audio_unittests.exe.pdb 
C:\b\depot_tools\python276_bin\python.exe gyp-win-tool link-with-manifests
environment.x64 True audio_unittests.exe
"C:\b\depot_tools\python276_bin\python.exe gyp-win-tool link-wrapper
environment.x64 False link.exe /nologo /OUT:audio_unittests.exe
@audio_unittests.exe.rsp" 1 mt.exe rc.exe
"obj\media\audio_unittests.audio_unittests.exe.intermediate.manifest"
obj\media\audio_unittests.audio_unittests.exe.generated.manifest
..\..\build\win\compatibility.manifest
media.lib(media.core_audio_util_win.obj) : error LNK2019: unresolved external
symbol CLSID_DxDiagProvider referenced in function "public: static bool __cdecl
media::CoreAudioUtil::GetDxDiagDetails(class std::basic_string<char,struct
std::char_traits<char>,class std::allocator<char> > *,class
std::basic_string<char,struct std::char_traits<char>,class std::allocator<char>
> *)"
(?GetDxDiagDetails@CoreAudioUtil@media@@SA_NPEAV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@0@Z)

media.lib(media.core_audio_util_win.obj) : error LNK2019: unresolved external
symbol IID_IDxDiagProvider referenced in function "public: static bool __cdecl
media::CoreAudioUtil::GetDxDiagDetails(class std::basic_string<char,struct
std::char_traits<char>,class std::allocator<char> > *,class
std::basic_string<char,struct std::char_traits<char>,class std::allocator<char>
> *)"
(?GetDxDiagDetails@CoreAudioUtil@media@@SA_NPEAV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@0@Z)

audio_unittests.exe : fatal error LNK1120: 2 unresolved externals
.