Use device Robot Account to access Cloud Print.

chrome/browser/resources/print_preview/cloud_print_interface.js

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 cr.define('cloudprint', function() {
   'use strict';
 
   /**
    * API to the Google Cloud Print service.
    * @param {string} baseUrl Base part of the Google Cloud Print service URL
    *     with no trailing slash. For example,
    *     'https://www.google.com/cloudprint'.
+   * @param {!print_preview.NativeLayer} nativeLayer Native layer used to get
+   *     Auth2 tokens.
    * @constructor
    * @extends {cr.EventTarget}
    */
-  function CloudPrintInterface(baseUrl) {
+  function CloudPrintInterface(baseUrl, nativeLayer) {
     /**
      * The base URL of the Google Cloud Print API.
      * @type {string}
      * @private
      */
     this.baseUrl_ = baseUrl;
 
     /**
+     * Used to get Auth2 tokens.
+     * @type {!print_preview.NativeLayer}
+     * @private
+     */
+    this.nativeLayer_ = nativeLayer;
+
+    /**
      * Last received XSRF token. Sent as a parameter in every request.
      * @type {string}
      * @private
      */
     this.xsrfToken_ = '';
 
     /**
+     * Pending requests delayed until we get access token.
+     * @type {!Array<objects>}

[Toscano, 2013/04/19 17:04:33]

The type looks like: {!Array.<Object>}

[Vitaly Buka (NO REVIEWS), 2013/04/19 22:14:29]

Actually i have type so * @type {!Array<!ClourPrintRequest>}

+     * @private
+     */
+    this.requestQueue_ = [];
+
+    /**
      * Number of outstanding cloud destination search requests.
      * @type {number}
      * @private
      */
     this.outstandingCloudSearchRequestCount_ = 0;
+
+    /**
+     * Event tracker used to keep track of native layer events.
+     * @type {!EventTracker}
+     * @private
+     */
+    this.tracker_ = new EventTracker();
+
+    this.addEventListeners_();
   };
 
   /**
    * Event types dispatched by the interface.
    * @enum {string}
    */
   CloudPrintInterface.EventType = {
     PRINTER_DONE: 'cloudprint.CloudPrintInterface.PRINTER_DONE',
     PRINTER_FAILED: 'cloudprint.CloudPrintInterface.PRINTER_FAILED',
     SEARCH_DONE: 'cloudprint.CloudPrintInterface.SEARCH_DONE',
@@ skipping 43 matching lines @@
 
   /**
    * Enumeration of JSON response fields from Google Cloud Print API.
    * @enum {string}
    * @private
    */
   CloudPrintInterface.JsonFields_ = {
     PRINTER: 'printer'
   };
 
+  /**
+   * Could Print origins used to search printers.
+   * @type {!Array<!print_preview.Destination.Origin>}

[Toscano, 2013/04/19 17:04:33]

Missing the "." between "Array" and "<!...":

{!Array.<!print...>}

[Vitaly Buka (NO REVIEWS), 2013/04/19 22:14:29]

Done.

+   * @const
+   * @private
+   */
+  CloudPrintInterface.CLOUD_ORIGINS_ = [
+      print_preview.Destination.Origin.COOKIES,
+      print_preview.Destination.Origin.DEVICE
+      // TODO(vitalybuka): Enable when implemented.
+      // ready print_preview.Destination.Origin.PROFILE
+  ];
+
   CloudPrintInterface.prototype = {
     __proto__: cr.EventTarget.prototype,
 
     /** @return {string} Base URL of the Google Cloud Print service. */
     get baseUrl() {
       return this.baseUrl_;
     },
 
     /**
      * @return {boolean} Whether a search for cloud destinations is in progress.
      */
     get isCloudDestinationSearchInProgress() {
       return this.outstandingCloudSearchRequestCount_ > 0;
     },
 
     /**
      * Sends a Google Cloud Print search API request.
      * @param {boolean} isRecent Whether to search for only recently used
      *     printers.
      */
     search: function(isRecent) {
       var params = [
         new HttpParam('connection_status', 'ALL'),
         new HttpParam('client', 'chrome'),
         new HttpParam('use_cdd', 'true')
       ];
       if (isRecent) {
         params.push(new HttpParam('q', '^recent'));
       }
-      ++this.outstandingCloudSearchRequestCount_;
-      this.sendRequest_('GET', 'search', params,
-                        this.onSearchDone_.bind(this, isRecent));
+      CloudPrintInterface.CLOUD_ORIGINS_.forEach(function(origin) {
+        ++this.outstandingCloudSearchRequestCount_;
+        var cpRequest = this.buildRequest_('GET', 'search', params, origin);
+        this.sendOrQueueRequest_(cpRequest,
+                                 this.onSearchDone_.bind(this, isRecent));
+      }, this);
     },
 
     /**
      * Sends a Google Cloud Print submit API request.
      * @param {!print_preview.Destination} destination Cloud destination to
      *     print to.
      * @param {!print_preview.PrintTicketStore} printTicketStore Contains the
      *     print ticket to print.
      * @param {string} data Base64 encoded data of the document.
      */
     submit: function(destination, printTicketStore, data) {
       var result =
           CloudPrintInterface.VERSION_REGEXP_.exec(navigator.userAgent);
       var chromeVersion = 'unknown';
       if (result && result.length == 2) {
         chromeVersion = result[1];
       }
       var params = [
         new HttpParam('printerid', destination.id),
         new HttpParam('contentType', 'dataUrl'),
         new HttpParam('title', printTicketStore.getDocumentTitle()),
         new HttpParam('ticket',
                       this.createPrintTicket_(destination, printTicketStore)),
         new HttpParam('content', 'data:application/pdf;base64,' + data),
         new HttpParam('tag',
                       '__google__chrome_version=' + chromeVersion),
         new HttpParam('tag', '__google__os=' + navigator.platform)
       ];
-      this.sendRequest_('POST', 'submit', params,
-                        this.onSubmitDone_.bind(this));
+      var cpRequest = this.buildRequest_('POST', 'submit', params,
+                                         destination.origin);
+      this.sendOrQueueRequest_(cpRequest, this.onSubmitDone_.bind(this));
     },
 
     /**
      * Sends a Google Cloud Print printer API request.
      * @param {string} printerId ID of the printer to lookup.
+     * @param {!print_preview.Destination.Origin} origin Origin of the printer.
      */
-    printer: function(printerId) {
+    printer: function(printerId, origin) {
       var params = [
         new HttpParam('printerid', printerId),
         new HttpParam('use_cdd', 'true')
       ];
-      this.sendRequest_('GET', 'printer', params,
-                        this.onPrinterDone_.bind(this, printerId));
+      var cpRequest = this.buildRequest_('GET', 'printer', params, origin);
+      this.sendOrQueueRequest_(cpRequest,
+                               this.onPrinterDone_.bind(this, printerId));
     },
 
     /**
      * Sends a Google Cloud Print update API request to accept (or reject) the
      * terms-of-service of the given printer.
      * @param {string} printerId ID of the printer to accept the
      *     terms-of-service for.
+     * @param {!print_preview.Destination.Origin} origin Origin of the printer.
      * @param {boolean} isAccepted Whether the user accepted the
      *     terms-of-service.
      */
-    updatePrinterTosAcceptance: function(printerId, isAccepted) {
+    updatePrinterTosAcceptance: function(printerId, origin, isAccepted) {
       var params = [
         new HttpParam('printerid', printerId),
         new HttpParam('is_tos_accepted', isAccepted)
       ];
-      this.sendRequest_('POST', 'update', params,
-                        this.onUpdatePrinterTosAcceptanceDone_.bind(this));
+      var cpRequest = this.buildRequest_('POST', 'update', params, origin);
+      this.sendOrQueueRequest_(cpRequest,
+                               this.onUpdateTosAcceptanceDone_.bind(this));

[Toscano, 2013/04/19 17:04:33]

Why did you change the name of the callback? Now it doesn't match the name of
this function.

[Vitaly Buka (NO REVIEWS), 2013/04/19 22:14:29]

Done.

     },
 
     /**
+     * Adds event listeners to the relevant native layer events.
+     * @private
+     */
+    addEventListeners_: function() {
+      this.tracker_.add(
+          this.nativeLayer_,
+          print_preview.NativeLayer.EventType.ACCESS_TOKEN_READY,
+          this.onAccessTokenReady_.bind(this));
+    },
+
+    /**
+     * Called when a native layer receives access token.
+     * @param {cr.Event} evt Contains the authetication type and access token.
+     * @private
+     */
+    onAccessTokenReady_: function(event) {

[Toscano, 2013/04/19 17:04:33]

private handlers should come after all other private methods.

[Vitaly Buka (NO REVIEWS), 2013/04/19 22:14:29]

Done.

+      assert(event.authType == print_preview.Destination.Origin.DEVICE);
+      this.requestQueue_ = this.requestQueue_.filter(function(request) {
+        assert(request.origin == print_preview.Destination.Origin.DEVICE);

[Toscano, 2013/04/19 17:04:33]

Can you add a TODO to remove this assert once we support multiple Origin types?

[Vitaly Buka (NO REVIEWS), 2013/04/19 22:14:29]

Done.

+        if (request.origin != event.authType)

[Toscano, 2013/04/19 17:04:33]

Please enclose "if" body with {}

[Vitaly Buka (NO REVIEWS), 2013/04/19 22:14:29]

Done.

+          return true;
+        if (event.accessToken) {
+          request.xhr.setRequestHeader('Authorization',
+                                       'Bearer ' + event.accessToken);
+          this.sendRequest_(request);
+        } else {
+          // No valid token.
+          request.callback(401, {

[Toscano, 2013/04/19 17:04:33]

Should be 403. With a string as the second parameter.

[Vitaly Buka (NO REVIEWS), 2013/04/19 22:14:29]

403 cause sign in promo
actually it's goes always to createErrorEvent_ 

And jsdoc there:
{@code null} if status was not 200.

+            'errorCode' : -1,
+            'message' : 'No access token.'
+          }, request.origin);
+        }
+        return false;
+      }, this);
+    },
+
+    /**
      * Creates an object that represents a Google Cloud Print print ticket.
      * @param {!print_preview.Destination} destination Destination to print to.
      * @param {!print_preview.PrintTicketStore} printTicketStore Used to create
      *     the state of the print ticket.
      * @return {!Object} Google Cloud Print print ticket.
      * @private
      */
     createPrintTicket_: function(destination, printTicketStore) {
       assert(!destination.isLocal,
              'Trying to create a Google Cloud Print print ticket for a local ' +
@@ skipping 34 matching lines @@
             {type: pts.isDuplexEnabled() ? 'LONG_EDGE' : 'NO_DUPLEX'};
       }
       if (pts.hasOrientationCapability()) {
         cjt.print.page_orientation =
           {type: pts.isLandscapeEnabled() ? 'LANDSCAPE' : 'PORTRAIT'};
       }
       return JSON.stringify(cjt);
     },
 
     /**
-     * Sends a request to the Google Cloud Print API.
+     * Builds request to the Google Cloud Print API.
      * @param {string} method HTTP method of the request.
      * @param {string} action Google Cloud Print action to perform.
      * @param {Array.<!HttpParam>} params HTTP parameters to include in the
      *     request.
-     * @param {function(number, Object)} callback Callback to invoke when
-     *     request completes.
+     * @param {!print_preview.Destination.Origin} origin Origin for destination.
+     * @private

[Toscano, 2013/04/19 17:04:33]

Where is the @return statement?

[Vitaly Buka (NO REVIEWS), 2013/04/19 22:14:29]

Done.

      */
-    sendRequest_: function(method, action, params, callback) {
-      if (!this.xsrfToken_) {
-        // TODO(rltoscano): Should throw an error if not a read-only action or
-        // issue an xsrf token request.
+    buildRequest_: function(method, action, params, origin) {
+      var url = this.baseUrl_ + '/' + action + '?xsrf=';

[Toscano, 2013/04/19 17:04:33]

You add "xsrf=" to the url, but you only provide a value in COOKIES case. This
might cause an invalid XSRF token error on the server.

[Vitaly Buka (NO REVIEWS), 2013/04/19 22:14:29]

that's how search request worked before, the first always was with ?xsrf=&...

On 2013/04/19 17:04:33, Toscano wrote:

[Toscano, 2013/04/19 22:32:33]

Ok, not a big deal. As long as it works.

On 2013/04/19 22:14:29, Vitaly Buka wrote:

+      if (origin == print_preview.Destination.Origin.COOKIES) {
+        if (!this.xsrfToken_) {
+          // TODO(rltoscano): Should throw an error if not a read-only action or
+          // issue an xsrf token request.
+        } else {
+          url = url + this.xsrfToken_;
+        }
       }
-      var url = this.baseUrl_ + '/' + action + '?xsrf=' + this.xsrfToken_;
       var body = null;
-
       if (params) {
         if (method == 'GET') {
           url = params.reduce(function(partialUrl, param) {
             return partialUrl + '&' + param.name + '=' +
                 encodeURIComponent(param.value);
           }, url);
         } else if (method == 'POST') {
           body = params.reduce(function(partialBody, param) {
             return partialBody + 'Content-Disposition: form-data; name=\"' +
                 param.name + '\"\r\n\r\n' + param.value + '\r\n--' +
                 CloudPrintInterface.MULTIPART_BOUNDARY_ + '\r\n';
           }, '--' + CloudPrintInterface.MULTIPART_BOUNDARY_ + '\r\n');
         }
       }
 
       var headers = {};
       headers['X-CloudPrint-Proxy'] = 'ChromePrintPreview';
       if (method == 'GET') {
         headers['Content-Type'] = CloudPrintInterface.URL_ENCODED_CONTENT_TYPE_;
       } else if (method == 'POST') {
         headers['Content-Type'] = CloudPrintInterface.MULTIPART_CONTENT_TYPE_;
       }
 
       var xhr = new XMLHttpRequest();
-      xhr.onreadystatechange =
-          this.onReadyStateChange_.bind(this, xhr, callback);
       xhr.open(method, url, true);
-      xhr.withCredentials = true;
+      xhr.withCredentials =
+          (origin == print_preview.Destination.Origin.COOKIES);
       for (var header in headers) {
         xhr.setRequestHeader(header, headers[header]);
       }
-      xhr.send(body);
+
+      return new ClourPrintRequest(xhr, body, origin);

[Toscano, 2013/04/19 17:04:33]

Cloud*PrintRequest.

[Vitaly Buka (NO REVIEWS), 2013/04/19 22:14:29]

Done.

     },
 
     /**
+     * Sends a request to the Google Cloud Print API. Or queue if it needs to

[Toscano, 2013/04/19 17:04:33]

This is one sentence. No need for a period before "Or".

[Vitaly Buka (NO REVIEWS), 2013/04/19 22:14:29]

Done.

+     *     wait OAuth2 access token.
+     * @param {!CloudPrintRequest} request Request to send or queue.
+     * @param {function(number, Object, !print_preview.Destination.Origin)}
+     *     callback Callback to invoke when request completes.
+     * @private
+     */
+    sendOrQueueRequest_: function(request, callback) {

[Toscano, 2013/04/19 17:04:33]

It actually looks like it makes more sense to include the callback parameter in
the buildRequest_ function. Let's move it there.

[Vitaly Buka (NO REVIEWS), 2013/04/19 22:14:29]

Done.

+      request.callback = callback;

[Toscano, 2013/04/19 17:04:33]

The CloudPrintRequest does not have a callback field, can you add it?

[Vitaly Buka (NO REVIEWS), 2013/04/19 22:14:29]

Done.

+      if (request.origin == print_preview.Destination.Origin.COOKIES) {
+        return this.sendRequest_(request);
+      } else {
+        this.requestQueue_.push(request);
+        this.nativeLayer_.startGetAccessToken(request.origin);
+      }
+    },
+
+    sendRequest_: function(request) {

[Toscano, 2013/04/19 17:04:33]

Please add jsdoc for this method.

[Vitaly Buka (NO REVIEWS), 2013/04/19 22:14:29]

Done.

+      request.xhr.onreadystatechange =
+          this.onReadyStateChange_.bind(this, request);
+      request.xhr.send(request.body);
+    },
+
+    /**
      * Creates a Google Cloud Print interface error that is ready to dispatch.
      * @param {!CloudPrintInterface.EventType} type Type of the error.
      * @param {number} status HTTP status code of the failed request.
      * @param {Object} result JSON response of the request. {@code null} if
      *     status was not 200.
      * @return {!cr.Event} Google Cloud Print interface error event.
      * @private
      */
     createErrorEvent_: function(type, status, result) {
       var errorEvent = new cr.Event(type);
       errorEvent.status = status;
       errorEvent.errorCode = status == 200 ? result['errorCode'] : 0;
       errorEvent.message = status == 200 ? result['message'] : '';
       return errorEvent;
     },
 
     /**
      * Called when the ready-state of a XML http request changes.
      * Calls the successCallback with the result or dispatches an ERROR event.
-     * @param {XMLHttpRequest} xhr XML http request that changed.
-     * @param {function(number, Object)} callback Callback to invoke when
-     *     request completes.
+     * @param {!CloudPrintRequest} request Request that changed.
      * @private
      */
-    onReadyStateChange_: function(xhr, callback) {
-      if (xhr.readyState == 4) {
-        if (xhr.status == 200) {
-          var result = JSON.parse(xhr.responseText);
-          if (result['success']) {
+    onReadyStateChange_: function(request) {
+      if (request.xhr.readyState == 4) {
+        if (request.xhr.status == 200) {
+          var result = JSON.parse(request.xhr.responseText);
+          if (request.origin == print_preview.Destination.Origin.COOKIES &&
+              result['success']) {
             this.xsrfToken_ = result['xsrf_token'];
           }
         }
-        callback(xhr.status, result);
+        request.callback(request.xhr.status, result, request.origin);

[Toscano, 2013/04/19 17:04:33]

Let's just add a result field to CloudPrintRequest and pass result to the
callback because we might add more fields later:

request.result = result;
request.callback(request);

[Vitaly Buka (NO REVIEWS), 2013/04/19 22:14:29]

Done.

       }
     },
 
     /**
      * Called when the search request completes.
      * @param {boolean} isRecent Whether the search request was for recent
      *     destinations.
      * @param {number} status Status of the HTTP request.
      * @param {Object} result JSON response.
+     * @param {!print_preview.Destination.Origin} origin Origin for destination.
      * @private
      */
-    onSearchDone_: function(isRecent, status, result) {
+    onSearchDone_: function(isRecent, status, result, origin) {
       --this.outstandingCloudSearchRequestCount_;
       if (status == 200 && result['success']) {
         var printerListJson = result['printers'] || [];
         var printerList = [];
         printerListJson.forEach(function(printerJson) {
           try {
             printerList.push(
-                cloudprint.CloudDestinationParser.parse(
-                    printerJson, print_preview.Destination.Origin.COOKIES));
+                cloudprint.CloudDestinationParser.parse(printerJson, origin));
           } catch (err) {
             console.error('Unable to parse cloud print destination: ' + err);
           }
         });
         var searchDoneEvent =
             new cr.Event(CloudPrintInterface.EventType.SEARCH_DONE);
         searchDoneEvent.printers = printerList;
+        searchDoneEvent.origin = origin;
         searchDoneEvent.isRecent = isRecent;
         searchDoneEvent.email = result['request']['user'];
         this.dispatchEvent(searchDoneEvent);
       } else {
         var errorEvent = this.createErrorEvent_(
             CloudPrintInterface.EventType.SEARCH_FAILED, status, result);
         this.dispatchEvent(errorEvent);
       }
     },
 
     /**
      * Called when the submit request completes.
      * @param {number} status Status of the HTTP request.
      * @param {Object} result JSON response.
+     * @param {!print_preview.Destination.Origin} origin Origin for destination.
      * @private
      */
-    onSubmitDone_: function(status, result) {
+    onSubmitDone_: function(status, result, origin) {
       if (status == 200 && result['success']) {
         var submitDoneEvent = new cr.Event(
             CloudPrintInterface.EventType.SUBMIT_DONE);
         submitDoneEvent.jobId = result['job']['id'];
         this.dispatchEvent(submitDoneEvent);
       } else {
         var errorEvent = this.createErrorEvent_(
             CloudPrintInterface.EventType.SUBMIT_FAILED, status, result);
         this.dispatchEvent(errorEvent);
       }
     },
 
     /**
      * Called when the printer request completes.
      * @param {string} destinationId ID of the destination that was looked up.
      * @param {number} status Status of the HTTP request.
      * @param {Object} result JSON response.
+     * @param {!print_preview.Destination.Origin} origin Origin for destination.
      * @private
      */
-    onPrinterDone_: function(destinationId, status, result) {
+    onPrinterDone_: function(destinationId, status, result, origin) {
       if (status == 200 && result['success']) {
         var printerJson = result['printers'][0];
         var printer;
         try {
-          printer = cloudprint.CloudDestinationParser.parse(
-              printerJson, print_preview.Destination.Origin.COOKIES);
+          printer = cloudprint.CloudDestinationParser.parse(printerJson,
+                                                            origin);
         } catch (err) {
           console.error('Failed to parse cloud print destination: ' +
               JSON.stringify(printerJson));
           return;
         }
         var printerDoneEvent =
             new cr.Event(CloudPrintInterface.EventType.PRINTER_DONE);
         printerDoneEvent.printer = printer;
         this.dispatchEvent(printerDoneEvent);
       } else {
         var errorEvent = this.createErrorEvent_(
             CloudPrintInterface.EventType.PRINTER_FAILED, status, result);
         errorEvent.destinationId = destinationId;
-        errorEvent.destinationOrigin = print_preview.Destination.Origin.COOKIES;
-        this.dispatchEvent(errorEvent);
+        errorEvent.destinationOrigin = origin;
+        this.dispatchEvent(errorEvent, origin);
       }
     },
 
     /**
      * Called when the update printer TOS acceptance request completes.
      * @param {number} status Status of the HTTP request.
      * @param {Object} result JSON response.
+     * @param {!print_preview.Destination.Origin} origin Origin for destination.
      * @private
      */
-    onUpdatePrinterTosAcceptanceDone_: function(status, result) {
+    onUpdateTosAcceptanceDone_: function(status, result, origin) {
       if (status == 200 && result['success']) {
         // Do nothing.
       } else {
         var errorEvent = this.createErrorEvent_(
             CloudPrintInterface.EventType.SUBMIT_FAILED, status, result);
         this.dispatchEvent(errorEvent);
       }
     }
   };
 
   /**
+   * Data structure that holds data for delayed Cloud Print requests.

[Toscano, 2013/04/19 17:04:33]

delayed: not all of them are delayed.

[Vitaly Buka (NO REVIEWS), 2013/04/19 22:14:29]

Done.

+   * @param {!XMLHttpRequest} xhr Partially prepared http request.
+   * @param {string} body Data to send with POST requests.
+   * @param {!print_preview.Destination.Origin} origin Origin for destination.
+   * @constructor
+   */
+  function ClourPrintRequest(xhr, body, origin) {

[Toscano, 2013/04/19 17:04:33]

Cloud*PrintRequest

[Toscano, 2013/04/19 17:04:33]

Please add callback parameter.

[Vitaly Buka (NO REVIEWS), 2013/04/19 22:14:29]

Done.

[Vitaly Buka (NO REVIEWS), 2013/04/19 22:14:29]

Done.

+    /**
+     * Partially prepared http request.
+     * @type {!XMLHttpRequest}
+     */
+    this.xhr = xhr;
+
+    /**
+     * Data to send with POST requests.
+     * @type {string}
+     */
+    this.body = body;
+
+    /**
+     * Origin for destination.
+     * @type {!print_preview.Destination.Origin}
+     */
+    this.origin = origin;
+  };

[Toscano, 2013/04/19 17:04:33]

Please add "callback" and "result" fields.

[Vitaly Buka (NO REVIEWS), 2013/04/19 22:14:29]

Done.

+
+  /**
    * Data structure that represents an HTTP parameter.
    * @param {string} name Name of the parameter.
    * @param {string} value Value of the parameter.
    * @constructor
    */
   function HttpParam(name, value) {
     /**
      * Name of the parameter.
      * @type {string}
      */
     this.name = name;
 
     /**
      * Name of the value.
      * @type {string}
      */
     this.value = value;
   };
 
   // Export
   return {
     CloudPrintInterface: CloudPrintInterface
   };
 });