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

Side by Side Diff: services/clipboard/clipboard_apptest.cc

Issue 1990603002: Make ApplicationTestBase not use ApplicationImpl. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: delete ApplicationImpl::WaitForInitialize() Created 4 years, 7 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
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 #include "base/bind.h" 5 #include "base/bind.h"
6 #include "base/run_loop.h" 6 #include "base/run_loop.h"
7 #include "mojo/converters/array_string/array_string_type_converters.h" 7 #include "mojo/converters/array_string/array_string_type_converters.h"
8 #include "mojo/public/cpp/application/application_impl.h"
9 #include "mojo/public/cpp/application/application_test_base.h" 8 #include "mojo/public/cpp/application/application_test_base.h"
10 #include "mojo/public/cpp/application/connect.h" 9 #include "mojo/public/cpp/application/connect.h"
11 #include "mojo/services/clipboard/interfaces/clipboard.mojom.h" 10 #include "mojo/services/clipboard/interfaces/clipboard.mojom.h"
12 11
13 using mojo::Array; 12 using mojo::Array;
14 using mojo::Clipboard; 13 using mojo::Clipboard;
15 using mojo::Map; 14 using mojo::Map;
16 using mojo::String; 15 using mojo::String;
17 16
18 namespace { 17 namespace {
(...skipping 29 matching lines...) Expand all
48 47
49 namespace clipboard { 48 namespace clipboard {
50 49
51 class ClipboardAppTest : public mojo::test::ApplicationTestBase { 50 class ClipboardAppTest : public mojo::test::ApplicationTestBase {
52 public: 51 public:
53 ClipboardAppTest() : ApplicationTestBase() {} 52 ClipboardAppTest() : ApplicationTestBase() {}
54 ~ClipboardAppTest() override {} 53 ~ClipboardAppTest() override {}
55 54
56 void SetUp() override { 55 void SetUp() override {
57 mojo::test::ApplicationTestBase::SetUp(); 56 mojo::test::ApplicationTestBase::SetUp();
58 mojo::ConnectToService(application_impl()->shell(), "mojo:clipboard", 57 mojo::ConnectToService(shell(), "mojo:clipboard", GetProxy(&clipboard_));
59 GetProxy(&clipboard_));
60 } 58 }
61 59
62 uint64_t GetSequenceNumber() { 60 uint64_t GetSequenceNumber() {
63 base::RunLoop run_loop; 61 base::RunLoop run_loop;
64 uint64_t sequence_num = 999999; 62 uint64_t sequence_num = 999999;
65 clipboard_->GetSequenceNumber( 63 clipboard_->GetSequenceNumber(
66 Clipboard::Type::COPY_PASTE, 64 Clipboard::Type::COPY_PASTE,
67 base::Bind(&CopyUint64AndEndRunloop, &sequence_num, &run_loop)); 65 base::Bind(&CopyUint64AndEndRunloop, &sequence_num, &run_loop));
68 run_loop.Run(); 66 run_loop.Run();
69 return sequence_num; 67 return sequence_num;
(...skipping 79 matching lines...) Expand 10 before | Expand all | Expand 10 after
149 EXPECT_EQ(kPlainTextData, data); 147 EXPECT_EQ(kPlainTextData, data);
150 148
151 Map<String, Array<uint8_t>> mime_data; 149 Map<String, Array<uint8_t>> mime_data;
152 clipboard_->WriteClipboardData(Clipboard::Type::COPY_PASTE, mime_data.Pass()); 150 clipboard_->WriteClipboardData(Clipboard::Type::COPY_PASTE, mime_data.Pass());
153 151
154 EXPECT_EQ(2ul, GetSequenceNumber()); 152 EXPECT_EQ(2ul, GetSequenceNumber());
155 EXPECT_FALSE(GetDataOfType(Clipboard::MIME_TYPE_TEXT, &data)); 153 EXPECT_FALSE(GetDataOfType(Clipboard::MIME_TYPE_TEXT, &data));
156 } 154 }
157 155
158 } // namespace clipboard 156 } // namespace clipboard
OLDNEW
« no previous file with comments | « services/authentication/accounts_db_manager_unittest.cc ('k') | services/contacts/contacts_apptest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698