Commit ffed0b28 authored by Bailey Berro's avatar Bailey Berro Committed by Commit Bot

Remove unused field from ProvidersModel

- Removes the largeIconUrl field from ProvidersModel which did not map
accurately to the Provider definition in file_manager_private.idl.

Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation

Test: tested manually w/ new clojure compiler provided by scottchen@
Change-Id: I4cea3c7633027cf1e3808aa2f895190630d2adf2
Reviewed-on: https://chromium-review.googlesource.com/994291
Commit-Queue: Bailey Berro <baileyberro@chromium.org>
Reviewed-by: default avatarTomasz Mikolajewski <mtomasz@chromium.org>
Reviewed-by: default avatarScott Chen <scottchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#548329}
parent b5ce7dd1
......@@ -8,7 +8,6 @@
*
* @param {string} providerId
* @param {!IconSet} iconSet
* @param {string} largeIconUrl
* @param {string} name
* @param {boolean} configurable
* @param {boolean} watchable
......@@ -18,8 +17,8 @@
* @struct
*/
function ProvidersModelItem(
providerId, iconSet, largeIconUrl, name, configurable, watchable,
multipleMounts, source) {
providerId, iconSet, name, configurable, watchable, multipleMounts,
source) {
/**
* @private {string}
* @const
......@@ -32,12 +31,6 @@ function ProvidersModelItem(
*/
this.iconSet_ = iconSet;
/**
* @private {string}
* @const
*/
this.largeIconUrl_ = largeIconUrl;
/**
* @private {string}
* @const
......@@ -84,13 +77,6 @@ ProvidersModelItem.prototype = {
return this.iconSet_;
},
/**
* @return {string}
*/
get largeIconUrl() {
return this.largeIconUrl_;
},
/**
* @return {string}
*/
......@@ -157,9 +143,9 @@ ProvidersModel.prototype.getInstalledProviders = function() {
var results = [];
providers.forEach(function(provider) {
results.push(new ProvidersModelItem(
provider.providerId, provider.iconSet, provider.largeIconUrl,
provider.name, provider.configurable, provider.watchable,
provider.multipleMounts, provider.source));
provider.providerId, provider.iconSet, provider.name,
provider.configurable, provider.watchable, provider.multipleMounts,
provider.source));
});
fulfill(results);
});
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment