Bug 58528

Summary: Web Inspector: Content area highlight does not cover scrollbars
Product: WebKit Reporter: Alexander Pavlov (apavlov) <apavlov>
Component: Web Inspector (Deprecated)Assignee: Alexander Pavlov (apavlov) <apavlov>
Severity: Normal CC: apavlov, bweinstein, joepeck, keishi, loislo, pfeldman, pmuellr, rik, timothy, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Description Flags
[PATCH] Suggested fix
pfeldman: review-
[PATCH] With ChangeLog
pfeldman: review+
[IMAGE] The fixed highlight screenshot none

Description Alexander Pavlov (apavlov) 2011-04-14 04:48:43 PDT
Scenario 1:
1.  Go to a page with an overflow:auto div. (I did it in w3schools tryit editor).
2.  Open developer tools.
3.  Browse to the DOM element in developer tools, look at the highlight on the web page.
The highlight does not cover the scrollbars (which also count towards the content area).
Upstreaming http://code.google.com/p/chromium/issues/detail?id=51314

Scenario 2:
1. Visit example URL (http://jsfiddle.net/54fAZ/)
2. Inspect the div in the result box, and open the Metrics view.
In the Elements/Metrics pane, the content box of the diagram is marked as 50x50 -- the content box is actually 10x10, the border box is 50x50.
Upstreaming http://code.google.com/p/chromium/issues/detail?id=78660
Comment 1 Alexander Pavlov (apavlov) 2011-04-14 09:34:11 PDT
The Scenario 2 will be fixed as a separate issue.
Comment 2 Alexander Pavlov (apavlov) 2011-04-14 09:35:33 PDT
Created attachment 89594 [details]
[PATCH] Suggested fix
Comment 3 Pavel Feldman 2011-04-14 09:37:09 PDT
Comment on attachment 89594 [details]
[PATCH] Suggested fix

ChangeLog is missing.
Comment 4 Alexander Pavlov (apavlov) 2011-04-14 09:37:44 PDT
Created attachment 89595 [details]
[PATCH] With ChangeLog
Comment 5 Alexander Pavlov (apavlov) 2011-04-14 09:42:19 PDT
Created attachment 89596 [details]
[IMAGE] The fixed highlight screenshot
Comment 6 Alexander Pavlov (apavlov) 2011-04-14 09:48:01 PDT
Committed r83865: <http://trac.webkit.org/changeset/83865>