RESOLVED FIXED 30253
Alter profile link format to enable multiple profile types 
https://bugs.webkit.org/show_bug.cgi?id=30253
Summary Alter profile link format to enable multiple profile types
Alexander Pavlov (apavlov)
Reported 2009-10-09 07:02:21 PDT
Currently, there is a single profile type (CPU profile) and its link URL format handled by WebInspector.documentClicked is as follows: webkit-profile://<title>/<uid> This does not warrant a possibility to introduce other types of profiles (say, Heap profiles). The suggested profile link URL format that enables multiple profile types looks like this: webkit-profile://<profileType>/<title>#<uid> where <profileType>, <title>, and <uid> are URL-encoded.
Attachments
patch (3.98 KB, patch)
2009-10-09 08:44 PDT, Alexander Pavlov (apavlov) 		timothy: review+
DetailsFormatted DiffDiff
[IMAGE] Screenshot with regression (83.72 KB, image/png)
2009-10-12 01:54 PDT, Pavel Feldman 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Alexander Pavlov (apavlov)
Comment 1 2009-10-09 08:44:07 PDT
Created attachment 40949 [details] patch
Pavel Feldman
Comment 2 2009-10-12 01:53:05 PDT
Not landing since this introduces a regression: CPU profiles with similar names are no longer grouped properly (see screenshot).
Pavel Feldman
Comment 3 2009-10-12 01:54:01 PDT
Created attachment 41025 [details] [IMAGE] Screenshot with regression
Alexander Pavlov (apavlov)
Comment 4 2009-10-12 02:40:02 PDT
(In reply to comment #2) > Not landing since this introduces a regression: CPU profiles with similar names > are no longer grouped properly (see screenshot). This is an unrelated regression. Filed as https://bugs.webkit.org/show_bug.cgi?id=30299
Pavel Feldman
Comment 5 2009-10-13 01:26:44 PDT
Committing to http://svn.webkit.org/repository/webkit/trunk ... M WebCore/ChangeLog M WebCore/inspector/InspectorController.cpp M WebCore/inspector/front-end/inspector.js Committed r49495
Note You need to log in before you can comment on or make changes to this bug.