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

Side by Side Diff: extensions/extensions.gyp

Issue 417603002: Remove duplicate extensions unit tests from chrome_tests_unit.gypi (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 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 | Annotate | Revision Log
« no previous file with comments | « chrome/chrome_tests_unit.gypi ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2013 The Chromium Authors. All rights reserved. 1 # Copyright 2013 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 'chromium_code': 1, 7 'chromium_code': 1,
8 }, 8 },
9 'targets': [ 9 'targets': [
10 { 10 {
(...skipping 671 matching lines...) Expand 10 before | Expand all | Expand 10 after
682 'common/test_util.h', 682 'common/test_util.h',
683 'common/value_builder.cc', 683 'common/value_builder.cc',
684 'common/value_builder.h', 684 'common/value_builder.h',
685 'renderer/test_extensions_renderer_client.cc', 685 'renderer/test_extensions_renderer_client.cc',
686 'renderer/test_extensions_renderer_client.h', 686 'renderer/test_extensions_renderer_client.h',
687 ], 687 ],
688 # Disable c4267 warnings until we fix size_t to int truncations. 688 # Disable c4267 warnings until we fix size_t to int truncations.
689 'msvs_disabled_warnings': [ 4267, ], 689 'msvs_disabled_warnings': [ 4267, ],
690 }, 690 },
691 { 691 {
692 # TODO(tfarina): Our plan is to build and run this target on Chromium bots 692 # TODO(tfarina): Many extension unit tests run as part of Chrome's
693 # (TS, CQ, Waterfall). First we will get this target passing all tests, 693 # unit_tests target. They should be moved here, which may require some
694 # after that we will start the work on buildbot to get this running there. 694 # refactoring (ExtensionsBrowserClient, TestingProfile, etc.).
695 # When we consider this stable in the bots, we can go to unit_tests target
696 # and remove the duplicated entries from there, otherwise if we just
697 # remove them right now we would be losing coverage.
698 # http://crbug.com/348066 695 # http://crbug.com/348066
699 'target_name': 'extensions_unittests', 696 'target_name': 'extensions_unittests',
700 'type': 'executable', 697 'type': 'executable',
701 'dependencies': [ 698 'dependencies': [
702 '../base/base.gyp:base', 699 '../base/base.gyp:base',
703 '../base/base.gyp:test_support_base', 700 '../base/base.gyp:test_support_base',
704 '../content/content_shell_and_tests.gyp:test_support_content', 701 '../content/content_shell_and_tests.gyp:test_support_content',
705 '../testing/gmock.gyp:gmock', 702 '../testing/gmock.gyp:gmock',
706 '../testing/gtest.gyp:gtest', 703 '../testing/gtest.gyp:gtest',
707 'extensions_common', 704 'extensions_common',
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
750 '<(SHARED_INTERMEDIATE_DIR)/extensions/extensions_renderer_resourc es.pak', 747 '<(SHARED_INTERMEDIATE_DIR)/extensions/extensions_renderer_resourc es.pak',
751 ], 748 ],
752 'pak_output': '<(PRODUCT_DIR)/extensions_unittests_resources.pak', 749 'pak_output': '<(PRODUCT_DIR)/extensions_unittests_resources.pak',
753 }, 750 },
754 'includes': [ '../build/repack_action.gypi' ], 751 'includes': [ '../build/repack_action.gypi' ],
755 }, 752 },
756 ], 753 ],
757 }, 754 },
758 ] 755 ]
759 } 756 }
OLDNEW
« no previous file with comments | « chrome/chrome_tests_unit.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698