Fix JavaScript style in ntp4

chrome/browser/resources/ntp4/page_list_view.js

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @fileoverview PageListView implementation.
  * PageListView manages page list, dot list, switcher buttons and handles apps
  * pages callbacks from backend.
  *
  * Note that you need to have AppLauncherHandler in your WebUI to use this code.
@@ skipping 200 matching lines @@
                'MostVisitedPage should be added as first tile page');
         this.mostVisitedPage = page;
       }
 
       // If we're appending an AppsPage and it's a temporary page, animate it.
       var animate = page instanceof ntp4.AppsPage &&
                     page.classList.contains('temporary');
       // Make a deep copy of the dot template to add a new one.
       var newDot = new ntp4.NavDot(page, title, titleIsEditable, animate);
       page.navigationDot = newDot;
-      this.dotList.insertBefore(newDot, opt_refNode ? opt_refNode.navigationDot
-                                                    : null);
+      this.dotList.insertBefore(newDot,
+                                opt_refNode ? opt_refNode.navigationDot :

[Evan Stade, 2012/02/02 23:48:42]

my spidey sense tells me you don't need this line break

[Tyler Breisacher (Chromium), 2012/02/02 23:54:47]

Done.

+                                              null);
       // Set a tab index on the first dot.
       if (this.dotList.dots.length == 1)
         newDot.tabIndex = 3;
 
       this.eventTracker.add(page, 'pagelayout', this.onPageLayout_.bind(this));
     },
 
     /**
      * Called by chrome when an existing app has been disabled or
      * removed/uninstalled from chrome.
@@ skipping 242 matching lines @@
       var page = this.cardSlider.currentCardValue;
 
       this.pageSwitcherStart.hidden = !page ||
           (this.cardSlider.currentCard == 0);
       this.pageSwitcherEnd.hidden = !page ||
           (this.cardSlider.currentCard == this.cardSlider.cardCount - 1);
 
       if (!page)
         return;
 
-      var pageSwitcherLeft = isRTL() ? this.pageSwitcherEnd
-                                     : this.pageSwitcherStart;
-      var pageSwitcherRight = isRTL() ? this.pageSwitcherStart
-                                      : this.pageSwitcherEnd;
+      var pageSwitcherLeft = isRTL() ? this.pageSwitcherEnd :
+                                       this.pageSwitcherStart;
+      var pageSwitcherRight = isRTL() ? this.pageSwitcherStart :
+                                        this.pageSwitcherEnd;
       var scrollbarWidth = page.scrollbarWidth;
       pageSwitcherLeft.style.width =
           (page.sideMargin + 13) + 'px';
       pageSwitcherLeft.style.left = '0';
       pageSwitcherRight.style.width =
           (page.sideMargin - scrollbarWidth + 13) + 'px';
       pageSwitcherRight.style.right = scrollbarWidth + 'px';
 
       var offsetTop = page.querySelector('.tile-page-content').offsetTop + 'px';
       pageSwitcherLeft.style.top = offsetTop;
@@ skipping 148 matching lines @@
       if (page.navigationDot)
         page.navigationDot.remove(opt_animate);
       this.cardSlider.removeCard(page);
     },
   };
 
   return {
     PageListView: PageListView
   };
 });