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

Side by Side Diff: chrome/browser/ui/views/bookmarks/bookmark_bar_view_test.cc

Issue 136643006: Clean up some ifdefs around Win Aura. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 6 years, 10 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
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #include "base/bind.h" 5 #include "base/bind.h"
6 #include "base/callback.h" 6 #include "base/callback.h"
7 #include "base/compiler_specific.h" 7 #include "base/compiler_specific.h"
8 #include "base/prefs/pref_service.h" 8 #include "base/prefs/pref_service.h"
9 #include "base/strings/string_number_conversions.h" 9 #include "base/strings/string_number_conversions.h"
10 #include "base/strings/utf_string_conversions.h" 10 #include "base/strings/utf_string_conversions.h"
(...skipping 699 matching lines...) Expand 10 before | Expand all | Expand 10 after
710 } 710 }
711 711
712 void Step6() { 712 void Step6() {
713 ASSERT_TRUE(model_->other_node()->GetChild(0)->url() == url_dragging_); 713 ASSERT_TRUE(model_->other_node()->GetChild(0)->url() == url_dragging_);
714 Done(); 714 Done();
715 } 715 }
716 716
717 GURL url_dragging_; 717 GURL url_dragging_;
718 }; 718 };
719 719
720 #if !(defined(OS_WIN) && defined(USE_AURA)) 720 #if !defined(OS_WIN)
721 // This test passes locally (on aero and non-aero) but fails on the trybots and 721 // This test passes locally (on aero and non-aero) but fails on the trybots and
722 // buildbot. 722 // buildbot.
723 // http://crbug.com/154081 723 // http://crbug.com/154081
724 VIEW_TEST(BookmarkBarViewTest7, MAYBE(DNDToDifferentMenu)) 724 VIEW_TEST(BookmarkBarViewTest7, MAYBE(DNDToDifferentMenu))
725 #endif 725 #endif
726 726
727 // Drags from one menu to next so that original menu closes, then back to 727 // Drags from one menu to next so that original menu closes, then back to
728 // original menu. 728 // original menu.
729 class BookmarkBarViewTest8 : public BookmarkBarViewEventTestBase { 729 class BookmarkBarViewTest8 : public BookmarkBarViewEventTestBase {
730 protected: 730 protected:
(...skipping 88 matching lines...) Expand 10 before | Expand all | Expand 10 after
819 // Make sure drop was processed. 819 // Make sure drop was processed.
820 GURL final_url = model_->bookmark_bar_node()->GetChild(0)->GetChild(0)-> 820 GURL final_url = model_->bookmark_bar_node()->GetChild(0)->GetChild(0)->
821 GetChild(1)->url(); 821 GetChild(1)->url();
822 ASSERT_TRUE(final_url == url_dragging_); 822 ASSERT_TRUE(final_url == url_dragging_);
823 Done(); 823 Done();
824 } 824 }
825 825
826 GURL url_dragging_; 826 GURL url_dragging_;
827 }; 827 };
828 828
829 #if !(defined(OS_WIN) && defined(USE_AURA)) 829 #if !defined(OS_WIN)
830 // This test passes locally (on aero and non-aero) but fails on the trybots and 830 // This test passes locally (on aero and non-aero) but fails on the trybots and
831 // buildbot. 831 // buildbot.
832 // http://crbug.com/154081 832 // http://crbug.com/154081
833 VIEW_TEST(BookmarkBarViewTest8, MAYBE(DNDBackToOriginatingMenu)) 833 VIEW_TEST(BookmarkBarViewTest8, MAYBE(DNDBackToOriginatingMenu))
834 #endif 834 #endif
835 835
836 // Moves the mouse over the scroll button and makes sure we get scrolling. 836 // Moves the mouse over the scroll button and makes sure we get scrolling.
837 class BookmarkBarViewTest9 : public BookmarkBarViewEventTestBase { 837 class BookmarkBarViewTest9 : public BookmarkBarViewEventTestBase {
838 protected: 838 protected:
839 virtual bool CreateBigMenu() OVERRIDE { return true; } 839 virtual bool CreateBigMenu() OVERRIDE { return true; }
(...skipping 980 matching lines...) Expand 10 before | Expand all | Expand 10 after
1820 EXPECT_TRUE(bb_view_->GetContextMenu() == NULL); 1820 EXPECT_TRUE(bb_view_->GetContextMenu() == NULL);
1821 EXPECT_TRUE(bb_view_->GetMenu() == NULL); 1821 EXPECT_TRUE(bb_view_->GetMenu() == NULL);
1822 1822
1823 Done(); 1823 Done();
1824 } 1824 }
1825 1825
1826 ContextMenuNotificationObserver observer_; 1826 ContextMenuNotificationObserver observer_;
1827 }; 1827 };
1828 1828
1829 VIEW_TEST(BookmarkBarViewTest21, ContextMenusForEmptyFolder) 1829 VIEW_TEST(BookmarkBarViewTest21, ContextMenusForEmptyFolder)
OLDNEW
« no previous file with comments | « chrome/browser/ui/toolbar/wrench_menu_model.cc ('k') | chrome/browser/ui/views/location_bar/star_view_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698