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

Unified Diff: third_party/WebKit/Source/devtools/front_end/components/ObjectPopoverHelper.js

Issue 2112673003: [DevTools] Move suspended generator location to internal properties (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: addressed comments Created 4 years, 5 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
Index: third_party/WebKit/Source/devtools/front_end/components/ObjectPopoverHelper.js
diff --git a/third_party/WebKit/Source/devtools/front_end/components/ObjectPopoverHelper.js b/third_party/WebKit/Source/devtools/front_end/components/ObjectPopoverHelper.js
index f2baae211d320e8dc4e830892735c13b742e5291..4b9b2fe516537c9d689888884baad2891d1000fa 100644
--- a/third_party/WebKit/Source/devtools/front_end/components/ObjectPopoverHelper.js
+++ b/third_party/WebKit/Source/devtools/front_end/components/ObjectPopoverHelper.js
@@ -112,23 +112,6 @@ WebInspector.ObjectPopoverHelper.prototype = {
}
/**
- * @param {?WebInspector.DebuggerModel.GeneratorObjectDetails} response
- * @this {WebInspector.ObjectPopoverHelper}
- */
- function didGetGeneratorObjectDetails(response)
- {
- if (!response || popover.disposed)
- return;
-
- var rawLocation = response.location;
- var sourceURL = response.sourceURL;
- if (rawLocation && sourceURL) {
- var link = this._lazyLinkifier().linkifyRawLocation(rawLocation, sourceURL, "function-location-link");
- this._titleElement.appendChild(link);
- }
- }
-
- /**
* @param {!WebInspector.RemoteObject} result
* @param {boolean} wasThrown
* @param {!Element=} anchorOverride
@@ -176,20 +159,16 @@ WebInspector.ObjectPopoverHelper.prototype = {
popoverContentElement = createElement("div");
this._titleElement = popoverContentElement.createChild("div", "monospace");
this._titleElement.createChild("span", "source-frame-popover-title").textContent = description;
- var section = new WebInspector.ObjectPropertiesSection(result, "");
+ var section = new WebInspector.ObjectPropertiesSection(result, "", this._lazyLinkifier());
section.element.classList.add("source-frame-popover-tree");
section.titleLessMode();
popoverContentElement.appendChild(section.element);
-
- if (result.subtype === "generator")
- result.generatorObjectDetails(didGetGeneratorObjectDetails.bind(this));
}
var popoverWidth = 300;
var popoverHeight = 250;
popover.showForAnchor(popoverContentElement, anchorElement, popoverWidth, popoverHeight);
}
}
-
this._queryObject(element, didQueryObject.bind(this), this._popoverObjectGroup);
},

Powered by Google App Engine
This is Rietveld 408576698