Bug 37340 – Web Inspector: Fixes "wrong parent" for GCEvents that come at the end of a timeline record node.

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector (Deprecated)

This patch fixes "wrong parent" for GCEvents that come at the end of a timeline record node.

The problem was as follows:

InspectorTimelineAgent::didCompleteRecord() pops the record stack and then
calls addRecordToTimeline()

addRecordToTimeline() pushes the GCEvents. This then attributes the GCEvents to
whatever is currently last on the record stack. Too bad the real parent has
already been popped :).

This means that the only time GCEvents are correctly parented are when they
happen to get pushed by some other peer node closing BEFORE their parent tries
to pop.

The solution:

Add a call to pushGCEventRecords() to didCompleteCurrentRecord(). Simply remove the call to pushGCEventRecords() from addRecordToTimeline() and just ensure that callers of addRecordToTimeline() push the GCEvents if they need to.

Created attachment 52959 [details]
patch

[Ilya Tikhonovsky]

> Index: WebCore/ChangeLog
> ===================================================================
> --- WebCore/ChangeLog	(revision 57339)
> +++ WebCore/ChangeLog	(working copy)
> @@ -1,3 +1,22 @@
> [phone]  Jaime Yap  <[e-mail]>
> +
> +        Reviewed by NOBODY (OOPS!).
> +
> +        Web Inspector: Fixes "wrong parent" for GCEvents that come at the end of a
> +        timeline record node.
> +        https://bugs.webkit.org/show_bug.cgi?id=37340
> +        
> +        No new tests. (OOPS!)

Please remove this line.


The other changes looks good to me.

Created attachment 52968 [details]
removes extra line from Changelog

[WebKit Commit Bot]

Comment on attachment 52968 [details]
removes extra line from Changelog

Clearing flags on attachment: 52968

Committed r57387: <http://trac.webkit.org/changeset/57387>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.