Bug 29897 – REGRESSION(r47820-r47822): Profiles aren't added to the inspector unless the inspector is already open when the profile completes

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

[Oliver Hunt]

When using the entry in the develop menu to start a profile, the profile is not inserted into the profile view unless the inspector has already been opened.

To reproduce:
1. Go to a site with js
2. Develop | Start profiling JavaScript
3. wait a bit
4. Develop | Stop profiling JavaScript

Result:
The inspector opens to the profiler pane, but doesn't display the profile

[Pavel Feldman]

Created attachment 40326 [details]
patch

Regression introduced in r47821. Thanks for the report.

[Oliver Hunt]

Comment on attachment 40326 [details]
patch

how does this interact with alloc profiles?  eg. if the first profile was an alloc profile would it display?

[Pavel Feldman]

(In reply to comment #2)
> (From update of attachment 40326 [details])
> how does this interact with alloc profiles?  eg. if the first profile was an
> alloc profile would it display?

This obviously needs to be resolved when we add different profile flavors (heap profiles are behind the flag now). Adding a FIXME note there and landing.

[Pavel Feldman]

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	WebCore/ChangeLog
	M	WebCore/inspector/front-end/ProfilesPanel.js
Committed r48907