From 580e10b024232aa76112750c759a5ecf0be8fc45 Mon Sep 17 00:00:00 2001 From: Victor Woeltjen Date: Fri, 7 Oct 2016 10:15:02 -0700 Subject: [PATCH] [API] Only use strategy where needed Satisfies JSHint. --- src/api/telemetry/TelemetryAPI.js | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/src/api/telemetry/TelemetryAPI.js b/src/api/telemetry/TelemetryAPI.js index 8b5ed95f6b..6e310b1aa8 100644 --- a/src/api/telemetry/TelemetryAPI.js +++ b/src/api/telemetry/TelemetryAPI.js @@ -201,9 +201,7 @@ define([ /** * @private */ - TelemetryAPI.prototype.findProvider = function (domainObject, options) { - var strategy = options.strategy; - + TelemetryAPI.prototype.findProvider = function (domainObject, strategy) { function supportsDomainObject(provider) { return provider.canProvideTelemetry(domainObject); } @@ -236,7 +234,7 @@ define([ * telemetry data */ TelemetryAPI.prototype.request = function (domainObject, options) { - var provider = this.findProvider(domainObject, options); + var provider = this.findProvider(domainObject, options.strategy); return provider ? provider.request(domainObject, options) : Promise.reject([]); @@ -310,7 +308,7 @@ define([ limitEvaluator: undefined }, function (defaultValue, method) { TelemetryAPI.prototype[method] = function (domainObject) { - var provider = this.findProvider(domainObject, options); + var provider = this.findProvider(domainObject); return provider ? provider[method].apply(provider, arguments) : defaultValue;