123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384385386387388389390391392393394395396397398399400401402403404405406407408409410411412413414415416417418419420421422423424425426427428429430431432433434435436437438439440441442443444445446447448449450451452453454455456457458459460461462463464465466467468469470471472473474475476477478479480481482483484485486487488489490491492493 |
- // Copyright 2020 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/clipboard/clipboard_nudge_controller.h"
- #include "ash/clipboard/clipboard_history.h"
- #include "ash/clipboard/clipboard_history_controller_impl.h"
- #include "ash/clipboard/clipboard_nudge.h"
- #include "ash/clipboard/clipboard_nudge_constants.h"
- #include "ash/constants/ash_features.h"
- #include "ash/session/session_controller_impl.h"
- #include "ash/shell.h"
- #include "ash/test/ash_test_base.h"
- #include "base/strings/string_number_conversions.h"
- #include "base/test/metrics/histogram_tester.h"
- #include "base/test/scoped_feature_list.h"
- #include "base/test/simple_test_clock.h"
- #include "chromeos/crosapi/mojom/clipboard_history.mojom.h"
- #include "ui/base/clipboard/clipboard_data.h"
- #include "ui/base/clipboard/clipboard_non_backed.h"
- #include "ui/compositor/layer.h"
- #include "ui/compositor/layer_animator.h"
- #include "ui/compositor/scoped_animation_duration_scale_mode.h"
- #include "ui/views/widget/widget_observer.h"
- namespace ash {
- class ClipboardNudgeControllerTest : public AshTestBase {
- public:
- ClipboardNudgeControllerTest() = default;
- ClipboardNudgeControllerTest(const ClipboardNudgeControllerTest&) = delete;
- ClipboardNudgeControllerTest& operator=(const ClipboardNudgeControllerTest&) =
- delete;
- ~ClipboardNudgeControllerTest() override = default;
- base::SimpleTestClock* clock() { return &test_clock_; }
- // AshTestBase:
- void SetUp() override {
- scoped_feature_list_.InitAndEnableFeature(
- chromeos::features::kClipboardHistoryContextMenuNudge);
- AshTestBase::SetUp();
- nudge_controller_ =
- Shell::Get()->clipboard_history_controller()->nudge_controller();
- nudge_controller_->OverrideClockForTesting(&test_clock_);
- test_clock_.Advance(base::Seconds(360));
- }
- void TearDown() override {
- nudge_controller_->ClearClockOverrideForTesting();
- AshTestBase::TearDown();
- }
- // Owned by ClipboardHistoryController.
- ClipboardNudgeController* nudge_controller_;
- void ShowNudgeForType(ClipboardNudgeType nudge_type) {
- switch (nudge_type) {
- case ClipboardNudgeType::kOnboardingNudge:
- case ClipboardNudgeType::kZeroStateNudge:
- nudge_controller_->ShowNudge(nudge_type);
- return;
- case ClipboardNudgeType::kNewFeatureBadge:
- nudge_controller_->MarkNewFeatureBadgeShown();
- return;
- case ClipboardNudgeType::kScreenshotNotificationNudge:
- nudge_controller_->MarkScreenshotNotificationShown();
- return;
- }
- }
- base::HistogramTester& histograms() { return histograms_; }
- private:
- base::test::ScopedFeatureList scoped_feature_list_;
- base::SimpleTestClock test_clock_;
- // Histogram value verifier.
- base::HistogramTester histograms_;
- };
- // Checks that clipboard state advances after the nudge controller is fed the
- // right event type.
- TEST_F(ClipboardNudgeControllerTest, ShouldShowNudgeAfterCorrectSequence) {
- EXPECT_EQ(ClipboardState::kInit,
- nudge_controller_->GetClipboardStateForTesting());
- // Checks that the first copy advances state as expected.
- nudge_controller_->OnClipboardHistoryItemAdded(
- ClipboardHistoryItem(ui::ClipboardData()));
- EXPECT_EQ(ClipboardState::kFirstCopy,
- nudge_controller_->GetClipboardStateForTesting());
- // Checks that the first paste advances state as expected.
- nudge_controller_->OnClipboardDataRead();
- EXPECT_EQ(ClipboardState::kFirstPaste,
- nudge_controller_->GetClipboardStateForTesting());
- // Checks that the second copy advances state as expected.
- nudge_controller_->OnClipboardHistoryItemAdded(
- ClipboardHistoryItem(ui::ClipboardData()));
- EXPECT_EQ(ClipboardState::kSecondCopy,
- nudge_controller_->GetClipboardStateForTesting());
- // Check that clipbaord nudge has not yet been created.
- EXPECT_FALSE(nudge_controller_->GetSystemNudgeForTesting());
- // Checks that the second paste resets state as expected.
- nudge_controller_->OnClipboardDataRead();
- EXPECT_EQ(ClipboardState::kInit,
- nudge_controller_->GetClipboardStateForTesting());
- // Check that clipbaord nudge has been created.
- EXPECT_TRUE(nudge_controller_->GetSystemNudgeForTesting());
- }
- // Checks that the clipboard state does not advace if too much time passes
- // during the copy paste sequence.
- TEST_F(ClipboardNudgeControllerTest, NudgeTimeOut) {
- // Perform copy -> paste -> copy sequence.
- nudge_controller_->OnClipboardHistoryItemAdded(
- ClipboardHistoryItem(ui::ClipboardData()));
- nudge_controller_->OnClipboardDataRead();
- nudge_controller_->OnClipboardHistoryItemAdded(
- ClipboardHistoryItem(ui::ClipboardData()));
- // Advance time to cause the nudge timer to time out.
- clock()->Advance(kMaxTimeBetweenPaste);
- nudge_controller_->OnClipboardDataRead();
- // Paste event should reset clipboard state to |kFirstPaste| instead of to
- // |kShouldShowNudge|.
- EXPECT_NE(ClipboardState::kShouldShowNudge,
- nudge_controller_->GetClipboardStateForTesting());
- EXPECT_EQ(ClipboardState::kFirstPaste,
- nudge_controller_->GetClipboardStateForTesting());
- }
- // Checks that multiple pastes refreshes the |kMaxTimeBetweenPaste| timer that
- // determines whether too much time has passed to show the nudge.
- TEST_F(ClipboardNudgeControllerTest, NudgeDoesNotTimeOutWithSparsePastes) {
- nudge_controller_->OnClipboardHistoryItemAdded(
- ClipboardHistoryItem(ui::ClipboardData()));
- nudge_controller_->OnClipboardDataRead();
- EXPECT_EQ(ClipboardState::kFirstPaste,
- nudge_controller_->GetClipboardStateForTesting());
- // Perform 5 pastes over 2.5*|kMaxTimeBetweenPaste|.
- for (int paste_cycle = 0; paste_cycle < 5; paste_cycle++) {
- SCOPED_TRACE("paste cycle " + base::NumberToString(paste_cycle));
- clock()->Advance(kMaxTimeBetweenPaste / 2);
- nudge_controller_->OnClipboardDataRead();
- EXPECT_EQ(ClipboardState::kFirstPaste,
- nudge_controller_->GetClipboardStateForTesting());
- }
- // Check that clipbaord nudge has not yet been created.
- EXPECT_FALSE(nudge_controller_->GetSystemNudgeForTesting());
- // Check that HandleClipboardChanged() will advance nudge_controller's
- // ClipboardState.
- nudge_controller_->OnClipboardHistoryItemAdded(
- ClipboardHistoryItem(ui::ClipboardData()));
- EXPECT_EQ(ClipboardState::kSecondCopy,
- nudge_controller_->GetClipboardStateForTesting());
- nudge_controller_->OnClipboardDataRead();
- EXPECT_EQ(ClipboardState::kInit,
- nudge_controller_->GetClipboardStateForTesting());
- // Check that clipbaord nudge has been created.
- EXPECT_TRUE(nudge_controller_->GetSystemNudgeForTesting());
- }
- // Checks that consecutive copy events does not advance the clipboard state.
- TEST_F(ClipboardNudgeControllerTest, RepeatedCopyDoesNotAdvanceState) {
- nudge_controller_->OnClipboardHistoryItemAdded(
- ClipboardHistoryItem(ui::ClipboardData()));
- EXPECT_EQ(ClipboardState::kFirstCopy,
- nudge_controller_->GetClipboardStateForTesting());
- nudge_controller_->OnClipboardHistoryItemAdded(
- ClipboardHistoryItem(ui::ClipboardData()));
- EXPECT_EQ(ClipboardState::kFirstCopy,
- nudge_controller_->GetClipboardStateForTesting());
- }
- // Checks that consecutive paste events does not advance the clipboard state.
- TEST_F(ClipboardNudgeControllerTest, RepeatedPasteDoesNotAdvanceState) {
- nudge_controller_->OnClipboardHistoryItemAdded(
- ClipboardHistoryItem(ui::ClipboardData()));
- EXPECT_EQ(ClipboardState::kFirstCopy,
- nudge_controller_->GetClipboardStateForTesting());
- nudge_controller_->OnClipboardDataRead();
- EXPECT_EQ(ClipboardState::kFirstPaste,
- nudge_controller_->GetClipboardStateForTesting());
- nudge_controller_->OnClipboardDataRead();
- EXPECT_EQ(ClipboardState::kFirstPaste,
- nudge_controller_->GetClipboardStateForTesting());
- }
- // Verifies that administrative events does not advance clipboard state.
- TEST_F(ClipboardNudgeControllerTest, AdminWriteDoesNotAdvanceState) {
- nudge_controller_->OnClipboardHistoryItemAdded(
- ClipboardHistoryItem(ui::ClipboardData()));
- nudge_controller_->OnClipboardDataRead();
- EXPECT_EQ(ClipboardState::kFirstPaste,
- nudge_controller_->GetClipboardStateForTesting());
- auto data = std::make_unique<ui::ClipboardData>();
- data->set_text("test");
- // Write the data to the clipboard, clipboard state should not advance.
- ui::ClipboardNonBacked::GetForCurrentThread()->WriteClipboardData(
- std::move(data));
- EXPECT_EQ(ClipboardState::kFirstPaste,
- nudge_controller_->GetClipboardStateForTesting());
- }
- // Verifies that the context menu new feature badge for the clipboard option
- // only shows |kContextMenuBadgeShowLimit| times.
- TEST_F(ClipboardNudgeControllerTest, ShowNewFeatureNudge) {
- ASSERT_TRUE(nudge_controller_->ShouldShowNewFeatureBadge());
- // Mark the badge shown |kContextMenuBadgeShowLimit| - 1 times.
- // Should expect to keep showing the badge
- for (int i = 0; i < kContextMenuBadgeShowLimit - 1; ++i) {
- nudge_controller_->MarkNewFeatureBadgeShown();
- EXPECT_TRUE(nudge_controller_->ShouldShowNewFeatureBadge());
- }
- // Marking the badge as shown |kContextMenuBadgeShowLimit| times should not
- // expect to show the badge the next time.
- nudge_controller_->MarkNewFeatureBadgeShown();
- EXPECT_FALSE(nudge_controller_->ShouldShowNewFeatureBadge());
- }
- // Verifies that controller cleans up and closes an old nudge before displaying
- // another one.
- TEST_F(ClipboardNudgeControllerTest, ShowZeroStateNudgeAfterOngoingNudge) {
- ui::ScopedAnimationDurationScaleMode test_duration_mode(
- ui::ScopedAnimationDurationScaleMode::NON_ZERO_DURATION);
- ShowNudgeForType(ClipboardNudgeType::kOnboardingNudge);
- ClipboardNudge* nudge = static_cast<ClipboardNudge*>(
- nudge_controller_->GetSystemNudgeForTesting());
- views::Widget* nudge_widget = nudge->widget();
- ASSERT_TRUE(nudge_widget->GetLayer()->GetAnimator()->is_animating());
- EXPECT_FALSE(nudge_widget->IsClosed());
- EXPECT_EQ(ClipboardNudgeType::kOnboardingNudge, nudge->nudge_type());
- ShowNudgeForType(ClipboardNudgeType::kZeroStateNudge);
- // Verify the old nudge widget was closed.
- EXPECT_TRUE(nudge_widget->IsClosed());
- nudge = static_cast<ClipboardNudge*>(
- nudge_controller_->GetSystemNudgeForTesting());
- EXPECT_FALSE(nudge->widget()->IsClosed());
- EXPECT_EQ(ClipboardNudgeType::kZeroStateNudge, nudge->nudge_type());
- }
- // Verifies that the nudge cleans up properly during shutdown while it is
- // animating to hide.
- TEST_F(ClipboardNudgeControllerTest, NudgeClosingDuringShutdown) {
- ShowNudgeForType(ClipboardNudgeType::kOnboardingNudge);
- ClipboardNudge* nudge = static_cast<ClipboardNudge*>(
- nudge_controller_->GetSystemNudgeForTesting());
- views::Widget* nudge_widget = nudge->widget();
- EXPECT_FALSE(nudge_widget->IsClosed());
- EXPECT_EQ(ClipboardNudgeType::kOnboardingNudge, nudge->nudge_type());
- // Slow down the duration of the nudge
- ui::ScopedAnimationDurationScaleMode test_duration_mode(
- ui::ScopedAnimationDurationScaleMode::SLOW_DURATION);
- nudge_controller_->FireHideNudgeTimerForTesting();
- ASSERT_TRUE(nudge_widget->GetLayer()->GetAnimator()->is_animating());
- }
- // Assert that all nudge metric related histograms start at 0.
- TEST_F(ClipboardNudgeControllerTest, NudgeMetrics_StartAtZero) {
- histograms().ExpectTotalCount(kOnboardingNudge_OpenTime, 0);
- histograms().ExpectTotalCount(kOnboardingNudge_PasteTime, 0);
- histograms().ExpectTotalCount(kZeroStateNudge_OpenTime, 0);
- histograms().ExpectTotalCount(kZeroStateNudge_PasteTime, 0);
- histograms().ExpectTotalCount(kNewBadge_OpenTime, 0);
- histograms().ExpectTotalCount(kNewBadge_PasteTime, 0);
- histograms().ExpectTotalCount(kScreenshotNotification_OpenTime, 0);
- histograms().ExpectTotalCount(kScreenshotNotification_PasteTime, 0);
- }
- // Test that opening the clipboard history after showing the nudges logs only
- // the metrics for the open time histograms. For this test, we only verify the
- // number of emits, but not the timing mechanism.
- TEST_F(ClipboardNudgeControllerTest, ShowMenuAfterNudges_LogsOpenNudgeMetrics) {
- ShowNudgeForType(ClipboardNudgeType::kOnboardingNudge);
- ShowNudgeForType(ClipboardNudgeType::kZeroStateNudge);
- ShowNudgeForType(ClipboardNudgeType::kNewFeatureBadge);
- ShowNudgeForType(ClipboardNudgeType::kScreenshotNotificationNudge);
- nudge_controller_->OnClipboardHistoryMenuShown(
- crosapi::mojom::ClipboardHistoryControllerShowSource::
- kRenderViewContextMenu);
- histograms().ExpectTotalCount(kOnboardingNudge_OpenTime, 1);
- histograms().ExpectTotalCount(kZeroStateNudge_OpenTime, 1);
- histograms().ExpectTotalCount(kNewBadge_OpenTime, 1);
- histograms().ExpectTotalCount(kScreenshotNotification_OpenTime, 1);
- histograms().ExpectTotalCount(kOnboardingNudge_PasteTime, 0);
- histograms().ExpectTotalCount(kZeroStateNudge_PasteTime, 0);
- histograms().ExpectTotalCount(kNewBadge_PasteTime, 0);
- histograms().ExpectTotalCount(kScreenshotNotification_PasteTime, 0);
- }
- // Test that pasting something from the clipboard history after showing the
- // nudges only the metrics for the paste time histograms. For this test, we only
- // verify the number of emits, but not the timing mechanism.
- TEST_F(ClipboardNudgeControllerTest, PasteAfterNudges_LogsPasteNudgeMetrics) {
- ShowNudgeForType(ClipboardNudgeType::kOnboardingNudge);
- ShowNudgeForType(ClipboardNudgeType::kZeroStateNudge);
- ShowNudgeForType(ClipboardNudgeType::kNewFeatureBadge);
- ShowNudgeForType(ClipboardNudgeType::kScreenshotNotificationNudge);
- nudge_controller_->OnClipboardHistoryMenuShown(
- crosapi::mojom::ClipboardHistoryControllerShowSource::
- kRenderViewContextMenu);
- nudge_controller_->OnClipboardHistoryPasted();
- histograms().ExpectTotalCount(kOnboardingNudge_OpenTime, 1);
- histograms().ExpectTotalCount(kZeroStateNudge_OpenTime, 1);
- histograms().ExpectTotalCount(kNewBadge_OpenTime, 1);
- histograms().ExpectTotalCount(kScreenshotNotification_OpenTime, 1);
- histograms().ExpectTotalCount(kOnboardingNudge_PasteTime, 1);
- histograms().ExpectTotalCount(kZeroStateNudge_PasteTime, 1);
- histograms().ExpectTotalCount(kNewBadge_PasteTime, 1);
- histograms().ExpectTotalCount(kScreenshotNotification_PasteTime, 1);
- }
- // Test that the onboarding nudge being shown only logs the metrics for the
- // onboarding nudge histograms,
- TEST_F(ClipboardNudgeControllerTest, OnboardingNudge_DoesNotLogOtherMetrics) {
- ShowNudgeForType(ClipboardNudgeType::kOnboardingNudge);
- nudge_controller_->OnClipboardHistoryMenuShown(
- crosapi::mojom::ClipboardHistoryControllerShowSource::
- kRenderViewContextMenu);
- nudge_controller_->OnClipboardHistoryPasted();
- histograms().ExpectTotalCount(kOnboardingNudge_OpenTime, 1);
- histograms().ExpectTotalCount(kZeroStateNudge_OpenTime, 0);
- histograms().ExpectTotalCount(kNewBadge_OpenTime, 0);
- histograms().ExpectTotalCount(kScreenshotNotification_OpenTime, 0);
- histograms().ExpectTotalCount(kOnboardingNudge_PasteTime, 1);
- histograms().ExpectTotalCount(kZeroStateNudge_PasteTime, 0);
- histograms().ExpectTotalCount(kNewBadge_PasteTime, 0);
- histograms().ExpectTotalCount(kScreenshotNotification_PasteTime, 0);
- }
- // Test that the zero state nudge being shown only logs the metrics for the zero
- // state nudge histograms,
- TEST_F(ClipboardNudgeControllerTest, ZeroStateNudge_DoesNotLogOtherMetrics) {
- ShowNudgeForType(ClipboardNudgeType::kZeroStateNudge);
- nudge_controller_->OnClipboardHistoryMenuShown(
- crosapi::mojom::ClipboardHistoryControllerShowSource::
- kRenderViewContextMenu);
- nudge_controller_->OnClipboardHistoryPasted();
- histograms().ExpectTotalCount(kOnboardingNudge_OpenTime, 0);
- histograms().ExpectTotalCount(kZeroStateNudge_OpenTime, 1);
- histograms().ExpectTotalCount(kNewBadge_OpenTime, 0);
- histograms().ExpectTotalCount(kScreenshotNotification_OpenTime, 0);
- histograms().ExpectTotalCount(kOnboardingNudge_PasteTime, 0);
- histograms().ExpectTotalCount(kZeroStateNudge_PasteTime, 1);
- histograms().ExpectTotalCount(kNewBadge_PasteTime, 0);
- histograms().ExpectTotalCount(kScreenshotNotification_PasteTime, 0);
- }
- // Test that the new feature badge being shown only logs the metrics for the new
- // feature badge histograms,
- TEST_F(ClipboardNudgeControllerTest, NewFeatureBadge_DoesNotLogOtherMetrics) {
- ShowNudgeForType(ClipboardNudgeType::kNewFeatureBadge);
- nudge_controller_->OnClipboardHistoryMenuShown(
- crosapi::mojom::ClipboardHistoryControllerShowSource::
- kRenderViewContextMenu);
- nudge_controller_->OnClipboardHistoryPasted();
- histograms().ExpectTotalCount(kOnboardingNudge_OpenTime, 0);
- histograms().ExpectTotalCount(kZeroStateNudge_OpenTime, 0);
- histograms().ExpectTotalCount(kNewBadge_OpenTime, 1);
- histograms().ExpectTotalCount(kScreenshotNotification_OpenTime, 0);
- histograms().ExpectTotalCount(kOnboardingNudge_PasteTime, 0);
- histograms().ExpectTotalCount(kZeroStateNudge_PasteTime, 0);
- histograms().ExpectTotalCount(kNewBadge_PasteTime, 1);
- histograms().ExpectTotalCount(kScreenshotNotification_PasteTime, 0);
- }
- // Test that the screenshot notification nudge being shown only logs the metrics
- // for the screenshot notification nudge histograms,
- TEST_F(ClipboardNudgeControllerTest,
- ScreenshotNotification_DoesNotLogOtherMetrics) {
- ShowNudgeForType(ClipboardNudgeType::kScreenshotNotificationNudge);
- nudge_controller_->OnClipboardHistoryMenuShown(
- crosapi::mojom::ClipboardHistoryControllerShowSource::
- kRenderViewContextMenu);
- nudge_controller_->OnClipboardHistoryPasted();
- histograms().ExpectTotalCount(kOnboardingNudge_OpenTime, 0);
- histograms().ExpectTotalCount(kZeroStateNudge_OpenTime, 0);
- histograms().ExpectTotalCount(kNewBadge_OpenTime, 0);
- histograms().ExpectTotalCount(kScreenshotNotification_OpenTime, 1);
- histograms().ExpectTotalCount(kOnboardingNudge_PasteTime, 0);
- histograms().ExpectTotalCount(kZeroStateNudge_PasteTime, 0);
- histograms().ExpectTotalCount(kNewBadge_PasteTime, 0);
- histograms().ExpectTotalCount(kScreenshotNotification_PasteTime, 1);
- }
- // Test that nudge metrics will not log multiple times if the nudges are not
- // shown before.
- TEST_F(ClipboardNudgeControllerTest, SecondTimeAction_DoesNotLogNudgeMetrics) {
- ShowNudgeForType(ClipboardNudgeType::kOnboardingNudge);
- ShowNudgeForType(ClipboardNudgeType::kZeroStateNudge);
- ShowNudgeForType(ClipboardNudgeType::kNewFeatureBadge);
- ShowNudgeForType(ClipboardNudgeType::kScreenshotNotificationNudge);
- nudge_controller_->OnClipboardHistoryMenuShown(
- crosapi::mojom::ClipboardHistoryControllerShowSource::
- kRenderViewContextMenu);
- nudge_controller_->OnClipboardHistoryPasted();
- nudge_controller_->OnClipboardHistoryMenuShown(
- crosapi::mojom::ClipboardHistoryControllerShowSource::
- kRenderViewContextMenu);
- nudge_controller_->OnClipboardHistoryPasted();
- histograms().ExpectTotalCount(kOnboardingNudge_OpenTime, 1);
- histograms().ExpectTotalCount(kZeroStateNudge_OpenTime, 1);
- histograms().ExpectTotalCount(kNewBadge_OpenTime, 1);
- histograms().ExpectTotalCount(kScreenshotNotification_OpenTime, 1);
- histograms().ExpectTotalCount(kOnboardingNudge_PasteTime, 1);
- histograms().ExpectTotalCount(kZeroStateNudge_PasteTime, 1);
- histograms().ExpectTotalCount(kNewBadge_PasteTime, 1);
- histograms().ExpectTotalCount(kScreenshotNotification_PasteTime, 1);
- }
- // Test that nudge metrics can log more times as the nudges are shown before.
- TEST_F(ClipboardNudgeControllerTest, ShowNudgeTwice_LogsMetricsTwoTimes) {
- ShowNudgeForType(ClipboardNudgeType::kOnboardingNudge);
- ShowNudgeForType(ClipboardNudgeType::kZeroStateNudge);
- ShowNudgeForType(ClipboardNudgeType::kNewFeatureBadge);
- ShowNudgeForType(ClipboardNudgeType::kScreenshotNotificationNudge);
- nudge_controller_->OnClipboardHistoryMenuShown(
- crosapi::mojom::ClipboardHistoryControllerShowSource::
- kRenderViewContextMenu);
- nudge_controller_->OnClipboardHistoryPasted();
- ShowNudgeForType(ClipboardNudgeType::kOnboardingNudge);
- ShowNudgeForType(ClipboardNudgeType::kZeroStateNudge);
- ShowNudgeForType(ClipboardNudgeType::kNewFeatureBadge);
- ShowNudgeForType(ClipboardNudgeType::kScreenshotNotificationNudge);
- nudge_controller_->OnClipboardHistoryMenuShown(
- crosapi::mojom::ClipboardHistoryControllerShowSource::
- kRenderViewContextMenu);
- nudge_controller_->OnClipboardHistoryPasted();
- histograms().ExpectTotalCount(kOnboardingNudge_OpenTime, 2);
- histograms().ExpectTotalCount(kZeroStateNudge_OpenTime, 2);
- histograms().ExpectTotalCount(kNewBadge_OpenTime, 2);
- histograms().ExpectTotalCount(kScreenshotNotification_OpenTime, 2);
- histograms().ExpectTotalCount(kOnboardingNudge_PasteTime, 2);
- histograms().ExpectTotalCount(kZeroStateNudge_PasteTime, 2);
- histograms().ExpectTotalCount(kNewBadge_PasteTime, 2);
- histograms().ExpectTotalCount(kScreenshotNotification_PasteTime, 2);
- }
- // For the new feature badge, metrics should only log for opening a menu by a
- // context menu source.
- TEST_F(ClipboardNudgeControllerTest,
- NewFeatureBadgeOpen_LogsByWithContextMenuSource) {
- ShowNudgeForType(ClipboardNudgeType::kNewFeatureBadge);
- nudge_controller_->OnClipboardHistoryMenuShown(
- crosapi::mojom::ClipboardHistoryControllerShowSource::
- kRenderViewContextMenu);
- ShowNudgeForType(ClipboardNudgeType::kNewFeatureBadge);
- nudge_controller_->OnClipboardHistoryMenuShown(
- crosapi::mojom::ClipboardHistoryControllerShowSource::
- kTextfieldContextMenu);
- histograms().ExpectTotalCount(kNewBadge_OpenTime, 2);
- }
- TEST_F(ClipboardNudgeControllerTest,
- NewFeatureBadgeOpen_DoesNotLogsWithNotContextMenuSource) {
- ShowNudgeForType(ClipboardNudgeType::kNewFeatureBadge);
- nudge_controller_->OnClipboardHistoryMenuShown(
- crosapi::mojom::ClipboardHistoryControllerShowSource::kAccelerator);
- nudge_controller_->OnClipboardHistoryMenuShown(
- crosapi::mojom::ClipboardHistoryControllerShowSource::kVirtualKeyboard);
- histograms().ExpectTotalCount(kNewBadge_OpenTime, 0);
- }
- } // namespace ash
|