Title: [147735] branches/chromium/1453/Source/WebCore/inspector/front-end/DOMBreakpointsSidebarPane.js
Revision
147735
Author
apav...@chromium.org
Date
2013-04-05 03:14:40 -0700 (Fri, 05 Apr 2013)

Log Message

Merge 147314 "Web Inspector: Fixed DOM Breakpoint pane styles."

> Web Inspector: Fixed DOM Breakpoint pane styles.
> https://bugs.webkit.org/show_bug.cgi?id=113688
> 
> Patch by Vladislav Kaznacheev <kaznach...@chromium.org> on 2013-04-01
> Reviewed by Pavel Feldman.
> 
> The required stylesheet (breakpointsList.css) was not loaded because of
> the peculiar way DOMBreakpointsSidebarPane is included in two panels
> (Sources and Elements) via a proxy pane. Since DOMBreakpointsSidebarPane
> was never shown directly the registerRequiredCSS call in its base class
> NativeBreakpointsSidebarPane constructor had no effect.
> Addin a registerRequiredCSS call to the proxy pane constructor fixes the
> problem.
> 
> * inspector/front-end/DOMBreakpointsSidebarPane.js:
> (WebInspector.DOMBreakpointsSidebarPane.Proxy):

TBR=commit-qu...@webkit.org
Review URL: https://codereview.chromium.org/13598005

Modified Paths

Diff

Modified: branches/chromium/1453/Source/WebCore/inspector/front-end/DOMBreakpointsSidebarPane.js (147734 => 147735)


--- branches/chromium/1453/Source/WebCore/inspector/front-end/DOMBreakpointsSidebarPane.js	2013-04-05 10:06:56 UTC (rev 147734)
+++ branches/chromium/1453/Source/WebCore/inspector/front-end/DOMBreakpointsSidebarPane.js	2013-04-05 10:14:40 UTC (rev 147735)
@@ -357,6 +357,7 @@
     WebInspector.View._assert(!pane.titleElement.firstChild, "Cannot create proxy for a sidebar pane with a toolbar");
 
     WebInspector.SidebarPane.call(this, pane.title());
+    this.registerRequiredCSS("breakpointsList.css");
 
     this._wrappedPane = pane;
     this._panel = panel;
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to