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

Unified Diff: webkit/glue/devtools/js/heap_profiler_panel.js

Issue 173339: DevTools: make "Take snapshot" button in heap profiler visible. (Closed)
Patch Set: Created 11 years, 4 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 | « webkit/glue/devtools/js/devtools.css ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/glue/devtools/js/heap_profiler_panel.js
diff --git a/webkit/glue/devtools/js/heap_profiler_panel.js b/webkit/glue/devtools/js/heap_profiler_panel.js
index 36dd3ab6c3714739dc6965abc9eab82875635dbd..67b09225bf608c64777eb9cb75ef5420c37552cb 100644
--- a/webkit/glue/devtools/js/heap_profiler_panel.js
+++ b/webkit/glue/devtools/js/heap_profiler_panel.js
@@ -31,9 +31,7 @@ WebInspector.HeapProfilerPanel = function() {
this.snapshotViews.id = "heap-snapshot-views";
this.element.appendChild(this.snapshotViews);
- this.snapshotButton = document.createElement("button");
- this.snapshotButton.title = WebInspector.UIString("Take heap snapshot.");
- this.snapshotButton.className = "heap-snapshot-status-bar-item status-bar-item";
+ this.snapshotButton = new WebInspector.StatusBarButton(WebInspector.UIString("Take heap snapshot."), "heap-snapshot-status-bar-item");
this.snapshotButton.addEventListener("click", this._snapshotClicked.bind(this), false);
this.snapshotViewStatusBarItemsContainer = document.createElement("div");
@@ -50,7 +48,7 @@ WebInspector.HeapProfilerPanel.prototype = {
},
get statusBarItems() {
- return [this.snapshotButton, this.snapshotViewStatusBarItemsContainer];
+ return [this.snapshotButton.element, this.snapshotViewStatusBarItemsContainer];
},
show: function() {
« no previous file with comments | « webkit/glue/devtools/js/devtools.css ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698