Bug 134415

Summary:

ScrollView::rootViewToTotalContents() needs to take topContentInset into account

Product:

WebKit

Reporter:

Beth Dakin <bdakin>

Component:

Layout and Rendering

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

bdakin, bunhere, cdumez, commit-queue, gyuyoung.kim, sam, sergio, simon.fraser, thorton

Priority:

Keywords:

InRadar

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
Patch	simon.fraser: review+

Description Beth Dakin 2014-06-27 15:06:05 PDT

Much like the other coordinate conversation functions, ScrollView::rootViewToTotalContents() needs to take topContentInset into account.

<rdar://problem/17473633>

Comment 1 Beth Dakin 2014-06-27 15:08:20 PDT

Created attachment 234022 [details]
Patch

Comment 2 Simon Fraser (smfr) 2014-06-27 15:10:53 PDT

Comment on attachment 234022 [details]
Patch

Maybe say what the visible impact of this was?

Comment 3 Beth Dakin 2014-06-27 15:13:36 PDT

(In reply to comment #2)
> (From update of attachment 234022 [details])
> Maybe say what the visible impact of this was?

Thanks! I added a line to the changelog about that.

http://trac.webkit.org/changeset/170558