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

Unified Diff: cloud_print/gcp20/prototype/cloud_print_requester.cc

Issue 22184007: GCP2.0 Device: Instant AccessToken update on AuthFailed. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years, 4 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: cloud_print/gcp20/prototype/cloud_print_requester.cc
diff --git a/cloud_print/gcp20/prototype/cloud_print_requester.cc b/cloud_print/gcp20/prototype/cloud_print_requester.cc
index 1154d62b38f5b867fad08e725c8bd4008a2e3807..bfdbddae9e9f73de14939df836bf6685490f7752 100644
--- a/cloud_print/gcp20/prototype/cloud_print_requester.cc
+++ b/cloud_print/gcp20/prototype/cloud_print_requester.cc
@@ -168,10 +168,14 @@ void CloudPrintRequester::OnFetchError(const std::string& server_api,
VLOG(3) << "Function: " << __FUNCTION__;
EraseRequest();
current_print_job_.reset();
- delegate_->OnServerError("Fetch error");
+
+ if (server_http_code == net::HTTP_FORBIDDEN) {
+ delegate_->OnAuthError();
+ } else {
+ delegate_->OnServerError("Fetch error");
+ }
// TODO(maksymb): |server_api| and other
gene 2013/08/05 20:56:28 still need this TODO?
maksymb 2013/08/06 21:45:54 Yes. NOTIMPLEMENTED was deleted because I already
gene 2013/08/08 02:01:55 What does "|server_api| and other" mean? Could you
maksymb 2013/08/08 18:35:46 Done.
- NOTIMPLEMENTED();
}
void CloudPrintRequester::OnFetchTimeoutReached() {

Powered by Google App Engine
This is Rietveld 408576698