Upend resource loading approach

Source/Core/BingMapsGeocoderService.js

@@ -5,15 +5,17 @@ define([
         './defined',
         './defineProperties',
         './loadJsonp',
-        './Rectangle'
+        './Rectangle',
+        './Resource'
     ], function(
         BingMapsApi,
         Check,
         defaultValue,
         defined,
         defineProperties,
         loadJsonp,
-        Rectangle) {
+        Rectangle,
+        Resource) {
     'use strict';
 
     var url = 'https://dev.virtualearth.net/REST/v1/Locations';
@@ -42,6 +44,13 @@ define([
                 options.scene._frameState.creditDisplay.addDefaultCredit(errorCredit);
             }
         }
+
+        this._resource = new Resource({
+            url: url,
+            queryParameters: {
+                key: this._key
+            }
+        });
     }
 
     defineProperties(BingMapsGeocoderService.prototype, {
@@ -81,16 +90,13 @@ define([
         Check.typeOf.string('query', query);
         //>>includeEnd('debug');
 
-        var key = this.key;
-        var promise = loadJsonp(url, {
-            parameters : {
-                query : query,
-                key : key
-            },
-            callbackParameterName : 'jsonp'
+        var resource = this._resource.getDerivedResource({
+            queryParameters: {
+                query: query
+            }
         });
 
-        return promise.then(function(result) {
+        return loadJsonp(resource, 'jsonp').then(function(result) {
             if (result.resourceSets.length === 0) {
                 return [];
             }

Source/Core/CesiumTerrainProvider.js

@@ -8,17 +8,18 @@ define([
         './defaultValue',
         './defined',
         './defineProperties',
+        './deprecationWarning',
         './DeveloperError',
         './Event',
         './GeographicTilingScheme',
         './HeightmapTerrainData',
         './IndexDatatype',
-        './joinUrls',
         './loadArrayBuffer',
         './loadJson',
         './Math',
         './OrientedBoundingBox',
         './QuantizedMeshTerrainData',
+        './Resource',
         './RuntimeError',
         './TerrainProvider',
         './TileAvailability',
@@ -33,17 +34,18 @@ define([
         defaultValue,
         defined,
         defineProperties,
+        deprecationWarning,
         DeveloperError,
         Event,
         GeographicTilingScheme,
         HeightmapTerrainData,
         IndexDatatype,
-        joinUrls,
         loadArrayBuffer,
         loadJson,
         CesiumMath,
         OrientedBoundingBox,
         QuantizedMeshTerrainData,
+        Resource,
         RuntimeError,
         TerrainProvider,
         TileAvailability,
@@ -68,8 +70,7 @@ define([
      * @constructor
      *
      * @param {Object} options Object with the following properties:
-     * @param {String} options.url The URL of the Cesium terrain server.
-     * @param {Proxy} [options.proxy] A proxy to use for requests. This object is expected to have a getURL function which returns the proxied URL, if needed.
+     * @param {Resource|String} options.url The URL of the Cesium terrain server.
      * @param {Boolean} [options.requestVertexNormals=false] Flag that indicates if the client should request additional lighting information from the server, in the form of per vertex normals if available.
      * @param {Boolean} [options.requestWaterMask=false] Flag that indicates if the client should request per tile water masks from the server,  if available.
      * @param {Ellipsoid} [options.ellipsoid] The ellipsoid.  If not specified, the WGS84 ellipsoid is used.
@@ -109,8 +110,16 @@ define([
         }
         //>>includeEnd('debug');
 
-        this._url = options.url;
-        this._proxy = options.proxy;
+        if (defined(options.proxy)) {
+            deprecationWarning('CesiumTerrainProvider.proxy', 'The options.proxy parameter has been deprecated. Specify options.url as a Resource instance and set the proxy property there.');
+        }
+
+        var resource = Resource.createIfNeeded(options.url, {
+            proxy: options.proxy
+        });
+        resource.isDirectory = true;
+
+        this._resource = resource;
 
         this._tilingScheme = new GeographicTilingScheme({
             numberOfLevelZeroTilesX : 2,
@@ -154,11 +163,10 @@ define([
         this._ready = false;
         this._readyPromise = when.defer();
 
-        var lastUrl = this._url;
-        var metadataUrl = joinUrls(this._url, 'layer.json');
-        if (defined(this._proxy)) {
-            metadataUrl = this._proxy.getURL(metadataUrl);
-        }
+        var lastResource = this._resource;
+        var metadataResource = lastResource.getDerivedResource({
+            url: 'layer.json'
+        });
 
         var that = this;
         var metadataError;
@@ -209,15 +217,6 @@ define([
             }
 
             var tileUrlTemplates = data.tiles;
-            for (var i = 0; i < tileUrlTemplates.length; ++i) {
-                var template = new Uri(tileUrlTemplates[i]);
-                var baseUri = new Uri(lastUrl);
-                if (template.authority && !baseUri.authority) {
-                    baseUri.authority = template.authority;
-                    baseUri.scheme = template.scheme;
-                }
-                tileUrlTemplates[i] = joinUrls(baseUri, template).toString().replace('{version}', data.version);
-            }
 
             var availableTiles = data.available;
             var availability;
@@ -267,6 +266,7 @@ define([
             }
 
             layers.push(new LayerInformation({
+                version: data.version,
                 isHeightmap: isHeightmap,
                 tileUrlTemplates: tileUrlTemplates,
                 availability: availability,
@@ -281,20 +281,21 @@ define([
                     console.log('A layer.json can\'t have a parentUrl if it does\'t have an available array.');
                     return when.resolve();
                 }
-                lastUrl = joinUrls(lastUrl, parentUrl);
-                metadataUrl = joinUrls(lastUrl, 'layer.json');
-                if (defined(that._proxy)) {
-                    metadataUrl = that._proxy.getURL(metadataUrl);
-                }
-                var parentMetadata = loadJson(metadataUrl);
+                lastResource = lastResource.getDerivedResource({
+                    url: parentUrl
+                });
+                metadataResource = lastResource.getDerivedResource({
+                    url: 'layer.json'
+                });
+                var parentMetadata = loadJson(metadataResource);
                 return when(parentMetadata, parseMetadataSuccess, parseMetadataFailure);
             }
 
             return when.resolve();
         }
 
         function parseMetadataFailure(data) {
-            var message = 'An error occurred while accessing ' + metadataUrl + '.';
+            var message = 'An error occurred while accessing ' + metadataResource.url + '.';
             metadataError = TileProviderError.handleError(metadataError, that, that._errorEvent, message, undefined, undefined, undefined, requestMetadata);
         }
 
@@ -344,7 +345,7 @@ define([
         }
 
         function requestMetadata() {
-            var metadata = loadJson(metadataUrl);
+            var metadata = loadJson(metadataResource);
             when(metadata, metadataSuccess, metadataFailure);
         }
 
@@ -600,13 +601,6 @@ define([
 
         var tmsY = (yTiles - y - 1);
 
-        var url = urlTemplates[(x + tmsY + level) % urlTemplates.length].replace('{z}', level).replace('{x}', x).replace('{y}', tmsY);
-
-        var proxy = this._proxy;
-        if (defined(proxy)) {
-            url = proxy.getURL(url);
-        }
-
         var extensionList = [];
         if (this._requestVertexNormals && layerToUse.hasVertexNormals) {
             extensionList.push(layerToUse.littleEndianExtensionSize ? 'octvertexnormals' : 'vertexnormals');
@@ -615,7 +609,19 @@ define([
             extensionList.push('watermask');
         }
 
-        var promise = loadArrayBuffer(url, getRequestHeader(extensionList), request);
+        var resource = this._resource.getDerivedResource({
+            url: urlTemplates[(x + tmsY + level) % urlTemplates.length],
+            templateValues: {
+                version: layerToUse.version,
+                z: level,
+                x: x,
+                y: tmsY
+            },
+            headers: getRequestHeader(extensionList),
+            request: request
+        });
+
+        var promise = loadArrayBuffer(resource);
 
         if (!defined(promise)) {
             return undefined;

Source/Core/EarthOrientationParameters.js

@@ -8,6 +8,7 @@ define([
         './JulianDate',
         './LeapSecond',
         './loadJson',
+        './Resource',
         './RuntimeError',
         './TimeConstants',
         './TimeStandard'
@@ -21,6 +22,7 @@ define([
         JulianDate,
         LeapSecond,
         loadJson,
+        Resource,
         RuntimeError,
         TimeConstants,
         TimeStandard) {
@@ -95,7 +97,7 @@ define([
         } else if (defined(options.url)) {
             // Download EOP data.
             var that = this;
-            this._downloadPromise = when(loadJson(options.url), function(eopData) {
+            this._downloadPromise = when(loadJson(new Resource({url: options.url})), function(eopData) {
                 onDataReady(that, eopData);
             }, function() {
                 that._dataError = 'An error occurred while retrieving the EOP data from the URL ' + options.url + '.';