[SCM] WebKit Debian packaging branch, debian/experimental, updated. upstream/1.3.3-9427-gc2be6fc
aroben at apple.com
aroben at apple.com
Wed Dec 22 13:45:07 UTC 2010
The following commit has been merged in the debian/experimental branch:
commit bd71f431729848456946c493b41875a802d250af
Author: aroben at apple.com <aroben at apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Fri Sep 24 22:11:29 2010 +0000
Add WebPageProxy::creationParameters
Fixes <http://webkit.org/b/46510> WebPageProxy has a bunch of
duplicated WebPageCreationParameters code
Reviewed by Anders Carlsson.
* UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::initializeWebPage):
(WebKit::WebPageProxy::reinitializeWebPage):
(WebKit::WebPageProxy::didReceiveSyncMessage):
Use creationParameters.
(WebKit::WebPageProxy::creationParameters):
* UIProcess/WebPageProxy.h:
Added creationParameters.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@68297 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/WebKit2/ChangeLog b/WebKit2/ChangeLog
index ca4ef3e..12a1313 100644
--- a/WebKit2/ChangeLog
+++ b/WebKit2/ChangeLog
@@ -1,5 +1,24 @@
2010-09-24 Adam Roben <aroben at apple.com>
+ Add WebPageProxy::creationParameters
+
+ Fixes <http://webkit.org/b/46510> WebPageProxy has a bunch of
+ duplicated WebPageCreationParameters code
+
+ Reviewed by Anders Carlsson.
+
+ * UIProcess/WebPageProxy.cpp:
+ (WebKit::WebPageProxy::initializeWebPage):
+ (WebKit::WebPageProxy::reinitializeWebPage):
+ (WebKit::WebPageProxy::didReceiveSyncMessage):
+ Use creationParameters.
+
+ (WebKit::WebPageProxy::creationParameters):
+ * UIProcess/WebPageProxy.h:
+ Added creationParameters.
+
+2010-09-24 Adam Roben <aroben at apple.com>
+
Make WebPage::create take a WebPageCreationParameter struct
It only makes sense, after all.
diff --git a/WebKit2/UIProcess/WebPageProxy.cpp b/WebKit2/UIProcess/WebPageProxy.cpp
index 4757d82..9ba516c 100644
--- a/WebKit2/UIProcess/WebPageProxy.cpp
+++ b/WebKit2/UIProcess/WebPageProxy.cpp
@@ -155,11 +155,7 @@ void WebPageProxy::initializeWebPage(const IntSize& size)
ASSERT(m_drawingArea);
- WebPageCreationParameters parameters;
- parameters.viewSize = size;
- parameters.store = pageNamespace()->context()->preferences()->store();
- parameters.drawingAreaInfo = m_drawingArea->info();
- process()->send(WebProcessMessage::Create, m_pageID, CoreIPC::In(parameters));
+ process()->send(WebProcessMessage::Create, m_pageID, CoreIPC::In(creationParameters(size)));
}
void WebPageProxy::reinitializeWebPage(const WebCore::IntSize& size)
@@ -169,11 +165,7 @@ void WebPageProxy::reinitializeWebPage(const WebCore::IntSize& size)
ASSERT(m_drawingArea);
- WebPageCreationParameters parameters;
- parameters.viewSize = size;
- parameters.store = pageNamespace()->context()->preferences()->store();
- parameters.drawingAreaInfo = m_drawingArea->info();
- process()->send(WebProcessMessage::Create, m_pageID, CoreIPC::In(parameters));
+ process()->send(WebProcessMessage::Create, m_pageID, CoreIPC::In(creationParameters(size)));
}
void WebPageProxy::close()
@@ -893,9 +885,7 @@ void WebPageProxy::didReceiveSyncMessage(CoreIPC::Connection* connection, CoreIP
WebPageCreationParameters parameters;
if (newPage) {
// FIXME: Pass the real size.
- parameters.viewSize = IntSize(100, 100);
- parameters.store = newPage->pageNamespace()->context()->preferences()->store();
- parameters.drawingAreaInfo = newPage->drawingArea()->info();
+ parameters = newPage->creationParameters(IntSize(100, 100));
reply->encode(CoreIPC::In(newPage->pageID(), parameters));
} else {
reply->encode(CoreIPC::In(static_cast<uint64_t>(0), parameters));
@@ -1354,6 +1344,15 @@ void WebPageProxy::processDidRevive()
m_pageClient->processDidRevive();
}
+WebPageCreationParameters WebPageProxy::creationParameters(const IntSize& size) const
+{
+ WebPageCreationParameters parameters;
+ parameters.viewSize = size;
+ parameters.store = pageNamespace()->context()->preferences()->store();
+ parameters.drawingAreaInfo = m_drawingArea->info();
+ return parameters;
+}
+
#if USE(ACCELERATED_COMPOSITING)
void WebPageProxy::didEnterAcceleratedCompositing()
{
diff --git a/WebKit2/UIProcess/WebPageProxy.h b/WebKit2/UIProcess/WebPageProxy.h
index 91635eb..98029aa 100644
--- a/WebKit2/UIProcess/WebPageProxy.h
+++ b/WebKit2/UIProcess/WebPageProxy.h
@@ -76,6 +76,7 @@ class WebProcessProxy;
class WebURLRequest;
class WebWheelEvent;
struct WebNavigationDataStore;
+struct WebPageCreationParameters;
typedef GenericCallback<WKStringRef, StringImpl*> FrameSourceCallback;
typedef GenericCallback<WKStringRef, StringImpl*> RenderTreeExternalRepresentationCallback;
@@ -260,6 +261,8 @@ private:
void didGetRenderTreeExternalRepresentation(const String&, uint64_t);
void didGetSourceForFrame(const String&, uint64_t);
+ WebPageCreationParameters creationParameters(const WebCore::IntSize&) const;
+
#if USE(ACCELERATED_COMPOSITING)
void didChangeAcceleratedCompositing(bool compositing);
#endif
--
WebKit Debian packaging
More information about the Pkg-webkit-commits
mailing list