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

Unified Diff: chrome/chrome_tests.gypi

Issue 10985069: [test fixlet] Add tests for the zoom icon in the location bar on GTK. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 8 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 side-by-side diff with in-line comments
Download patch
« chrome/browser/ui/view_ids.h ('K') | « chrome/browser/ui/view_ids.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/chrome_tests.gypi
diff --git a/chrome/chrome_tests.gypi b/chrome/chrome_tests.gypi
index f1d905d5ea49285e21bd136c09928a42a64c0a16..c18fb0c04fdd91ddfe2436463b9aff2407432f48 100644
--- a/chrome/chrome_tests.gypi
+++ b/chrome/chrome_tests.gypi
@@ -509,6 +509,7 @@
'browser/task_manager/task_manager_browsertest_util.cc',
'browser/ui/fullscreen/fullscreen_controller_interactive_browsertest.cc',
'browser/ui/gtk/bookmarks/bookmark_bar_gtk_interactive_uitest.cc',
+ 'browser/ui/gtk/zoom_bubble_gtk_browsertest.cc',
'browser/ui/omnibox/omnibox_view_browsertest.cc',
'browser/ui/panels/detached_panel_browsertest.cc',
'browser/ui/panels/docked_panel_browsertest.cc',
@@ -3046,6 +3047,9 @@
'browser/ui/find_bar/find_bar_host_browsertest.cc',
'browser/ui/fullscreen/fullscreen_controller_browsertest.cc',
'browser/ui/global_error/global_error_service_browsertest.cc',
+ 'browser/ui/gtk/bubble/bubble_gtk_browsertest.cc',
+ 'browser/ui/gtk/confirm_bubble_gtk_browsertest.cc',
+ 'browser/ui/gtk/location_bar_view_gtk_browsertest.cc',
'browser/ui/gtk/one_click_signin_bubble_gtk_browsertest.cc',
'browser/ui/gtk/view_id_util_browsertest.cc',
'browser/ui/intents/web_intent_picker_controller_browsertest.cc',
@@ -3360,13 +3364,6 @@
'../build/linux/system.gyp:ssl',
],
}],
- ['toolkit_uses_gtk == 1 and toolkit_views == 0', {
- 'sources': [
- # BubbleGtk is used only on Linux/GTK.
- 'browser/ui/gtk/bubble/bubble_gtk_browsertest.cc',
- 'browser/ui/gtk/confirm_bubble_gtk_browsertest.cc',
Evan Stade 2012/10/02 18:35:59 shouldn't the /gtk/ directory be enough to make th
Dan Beam 2012/10/02 19:34:14 you'd think so. so that means this is OK to remove
Lei Zhang 2012/10/02 19:56:25 Ya. I'd still leave the filename_rules.gypi change
- ],
- }],
['OS=="mac"', {
'include_dirs': [
'../third_party/GTM',
« chrome/browser/ui/view_ids.h ('K') | « chrome/browser/ui/view_ids.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698