DevTools: added support for image url & node in tracing-based timeline

DevTools: added support for image url & node in tracing-based timeline

- add trace event for RenderImage::paintIntoRect;
- fix name of decode & resize image events for tracing vs. old timeline;
- add processing of decode/resize/paint/imageref events to TimelineTraceEventBindings;
- generalize handling of additional event properties when building details, so that
    we avoid extra switches over event type by extracting properties relevant for
    details into TraceEvent properties;
- temporary workaround for crash on record stop in tests by cloning an AtomicString
    (see crbug.com/375242 for more details).

BUG=361045, 375242
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=175284

[yurys, 2014-06-02 11:22:23]

lgtm

https://codereview.chromium.org/308123008/diff/1/Source/devtools/front_end/ti...
File Source/devtools/front_end/timeline/TimelineTraceEventBindings.js (right):

https://codereview.chromium.org/308123008/diff/1/Source/devtools/front_end/ti...
Source/devtools/front_end/timeline/TimelineTraceEventBindings.js:212: if
(event.args["endData"])
endData always exists for Layout event, no need to check.

https://codereview.chromium.org/308123008/diff/1/Source/devtools/front_end/ti...
Source/devtools/front_end/timeline/TimelineTraceEventBindings.js:262:
event.backendNodeId = paintImageEvent.backendNodeId;
Would it make sense to write initiator instead:

event.initiator = paintImageEvent?

[caseq, 2014-06-02 11:57:06]

The CQ bit was checked by caseq@chromium.org

[caseq, 2014-06-02 11:57:08]

The CQ bit was unchecked by caseq@chromium.org

[caseq, 2014-06-02 11:57:19]

https://codereview.chromium.org/308123008/diff/1/Source/devtools/front_end/ti...
File Source/devtools/front_end/timeline/TimelineTraceEventBindings.js (right):

https://codereview.chromium.org/308123008/diff/1/Source/devtools/front_end/ti...
Source/devtools/front_end/timeline/TimelineTraceEventBindings.js:212: if
(event.args["endData"])
On 2014/06/02 11:22:24, yurys wrote:
> endData always exists for Layout event, no need to check.

Done.

https://codereview.chromium.org/308123008/diff/1/Source/devtools/front_end/ti...
Source/devtools/front_end/timeline/TimelineTraceEventBindings.js:262:
event.backendNodeId = paintImageEvent.backendNodeId;
On 2014/06/02 11:22:24, yurys wrote:
> Would it make sense to write initiator instead:
> 
> event.initiator = paintImageEvent?

Discussed this offline and decided not to do it now.

[caseq, 2014-06-02 11:57:22]

The CQ bit was checked by caseq@chromium.org

[commit-bot: I haz the power, 2014-06-02 11:57:38]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/caseq@chromium.org/308123008/20001

[commit-bot: I haz the power, 2014-06-02 14:20:03]

Change committed as 175284