Bug 58528 - Web Inspector: Content area highlight does not cover scrollbars
Summary: Web Inspector: Content area highlight does not cover scrollbars
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Alexander Pavlov (apavlov)
Depends on:
Reported: 2011-04-14 04:48 PDT by Alexander Pavlov (apavlov)
Modified: 2011-04-14 09:48 PDT (History)
10 users (show)

See Also:

[PATCH] Suggested fix (1.24 KB, patch)
2011-04-14 09:35 PDT, Alexander Pavlov (apavlov)
pfeldman: review-
Details | Formatted Diff | Diff
[PATCH] With ChangeLog (1.83 KB, patch)
2011-04-14 09:37 PDT, Alexander Pavlov (apavlov)
pfeldman: review+
Details | Formatted Diff | Diff
[IMAGE] The fixed highlight screenshot (3.36 KB, image/png)
2011-04-14 09:42 PDT, Alexander Pavlov (apavlov)
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
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>