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

Unified Diff: Source/WebCore/inspector/InspectorOverlayPage.html

Issue 11434008: Merge 135720 - Web Inspector: object preview does not render node id, className; logs too many func… (Closed) Base URL: http://svn.webkit.org/repository/webkit/branches/chromium/1312/
Patch Set: Created 8 years, 1 month 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: Source/WebCore/inspector/InspectorOverlayPage.html
===================================================================
--- Source/WebCore/inspector/InspectorOverlayPage.html (revision 136041)
+++ Source/WebCore/inspector/InspectorOverlayPage.html (working copy)
@@ -83,13 +83,20 @@
}
#tag-name {
+ /* Keep this in sync with view-source.css (.webkit-html-tag-name) */
color: rgb(136, 18, 128);
}
#node-id {
+ /* Keep this in sync with view-source.css (.webkit-html-attribute-value) */
color: rgb(26, 26, 166);
}
+#class-name {
+ /* Keep this in sync with view-source.css (.webkit-html-attribute-name) */
+ color: rgb(153, 69, 0);
+}
+
#right-gutter {
display: none;
right: 0;
@@ -335,8 +342,11 @@
return;
document.getElementById("tag-name").textContent = elementInfo.tagName;
- document.getElementById("node-id").textContent = elementInfo.nodeId ? "#" + elementInfo.nodeId : "";
- document.getElementById("class-name").textContent = elementInfo.className || "";
+ document.getElementById("node-id").textContent = elementInfo.idValue ? "#" + elementInfo.idValue : "";
+ var className = elementInfo.className;
+ if (className.length > 50)
+ className = className.substring(0, 50) + "\u2026";
+ document.getElementById("class-name").textContent = className;
document.getElementById("node-width").textContent = elementInfo.nodeWidth;
document.getElementById("node-height").textContent = elementInfo.nodeHeight;
var elementTitle = document.getElementById("element-title");
« no previous file with comments | « Source/WebCore/inspector/InjectedScriptSource.js ('k') | Source/WebCore/inspector/front-end/ConsoleMessage.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698