WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
81821
TileCache needs to know if its containing page is in a window
https://bugs.webkit.org/show_bug.cgi?id=81821
Summary
TileCache needs to know if its containing page is in a window
Anders Carlsson
Reported
2012-03-21 12:09:56 PDT
TileCache needs to know if its containing page is in a window
Attachments
Patch
(7.85 KB, patch)
2012-03-21 12:11 PDT
,
Anders Carlsson
kling
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Anders Carlsson
Comment 1
2012-03-21 12:11:23 PDT
Created
attachment 133090
[details]
Patch
Andreas Kling
Comment 2
2012-03-21 12:19:24 PDT
Comment on
attachment 133090
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=133090&action=review
r=me buuut..
> Source/WebCore/page/FrameView.cpp:855 > + tiledBacking->setIsInWindow(true);
Should be setIsInWindow(false).
> Source/WebCore/page/Page.h:423 > + bool m_isOnscreen;
My gut says s/m_isOnscreen/m_isOnScreen/g. But my gut has been wrong before.
> Source/WebCore/platform/graphics/ca/mac/TileCache.mm:227 > + if (m_isInWindow == isInWindow) > + return;
I assume there's more code coming to this function, otherwise this is pretty overkill.
Anders Carlsson
Comment 3
2012-03-21 12:24:57 PDT
Committed
r111593
: <
http://trac.webkit.org/changeset/111593
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug