JSLint cleanups. Removed a TODO and added API note for inline widgets.

Author: Narciso Jaramillo

src/editor/Editor.js

@@ -807,10 +807,10 @@ define(function (require, exports, module) {
         if (start.line <= pos.line && end.line >= pos.line) {
             if (endInclusive) {
                 return (start.line < pos.line || start.ch <= pos.ch) &&  // inclusive
-                       (end.line > pos.line   || end.ch >= pos.ch);      // inclusive
+                    (end.line > pos.line   || end.ch >= pos.ch);      // inclusive
             } else {
                 return (start.line < pos.line || start.ch <= pos.ch) &&  // inclusive
-                       (end.line > pos.line   || end.ch > pos.ch);       // exclusive
+                    (end.line > pos.line   || end.ch > pos.ch);       // exclusive
             }
 
         }
@@ -1060,10 +1060,7 @@ define(function (require, exports, module) {
 
             // Callback to widget once parented to the editor. The widget should call back to
             // setInlineWidgetHeight() in order to set its initial height and animate open.
-            // TODO: Should we make onAdded() return the desired height so that it's a
-            // required part of the API, instead of people just having to know that they
-            // need to call setInlineWidgetHeight() at least once?
-            inlineWidget.onAdded();    
+            inlineWidget.onAdded();
         });
     };
 

src/editor/InlineWidget.js

@@ -89,6 +89,8 @@ define(function (require, exports, module) {
     /**
      * Called once content is parented in the host editor's DOM. Useful for performing tasks like setting
      * focus or measuring content, which require htmlContent to be in the DOM tree.
+     * IMPORTANT: onAdded() must ensure that hostEditor.setInlineWidgetHeight() is called at least once in order
+     * to set the initial height of the widget and animate it open.
      */
     InlineWidget.prototype.onAdded = function () {
         // Does nothing in base implementation.

src/extensions/default/QuickView/unittests.js

@@ -308,9 +308,9 @@ define(function (require, exports, module) {
                 var bounds = getBounds(object, false),
                     editorBounds = getBounds(testWindow.$("#editor-holder"), true);
                 return bounds.left   >= editorBounds.left   &&
-                       bounds.right  <= editorBounds.right  &&
-                       bounds.top    >= editorBounds.top    &&
-                       bounds.bottom <= editorBounds.bottom;
+                    bounds.right  <= editorBounds.right  &&
+                    bounds.top    >= editorBounds.top    &&
+                    bounds.bottom <= editorBounds.bottom;
             }
 
             function toggleOption(commandID, text) {