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

Unified Diff: chrome/browser/custom_handlers/protocol_handler_registry_unittest.cc

Issue 10539169: Prototype version of the first-run dialog for Windows 8 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed some of reviewer's remarks. Created 8 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/custom_handlers/protocol_handler_registry_unittest.cc
diff --git a/chrome/browser/custom_handlers/protocol_handler_registry_unittest.cc b/chrome/browser/custom_handlers/protocol_handler_registry_unittest.cc
index d0c42284502c13f8da233591aa9d8a3314d75ef3..6dbc01d9259c4e5a983d48c352830e6f0b32e493 100644
--- a/chrome/browser/custom_handlers/protocol_handler_registry_unittest.cc
+++ b/chrome/browser/custom_handlers/protocol_handler_registry_unittest.cc
@@ -91,9 +91,10 @@ class FakeClientObserver
delegate_(registry_delegate) {}
virtual void SetDefaultWebClientUIState(
- ShellIntegration::DefaultWebClientUIState state) {
+ ShellIntegration::DefaultWebClientUIState state,
+ ShellIntegration::SetDefaultWebClientResult call_result) {
ProtocolHandlerRegistry::DefaultClientObserver::SetDefaultWebClientUIState(
- state);
+ state, call_result);
if (state == ShellIntegration::STATE_IS_DEFAULT) {
delegate_->FakeRegisterWithOS(worker_->protocol());
}
@@ -126,7 +127,10 @@ class FakeProtocolClientWorker
}
}
- virtual void SetAsDefault(bool interactive_permitted) {}
+ virtual ShellIntegration::SetDefaultWebClientResult
+ SetAsDefault(bool interactive_permitted) {
+ return ShellIntegration::RESULT_SET_DEFAULT_OK;
+ }
private:
bool force_failure_;

Powered by Google App Engine
This is Rietveld 408576698