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

Unified Diff: chrome/browser/printing/cloud_print/cloud_print_url.cc

Issue 7518002: Add sign in dialog for GCP->print preview integration. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge Created 9 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: chrome/browser/printing/cloud_print/cloud_print_url.cc
diff --git a/chrome/browser/printing/cloud_print/cloud_print_url.cc b/chrome/browser/printing/cloud_print/cloud_print_url.cc
index 209fe86975333e13845ab1916ec3e8102bd0306b..c4312f344eb2b1d09849d335e55f1ad8502a48ea 100644
--- a/chrome/browser/printing/cloud_print/cloud_print_url.cc
+++ b/chrome/browser/printing/cloud_print/cloud_print_url.cc
@@ -15,6 +15,10 @@
#include "googleurl/src/gurl.h"
const char kDefaultCloudPrintServiceURL[] = "https://www.google.com/cloudprint";
+const char kSigninURL[] = "https://www.google.com/accounts/ServiceLogin?"
+ "service=cloudprint&continue="
+ "https%3A%2F%2Fwww.google.com%2Fcloudprint";
Scott Byer 2011/08/23 23:57:33 Should this be over-ridable in the prefs?
Albert Bodenhamer 2011/08/24 01:00:07 Done.
+
const char kLearnMoreURL[] =
"https://www.google.com/support/cloudprint";
const char kTestPageURL[] =
@@ -47,6 +51,11 @@ GURL CloudPrintURL::GetCloudPrintServiceURL() {
return cloud_print_service_url;
}
+GURL CloudPrintURL::GetCloudPrintSigninURL() {
+ GURL url(kSigninURL);
+ return google_util::AppendGoogleLocaleParam(url);
+}
+
GURL CloudPrintURL::GetCloudPrintServiceDialogURL() {
GURL cloud_print_service_url = GetCloudPrintServiceURL();
std::string path(cloud_print_service_url.path() + "/client/dialog.html");

Powered by Google App Engine
This is Rietveld 408576698