[SCM] WebKit Debian packaging branch, webkit-1.1, updated. upstream/1.1.16-1409-g5afdf4d
eric at webkit.org
eric at webkit.org
Thu Dec 3 13:25:40 UTC 2009
The following commit has been merged in the webkit-1.1 branch:
commit 0fc1eec0acfa4fd2f9678af844862a83936b16e7
Author: eric at webkit.org <eric at webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Tue Nov 3 17:47:39 2009 +0000
2009-11-03 Mikhail Naganov <mnaganov at chromium.org>
Reviewed by Timothy Hatcher.
Prepare for heap profiles upstreaming:
- pass profile type id from InspectorController;
- this makes WebInspector.CPUProfile redundant---removed;
- support multiple profile types when populating profiles.
https://bugs.webkit.org/show_bug.cgi?id=31052
* inspector/InspectorController.cpp:
(WebCore::InspectorController::createProfileHeader):
* inspector/front-end/ProfileView.js:
(WebInspector.CPUProfileView.profileCallback):
(WebInspector.CPUProfileView):
(WebInspector.CPUProfileView.prototype._sortData):
* inspector/front-end/ProfilesPanel.js:
(WebInspector.ProfilesPanel.prototype.addProfileHeader):
* inspector/front-end/inspector.js:
(WebInspector.addProfileHeader):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@50460 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 06290b7..964d709 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,25 @@
+2009-11-03 Mikhail Naganov <mnaganov at chromium.org>
+
+ Reviewed by Timothy Hatcher.
+
+ Prepare for heap profiles upstreaming:
+ - pass profile type id from InspectorController;
+ - this makes WebInspector.CPUProfile redundant---removed;
+ - support multiple profile types when populating profiles.
+
+ https://bugs.webkit.org/show_bug.cgi?id=31052
+
+ * inspector/InspectorController.cpp:
+ (WebCore::InspectorController::createProfileHeader):
+ * inspector/front-end/ProfileView.js:
+ (WebInspector.CPUProfileView.profileCallback):
+ (WebInspector.CPUProfileView):
+ (WebInspector.CPUProfileView.prototype._sortData):
+ * inspector/front-end/ProfilesPanel.js:
+ (WebInspector.ProfilesPanel.prototype.addProfileHeader):
+ * inspector/front-end/inspector.js:
+ (WebInspector.addProfileHeader):
+
2009-11-03 Dan Kegel <dank at chromium.org>
Reviewed by Dimitri Glazkov.
diff --git a/WebCore/inspector/InspectorController.cpp b/WebCore/inspector/InspectorController.cpp
index eff8014..0327017 100644
--- a/WebCore/inspector/InspectorController.cpp
+++ b/WebCore/inspector/InspectorController.cpp
@@ -1405,6 +1405,7 @@ ScriptObject InspectorController::createProfileHeader(const JSC::Profile& profil
ScriptObject header = m_frontend->newScriptObject();
header.set("title", profile.title());
header.set("uid", profile.uid());
+ header.set("typeId", UString(CPUProfileType));
return header;
}
diff --git a/WebCore/inspector/front-end/ProfileView.js b/WebCore/inspector/front-end/ProfileView.js
index afced41..2b1d236 100644
--- a/WebCore/inspector/front-end/ProfileView.js
+++ b/WebCore/inspector/front-end/ProfileView.js
@@ -83,7 +83,7 @@ WebInspector.CPUProfileView = function(profile)
var self = this;
function profileCallback(profile)
{
- self.profile.representedObject = profile;
+ self.profile = profile;
self._assignParentsInProfile();
self.profileDataGridTree = self.bottomUpProfileDataGridTree;
@@ -505,7 +505,7 @@ WebInspector.CPUProfileView.prototype = {
_sortData: function(event)
{
- this._sortProfile(this.profile.representedObject);
+ this._sortProfile(this.profile);
},
_sortProfile: function()
@@ -616,26 +616,3 @@ WebInspector.CPUProfileType.prototype = {
}
WebInspector.CPUProfileType.prototype.__proto__ = WebInspector.ProfileType.prototype;
-
-WebInspector.CPUProfile = function(profile)
-{
- this.representedObject = profile;
- this.typeId = WebInspector.CPUProfileType.TypeId;
-}
-
-WebInspector.CPUProfile.prototype = {
- get title()
- {
- return this.representedObject.title;
- },
-
- get uid()
- {
- return this.representedObject.uid;
- },
-
- get head()
- {
- return this.representedObject.head;
- }
-}
diff --git a/WebCore/inspector/front-end/ProfilesPanel.js b/WebCore/inspector/front-end/ProfilesPanel.js
index bac4702..adf0a29 100644
--- a/WebCore/inspector/front-end/ProfilesPanel.js
+++ b/WebCore/inspector/front-end/ProfilesPanel.js
@@ -210,8 +210,9 @@ WebInspector.ProfilesPanel.prototype = {
return escape(text) + '/' + escape(profileTypeId);
},
- addProfileHeader: function(typeId, profile)
+ addProfileHeader: function(profile)
{
+ var typeId = profile.typeId;
var profileType = this.getProfileType(typeId);
var sidebarParent = profileType.treeElement;
var small = false;
@@ -433,11 +434,12 @@ WebInspector.ProfilesPanel.prototype = {
_populateProfiles: function()
{
- // FIXME: This code needs to be adjusted when more profiling types are added.
- // Currently defaults to CPU profiles.
- var cpuProfiles = this.getProfileType(WebInspector.CPUProfileType.TypeId).treeElement;
- if (cpuProfiles.children.length)
- return;
+ var sidebarTreeChildrenCount = this.sidebarTree.children.length;
+ for (var i = 0; i < sidebarTreeChildrenCount; ++i) {
+ var treeElement = this.sidebarTree.children[i];
+ if (treeElement.children.length)
+ return;
+ }
function populateCallback(profileHeaders) {
profileHeaders.sort(function(a, b) { return a.uid - b.uid; });
diff --git a/WebCore/inspector/front-end/inspector.js b/WebCore/inspector/front-end/inspector.js
index e259bcc..b9852fe 100644
--- a/WebCore/inspector/front-end/inspector.js
+++ b/WebCore/inspector/front-end/inspector.js
@@ -1290,7 +1290,7 @@ WebInspector.log = function(message)
WebInspector.addProfileHeader = function(profile)
{
- this.panels.profiles.addProfileHeader(WebInspector.CPUProfileType.TypeId, new WebInspector.CPUProfile(profile));
+ this.panels.profiles.addProfileHeader(profile);
}
WebInspector.setRecordingProfile = function(isProfiling)
--
WebKit Debian packaging
More information about the Pkg-webkit-commits
mailing list