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

Unified Diff: chrome/service/cloud_print/cloud_print_proxy.cc

Issue 6245005: Code to send diagnostic messages about cloud print proxy. Currently diagnosti... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Fixed build errors yet again Created 9 years, 11 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/service/cloud_print/cloud_print_proxy.cc
===================================================================
--- chrome/service/cloud_print/cloud_print_proxy.cc (revision 70946)
+++ chrome/service/cloud_print/cloud_print_proxy.cc (working copy)
@@ -106,7 +106,7 @@
cloud_print_email_, proxy_id);
}
if (client_) {
- client_->OnCloudPrintProxyEnabled();
+ client_->OnCloudPrintProxyEnabled(true);
}
}
@@ -115,7 +115,7 @@
cloud_print_email_.clear();
Shutdown();
if (client_) {
- client_->OnCloudPrintProxyDisabled();
+ client_->OnCloudPrintProxyDisabled(true);
}
}
@@ -161,6 +161,15 @@
FROM_HERE, NewRunnableFunction(&ShowTokenExpiredNotificationInBrowser));
}
+void CloudPrintProxy::OnPrintSystemUnavailable() {
+ // If the print system is unavailable, we want to shutdown the proxy and
+ // disable it non-persistently.
+ Shutdown();
+ if (client_) {
+ client_->OnCloudPrintProxyDisabled(false);
+ }
+}
+
void CloudPrintProxy::Shutdown() {
DCHECK(CalledOnValidThread());
if (backend_.get())

Powered by Google App Engine
This is Rietveld 408576698