[Webkit-unassigned] [Bug 119435] FrameView should have an isMainFrameView()

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=119435





--- Comment #3 from Tim Horton <thorton at apple.com>  2013-08-02 03:22:18 PST ---
(In reply to comment #2)
> Yay. I wanted to do this for such a long time :) (I was a bit hesitant though, whether WebCore::Frame should have an isMainFrame() function instead (copy what WK2 already has) and use that in FrameView (and in other classes))
> 
> Anyway there are a few more places where you check for isMainFrameView()
> 
> FrameView::setFrameRect()
> FrameView::performPostLayoutTasks()
> FrameView::pagination()
> FrameView::paintOverhangAreas()

There are even more than that! I will post a new patch in a moment.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.