diff options
author | Simon Hausmann <simon.hausmann@nokia.com> | 2009-06-24 11:42:15 (GMT) |
---|---|---|
committer | Simon Hausmann <simon.hausmann@nokia.com> | 2009-06-24 11:42:15 (GMT) |
commit | 259c32cd921fcbb85f79f21923d0efa0d6743d8a (patch) | |
tree | f9629c9be4b8488eb3221b48c0629a42a6ffce4c /src/3rdparty/webkit/WebKit/qt | |
parent | ff2b98cf81daf585bb657bee7f5f131769b38eca (diff) | |
download | Qt-259c32cd921fcbb85f79f21923d0efa0d6743d8a.zip Qt-259c32cd921fcbb85f79f21923d0efa0d6743d8a.tar.gz Qt-259c32cd921fcbb85f79f21923d0efa0d6743d8a.tar.bz2 |
Updated WebKit from /home/shausman/src/webkit/trunk to qtwebkit-4.6-snapshot-24062009 ( 6d5a2a0472a6af0b7f781da018e76bb8522d57a5 )
++ b/WebKit/qt/ChangeLog
2009-06-19 Daniel <qt-info@nokia.com>
Reviewed by Simon Hausmann.
Remove warnings for QString() constructions from const char *
By explicitly wrapping it with QLatin1String() / QLatin1Char()
* Api/qwebelement.cpp:
(QWebElement::classes): Use QLatin1String.
(QWebElement::addClass): Ditto.
(QWebElement::removeClass): Ditto.
(QWebElement::toggleClass): Ditto.
2009-06-18 Friedemann Kleint <Friedemann.Kleint@nokia.com>
Reviewed by Simon Hausmann.
Fixed MinGW compilation.
* Api/qwebelement.cpp:
(QWebElement::evaluateScript):
2009-06-18 Markus Goetz <Markus.Goetz@nokia.com>
Reviewed by Simon Hausman.
Clarify in docs how to compile with debug information.
* docs/qtwebkit.qdoc:
2009-06-17 Markus Goetz <Markus.Goetz@nokia.com>
Reviewed by Simon Hausmann.
QWebPage: Don't call supportsSsl()
This stops QWebPage from loading the OpenSSL libs,
certificates etc. when they are not needed for the non-HTTPS case.
Loading the SSL libraries can be a very slow operation.
* Api/qwebpage.cpp:
(QWebPage::userAgentForUrl):
2009-06-15 Benjamin C Meyer <benjamin.meyer@torchmobile.com>
Reviewed by Adam Treat.
Support the back/forward/stop/refresh multimedia keys and accept the
event when handling backspace and shift backspace as we should.
* Api/qwebpage.cpp:
(QWebPagePrivate::keyPressEvent):
2009-06-15 Andre Pedralho <andre.pedralho@openbossa.org>
Reviewed by Adam Treat.
https://bugs.webkit.org/show_bug.cgi?id=26351
Remove bool QWebHitTestResult::isScrollBar() const and make sure a null QWebHitTestResult is returned instead.
* Api/qwebframe.cpp:
(QWebFrame::hitTestContent):
* Api/qwebframe.h:
* Api/qwebpage.cpp:
(QWebPage::updatePositionDependentActions):
2009-06-15 Simon Hausmann <simon.hausmann@nokia.com>
Reviewed by Adam Treat.
Fix the logic for disabling the fixed layout feature, when an invalid
QSize is set.
* Api/qwebpage.cpp:
(QWebPage::setFixedContentsSize):
2009-06-13 Adam Barth <abarth@webkit.org>
Reviewed by Darin Fisher.
https://bugs.webkit.org/show_bug.cgi?id=24492
Move registerURLSchemeAsLocal from FrameLoader to SecurityOrigin.
* Api/qwebpage.cpp:
(QWebPage::acceptNavigationRequest):
Rubber-stamped by Simon Hausmann.
2009-06-09 Simon Hausmann <simon.hausmann@nokia.com>
Reviewed by Ariya Hidayat.
Renamed QWebSettings::AllowUniversalAccessFromFileUrls to
LocalContentCanAccessRemoteUrls, as discussed in the API review.
* Api/qwebsettings.cpp:
(QWebSettingsPrivate::apply):
(QWebSettings::QWebSettings):
* Api/qwebsettings.h:
2009-06-09 Simon Hausmann <simon.hausmann@nokia.com>
Reviewed by Ariya Hidayat.
Merged useFixedLayout property with fixedLayoutSize and
renamed the latter to fixedContentsSize.
* Api/qwebpage.cpp:
(QWebPage::fixedContentsSize):
(QWebPage::setFixedContentsSize):
* Api/qwebpage.h:
* WebCoreSupport/FrameLoaderClientQt.cpp:
(WebCore::FrameLoaderClientQt::transitionToCommittedForNewPage):
2009-06-09 Simon Hausmann <simon.hausmann@nokia.com>
Reviewed by Ariya Hidayat.
Renamed QWebHitTestResult::linkTarget to linkElement() and made it return
a QWebElement. The link target itself is always the target DOM attribute.
* Api/qwebframe.cpp:
(QWebHitTestResultPrivate::QWebHitTestResultPrivate):
(QWebHitTestResult::linkElement):
* Api/qwebframe.h:
* Api/qwebframe_p.h:
* tests/qwebframe/tst_qwebframe.cpp:
Diffstat (limited to 'src/3rdparty/webkit/WebKit/qt')
-rw-r--r-- | src/3rdparty/webkit/WebKit/qt/Api/qwebelement.cpp | 16 | ||||
-rw-r--r-- | src/3rdparty/webkit/WebKit/qt/Api/qwebframe.cpp | 28 | ||||
-rw-r--r-- | src/3rdparty/webkit/WebKit/qt/Api/qwebframe.h | 4 | ||||
-rw-r--r-- | src/3rdparty/webkit/WebKit/qt/Api/qwebframe_p.h | 2 | ||||
-rw-r--r-- | src/3rdparty/webkit/WebKit/qt/Api/qwebpage.cpp | 67 | ||||
-rw-r--r-- | src/3rdparty/webkit/WebKit/qt/Api/qwebpage.h | 10 | ||||
-rw-r--r-- | src/3rdparty/webkit/WebKit/qt/Api/qwebsettings.cpp | 9 | ||||
-rw-r--r-- | src/3rdparty/webkit/WebKit/qt/Api/qwebsettings.h | 2 | ||||
-rw-r--r-- | src/3rdparty/webkit/WebKit/qt/ChangeLog | 131 | ||||
-rw-r--r-- | src/3rdparty/webkit/WebKit/qt/WebCoreSupport/FrameLoaderClientQt.cpp | 8 | ||||
-rw-r--r-- | src/3rdparty/webkit/WebKit/qt/docs/qtwebkit.qdoc | 5 | ||||
-rw-r--r-- | src/3rdparty/webkit/WebKit/qt/tests/qwebframe/tst_qwebframe.cpp | 3 |
12 files changed, 207 insertions, 78 deletions
diff --git a/src/3rdparty/webkit/WebKit/qt/Api/qwebelement.cpp b/src/3rdparty/webkit/WebKit/qt/Api/qwebelement.cpp index adc55ae..413a662 100644 --- a/src/3rdparty/webkit/WebKit/qt/Api/qwebelement.cpp +++ b/src/3rdparty/webkit/WebKit/qt/Api/qwebelement.cpp @@ -1046,10 +1046,10 @@ QString QWebElement::computedStyleProperty(const QString &name) const */ QStringList QWebElement::classes() const { - if (!hasAttribute("class")) + if (!hasAttribute(QLatin1String("class"))) return QStringList(); - QStringList classes = attribute("class").simplified().split(' ', QString::SkipEmptyParts); + QStringList classes = attribute(QLatin1String("class")).simplified().split(QLatin1Char(' '), QString::SkipEmptyParts); #if QT_VERSION >= 0x040500 classes.removeDuplicates(); #else @@ -1089,8 +1089,8 @@ void QWebElement::addClass(const QString &name) QStringList list = classes(); if (!list.contains(name)) { list.append(name); - QString value = list.join(" "); - setAttribute("class", value); + QString value = list.join(QLatin1String(" ")); + setAttribute(QLatin1String("class"), value); } } @@ -1102,8 +1102,8 @@ void QWebElement::removeClass(const QString &name) QStringList list = classes(); if (list.contains(name)) { list.removeAll(name); - QString value = list.join(" "); - setAttribute("class", value); + QString value = list.join(QLatin1String(" ")); + setAttribute(QLatin1String("class"), value); } } @@ -1119,8 +1119,8 @@ void QWebElement::toggleClass(const QString &name) else list.append(name); - QString value = list.join(" "); - setAttribute("class", value); + QString value = list.join(QLatin1String(" ")); + setAttribute(QLatin1String("class"), value); } /*! diff --git a/src/3rdparty/webkit/WebKit/qt/Api/qwebframe.cpp b/src/3rdparty/webkit/WebKit/qt/Api/qwebframe.cpp index cedd8ae..381251d 100644 --- a/src/3rdparty/webkit/WebKit/qt/Api/qwebframe.cpp +++ b/src/3rdparty/webkit/WebKit/qt/Api/qwebframe.cpp @@ -1017,6 +1017,10 @@ QWebHitTestResult QWebFrame::hitTestContent(const QPoint &pos) const return QWebHitTestResult(); HitTestResult result = d->frame->eventHandler()->hitTestResultAtPoint(d->frame->view()->windowToContents(pos), /*allowShadowContent*/ false, /*ignoreClipping*/ true); + + if (result.scrollbar()) + return QWebHitTestResult(); + return QWebHitTestResult(new QWebHitTestResultPrivate(result)); } @@ -1268,9 +1272,7 @@ QWebHitTestResultPrivate::QWebHitTestResultPrivate(const WebCore::HitTestResult WebCore::Frame *wframe = hitTest.targetFrame(); if (wframe) linkTargetFrame = QWebFramePrivate::kit(wframe); - Element* urlElement = hitTest.URLElement(); - if (urlElement) - linkTarget = urlElement->target(); + linkElement = QWebElement(hitTest.URLElement()); isContentEditable = hitTest.isContentEditable(); isContentSelected = hitTest.isSelected(); @@ -1413,21 +1415,21 @@ QUrl QWebHitTestResult::linkTitle() const /*! \since 4.6 - Returns the name of the target frame that will load the link if it is activated. + Returns the element that represents the link. \sa linkTargetFrame() */ -QString QWebHitTestResult::linkTarget() const +QWebElement QWebHitTestResult::linkElement() const { if (!d) - return 0; - return d->linkTarget; + return QWebElement(); + return d->linkElement; } /*! Returns the frame that will load the link if it is activated. - \sa linkTarget() + \sa linkElement() */ QWebFrame *QWebHitTestResult::linkTargetFrame() const { @@ -1509,13 +1511,3 @@ QWebFrame *QWebHitTestResult::frame() const return d->frame; } -/*! - \since 4.6 - Returns true if the test includes a scrollbar. -*/ -bool QWebHitTestResult::isScrollBar() const -{ - if (!d) - return false; - return d->isScrollBar; -} diff --git a/src/3rdparty/webkit/WebKit/qt/Api/qwebframe.h b/src/3rdparty/webkit/WebKit/qt/Api/qwebframe.h index ebc22fe..3ed453e 100644 --- a/src/3rdparty/webkit/WebKit/qt/Api/qwebframe.h +++ b/src/3rdparty/webkit/WebKit/qt/Api/qwebframe.h @@ -79,7 +79,7 @@ public: QUrl linkUrl() const; QUrl linkTitle() const; QWebFrame *linkTargetFrame() const; - QString linkTarget() const; + QWebElement linkElement() const; QString alternateText() const; // for img, area, input and applet @@ -93,8 +93,6 @@ public: QWebFrame *frame() const; - bool isScrollBar() const; - private: QWebHitTestResult(QWebHitTestResultPrivate *priv); QWebHitTestResultPrivate *d; diff --git a/src/3rdparty/webkit/WebKit/qt/Api/qwebframe_p.h b/src/3rdparty/webkit/WebKit/qt/Api/qwebframe_p.h index 325e6e0..0c7da90 100644 --- a/src/3rdparty/webkit/WebKit/qt/Api/qwebframe_p.h +++ b/src/3rdparty/webkit/WebKit/qt/Api/qwebframe_p.h @@ -107,7 +107,7 @@ public: QUrl linkUrl; QString linkTitle; QPointer<QWebFrame> linkTargetFrame; - QString linkTarget; + QWebElement linkElement; QString alternateText; QUrl imageUrl; QPixmap pixmap; diff --git a/src/3rdparty/webkit/WebKit/qt/Api/qwebpage.cpp b/src/3rdparty/webkit/WebKit/qt/Api/qwebpage.cpp index 2decbca..3370f15 100644 --- a/src/3rdparty/webkit/WebKit/qt/Api/qwebpage.cpp +++ b/src/3rdparty/webkit/WebKit/qt/Api/qwebpage.cpp @@ -43,6 +43,7 @@ #include "DragController.h" #include "DragData.h" #include "EditorClientQt.h" +#include "SecurityOrigin.h" #include "Settings.h" #include "Page.h" #include "Pasteboard.h" @@ -797,11 +798,24 @@ void QWebPagePrivate::keyPressEvent(QKeyEvent *ev) int fontHeight = fm.height(); if (!handleScrolling(ev)) { switch (ev->key()) { + case Qt::Key_Back: + q->triggerAction(QWebPage::Back); + break; + case Qt::Key_Forward: + q->triggerAction(QWebPage::Forward); + break; + case Qt::Key_Stop: + q->triggerAction(QWebPage::Stop); + break; + case Qt::Key_Refresh: + q->triggerAction(QWebPage::Reload); + break; case Qt::Key_Backspace: if (ev->modifiers() == Qt::ShiftModifier) q->triggerAction(QWebPage::Forward); else q->triggerAction(QWebPage::Back); + break; default: handled = false; break; @@ -1598,16 +1612,21 @@ void QWebPage::setViewportSize(const QSize &size) const } } -QSize QWebPage::fixedLayoutSize() const +QSize QWebPage::fixedContentsSize() const { - if (d->mainFrame && d->mainFrame->d->frame->view()) - return d->mainFrame->d->frame->view()->fixedLayoutSize(); + QWebFrame* frame = d->mainFrame; + if (frame) { + WebCore::FrameView* view = frame->d->frame->view(); + if (view && view->useFixedLayout()) { + return d->mainFrame->d->frame->view()->fixedLayoutSize(); + } + } return d->fixedLayoutSize; } /*! - \property QWebPage::fixedLayoutSize + \property QWebPage::fixedContentsSize \since 4.6 \brief the size of the fixed layout @@ -1615,37 +1634,22 @@ QSize QWebPage::fixedLayoutSize() const 1024x768 for example then webkit will layout the page as if the viewport were that size rather than something different. */ -void QWebPage::setFixedLayoutSize(const QSize &size) const +void QWebPage::setFixedContentsSize(const QSize &size) const { d->fixedLayoutSize = size; QWebFrame *frame = mainFrame(); if (frame->d->frame && frame->d->frame->view()) { WebCore::FrameView* view = frame->d->frame->view(); - view->setFixedLayoutSize(size); - view->forceLayout(); - } -} -bool QWebPage::useFixedLayout() const -{ - return d->useFixedLayout; -} - -/*! - \property QWebPage::useFixedLayout - \since 4.6 - \brief whether to use a fixed layout size -*/ -void QWebPage::setUseFixedLayout(bool useFixedLayout) -{ - d->useFixedLayout = useFixedLayout; - - QWebFrame *frame = mainFrame(); - if (frame->d->frame && frame->d->frame->view()) { - WebCore::FrameView* view = frame->d->frame->view(); - view->setUseFixedLayout(useFixedLayout); - view->forceLayout(); + if (size.isValid()) { + view->setUseFixedLayout(true); + view->setFixedLayoutSize(size); + view->forceLayout(); + } else if (view->useFixedLayout()) { + view->setUseFixedLayout(false); + view->forceLayout(); + } } } @@ -1675,7 +1679,7 @@ bool QWebPage::acceptNavigationRequest(QWebFrame *frame, const QWebNetworkReques return true; case DelegateExternalLinks: - if (WebCore::FrameLoader::shouldTreatURLSchemeAsLocal(request.url().scheme())) + if (WebCore::SecurityOrigin::shouldTreatURLSchemeAsLocal(request.url().scheme())) return true; emit linkClicked(request.url()); return false; @@ -2228,7 +2232,10 @@ void QWebPage::updatePositionDependentActions(const QPoint &pos) WebCore::Frame* focusedFrame = d->page->focusController()->focusedOrMainFrame(); HitTestResult result = focusedFrame->eventHandler()->hitTestResultAtPoint(focusedFrame->view()->windowToContents(pos), /*allowShadowContent*/ false); - d->hitTestResult = QWebHitTestResult(new QWebHitTestResultPrivate(result)); + if (result.scrollbar()) + d->hitTestResult = QWebHitTestResult(); + else + d->hitTestResult = QWebHitTestResult(new QWebHitTestResultPrivate(result)); WebCore::ContextMenu menu(result); menu.populate(); if (d->page->inspectorController()->enabled()) diff --git a/src/3rdparty/webkit/WebKit/qt/Api/qwebpage.h b/src/3rdparty/webkit/WebKit/qt/Api/qwebpage.h index 2152865..7edc060 100644 --- a/src/3rdparty/webkit/WebKit/qt/Api/qwebpage.h +++ b/src/3rdparty/webkit/WebKit/qt/Api/qwebpage.h @@ -66,8 +66,7 @@ class QWEBKIT_EXPORT QWebPage : public QObject Q_PROPERTY(bool modified READ isModified) Q_PROPERTY(QString selectedText READ selectedText) Q_PROPERTY(QSize viewportSize READ viewportSize WRITE setViewportSize) - Q_PROPERTY(QSize fixedLayoutSize READ fixedLayoutSize WRITE setFixedLayoutSize) - Q_PROPERTY(bool useFixedLayout READ useFixedLayout WRITE setUseFixedLayout) + Q_PROPERTY(QSize fixedContentsSize READ fixedContentsSize WRITE setFixedContentsSize) Q_PROPERTY(bool forwardUnsupportedContent READ forwardUnsupportedContent WRITE setForwardUnsupportedContent) Q_PROPERTY(LinkDelegationPolicy linkDelegationPolicy READ linkDelegationPolicy WRITE setLinkDelegationPolicy) Q_PROPERTY(QPalette palette READ palette WRITE setPalette) @@ -234,11 +233,8 @@ public: QSize viewportSize() const; void setViewportSize(const QSize &size) const; - QSize fixedLayoutSize() const; - void setFixedLayoutSize(const QSize &size) const; - - bool useFixedLayout() const; - void setUseFixedLayout(bool useFixedLayout); + QSize fixedContentsSize() const; + void setFixedContentsSize(const QSize &size) const; virtual bool event(QEvent*); bool focusNextPrevChild(bool next); diff --git a/src/3rdparty/webkit/WebKit/qt/Api/qwebsettings.cpp b/src/3rdparty/webkit/WebKit/qt/Api/qwebsettings.cpp index cc49499..02ef4a0 100644 --- a/src/3rdparty/webkit/WebKit/qt/Api/qwebsettings.cpp +++ b/src/3rdparty/webkit/WebKit/qt/Api/qwebsettings.cpp @@ -190,8 +190,8 @@ void QWebSettingsPrivate::apply() global->attributes.value(QWebSettings::LocalStorageDatabaseEnabled)); settings->setLocalStorageEnabled(value); - value = attributes.value(QWebSettings::AllowUniversalAccessFromFileUrls, - global->attributes.value(QWebSettings::AllowUniversalAccessFromFileUrls)); + value = attributes.value(QWebSettings::LocalContentCanAccessRemoteUrls, + global->attributes.value(QWebSettings::LocalContentCanAccessRemoteUrls)); settings->setAllowUniversalAccessFromFileURLs(value); } else { QList<QWebSettingsPrivate *> settings = *::allSettings(); @@ -328,8 +328,7 @@ QWebSettings *QWebSettings::globalSettings() web application cache feature is enabled or not. \value LocalStorageDatabaseEnabled Specifies whether support for the HTML 5 local storage feature is enabled or not. - \value AllowUniversalAccessFromFileUrls Specifies whether documents from file - Urls should be granted universal access (e.g., to HTTP and HTTPS documents). + \value LocalContentCanAccessRemoteUrls Specifies whether locally loaded documents are allowed to access remote urls. */ /*! @@ -358,7 +357,7 @@ QWebSettings::QWebSettings() d->attributes.insert(QWebSettings::OfflineStorageDatabaseEnabled, true); d->attributes.insert(QWebSettings::OfflineWebApplicationCacheEnabled, true); d->attributes.insert(QWebSettings::LocalStorageDatabaseEnabled, true); - d->attributes.insert(QWebSettings::AllowUniversalAccessFromFileUrls, true); + d->attributes.insert(QWebSettings::LocalContentCanAccessRemoteUrls, true); d->offlineStorageDefaultQuota = 5 * 1024 * 1024; } diff --git a/src/3rdparty/webkit/WebKit/qt/Api/qwebsettings.h b/src/3rdparty/webkit/WebKit/qt/Api/qwebsettings.h index da0c9f2..7cbca25 100644 --- a/src/3rdparty/webkit/WebKit/qt/Api/qwebsettings.h +++ b/src/3rdparty/webkit/WebKit/qt/Api/qwebsettings.h @@ -64,7 +64,7 @@ public: OfflineStorageDatabaseEnabled, OfflineWebApplicationCacheEnabled, LocalStorageDatabaseEnabled, - AllowUniversalAccessFromFileUrls + LocalContentCanAccessRemoteUrls }; enum WebGraphic { MissingImageGraphic, diff --git a/src/3rdparty/webkit/WebKit/qt/ChangeLog b/src/3rdparty/webkit/WebKit/qt/ChangeLog index 8d50ddd..c2ed475 100644 --- a/src/3rdparty/webkit/WebKit/qt/ChangeLog +++ b/src/3rdparty/webkit/WebKit/qt/ChangeLog @@ -1,3 +1,48 @@ +2009-06-19 Daniel Teske <qt-info@nokia.com> + + Reviewed by Simon Hausmann. + + Remove warnings for QString() constructions from const char * + + By explicitly wrapping it with QLatin1String() / QLatin1Char() + + * Api/qwebelement.cpp: + (QWebElement::classes): Use QLatin1String. + (QWebElement::addClass): Ditto. + (QWebElement::removeClass): Ditto. + (QWebElement::toggleClass): Ditto. + +2009-06-18 Friedemann Kleint <Friedemann.Kleint@nokia.com> + + Reviewed by Simon Hausmann. + + Fixed MinGW compilation. + + * Api/qwebelement.cpp: + (QWebElement::evaluateScript): + +2009-06-18 Markus Goetz <Markus.Goetz@nokia.com> + + Reviewed by Simon Hausman. + + Clarify in docs how to compile with debug information. + + * docs/qtwebkit.qdoc: + +2009-06-17 Markus Goetz <Markus.Goetz@nokia.com> + + Reviewed by Simon Hausmann. + + QWebPage: Don't call supportsSsl() + + This stops QWebPage from loading the OpenSSL libs, + certificates etc. when they are not needed for the non-HTTPS case. + + Loading the SSL libraries can be a very slow operation. + + * Api/qwebpage.cpp: + (QWebPage::userAgentForUrl): + 2009-06-16 David Boddie <dboddie@trolltech.com> Reviewed by Simon Hausmann. @@ -19,6 +64,50 @@ * Api/qwebframe.cpp: +2009-06-15 Benjamin C Meyer <benjamin.meyer@torchmobile.com> + + Reviewed by Adam Treat. + + Support the back/forward/stop/refresh multimedia keys and accept the + event when handling backspace and shift backspace as we should. + + * Api/qwebpage.cpp: + (QWebPagePrivate::keyPressEvent): + +2009-06-15 Andre Pedralho <andre.pedralho@openbossa.org> + + Reviewed by Adam Treat. + + https://bugs.webkit.org/show_bug.cgi?id=26351 + Remove bool QWebHitTestResult::isScrollBar() const and make sure a null QWebHitTestResult is returned instead. + + * Api/qwebframe.cpp: + (QWebFrame::hitTestContent): + * Api/qwebframe.h: + * Api/qwebpage.cpp: + (QWebPage::updatePositionDependentActions): + +2009-06-15 Simon Hausmann <simon.hausmann@nokia.com> + + Reviewed by Adam Treat. + + Fix the logic for disabling the fixed layout feature, when an invalid + QSize is set. + + * Api/qwebpage.cpp: + (QWebPage::setFixedContentsSize): + +2009-06-13 Adam Barth <abarth@webkit.org> + + Reviewed by Darin Fisher. + + https://bugs.webkit.org/show_bug.cgi?id=24492 + + Move registerURLSchemeAsLocal from FrameLoader to SecurityOrigin. + + * Api/qwebpage.cpp: + (QWebPage::acceptNavigationRequest): + 2009-06-10 Antonio Gomes <antonio.gomes@openbossa.org> Reviewed by Ariya Hidayat. @@ -35,12 +124,52 @@ 2009-06-09 Ariya Hidayat <ariya.hidayat@nokia.com> - Rubber-stamped by Tor Arne Vestbø. + Rubber-stamped by Simon Hausmann. Fix qdoc warning, function parameter string must be referred. * Api/qwebview.cpp: +2009-06-09 Simon Hausmann <simon.hausmann@nokia.com> + + Reviewed by Ariya Hidayat. + + Renamed QWebSettings::AllowUniversalAccessFromFileUrls to + LocalContentCanAccessRemoteUrls, as discussed in the API review. + + * Api/qwebsettings.cpp: + (QWebSettingsPrivate::apply): + (QWebSettings::QWebSettings): + * Api/qwebsettings.h: + +2009-06-09 Simon Hausmann <simon.hausmann@nokia.com> + + Reviewed by Ariya Hidayat. + + Merged useFixedLayout property with fixedLayoutSize and + renamed the latter to fixedContentsSize. + + * Api/qwebpage.cpp: + (QWebPage::fixedContentsSize): + (QWebPage::setFixedContentsSize): + * Api/qwebpage.h: + * WebCoreSupport/FrameLoaderClientQt.cpp: + (WebCore::FrameLoaderClientQt::transitionToCommittedForNewPage): + +2009-06-09 Simon Hausmann <simon.hausmann@nokia.com> + + Reviewed by Ariya Hidayat. + + Renamed QWebHitTestResult::linkTarget to linkElement() and made it return + a QWebElement. The link target itself is always the target DOM attribute. + + * Api/qwebframe.cpp: + (QWebHitTestResultPrivate::QWebHitTestResultPrivate): + (QWebHitTestResult::linkElement): + * Api/qwebframe.h: + * Api/qwebframe_p.h: + * tests/qwebframe/tst_qwebframe.cpp: + 2009-06-08 Simon Hausmann <simon.hausmann@nokia.com> Reviewed by Holger Freyther. diff --git a/src/3rdparty/webkit/WebKit/qt/WebCoreSupport/FrameLoaderClientQt.cpp b/src/3rdparty/webkit/WebKit/qt/WebCoreSupport/FrameLoaderClientQt.cpp index 8f29523..c6c4b9e 100644 --- a/src/3rdparty/webkit/WebKit/qt/WebCoreSupport/FrameLoaderClientQt.cpp +++ b/src/3rdparty/webkit/WebKit/qt/WebCoreSupport/FrameLoaderClientQt.cpp @@ -219,10 +219,14 @@ void FrameLoaderClientQt::transitionToCommittedForNewPage() QBrush brush = m_webFrame->page()->palette().brush(QPalette::Base); QColor backgroundColor = brush.style() == Qt::SolidPattern ? brush.color() : QColor(); + + QWebPage* page = m_webFrame->page(); + const QSize fixedLayoutSize = page->fixedContentsSize(); + m_frame->createView(m_webFrame->page()->viewportSize(), backgroundColor, !backgroundColor.alpha(), - m_webFrame->page()->fixedLayoutSize(), - m_webFrame->page()->useFixedLayout(), + fixedLayoutSize, + fixedLayoutSize.isValid(), (ScrollbarMode)m_webFrame->scrollBarPolicy(Qt::Horizontal), (ScrollbarMode)m_webFrame->scrollBarPolicy(Qt::Vertical)); } diff --git a/src/3rdparty/webkit/WebKit/qt/docs/qtwebkit.qdoc b/src/3rdparty/webkit/WebKit/qt/docs/qtwebkit.qdoc index 119c126..3def65c 100644 --- a/src/3rdparty/webkit/WebKit/qt/docs/qtwebkit.qdoc +++ b/src/3rdparty/webkit/WebKit/qt/docs/qtwebkit.qdoc @@ -39,7 +39,10 @@ \note Building the QtWebKit module with debugging symbols is problematic on many platforms due to the size of the WebKit engine. We recommend - building the module in release mode only for embedded platforms. + building the module only in release mode for embedded platforms. + Currently QtWebKit will always be compiled without debugging symbols + when using gcc. Take a look at the last lines of + \c{src/3rdparty/webkit/WebCore/WebCore.pro} if you need to change this. \note Web site icons, also known as "FavIcons", are currently not supported on Windows. We plan to address this in a future release. diff --git a/src/3rdparty/webkit/WebKit/qt/tests/qwebframe/tst_qwebframe.cpp b/src/3rdparty/webkit/WebKit/qt/tests/qwebframe/tst_qwebframe.cpp index 30e3364..cc4d313 100644 --- a/src/3rdparty/webkit/WebKit/qt/tests/qwebframe/tst_qwebframe.cpp +++ b/src/3rdparty/webkit/WebKit/qt/tests/qwebframe/tst_qwebframe.cpp @@ -2326,7 +2326,8 @@ void tst_QWebFrame::hitTestContent() page.setViewportSize(QSize(200, 0)); //no height so link is not visible QWebHitTestResult result = frame->hitTestContent(QPoint(10, 100)); QCOMPARE(result.linkText(), QString("link text")); - QCOMPARE(result.linkTarget(), QString("_foo")); + QWebElement link = result.linkElement(); + QCOMPARE(link.attribute("target"), QString("_foo")); } void tst_QWebFrame::jsByteArray() |