Tons of timeline tweaks

chrome/browser/resources/tracing/timeline_model.js

Index: chrome/browser/resources/tracing/timeline_model.js
diff --git a/chrome/browser/resources/tracing/timeline_model.js b/chrome/browser/resources/tracing/timeline_model.js
index a87370c17821a5b212ed8c2297fe83d4201841c4..40fff807b5de1fe44b7f8b9e2728b7a436805a51 100644
--- a/chrome/browser/resources/tracing/timeline_model.js
+++ b/chrome/browser/resources/tracing/timeline_model.js
@@ -283,7 +283,7 @@ cr.define('tracing', function() {
 
           // Store the slice in a non-nested subrow.
           var thread = self.getOrCreateProcess(event.pid).
-                            getOrCreateThread(event.tid);
+                           getOrCreateThread(event.tid);

[James Hawkins, 2011/11/11 23:37:40]

nit: Indentation for wrapped lines is 4 spaces from start of previous line.

In fact, if you wrap the line after '=', this would be much cleaner. Here and
line 300.

[nduca, 2011/11/12 01:28:27]

Done.

           thread.addNonNestedSlice(slice.slice);
           delete state.openNonNestedSlices[name];
         } else {
@@ -298,7 +298,7 @@ cr.define('tracing', function() {
 
           // Store the slice on the correct subrow.
           var thread = self.getOrCreateProcess(event.pid)
-                           .getOrCreateThread(event.tid);
+              .getOrCreateThread(event.tid);
           var subRowIndex = state.openSlices.length;
           thread.getSubrow(subRowIndex).push(slice);
 
@@ -338,7 +338,7 @@ cr.define('tracing', function() {
           }
         } else {
           this.importErrors.push('Unrecognized event phase: ' + event.ph +
-                                 '(' + event.name + ')');
+              '(' + event.name + ')');
         }
       }
       this.pruneEmptyThreads();