Cleaning up not useful TODOs

src/js/background/collection/streamItems.js

@@ -79,7 +79,7 @@
         },
 
         addSongs: function(songs, options) {
-            //  TODO: reduce cyclomatic complexity.
+            //  TODO: Reduce cyclomatic complexity.
             /* jshint ignore:start */
             options = _.isUndefined(options) ? {} : options;
             songs = songs instanceof Backbone.Collection ? songs.models : _.isArray(songs) ? songs : [songs];

src/js/background/model/dataSource.js

@@ -16,7 +16,7 @@
             url: ''
         },
 
-        //  TODO: Function is way too big
+        //  TODO: Reduce cyclomatic complexity
         //  Take the URL given to the dataSource and parse it for relevant information.
         //  If the URL is for a Playlist -- just get the title and set the ID. If it's a Channel,
         //  need to fetch the Channel's Uploads playlist first.

src/js/foreground/model/selectionBar/selectionBar.js

@@ -46,7 +46,6 @@
             this.on('change:activeCollection', this._onChangeActiveCollection);
             this.listenTo(signInManager, 'change:signedInUser', this._onSignInManagerChangeSignedInUser);
 
-            //  TODO: Make bindEntityEvents a part of BB model
             Marionette.bindEntityEvents(this, this.get('streamItems'), this.streamItemsEvents);
         },
 

src/js/foreground/view/behavior/listItemButton.js

@@ -47,8 +47,8 @@
         _debounceOnClickAction: null,
 
         _doOnClickAction: function() {
+            //  TODO: Prefer not to check hasClass and leverage state of a model.
             if (!this.$el.hasClass('is-disabled')) {
-                //  TODO: This should probably be a triggerMethod?
                 this.view.doOnClickAction();
             }
         }

src/js/foreground/view/behavior/scrollable.js

@@ -24,7 +24,6 @@
             this._updateScrollbar();
         },
 
-        //  TODO: This wouldn't be necessary (and bad) if I calculate the height of the view before sliding it out and use transforms.
         onListHeightUpdated: function() {
             requestAnimationFrame(this._updateScrollbar.bind(this));
         },

src/js/foreground/view/behavior/slidingRender.js

@@ -127,7 +127,7 @@
         },
 
         _setRenderedElements: function(scrollTop) {
-            //  TODO: Clean this up. It's still such a huge function.
+            //  TODO: Reduce cyclomatic complexity
             /* jshint ignore:start */
             //  Figure out the range of items currently rendered:
             var currentMinRenderIndex = this.minRenderIndex;

src/js/foreground/view/dialog/createPlaylistView.js

@@ -8,7 +8,6 @@
     var CreatePlaylistView = Marionette.LayoutView.extend({
         id: 'createPlaylist',
         template: _.template(CreatePlaylistTemplate),
-        //  TODO: Would also be nice to pull this from the DB instead, need to truncate DB column to 150.
         titleMaxLength: 150,
 
         templateHelpers: function() {

src/js/foreground/view/dialog/dialogRegion.js

@@ -33,7 +33,6 @@
         //  http://developer.chrome.com/extensions/runtime#method-requestUpdateCheck
         _showDialogIfUpdateAvailable: function() {
             //  Calling requestUpdateCheck will cause onUpdateAvailable to trigger if an update is available.
-            //  TODO: Check if the response is 'throttled' and, if so, wait 5 seconds and try again. Potentially use exponential back-off instead of linear 5s.
             chrome.runtime.requestUpdateCheck(_.noop);
         },
 

src/js/foreground/view/dialog/editPlaylistView.js

@@ -7,7 +7,6 @@
     var EditPlaylistView = Marionette.LayoutView.extend({
         id: 'editPlaylist',
         template: _.template(EditPlaylistTemplate),
-        //  TODO: Not DRY w/ CreatePlaylistView -- pull from DB?
         titleMaxLength: 150,
 
         templateHelpers: function() {