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

Unified Diff: chrome/browser/safe_browsing/chrome_cleaner/chrome_cleaner_runner_win_unittest.cc

Issue 2966453002: Chrome Cleaner UI: Add logs upload permission checkbox to the dialog (Closed)
Patch Set: More comments Created 3 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
Index: chrome/browser/safe_browsing/chrome_cleaner/chrome_cleaner_runner_win_unittest.cc
diff --git a/chrome/browser/safe_browsing/chrome_cleaner/chrome_cleaner_runner_win_unittest.cc b/chrome/browser/safe_browsing/chrome_cleaner/chrome_cleaner_runner_win_unittest.cc
index 0c50ae0cd1f97dd61e48eb8fc7b1d124ee969323..93f3539e91a2b3ecc46b89e2029ba90bd751e901 100644
--- a/chrome/browser/safe_browsing/chrome_cleaner/chrome_cleaner_runner_win_unittest.cc
+++ b/chrome/browser/safe_browsing/chrome_cleaner/chrome_cleaner_runner_win_unittest.cc
@@ -115,6 +115,9 @@ class ChromeCleanerRunnerSimpleTest
return base::Process();
}
+ void OnCleanerProcessDone(
+ const ChromeCleanerRunner::ProcessStatus& process_status) override {}
+
// IPC callbacks.
void OnPromptUser(std::unique_ptr<std::set<base::FilePath>> files_to_delete,
@@ -235,6 +238,8 @@ class ChromeCleanerRunnerTest
cleaner_process_options_.set_reboot_required(
uws_found_state == UwsFoundState::kUwsFoundRebootRequired);
cleaner_process_options_.set_crash_point(crash_point);
+ cleaner_process_options_.set_expected_user_response(
+ prompt_acceptance_to_send);
prompt_acceptance_to_send_ = prompt_acceptance_to_send;
SetChromeCleanerRunnerTestDelegateForTesting(this);
@@ -278,6 +283,9 @@ class ChromeCleanerRunnerTest
return std::move(result.process);
}
+ void OnCleanerProcessDone(
+ const ChromeCleanerRunner::ProcessStatus& process_status) override {}
+
// IPC callbacks.
// Will receive the main Mojo message from the Mock Chrome Cleaner process.
@@ -395,7 +403,9 @@ INSTANTIATE_TEST_CASE_P(
MockChromeCleanerProcess::CrashPoint::kAfterConnection,
MockChromeCleanerProcess::CrashPoint::kAfterRequestSent,
MockChromeCleanerProcess::CrashPoint::kAfterResponseReceived),
- Values(PromptAcceptance::DENIED, PromptAcceptance::ACCEPTED)));
+ Values(PromptAcceptance::DENIED,
+ PromptAcceptance::ACCEPTED_WITH_LOGS,
+ PromptAcceptance::ACCEPTED_WITHOUT_LOGS)));
} // namespace
} // namespace safe_browsing

Powered by Google App Engine
This is Rietveld 408576698