Index: ui/arc/notification/arc_notification_manager_unittest.cc |
diff --git a/ui/arc/notification/arc_notification_manager_unittest.cc b/ui/arc/notification/arc_notification_manager_unittest.cc |
index 4ce88b6805103b6fa87d6f5faed2429fc3c117e7..43c9662bdde868ddf9e86a8b16d7f9529ae8db7a 100644 |
--- a/ui/arc/notification/arc_notification_manager_unittest.cc |
+++ b/ui/arc/notification/arc_notification_manager_unittest.cc |
@@ -2,6 +2,8 @@ |
// Use of this source code is governed by a BSD-style license that can be |
// found in the LICENSE file. |
+#include <vector> |
hidehiko
2016/07/11 06:05:18
nit: <string> and <memory>, too?
Luis Héctor Chávez
2016/07/11 17:15:08
Addressed in previous change.
|
+ |
#include "base/message_loop/message_loop.h" |
#include "base/run_loop.h" |
#include "components/arc/test/fake_arc_bridge_instance.h" |
@@ -112,9 +114,7 @@ class ArcNotificationManagerTest : public testing::Test { |
std::unique_ptr<MockMessageCenter> message_center_; |
void SetUp() override { |
- mojom::NotificationsInstancePtr arc_notifications_instance; |
- arc_notifications_instance_.reset( |
- new FakeNotificationsInstance(GetProxy(&arc_notifications_instance))); |
+ arc_notifications_instance_.reset(new FakeNotificationsInstance()); |
service_.reset(new FakeArcBridgeService()); |
message_center_.reset(new MockMessageCenter()); |
@@ -123,8 +123,7 @@ class ArcNotificationManagerTest : public testing::Test { |
NotificationsObserver observer; |
service_->notifications()->AddObserver(&observer); |
- service_->OnNotificationsInstanceReady( |
- std::move(arc_notifications_instance)); |
+ service_->notifications()->SetInstance(arc_notifications_instance_.get()); |
while (!observer.IsReady()) |
loop_.RunUntilIdle(); |
@@ -165,8 +164,6 @@ TEST_F(ArcNotificationManagerTest, NotificationRemovedByChrome) { |
// |notification| gets stale here. |
} |
- arc_notifications_instance()->WaitForIncomingMethodCall(); |
- |
ASSERT_EQ(1u, arc_notifications_instance()->events().size()); |
EXPECT_EQ(key, arc_notifications_instance()->events().at(0).first); |
EXPECT_EQ(mojom::ArcNotificationEvent::CLOSED, |