From 0343f7d1ee0c15743dc96f16c4946f2b186f8504 Mon Sep 17 00:00:00 2001
From: Nicholas Nethercote <nnethercote@mozilla.com>
Date: Mon, 20 Feb 2012 14:42:22 -0800
Subject: [PATCH] Bug 728780 - Define String.prototype.startsWith and use it in
 about:memory.  r=jschoenick.

--HG--
extra : rebase_source : 8b3f4476cc93279ebb481e529bbf35b353d938c5
---
 .../aboutmemory/content/aboutMemory.js         | 18 +++++++++++-------
 1 file changed, 11 insertions(+), 7 deletions(-)

diff --git a/toolkit/components/aboutmemory/content/aboutMemory.js b/toolkit/components/aboutmemory/content/aboutMemory.js
index 8d30f87438e..d7bf490c8a7 100644
--- a/toolkit/components/aboutmemory/content/aboutMemory.js
+++ b/toolkit/components/aboutmemory/content/aboutMemory.js
@@ -63,6 +63,10 @@ const gVerbose = location.href === "about:memory?verbose" ||
 
 let gChildMemoryListener = undefined;
 
+// This is a useful function and an efficient way to implement it.
+String.prototype.startsWith =
+  function(s) { return this.lastIndexOf(s, 0) === 0; }
+
 //---------------------------------------------------------------------------
 
 // Forward slashes in URLs in paths are represented with backslashes to avoid
@@ -101,10 +105,10 @@ function addChildObserversAndUpdate(aUpdateFn)
 
 function onLoad()
 {
-  if (location.href.indexOf("about:memory") === 0) {
+  if (location.href.startsWith("about:memory")) {
     document.title = "about:memory";
     onLoadAboutMemory();
-  } else if (location.href.indexOf("about:compartment") === 0) {
+  } else if (location.href.startsWith("about:compartment")) {
     document.title = "about:compartments";
     onLoadAboutCompartments();
   } else {
@@ -452,7 +456,7 @@ Report.prototype = {
   treeNameMatches: function(aTreeName) {
     // Nb: the '/' must be present, because we have a KIND_OTHER reporter
     // called "explicit" which is not part of the "explicit" tree.
-    return this._unsafePath.indexOf(aTreeName) === 0 &&
+    return this._unsafePath.startsWith(aTreeName) &&
            this._unsafePath.charAt(aTreeName.length) === '/';
   }
 };
@@ -466,8 +470,8 @@ function getReportsByProcess(aMgr)
 
   function ignoreSingle(aPath) 
   {
-    return (aPath.indexOf("smaps/") === 0 && !gVerbose) ||
-           (aPath.indexOf("compartments/") === 0)
+    return (aPath.startsWith("smaps/") && !gVerbose) ||
+           (aPath.startsWith("compartments/"))
   }
 
   function ignoreMulti(aName)
@@ -1525,7 +1529,7 @@ function getCompartmentsByProcess(aMgr)
 
   function ignoreSingle(aPath) 
   {
-    return aPath.indexOf("compartments/") !== 0;
+    return !aPath.startsWith("compartments/");
   }
 
   function ignoreMulti(aName)
@@ -1560,7 +1564,7 @@ function getCompartmentsByProcess(aMgr)
       // These null principal compartments are user compartments according to
       // the JS engine, but they look odd being shown with content
       // compartments, so we put them in the system compartments list.
-      if (unsafeNames[2].indexOf("moz-nullprincipal:{") === 0) {
+      if (unsafeNames[2].startsWith("moz-nullprincipal:{")) {
         isSystemCompartment = true;
       }