Android: adds missing dependencies for gfx_unittests.

ui/gfx/gfx_tests.gyp

 # Copyright 2014 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.
 
 {
   'variables': {
     'chromium_code': 1,
   },
   'targets': [
     {
@@ skipping 97 matching lines @@
             }],
           ],
         }],
         ['use_ozone==1 and use_pango==0', {
           'sources!': [
             'font_list_unittest.cc',
             'font_unittest.cc',
             'text_elider_unittest.cc',
           ],
         }],
+        ['OS == "android"', {
+          'dependencies': [
+            '../../testing/android/native_test.gyp:native_test_native_code',

[msw, 2014/06/02 17:36:08]

Why is this specified as an explicit dependency in so many gyp files? Shouldn't
this be a dependency of base/base.gyp:test_support_base or something similar, to
avoid having separate includes for every test target like this?

[cjhopman, 2014/06/02 21:27:10]

Yeah, I'd like to see us come up with something where we don't have to
explicitly depend on this everywhere. I'd say that this stuff is similar to
base/test/launcher/* and, so, including it in base_test_support should be fine.

No need to block this change on that, though.

[msw, 2014/06/02 21:32:47]

cool, please do follow up or file a bug about this.

+          ],
+        }],
       ],
     }
   ],
   'conditions': [
     ['OS == "android"', {
       'targets': [
         {
           'target_name': 'gfx_unittests_apk',
           'type': 'none',
           'dependencies': [
+            '../android/ui_android.gyp:ui_java',
             'gfx_unittests',
           ],
           'variables': {
             'test_suite_name': 'gfx_unittests',
           },
           'includes': [ '../../build/apk_test.gypi' ],
         },
       ],
     }],
   ],
 }