Changeset 83865 in webkit

Timestamp:

Apr 14, 2011 9:47:34 AM (13 years ago)

Author:

apavlov@chromium.org

Message:

2011-04-14 Alexander Pavlov <​apavlov@chromium.org>

Reviewed by Pavel Feldman.

Web Inspector: Content area highlight does not cover scrollbars
​https://bugs.webkit.org/show_bug.cgi?id=58528

• inspector/DOMNodeHighlighter.cpp: (WebCore::DOMNodeHighlighter::DrawNodeHighlight):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• inspector/DOMNodeHighlighter.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2011-04-14  Alexander Pavlov  <apavlov@chromium.org>
+
+        Reviewed by Pavel Feldman.
+
+        Web Inspector: Content area highlight does not cover scrollbars
+        https://bugs.webkit.org/show_bug.cgi?id=58528
+
+        * inspector/DOMNodeHighlighter.cpp:
+        (WebCore::DOMNodeHighlighter::DrawNodeHighlight):
+
 2011-04-14  Pavel Podivilov  <podivilov@chromium.org>
 

trunk/Source/WebCore/inspector/DOMNodeHighlighter.cpp

@@ -244 +244 @@
         RenderBox* renderBox = toRenderBox(renderer);
 
+        // RenderBox returns the "pure" content area box, exclusive of the scrollbars (if present), which also count towards the content area in CSS.
         IntRect contentBox = renderBox->contentBoxRect();
+        contentBox.setWidth(contentBox.width() + renderBox->verticalScrollbarWidth());
+        contentBox.setHeight(contentBox.height() + renderBox->horizontalScrollbarHeight());
 
         IntRect paddingBox(contentBox.x() - renderBox->paddingLeft(), contentBox.y() - renderBox->paddingTop(),