Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(605)

Side by Side Diff: components/components_tests.gyp

Issue 563923002: display error message for empty distilled content (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: move test to browsertest to bypass missing resources on ios/android Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | components/dom_distiller/content/distiller_page_web_contents_browsertest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 { 5 {
6 'variables': { 6 'variables': {
7 # This turns on e.g. the filename-based detection of which 7 # This turns on e.g. the filename-based detection of which
8 # platforms to include source files on (e.g. files ending in 8 # platforms to include source files on (e.g. files ending in
9 # _mac.h or _mac.cc are only compiled on MacOSX). 9 # _mac.h or _mac.cc are only compiled on MacOSX).
10 'chromium_code': 1, 10 'chromium_code': 1,
(...skipping 813 matching lines...) Expand 10 before | Expand all | Expand 10 after
824 ], 824 ],
825 }], 825 }],
826 ['OS!="ios"', { 826 ['OS!="ios"', {
827 'targets': [ 827 'targets': [
828 { 828 {
829 'target_name': 'components_browsertests', 829 'target_name': 'components_browsertests',
830 'type': '<(gtest_target_type)', 830 'type': '<(gtest_target_type)',
831 'defines!': ['CONTENT_IMPLEMENTATION'], 831 'defines!': ['CONTENT_IMPLEMENTATION'],
832 'dependencies': [ 832 'dependencies': [
833 'components.gyp:autofill_content_browser', 833 'components.gyp:autofill_content_browser',
834 'components.gyp:dom_distiller_content',
835 'components.gyp:dom_distiller_core',
836 'components.gyp:password_manager_content_renderer', 834 'components.gyp:password_manager_content_renderer',
837 'components.gyp:pref_registry_test_support', 835 'components.gyp:pref_registry_test_support',
838 'components_resources.gyp:components_resources', 836 'components_resources.gyp:components_resources',
839 '../content/content.gyp:content_common', 837 '../content/content.gyp:content_common',
840 '../content/content.gyp:content_gpu', 838 '../content/content.gyp:content_gpu',
841 '../content/content.gyp:content_plugin', 839 '../content/content.gyp:content_plugin',
842 '../content/content.gyp:content_renderer', 840 '../content/content.gyp:content_renderer',
843 '../content/content_shell_and_tests.gyp:content_browser_test_support ', 841 '../content/content_shell_and_tests.gyp:content_browser_test_support ',
844 '../content/content_shell_and_tests.gyp:content_shell_lib', 842 '../content/content_shell_and_tests.gyp:content_shell_lib',
845 '../content/content_shell_and_tests.gyp:content_shell_pak', 843 '../content/content_shell_and_tests.gyp:content_shell_pak',
846 '../content/content_shell_and_tests.gyp:test_support_content', 844 '../content/content_shell_and_tests.gyp:test_support_content',
847 '../skia/skia.gyp:skia', 845 '../skia/skia.gyp:skia',
848 '../testing/gmock.gyp:gmock', 846 '../testing/gmock.gyp:gmock',
849 '../testing/gtest.gyp:gtest', 847 '../testing/gtest.gyp:gtest',
848
849 # Dependencies of dom_distiller
nyquist 2014/09/25 04:55:59 These now become out of order. Was your thought th
kuan 2014/09/25 14:53:06 yes, like @304.
850 'components.gyp:dom_distiller_content',
851 'components.gyp:dom_distiller_core',
852 'components_strings.gyp:components_strings',
850 ], 853 ],
851 'include_dirs': [ 854 'include_dirs': [
852 '..', 855 '..',
853 ], 856 ],
854 'defines': [ 857 'defines': [
855 'HAS_OUT_OF_PROC_TEST_RUNNER', 858 'HAS_OUT_OF_PROC_TEST_RUNNER',
856 ], 859 ],
857 'sources': [ 860 'sources': [
858 'autofill/content/browser/risk/fingerprint_browsertest.cc', 861 'autofill/content/browser/risk/fingerprint_browsertest.cc',
859 'dom_distiller/content/distiller_page_web_contents_browsertest.cc', 862 'dom_distiller/content/distiller_page_web_contents_browsertest.cc',
(...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
926 'dependencies': [ 929 'dependencies': [
927 '../base/allocator/allocator.gyp:allocator', 930 '../base/allocator/allocator.gyp:allocator',
928 ], 931 ],
929 }], 932 }],
930 ], 933 ],
931 }, 934 },
932 ], 935 ],
933 }], 936 }],
934 ], 937 ],
935 } 938 }
OLDNEW
« no previous file with comments | « no previous file | components/dom_distiller/content/distiller_page_web_contents_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698