Add support for creating HW codecs in VideoProcessor tests.

webrtc/modules/video_coding/DEPS

Index: webrtc/modules/video_coding/DEPS
diff --git a/webrtc/modules/video_coding/DEPS b/webrtc/modules/video_coding/DEPS
index d8e3c22cd66a387ed6ba52764707ddd594318bd3..760a7b783f3390fbdcb2a8de7189499cd8ab8106 100644
--- a/webrtc/modules/video_coding/DEPS
+++ b/webrtc/modules/video_coding/DEPS
@@ -8,3 +8,14 @@ include_rules = [
   "+webrtc/system_wrappers",
   "+webrtc/tools",
 ]
+
+specific_include_rules = {
+  "android_test_initializer\.cc": [

[kjellander_webrtc, 2017/02/14 11:00:15]

Don't you need the codecs/test/ in the path? Maybe it's not needed...

[brandtr, 2017/02/14 14:22:37]

Without that prefix, 'git cl presubmit' passes, but with the prefix I get
errors:

** Presubmit ERRORS **
You added one or more #includes that violate checkdeps rules.
  webrtc/modules/video_coding/codecs/test/android_test_initializer.cc
      Illegal include: "webrtc/sdk/android/src/jni/classreferenceholder.h"
    Because of "-webrtc" from webrtc's include_rules. \
  webrtc/modules/video_coding/codecs/test/android_test_initializer.cc
      Illegal include: "webrtc/sdk/android/src/jni/jni_helpers.h"
    Because of "-webrtc" from webrtc's include_rules. \
  webrtc/modules/video_coding/codecs/test/android_test_initializer.cc
      Illegal include: "base/android/jni_android.h"
    Because of "-base" from webrtc's include_rules. \
  webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.h
      Illegal include: "webrtc/sdk/android/src/jni/androidmediadecoder_jni.h"
    Because of "-webrtc" from webrtc's include_rules. \
  webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.h
      Illegal include: "webrtc/sdk/android/src/jni/androidmediaencoder_jni.h"
    Because of "-webrtc" from webrtc's include_rules. \
  webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.h
      Illegal include:
"webrtc/sdk/objc/Framework/Classes/h264_video_toolbox_decoder.h"
    Because of "-webrtc" from webrtc's include_rules. \
  webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.h
      Illegal include:
"webrtc/sdk/objc/Framework/Classes/h264_video_toolbox_encoder.h"
    Because of "-webrtc" from webrtc's include_rules. \
  webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.h
      Illegal include: "webrtc/media/engine/webrtcvideodecoderfactory.h"
    Because of "-webrtc" from webrtc's include_rules. \
  webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.h
      Illegal include: "webrtc/media/engine/webrtcvideoencoderfactory.h"
    Because of "-webrtc" from webrtc's include_rules.

[kjellander_webrtc, 2017/02/14 14:30:13]

I see. It doesn't take the path into account then (I didn't look into the
details on how the check is implemented).

+    "+base/android",
+    "+webrtc/sdk",
+  ],
+  "videoprocessor_integrationtest\.h": [

[kjellander_webrtc, 2017/02/14 11:00:15]

I'd recommend ".*test\.cc" instead. It would avoid the extra hassle when adding
more tests in the future.

[brandtr, 2017/02/14 14:22:37]

Done. In order to get it to pass, I had to do ".*test\.h" however. Do you think
I should add ".*test\.cc" as well, even though it's not strictly needed here?

(I tried to add ".*test\.{cc,h}", to no avail.)

[kjellander_webrtc, 2017/02/14 14:30:13]

Hmm, 
By looking at other examples
(https://cs.chromium.org/chromium/src/apps/DEPS?rcl=ea69b8739b1bf1a220ea541cca...)
it seems this should work at least:
"(.*test\.cc|.*test\.h)"

[brandtr, 2017/02/14 14:34:20]

It did. Thanks!

+     "+webrtc/media/engine",
+     "+webrtc/sdk",
+  ],
+}