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

Side by Side Diff: chrome/browser/cocoa/bug_report_window_controller_unittest.mm

Issue 438005: Enable sync by default on all platforms. (Closed)
Patch Set: Synced to head Created 11 years 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 | chrome/browser/sync/profile_sync_service.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 (c) 2009 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2009 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 #import <Cocoa/Cocoa.h> 5 #import <Cocoa/Cocoa.h>
6 6
7 #include "base/ref_counted.h" 7 #include "base/ref_counted.h"
8 #import "chrome/browser/cocoa/bug_report_window_controller.h" 8 #import "chrome/browser/cocoa/bug_report_window_controller.h"
9 #include "chrome/browser/renderer_host/site_instance.h" 9 #include "chrome/browser/renderer_host/site_instance.h"
10 #include "chrome/browser/renderer_host/test/test_render_view_host.h" 10 #include "chrome/browser/renderer_host/test/test_render_view_host.h"
11 #include "chrome/browser/tab_contents/test_tab_contents.h" 11 #include "chrome/browser/tab_contents/test_tab_contents.h"
12 #include "chrome/browser/profile.h" 12 #include "chrome/browser/profile.h"
13 13
14 namespace { 14 namespace {
15 15
16 class BugReportWindowControllerUnittest : public RenderViewHostTestHarness { 16 class BugReportWindowControllerUnittest : public RenderViewHostTestHarness {
17 public:
18 virtual void SetUp() {
19 RenderViewHostTestHarness::SetUp();
20 // This is needed since the new tab page queries the sync service.
21 profile_->CreateProfileSyncService();
22 }
17 }; 23 };
18 24
19 // See http://crbug.com/29019 for why it's disabled. 25 // See http://crbug.com/29019 for why it's disabled.
20 TEST_F(BugReportWindowControllerUnittest, DISABLED_ReportBugWithNewTabPageOpen) { 26 TEST_F(BugReportWindowControllerUnittest, DISABLED_ReportBugWithNewTabPageOpen) {
21 ChromeThread ui_thread(ChromeThread::UI, MessageLoop::current()); 27 ChromeThread ui_thread(ChromeThread::UI, MessageLoop::current());
22 // Create a "chrome://newtab" test tab. SiteInstance will be deleted when 28 // Create a "chrome://newtab" test tab. SiteInstance will be deleted when
23 // tabContents is deleted. 29 // tabContents is deleted.
24 SiteInstance* instance = 30 SiteInstance* instance =
25 SiteInstance::CreateSiteInstance(profile_.get()); 31 SiteInstance::CreateSiteInstance(profile_.get());
26 TestTabContents* tabContents = new TestTabContents(profile_.get(), 32 TestTabContents* tabContents = new TestTabContents(profile_.get(),
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
66 // that don't refer to a specific page should be available, and the send 72 // that don't refer to a specific page should be available, and the send
67 // screenshot option should be turned off. 73 // screenshot option should be turned off.
68 EXPECT_EQ([[controller bugTypeList] count], 4U); 74 EXPECT_EQ([[controller bugTypeList] count], 4U);
69 EXPECT_FALSE([controller sendScreenshot]); 75 EXPECT_FALSE([controller sendScreenshot]);
70 76
71 [controller release]; 77 [controller release];
72 } 78 }
73 79
74 } // namespace 80 } // namespace
75 81
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/sync/profile_sync_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698