Bug 1100328 - remove $cookies and localStorage from dependency

injection where not needed
mozilla · Nov 17, 2014 · 31c6859 · 31c6859
1 parent 2df8878
commit 31c6859
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 10 deletions.
diff --git a/ui/js/controllers/filters.js b/ui/js/controllers/filters.js
@@ -6,11 +6,11 @@
 
 treeherder.controller('FilterPanelCtrl', [
     '$scope', '$rootScope', '$route', '$routeParams', '$location', 'ThLog',
-    'localStorageService', 'thResultStatusList', 'thEvents', 'thJobFilters',
+    'thResultStatusList', 'thEvents', 'thJobFilters',
     'ThResultSetModel', 'thPinboard', 'thNotify', 'thFailureResults',
     function FilterPanelCtrl(
         $scope, $rootScope, $route, $routeParams, $location, ThLog,
-        localStorageService, thResultStatusList, thEvents, thJobFilters,
+        thResultStatusList, thEvents, thJobFilters,
         ThResultSetModel, thPinboard, thNotify, thFailureResults) {
 
         var $log = new ThLog(this.constructor.name);

diff --git a/ui/js/controllers/jobs.js b/ui/js/controllers/jobs.js
@@ -5,13 +5,13 @@
 "use strict";
 
 treeherder.controller('JobsCtrl', [
-    '$scope', '$http', '$rootScope', '$routeParams', 'ThLog', '$cookies',
-    'localStorageService', 'thUrl', 'ThRepositoryModel', 'thDefaultRepo',
+    '$scope', '$http', '$rootScope', '$routeParams', 'ThLog',
+    'thUrl', 'ThRepositoryModel', 'thDefaultRepo',
     'ThResultSetModel', 'thResultStatusList', '$location', 'thEvents',
     'ThJobModel',
     function JobsCtrl(
-        $scope, $http, $rootScope, $routeParams, ThLog, $cookies,
-        localStorageService, thUrl, ThRepositoryModel, thDefaultRepo,
+        $scope, $http, $rootScope, $routeParams, ThLog,
+        thUrl, ThRepositoryModel, thDefaultRepo,
         ThResultSetModel, thResultStatusList, $location, thEvents, ThJobModel) {
 
         var $log = new ThLog(this.constructor.name);

diff --git a/ui/js/controllers/main.js b/ui/js/controllers/main.js
@@ -6,13 +6,13 @@
 
 treeherder.controller('MainCtrl', [
     '$scope', '$rootScope', '$routeParams', '$location', 'ThLog',
-    'localStorageService', 'ThRepositoryModel', 'thPinboard',
+    'ThRepositoryModel', 'thPinboard',
     'thClassificationTypes', 'thEvents', '$interval', '$window',
     'ThExclusionProfileModel', 'thJobFilters', 'ThResultSetModel',
     'thDefaultRepo',
     function MainController(
         $scope, $rootScope, $routeParams, $location, ThLog,
-        localStorageService, ThRepositoryModel, thPinboard,
+        ThRepositoryModel, thPinboard,
         thClassificationTypes, thEvents, $interval, $window,
         ThExclusionProfileModel, thJobFilters, ThResultSetModel,
         thDefaultRepo) {

diff --git a/ui/js/directives/persona.js b/ui/js/directives/persona.js
@@ -5,10 +5,10 @@
 'use strict';
 
 treeherder.directive('personaButtons', [
-    '$http', '$q', '$log', '$rootScope', 'localStorageService',
+    '$http', '$q', '$log', '$rootScope',
     'thServiceDomain', 'BrowserId', 'ThUserModel',
     function(
-        $http, $q, $log, $rootScope, localStorageService, thServiceDomain,
+        $http, $q, $log, $rootScope, thServiceDomain,
         BrowserId, ThUserModel) {
 
     return {