[webkit-reviews] review requested: [Bug 133818] Web Inspector: Selected DOM element highlights invisible near bottom of the viewport (topContentInset?) : [Attachment 234130] [PATCH] Proposed Fix

[bugzilla-daemon at webkit.org]

Joseph Pecoraro <joepeck at webkit.org> has asked	for review:
Bug 133818: Web Inspector: Selected DOM element highlights invisible near
bottom of the viewport (topContentInset?)
https://bugs.webkit.org/show_bug.cgi?id=133818

Attachment 234130: [PATCH] Proposed Fix
https://bugs.webkit.org/attachment.cgi?id=234130&action=review

------- Additional Comments from Joseph Pecoraro <joepeck at webkit.org>
Dealing with ScrollView / FrameView sizes is not my normal area of expertise.
Simon / Benjamin, is there a better way to do this? In fact, the whole
InspectorOverlay page approach could use an audit by someone who knows more
about the page sizing. This fixes the issue in what I considered a safe way if
there is no topContentInset.