Title: [92170] trunk/Source/WebCore
Revision
92170
Author
pfeld...@chromium.org
Date
2011-08-02 02:20:57 -0700 (Tue, 02 Aug 2011)

Log Message

Web Inspector: 404/errors should get a RED visual treatment so easier to identify.
https://bugs.webkit.org/show_bug.cgi?id=65471

Reviewed by Yury Semikhatsky.

* inspector/front-end/NetworkPanel.js:
(WebInspector.NetworkDataGridNode.prototype._refreshStatusCell):
* inspector/front-end/networkLogView.css:
(.network-error-cell):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (92169 => 92170)


--- trunk/Source/WebCore/ChangeLog	2011-08-02 08:46:48 UTC (rev 92169)
+++ trunk/Source/WebCore/ChangeLog	2011-08-02 09:20:57 UTC (rev 92170)
@@ -1,5 +1,17 @@
 2011-08-02  Pavel Feldman  <pfeld...@google.com>
 
+        Web Inspector: 404/errors should get a RED visual treatment so easier to identify.
+        https://bugs.webkit.org/show_bug.cgi?id=65471
+
+        Reviewed by Yury Semikhatsky.
+
+        * inspector/front-end/NetworkPanel.js:
+        (WebInspector.NetworkDataGridNode.prototype._refreshStatusCell):
+        * inspector/front-end/networkLogView.css:
+        (.network-error-cell):
+
+2011-08-02  Pavel Feldman  <pfeld...@google.com>
+
         Web Inspector: arrow is a pain to click in console. Drive-by selection reset fix.
         https://bugs.webkit.org/show_bug.cgi?id=65467
 

Modified: trunk/Source/WebCore/inspector/front-end/NetworkPanel.js (92169 => 92170)


--- trunk/Source/WebCore/inspector/front-end/NetworkPanel.js	2011-08-02 08:46:48 UTC (rev 92169)
+++ trunk/Source/WebCore/inspector/front-end/NetworkPanel.js	2011-08-02 09:20:57 UTC (rev 92170)
@@ -1889,6 +1889,7 @@
             else
                 this._statusCell.setTextAndTitle(WebInspector.UIString("(failed)"));
             this._statusCell.addStyleClass("network-dim-cell");
+            this.element.addStyleClass("network-error-row");
             return;
         }
 
@@ -1896,15 +1897,20 @@
         if (fromCache) {
             this._statusCell.setTextAndTitle(WebInspector.UIString("(from cache)"));
             this._statusCell.addStyleClass("network-dim-cell");
+            this.element.removeStyleClass("network-error-row");
             return;
         }
 
         this._statusCell.removeStyleClass("network-dim-cell");
+        this.element.removeStyleClass("network-error-row");
+
         if (this._resource.statusCode) {
             this._statusCell.appendChild(document.createTextNode(this._resource.statusCode));
             this._statusCell.removeStyleClass("network-dim-cell");
             this._appendSubtitle(this._statusCell, this._resource.statusText);
             this._statusCell.title = this._resource.statusCode + " " + this._resource.statusText;
+            if (this._resource.statusCode >= 400)
+                this.element.addStyleClass("network-error-row");
         } else {
             if (this._resource.isDataURL() && this._resource.finished)
                 this._statusCell.setTextAndTitle(WebInspector.UIString("(data url)"));

Modified: trunk/Source/WebCore/inspector/front-end/networkLogView.css (92169 => 92170)


--- trunk/Source/WebCore/inspector/front-end/networkLogView.css	2011-08-02 08:46:48 UTC (rev 92169)
+++ trunk/Source/WebCore/inspector/front-end/networkLogView.css	2011-08-02 09:20:57 UTC (rev 92170)
@@ -375,6 +375,10 @@
     color: grey;
 }
 
+.network-error-row {
+    color: rgb(230, 0, 0);
+}
+
 /* Dividers */
 
 .network-timeline-grid {
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to