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

Unified Diff: ash/system/chromeos/screen_security/screen_tray_item_unittest.cc

Issue 16472006: Add unit tests for screen capture and share notification UI. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years, 6 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
« no previous file with comments | « ash/system/chromeos/screen_security/screen_tray_item.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/system/chromeos/screen_security/screen_tray_item_unittest.cc
diff --git a/ash/system/chromeos/screen_security/screen_tray_item_unittest.cc b/ash/system/chromeos/screen_security/screen_tray_item_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..705b99c74b1e6642ef0cf42e81933df94bded502
--- /dev/null
+++ b/ash/system/chromeos/screen_security/screen_tray_item_unittest.cc
@@ -0,0 +1,182 @@
+// Copyright 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "ash/system/chromeos/screen_security/screen_tray_item.h"
+
+#include "ash/system/chromeos/screen_security/screen_capture_tray_item.h"
+#include "ash/system/chromeos/screen_security/screen_share_tray_item.h"
+#include "ash/test/ash_test_base.h"
+#include "base/callback.h"
+#include "base/strings/utf_string_conversions.h"
+#include "ui/base/events/event.h"
+#include "ui/gfx/point.h"
+#include "ui/views/view.h"
+
+namespace ash {
+namespace internal {
+
+// Test with unicode strings.
+const char kTestScreenCaptureAppName[] = "Screen Capture Unit Test ಠ_ಠ";
James Cook 2013/06/07 18:45:56 I don't think you should put raw Unicode character
+const char kTestScreenShareHelperName[] = "宋腾";
+
+SystemTray* GetSystemTray() {
+ return Shell::GetInstance()->GetPrimarySystemTray();
+}
+
+SystemTrayNotifier* GetSystemTrayNotifier() {
+ return Shell::GetInstance()->system_tray_notifier();
+}
+
+void ClickViewCenter(views::View* view) {
+ gfx::Point click_location_in_local =
+ gfx::Point(view->width() / 2, view->height() / 2);
+ view->OnMousePressed(ui::MouseEvent(ui::ET_MOUSE_PRESSED,
+ click_location_in_local,
+ click_location_in_local,
+ ui::EF_NONE));
+}
+
+class ScreenTrayItemTest : public ash::test::AshTestBase {
+ public:
+ ScreenTrayItemTest()
+ : tray_item_(NULL), stop_callback_hit_count_(0) {}
+ virtual ~ScreenTrayItemTest() {}
+
+ ScreenTrayItem* tray_item() { return tray_item_; }
+ void set_tray_item(ScreenTrayItem* tray_item) { tray_item_ = tray_item; }
+
+ int stop_callback_hit_count() const { return stop_callback_hit_count_; }
+
+ void StartSession() {
+ tray_item_->Start(
+ base::Bind(&ScreenTrayItemTest::StopCallback, base::Unretained(this)));
+ }
+
+ void StopSession() {
+ tray_item_->Stop();
+ }
+
+ void StopCallback() {
+ stop_callback_hit_count_++;
+ }
+
+ private:
+ ScreenTrayItem* tray_item_;
+ int stop_callback_hit_count_;
+
+ DISALLOW_COPY_AND_ASSIGN(ScreenTrayItemTest);
+};
+
+class ScreenCaptureTest : public ScreenTrayItemTest {
+ public:
+ ScreenCaptureTest() {}
+ virtual ~ScreenCaptureTest() {}
+
+ virtual void SetUp() OVERRIDE {
+ test::AshTestBase::SetUp();
+ // This tray item is owned by its parent system tray view and will
+ // be deleted automatically when its parent is destroyed in AshTestBase.
+ set_tray_item(new ScreenCaptureTrayItem(GetSystemTray()));
+ }
+
+ DISALLOW_COPY_AND_ASSIGN(ScreenCaptureTest);
+};
+
+class ScreenShareTest : public ScreenTrayItemTest {
+ public:
+ ScreenShareTest() {}
+ virtual ~ScreenShareTest() {}
+
+ virtual void SetUp() OVERRIDE {
+ test::AshTestBase::SetUp();
+ // This tray item is owned by its parent system tray view and will
+ // be deleted automatically when its parent is destroyed in AshTestBase.
+ set_tray_item(new ScreenShareTrayItem(GetSystemTray()));
+ }
+
+ DISALLOW_COPY_AND_ASSIGN(ScreenShareTest);
+};
+
+void TestStartAndStop(ScreenTrayItemTest* test) {
+ ScreenTrayItem* tray_item = test->tray_item();
+
+ EXPECT_FALSE(tray_item->is_started());
+ EXPECT_EQ(0, test->stop_callback_hit_count());
+
+ test->StartSession();
+ EXPECT_TRUE(tray_item->is_started());
+
+ test->StopSession();
+ EXPECT_FALSE(tray_item->is_started());
+ EXPECT_EQ(1, test->stop_callback_hit_count());
+}
+
+TEST_F(ScreenCaptureTest, StartAndStop) { TestStartAndStop(this); }
+TEST_F(ScreenShareTest, StartAndStop) { TestStartAndStop(this); }
+
+void TestNotificationStartAndStop(ScreenTrayItemTest* test,
+ const base::Closure& start_function,
+ const base::Closure& stop_function) {
+ ScreenTrayItem* tray_item = test->tray_item();
+ EXPECT_FALSE(tray_item->is_started());
+
+ start_function.Run();
+ EXPECT_TRUE(tray_item->is_started());
+
+ // The stop callback shouldn't be called because we stopped
+ // through the notification system.
+ stop_function.Run();
+ EXPECT_FALSE(tray_item->is_started());
+ EXPECT_EQ(0, test->stop_callback_hit_count());
+}
+
+TEST_F(ScreenCaptureTest, NotificationStartAndStop) {
+ base::Closure start_function =
+ base::Bind(&SystemTrayNotifier::NotifyScreenCaptureStart,
+ base::Unretained(GetSystemTrayNotifier()),
+ base::Bind(&ScreenTrayItemTest::StopCallback,
+ base::Unretained(this)),
+ base::UTF8ToUTF16(kTestScreenCaptureAppName));
+
+ base::Closure stop_function =
+ base::Bind(&SystemTrayNotifier::NotifyScreenCaptureStop,
+ base::Unretained(GetSystemTrayNotifier()));
+
+ TestNotificationStartAndStop(this, start_function, stop_function);
+}
+
+TEST_F(ScreenShareTest, NotificationStartAndStop) {
+ base::Closure start_func =
+ base::Bind(&SystemTrayNotifier::NotifyScreenShareStart,
+ base::Unretained(GetSystemTrayNotifier()),
+ base::Bind(&ScreenTrayItemTest::StopCallback,
+ base::Unretained(this)),
+ base::UTF8ToUTF16(kTestScreenShareHelperName));
+
+ base::Closure stop_func =
+ base::Bind(&SystemTrayNotifier::NotifyScreenShareStop,
+ base::Unretained(GetSystemTrayNotifier()));
+
+ TestNotificationStartAndStop(this, start_func, stop_func);
+}
+
+
+void TestNotificationView(ScreenTrayItemTest* test) {
+ ScreenTrayItem* tray_item = test->tray_item();
+
+ test->StartSession();
+ EXPECT_TRUE(tray_item->notification_view()->visible());
+
+ // Clicking on the notification view should dismiss the view
+ ClickViewCenter(tray_item->notification_view());
+ EXPECT_FALSE(tray_item->notification_view());
+
+ test->StopSession();
+}
+
+TEST_F(ScreenCaptureTest, NotificationView) { TestNotificationView(this); }
+TEST_F(ScreenShareTest, NotificationView) { TestNotificationView(this); }
+
+} // namespace internal
+} // namespace ash
« no previous file with comments | « ash/system/chromeos/screen_security/screen_tray_item.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698