Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(66)

Unified Diff: Source/devtools/front_end/HeapSnapshotProxy.js

Issue 212773005: DevTools: Show all objects in class view in advanced heap snapshot mode. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebaseline Created 6 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/devtools/front_end/HeapSnapshotLoader.js ('k') | Source/devtools/front_end/HeapSnapshotView.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/devtools/front_end/HeapSnapshotProxy.js
diff --git a/Source/devtools/front_end/HeapSnapshotProxy.js b/Source/devtools/front_end/HeapSnapshotProxy.js
index 239653a77be77ed1821c25d8002150cd47bc4d54..20e73604874dbac86b533538638a762d56e1ec44 100644
--- a/Source/devtools/front_end/HeapSnapshotProxy.js
+++ b/Source/devtools/front_end/HeapSnapshotProxy.js
@@ -216,7 +216,7 @@ WebInspector.HeapSnapshotProxyObject.prototype = {
* @param {string} methodName
* @param {function (new:T, ...[?])} proxyConstructor
* @param {...*} var_args
- * @return {?WebInspector.HeapSnapshotProviderProxy}
+ * @return {!T}
* @template T
*/
callFactoryMethod: function(callback, methodName, proxyConstructor, var_args)
@@ -274,7 +274,8 @@ WebInspector.HeapSnapshotLoaderProxy.prototype = {
{
if (callback)
callback();
- this.callFactoryMethod(updateStaticData.bind(this), "buildSnapshot", WebInspector.HeapSnapshotProxy);
+ var showHiddenData = WebInspector.settings.showAdvancedHeapSnapshotProperties.get();
+ this.callFactoryMethod(updateStaticData.bind(this), "buildSnapshot", WebInspector.HeapSnapshotProxy, showHiddenData);
}
/**
@@ -340,22 +341,20 @@ WebInspector.HeapSnapshotProxy.prototype = {
/**
* @param {number} nodeIndex
- * @param {boolean} showHiddenData
* @return {!WebInspector.HeapSnapshotProviderProxy}
*/
- createEdgesProvider: function(nodeIndex, showHiddenData)
+ createEdgesProvider: function(nodeIndex)
{
- return /** @type {!WebInspector.HeapSnapshotProviderProxy} */ (this.callFactoryMethod(null, "createEdgesProvider", WebInspector.HeapSnapshotProviderProxy, nodeIndex, showHiddenData));
+ return this.callFactoryMethod(null, "createEdgesProvider", WebInspector.HeapSnapshotProviderProxy, nodeIndex);
},
/**
* @param {number} nodeIndex
- * @param {boolean} showHiddenData
* @return {!WebInspector.HeapSnapshotProviderProxy}
*/
- createRetainingEdgesProvider: function(nodeIndex, showHiddenData)
+ createRetainingEdgesProvider: function(nodeIndex)
{
- return /** @type {!WebInspector.HeapSnapshotProviderProxy} */ (this.callFactoryMethod(null, "createRetainingEdgesProvider", WebInspector.HeapSnapshotProviderProxy, nodeIndex, showHiddenData));
+ return this.callFactoryMethod(null, "createRetainingEdgesProvider", WebInspector.HeapSnapshotProviderProxy, nodeIndex);
},
/**
« no previous file with comments | « Source/devtools/front_end/HeapSnapshotLoader.js ('k') | Source/devtools/front_end/HeapSnapshotView.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698