Handle media access permission request

android_webview/android_webview_tests.gypi

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 {
   'targets': [
     {
       'target_name': 'android_webview_apk',
       'type': 'none',
       'dependencies': [
         'libwebviewchromium',
@@ skipping 71 matching lines @@
       'dependencies': [
         '../base/base.gyp:test_support_base',
         '../content/content_shell_and_tests.gyp:test_support_content',
         '../net/net.gyp:net_test_support',
         '../testing/android/native_test.gyp:native_test_native_code',
         '../testing/gmock.gyp:gmock',
         '../testing/gtest.gyp:gtest',
         '../ui/base/ui_base.gyp:ui_base_jni_headers',
         'android_webview_common',
         'android_webview_unittests_jni',
+        'libwebviewchromium',

[mkosiba (inactive), 2014/04/23 18:04:40]

I'm surprised it doesn't crash. - you've just introduced a 2nd entry point to
the test binary.

what error do you get without this? All the webview code the tests depend on
should already be part of android_webview_common.

[michaelbai, 2014/04/24 00:56:43]

oops, it should be removed, I were hoping it could avoid call RegisterJni. there
is no crash, why you think it is add 2nd entry point, were you talking about
Jni_onload?

On 2014/04/23 18:04:40, mkosiba wrote:

[mkosiba (inactive), 2014/04/24 09:24:58]

yes, testing/android/native_test_launcher.cc defines JNI_OnLoad which I thought
would be shadowed by the one from
android_webview/lib/main/webview_entry_point.cc

       ],
       'include_dirs': [
         '..',
         '../skia/config',
         '<(SHARED_INTERMEDIATE_DIR)/android_webview_unittests',
       ],
       'sources': [
         'browser/aw_form_database_service_unittest.cc',
         'browser/net/android_stream_reader_url_request_job_unittest.cc',
         'browser/net/input_stream_reader_unittest.cc',
         'lib/main/webview_tests.cc',
         'native/input_stream_unittest.cc',
+        'native/permission/media_access_permission_request_unittest.cc',
+        'native/permission/permission_request_handler_unittest.cc',
         'native/state_serializer_unittest.cc',
       ],
     },
     {
       'target_name': 'android_webview_unittest_java',
       'type': 'none',
       'dependencies': [
         '../base/base.gyp:base_java_test_support',
         '../content/content_shell_and_tests.gyp:content_java_test_support',
         'android_webview_java',
@@ skipping 22 matching lines @@
         'android_webview_unittest_java',
         'android_webview_unittests',
       ],
       'variables': {
         'test_suite_name': 'android_webview_unittests',
       },
       'includes': [ '../build/apk_test.gypi' ],
     },
   ],
 }