Add tracing for RenderView::hitTest and FrameView::invalidateTree.

Source/core/frame/FrameView.cpp

 /*
  * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org>
  *                     1999 Lars Knoll <knoll@kde.org>
  *                     1999 Antti Koivisto <koivisto@kde.org>
  *                     2000 Dirk Mueller <mueller@kde.org>
  * Copyright (C) 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserved.
  *           (C) 2006 Graham Dennis (graham.dennis@gmail.com)
  *           (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
@@ skipping 1018 matching lines @@
     if (frame().page())
         frame().page()->chrome().client().layoutUpdated(m_frame.get());
 }
 
 // The plan is to move to compositor-queried paint invalidation, in which case this
 // method would setNeedsRedraw on the GraphicsLayers with invalidations and
 // let the compositor pick which to actually draw.
 // See http://crbug.com/306706
 void FrameView::invalidateTree(RenderObject* root)
 {
+    TRACE_EVENT0("blink", "FrameView::invalidateTree");

[dsinclair, 2014/06/06 01:59:03]

This already exists on line 1047.

[chrishtr, 2014/06/06 02:08:03]

Done.

[ojan, 2014/06/06 03:22:38]

Why is the one on line 1047 disabled by default?

[dsinclair, 2014/06/06 03:29:27]

The TRACE_STR_COPY has a cost which I didn't want to incur in the default case.

[ojan, 2014/06/06 05:53:11]

Can we put in a trace (instead or in addition the the one below) that is enabled
by default that doesn't add the root parameter? Having the trace by default is
more useful than having the trace with the extra information. When we were
looking at the trace today, there was this untraced block and we had to stare at
the code and add speculative traces to see where the time was going.

+
     ASSERT(RuntimeEnabledFeatures::repaintAfterLayoutEnabled());
     ASSERT(!root->needsLayout());
     // We should only invalidate paints for the outer most layout. This works as
     // we continue to track paint invalidation rects until this function is called.
     ASSERT(!m_nestedLayoutCount);
 
     TRACE_EVENT1(TRACE_DISABLED_BY_DEFAULT("blink.invalidation"), "FrameView::invalidateTree",
         "root", TRACE_STR_COPY(root->debugName().ascii().data()));
 
     // FIXME: really, we're in the paint invalidation phase here, and the compositing queries are legal.
@@ skipping 2224 matching lines @@
 void FrameView::willRemoveScrollbar(Scrollbar* scrollbar, ScrollbarOrientation orientation)
 {
     ScrollableArea::willRemoveScrollbar(scrollbar, orientation);
     if (AXObjectCache* cache = axObjectCache()) {
         cache->remove(scrollbar);
         cache->handleScrollbarUpdate(this);
     }
 }
 
 } // namespace WebCore