Changeset 35756 in webkit for trunk/JavaScriptCore/profiler/Profile.cpp

Timestamp:

Aug 14, 2008, 11:11:20 AM (17 years ago)

Author:

[email protected]

Message:

JavaScriptCore:

2008-08-14 Kevin McCullough <​[email protected]>

Reviewed by Tim.

<rdar://problem/6115819> Notify of profile in console

• Profiles now have a unique ID so that they can be linked to the console message that announces that a profile completed.

• profiler/HeavyProfile.cpp: (KJS::HeavyProfile::HeavyProfile):
• profiler/Profile.cpp: (KJS::Profile::create): (KJS::Profile::Profile):
• profiler/Profile.h: (KJS::Profile::uid):
• profiler/ProfileGenerator.cpp: (KJS::ProfileGenerator::create): (KJS::ProfileGenerator::ProfileGenerator):
• profiler/ProfileGenerator.h:
• profiler/Profiler.cpp: (KJS::Profiler::startProfiling):
• profiler/TreeProfile.cpp: (KJS::TreeProfile::create): (KJS::TreeProfile::TreeProfile):
• profiler/TreeProfile.h:

WebCore:

2008-08-14 Kevin McCullough <​[email protected]>

Reviewed by Tim.

<rdar://problem/6115819> Notify of profile in console

• page/InspectorController.cpp: (WebCore::InspectorController::addProfile): (WebCore::InspectorController::addProfileMessageToConsole): Called by addProfile this is the function that adds a message to the console that a profile finished.
• page/InspectorController.h:
• page/JavaScriptProfile.cpp: Expose the profiler's unique ID to match the console log to the profile in the web inspector. (WebCore::getUniqueIdCallback): (WebCore::ProfileClass):
• page/inspector/ProfilesPanel.js: Created a map of all the profiles by Id to bring up the requested profile. Also select and reveal the profile in the profile panel. And created displayTitleForProfileLink() which formats a title taking into account if it's user initiated or if there are multiples. Lasty, I put the user initiated profile in a variable.
• page/inspector/inspector.js: Make the profile title be a clickable link that will take the user to the identified profile. Also expose the count of user initiated profiles so they can be displayed in the console with the correct count.

File:

• trunk/JavaScriptCore/profiler/Profile.cpp (modified) (1 diff)

trunk/JavaScriptCore/profiler/Profile.cpp

@@ -33 +33 @@
 namespace KJS {
 
-PassRefPtr<Profile> Profile::create(const UString& title)
+PassRefPtr<Profile> Profile::create(const UString& title, unsigned uid)
 {
-    return TreeProfile::create(title);
+    return TreeProfile::create(title, uid);
 }
 
-Profile::Profile(const UString& title)
+Profile::Profile(const UString& title, unsigned uid)
     : m_title(title)
+    , m_uid(uid)
 {
     // FIXME: When multi-threading is supported this will be a vector and calls