Bug 118115

Summary: Avoid duplicating hostWindow() call in FrameView::setFrameRect()
Product: WebKit Reporter: Seokju Kwon <seokju>
Component: WebCore Misc.Assignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: cdumez, commit-queue
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

Description Seokju Kwon 2013-06-26 22:09:52 PDT 
Patch follows.
Comment 1 Seokju Kwon 2013-06-26 22:11:40 PDT 
Created attachment 205561 [details]
Patch
Comment 2 Chris Dumez 2013-06-27 04:21:35 PDT 
Comment on attachment 205561 [details]
Patch

FrameView::hostWindow() is not trivial/inlined so ok. r=me.
Comment 3 Chris Dumez 2013-06-27 04:22:04 PDT 
Comment on attachment 205561 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=205561&action=review

> Source/WebCore/ChangeLog:7
> +

Please add a Changelog.
Comment 4 Seokju Kwon 2013-06-27 06:50:55 PDT 
Created attachment 205598 [details]
Patch
Comment 5 Seokju Kwon 2013-06-27 06:51:30 PDT 
Comment on attachment 205598 [details]
Patch

Fixed it. Thanks!
Comment 6 WebKit Commit Bot 2013-06-27 08:12:09 PDT 
Comment on attachment 205598 [details]
Patch

Clearing flags on attachment: 205598

Committed r152098: <http://trac.webkit.org/changeset/152098>
Comment 7 WebKit Commit Bot 2013-06-27 08:12:11 PDT 
All reviewed patches have been landed.  Closing bug.