Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(116)

Unified Diff: chrome/browser/resources/ntp4/page_list_view.js

Issue 9320051: Fix JavaScript style in ntp4 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/ntp4/page_list_view.js
diff --git a/chrome/browser/resources/ntp4/page_list_view.js b/chrome/browser/resources/ntp4/page_list_view.js
index 1a8d7ed6c088931b11eb4a80374016c4ef469d2b..824182abdc3c4361ceb6d185269d5e9edcb632db 100644
--- a/chrome/browser/resources/ntp4/page_list_view.js
+++ b/chrome/browser/resources/ntp4/page_list_view.js
@@ -218,8 +218,9 @@ cr.define('ntp4', function() {
// 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
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;
@@ -482,10 +483,10 @@ cr.define('ntp4', function() {
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';

Powered by Google App Engine
This is Rietveld 408576698