better

Author: gaearon

src/renderers/shared/fiber/ReactDebugFiberPerf.js

@@ -31,31 +31,24 @@ let isProfiling = true;
 
 // TODO: individual render methods
 
-function getLabel(fiber) {
+function getMarkName(fiber) {
+  return `react:${fiber._debugID}`;
+}
+
+function shouldIgnore(fiber) {
   switch (fiber.tag) {
     case HostRoot:
-      return '(root)';
+    case HostComponent:
     case HostText:
-      return '(text)';
     case HostPortal:
-      return '(portal)';
     case YieldComponent:
-      return '(yield)';
     case Fragment:
-      return '(fragment)';
+      return true;
     default:
-      return getComponentName(fiber);
+      return false;
   }
 }
 
-function getMarkName(fiber) {
-  return `react:${fiber._debugID}`;
-}
-
-function shouldIgnore(fiber) {
-  return typeof fiber.type === 'string';
-}
-
 let bailedFibers = new Set();
 
 function markBeginWork(fiber) {
@@ -80,7 +73,7 @@ function markCompleteWork(fiber) {
   if (bailedFibers.has(fiber)) {
     bailedFibers.delete(fiber);
   } else {
-    performance.measure(getLabel(fiber), getMarkName(fiber));
+    performance.measure(getComponentName(fiber), getMarkName(fiber));
   }
 }
 
@@ -89,15 +82,15 @@ function markWillCommit() {
 }
 
 function markDidCommit() {
-  performance.measure('React: Commit Tree', 'react:commit');
+  performance.measure('Commit React Tree', 'react:commit');
 }
 
 function markWillReconcile() {
   performance.mark('react:reconcile');
 }
 
 function markDidReconcile() {
-  performance.measure('React: Reconcile Tree', 'react:reconcile');
+  performance.measure('Reconcile React Tree', 'react:reconcile');
 }
 
 exports.markBeginWork = markBeginWork;