| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| | |
Change-Id: Id8fd457ab1eb501f8f5622cb5f7d88c352e3758f
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Not all QImage plugins are safe to load from the internet. We should
only load formats that are well-used on the internet and we can be
reasonably sure are safe.
[ChangeLog][WebKit][Behavior Change] QtWebkit will no longer support
any QImage plugin with the Size option, but instead only decode formats
that have been whitelisted. If you are using QtWebKit for controlled
content and wish to override the white-listed it can now be done with
the environment variable QTWEBKIT_IMAGEFORMAT_WHITELIST which takes a
comma-separated list of QImageIO formats.
Change-Id: Ifc4f1a3addfa4ec117697a12000db3c265422314
Reviewed-by: Richard J. Moore <rich@kde.org>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
https://bugs.webkit.org/show_bug.cgi?id=122412
Patch by Maurice van der Pot <griffon26@kfk4ever.com> on 2014-03-31
Reviewed by Darin Adler.
Trivial fix for compilation error; no new tests.
* platform/image-decoders/jpeg/JPEGImageDecoder.cpp:
(WebCore::JPEGImageReader::decode):
(WebCore::fill_input_buffer):
Use TRUE/FALSE defined by libjpeg for libjpeg booleans
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@166490 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Change-Id: I6c669c951fa4bc87862b261ad1a9dd05016086e3
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
https://bugs.webkit.org/show_bug.cgi?id=156445
Patch by Konstantin Tokarev <annulen@yandex.ru> on 2016-04-09
Reviewed by Michael Catanzaro.
ICU defines TRUE and FALSE macros, breaking libjpeg v9 headers.
No new tests needed.
* platform/image-decoders/jpeg/JPEGImageDecoder.h:
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@199278 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Change-Id: I82db8bae210f8b03bd472a82925bd308fa01b6ca
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Also, from now use_wchar_unicode is never silently enabled
when ICU config test fails, and requires qmake argument
WEBKIT_CONFIG+=use_wchar_unicode.
Change-Id: I434f5245c796b723a3bb116f62f8d53d05c3b4f7
Reviewed-by: Michael Brüning <michael.bruning@theqtcompany.com>
|
| |
| |
| |
| |
| | |
Change-Id: Ifff0f8877a2d2d77a04468c205c3353f043e7738
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
garbage collection happens quickly.
Based on upstream fix by said@apple.com at http://trac.webkit.org/changeset/197125
Since the whole document was leaking once an SVGAnimatedProperty was created
so there was no way to produce this bug. After fixing the leak, one crash
and one assert got uncovered. Both of them happen because of the fact:
"if an SVGAnimatedProperty is not referenced it will be deleted."
* svg/SVGPathElement.cpp:
(WebCore::SVGPathElement::lookupOrCreateDWrapper):
The code in this function was assuming that the wrapper will be created
only once which happens when SVGAnimatedProperty::lookupOrCreateWrapper()
is called. Before making this single call, lookupOrCreateDWrapper() was
building an initial SVGPathSegList from byte stream. But now
SVGAnimatedProperty::lookupWrapper() can return false even after creating
the SVGAnimatedProperty because it was deleted later. Calling
buildSVGPathSegListFromByteStream() more than once was causing
SVGAnimatedListPropertyTearOff::animationStarted() to fire the assertion
ASSERT(m_values.size() == m_wrappers.size()) because the path segments were
appended twice to m_values which is in fact SVGPathElement::m_pathSegList.value.
The fix is to build the initial SVGPathSegList only once which should happen
when m_pathSegList.value.isEmpty().
(WebCore::SVGPathElement::animatedPropertyWillBeDeleted):
* svg/SVGPathElement.h:
* svg/properties/SVGAnimatedPathSegListPropertyTearOff.h:
(WebCore::SVGAnimatedPathSegListPropertyTearOff::~SVGAnimatedPathSegListPropertyTearOff):
SVGPathElement is assuming the following equivalence relation:
m_pathSegList.shouldSynchronize ~ SVGAnimatedProperty_is_created_and_not_null.
SVGPathElement::animatedPathSegList() and animatedNormalizedPathSegList()
set m_pathSegList.shouldSynchronize to true when SVGAnimatedProperty is
created but nothing sets m_pathSegList.shouldSynchronize back to false.
This was not a problem when the SVGAnimatedProperty was leaking but after
ensuring it is deleted when it is not referenced this equivalence relation
becomes untrue sometimes. This caused SVGPathElement::svgAttributeChanged()
to crash when we check m_pathSegList.shouldSynchronize and if it is true we
assume that SVGAnimatedProperty::lookupWrapper() will return a non-null pointer
and therefore we deference this pointer and call SVGAnimatedProperty::isAnimating().
To fix this crash we need to set m_pathSegList.shouldSynchronize back to false
when the associated SVGAnimatedProperty is deleted.
Change-Id: I05be755635b02d0d76105fc2eb21c2f013298c4e
Reviewed-by: Michael Brüning <michael.bruning@theqtcompany.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Based on upstream fix by said@apple.com at http://trac.webkit.org/changeset/196670
A destructor was added to SVGListPropertyTearOff that notifies its
wrapper (the SVGAnimatedListPropertyTearoff) about its deletion. This
allows the wrapper to nullify any references to the wrapped content.
We needed to do the same thing for SVGPathSegListPropertyTearOff. Both
SVGPathSegListPropertyTearOff and SVGListPropertyTearOff inherit from
SVGListProperty and both hold pointers to SVGAnimatedListPropertyTearOff
which needs to be notified.
Change-Id: I1873825c7bdc07bf06cd5c300156ebe084f2607e
Reviewed-by: Konstantin Tokarev <annulen@yandex.ru>
Reviewed-by: Michael Brüning <michael.bruning@theqtcompany.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Based on upstream fix by sabouhallawa@apple.com at http://trac.webkit.org/changeset/196268
The leak happens because of cyclic reference between SVGListPropertyTearOff
and SVGAnimatedListPropertyTearOff which is derived from SVGAnimatedProperty.
There is also cyclic reference between SVGAnimatedProperty and SVGElement and
this causes the whole document to be leaked. So if the JS requests, for
example, an instance of SVGPolylineElement.points, the whole document will be
leaked.
The fix depends on having the cyclic reference as is since the owning and the
owned classes have to live together if any of them is referenced. But the owning
class caches a raw 'ref-counted' pointer of the owned class. If it is requested
for an instance of the owned class it returned a RefPtr<> of it. Once the owned
class is not used, it can delete itself. The only thing needed here is to notify
the owner class of the deletion so it cleans its caches and be able to create a
new pointer if it is requested for an instance of the owned class later.
Revert the change of r181345 in SVGAnimatedProperty::lookupOrCreateWrapper()
to break the cyclic reference between SVGElement and SVGAnimatedProperty.
Also apply the same approach in SVGAnimatedListPropertyTearOff::baseVal() and
animVal() to break cyclic reference between SVGListPropertyTearOff and
SVGAnimatedListPropertyTearOff.
Change-Id: Ied6a077299e47855feb235a1c9310f1a58aad91b
Reviewed-by: Konstantin Tokarev <annulen@yandex.ru>
Reviewed-by: Michael Brüning <michael.bruning@theqtcompany.com>
|
|\ \
| |/
| |
| |
| |
| |
| | |
Conflicts:
.qmake.conf
Change-Id: Ib65e8e98ec09b9cc934c8ab7d9f1df3bc377a718
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
WebCore::BidiRunList<WebCore::BidiRun>::reverseRuns
https://bugs.webkit.org/show_bug.cgi?id=123863
Reviewed by Andreas Kling.
Source/WebCore:
Merge https://chromium.googlesource.com/chromium/blink/+/cbaa92c763a37d89eeabd01658e522219299290c
Test: fast/text/bidi-reverse-runs-crash.html
* platform/text/BidiResolver.h:
(WebCore::BidiResolver<Iterator, Run>::createBidiRunsForLine): Don't reverse the runs if there's
nothing to be reversed.
LayoutTests:
* fast/text/bidi-reverse-runs-crash-expected.txt: Added.
* fast/text/bidi-reverse-runs-crash.html: Added.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@158729 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Change-Id: I6bf0902444acf98db29c3cfa2cd3535e1a0c1bef
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
https://bugs.webkit.org/show_bug.cgi?id=129489
Reviewed by David Hyatt.
A use-after-free issue was caught in Blink because m_logicallyLastRun
is not cleared when the item it points to is deleted. Clearing it
turns the use-after-free into a segfault, and prevents any future
use-after-frees from happening.
* platform/text/BidiRunList.h:
(WebCore::BidiRunList<Run>::deleteRuns):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@164876 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Change-Id: Ia76a5723ea649e7a3609fc26025dd5bbd96f3302
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
https://bugs.webkit.org/show_bug.cgi?id=128533
The DNT header should be set by web engines according user preferences.
That includes all HTTP requests, including XHR requests.
Unpriviledged web apps should not be allowed to override/interfere with
user preferences.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@163915 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Change-Id: I80bed1eb13826cdb3cfade3d51297f439b5016f4
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
https://bugs.webkit.org/show_bug.cgi?id=18167 explains that 0.0
value was added specifically to create Windows-like behavior.
In Qt port OSX-like behavior was added in r59846, but this behavior
was lost after transition to Qt 5.
Change-Id: I2d618a356d494805c2d8f9d2fc7751744e040741
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
WebCore::SVGRenderStyle::diff
https://bugs.webkit.org/show_bug.cgi?id=119623
Reviewed by Dirk Schulze.
Source/WebCore:
Include all the stroke attributes in the style diff comparison, the visited links ones were missing.
Test: svg/animations/animate-stroke-crasher.html
* rendering/style/SVGRenderStyle.cpp:
(WebCore::SVGRenderStyle::diff):
LayoutTests:
Add testcase from bug with small adjustments.
* svg/animations/animate-stroke-crasher-expected.txt: Added.
* svg/animations/animate-stroke-crasher.html: Added.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@153914 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Change-Id: I71ff999dab89127600b4a57b5cee58a6fa4687fd
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Prior to the fix, quality would default to 100 irrespective of the image
type, if an explicit quality-argument was not provided, or if the
quality was outside the valid range. In the case where toDataURL is
called without any arguments, the image type defaults to "image/png" and
quality eventually defaults to 100. However, quality in the context of a
PNG applies to the quality of compression and not the quality of the
image. Since PNG is a lossless format, compression only affects the size
of the image and not its quality. This resulted in PNG images of a large
size, with no compression at all. The same behavior could be observed
when toDataURL is called with the image type explicitly set to
"image/png", without a quality argument. The expected behavior is only
observed if toDataURL is called with the image type set to "image/png"
and the quality set to 0, since this provides the highest level of
compression. According to section 4.12.4.4 of the HTML5 spec, the
quality argument should only apply to images of type "image/jpeg", and
if quality is not provided, the user-agent should use a default value.
This means that the spec was being violated, since the quality was set
to 100 regardless of the image type.
The fix was to consider the quality argument (along with the associated
sanity-checks) only if the image type is "jpeg"; otherwise quality is
set to -1. This change results in PNG images being encoded to
properly-sized base64 strings.
[ChangeLog][WebKit][Behavior Change] The quality parameter
in canvas.toDataURL only applies to JPEG images now, in accordance with
section 4.12.4.4 of the HTML5 spec.
Change-Id: Ie87a32ec368e70e7736d4d2e684e2528ce37f745
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
https://bugs.webkit.org/show_bug.cgi?id=118516
Patch by Gurpreet Kaur <k.gurpreet@samsung.com> on 2013-12-04
Reviewed by Simon Fraser.
From Blink r156449 by <srinivasa.ragavan.venkateswaran@intel.com>
Source/WebCore:
An element having height as percentage needs to have the
containingblock's height or availableheight to calculate its
own height. The containing block having a height set in vh
unit was not being considered for calculating the child's
height.
* rendering/RenderBox.cpp:
(WebCore::RenderBox::computePercentageLogicalHeight):
Correct child's height(in pecentage) was not being calculated
incase of parent having height set in vh unit. Added condition
to calculate the containing block height in terms of viewport size.
Change-Id: Id0158323c6d1b5a43b267133bfe0c7ce5fe4b472
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
|
| |\
| | |
| | |
| | | |
Change-Id: If31212a273bc60692eb21169f24d136763409ead
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
When QT_NO_OPENSSL is checked, the user agent says "no SSL support" even when
there *is* SSL support via Secure Transport on OS X.
Change-Id: Idb8bf3ee95689f60adab332b3cbac7b6bdf6a738
Reviewed-by: Timur Pocheptsov <timur.pocheptsov@theqtcompany.com>
Reviewed-by: Michael Brüning <michael.bruning@theqtcompany.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Since Qt 5.5, Secure Transport is used instead of OpenSSL on OS X. This means
secure websockets (wss://) were disabled on OS X despite QSslSocket being
available.
Change-Id: Ic584a6ed81b625c84a85e54dba84c8fc920b861a
Task-number: QTBUG-49027
Reviewed-by: Richard J. Moore <rich@kde.org>
Reviewed-by: Timur Pocheptsov <timur.pocheptsov@theqtcompany.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
m_socket can be set to 0 in the constructor, e.g. when Qt is compiled without
SSL support.
Change-Id: Ic3bb18f6c801d463e2277b4c19ef2c790216bf69
Task-number: QTBUG-49027
Reviewed-by: Richard J. Moore <rich@kde.org>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Previously, buttonHeightRatio was defined as an int with a value of 1.5.
Hence, the value was actually trimmed to 1.
Change-Id: Icbc628f6c5495774ba4476d0ad53d6164ec62213
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
|
|/ /
| |
| |
| |
| |
| |
| |
| | |
This is enough to trick http://youtube/tv into loading (it blocks UAs
without MediaSource).
Change-Id: I7163c8ef65cbbe65c23deb84ad2e4d5300f41cf2
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
|
|\ \
| |/
| |
| | |
Change-Id: I33f49683dbb3f99f4149accfeddeea95b309657f
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Don't read from closed QNetworkReply instances. This fixes
QWebView hangs on some of the web sites when
reply->abort() or reply->close() is called.
[ChangeLog][QtWebKit] fixed reply->abort() and reply->close() handling.
Change-Id: I8d46ad115954369b9de0b5a74b118fc3bf2f1092
Task-number: QTBUG-47654
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
|
|/
|
|
|
|
|
|
|
|
| |
Implements the GraphicsContext::setURLForRect method using the new
QPdfEngine::drawHyperlink method added in version 5.6.0 so that
anchors are rendered as clickable links in PDF documents.
Task-number: QTBUG-44563
Change-Id: Ic45399ba2d97be28816e54f6bd169e90de236e91
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
|
|
|
|
|
|
|
|
|
| |
Without this patch, i.e. without removing -GL from linker flags, WebCore fails
to link. Checked with MSVC 2015 RTM (latest version right now).
Change-Id: If36ea0e7d8f6e2e6026033240e547e90be70d335
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
Reviewed-by: Simon Hausmann <simon.hausmann@theqtcompany.com>
|
|
|
|
|
|
|
|
|
| |
Also see a related Chromium fix:
http://trac.webkit.org/changeset/111656/trunk/Source/WebKit/chromium/src/SocketStreamHandle.cpp
Change-Id: I14d4279dad5db56a2a4a273e89e76456d8eb1702
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When a subframe is not composited it may be painted immediately when its
parent is painted. This means that if it is not laid out, it may cause
asserts during painting, if we only check the parent is laid out.
The patch is based on
http://svn.webkit.org/repository/webkit/trunk@180063 by Simon Fraser.
Task-number: QTBUG-45428
Change-Id: Ib7e283485bef68375c7b023264f79acd7490e2c5
Reviewed-by: Andras Becsi <andras.becsi@theqtcompany.com>
|
|\
| |
| |
| | |
Change-Id: Icc1f3caf891dd114064f0fd37931d9bf6c7ccfc1
|
| |
| |
| |
| |
| |
| |
| |
| | |
The VLC plugin may load a plugins of its own that uses Qt4.
Change-Id: I5229a360f7d08d024fb93df1a2993fd7c0176848
Task-number: QTBUG-45431
Reviewed-by: Simon Hausmann <simon.hausmann@theqtcompany.com>
|
|\ \ |
|
| |\ \
| | |/
| | |
| | | |
Change-Id: Iceb5c402acfd83632007cb5f7673b4d6cda08e6b
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Fixes building qtwebkit with MSVC 2015
Change-Id: I94b085ab822b3cdfa911814467e051c11c93d79e
Task-number: QTBUG-46344
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@theqtcompany.com>
Reviewed-by: Simon Hausmann <simon.hausmann@theqtcompany.com>
|
|/ /
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Now that the QOpenGLContext is watched and set to 0 if deleted, we
might have a private class, but no context on deletion, we must
therefore test the pointer before trying to make it active to do the
final cleanups.
Change-Id: Ie970265adfba5d0405a02a157bee344665d2092f
Reviewed-by: Yohan Chuzeville <ychuzevi@cisco.com>
Reviewed-by: Michael Brüning <michael.bruning@theqtcompany.com>
|
|\ \
| |/
| |
| | |
Change-Id: Icfe7bccfeba4697fc1fa7b3ddd082ffa30909900
|
| |\
| | |
| | |
| | | |
Change-Id: Iafbf406788fa3b89d01bc11c45c56727f058b189
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Adds a special content layer that can used when we drop the backing
store due to size, but still need to draw the content that would have
been in the backing store.
This fixes a regression introduced with the recent patch to prevent
large backing stores.
Change-Id: I9a61b3cc978bccdaa423c0d076e3aeaa7d9ebc99
Task-number: QTBUG-46178
Reviewed-by: Michael Brüning <michael.bruning@theqtcompany.com>
|
| |\ \
| | |/
| | |
| | | |
Change-Id: I5bacbd4ac254149e45003fea759855df598a2f36
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Our webkit examples all assert on resize due to qt.io trying to do
an animation from an undefined value.
Change-Id: I95aaee7834b2caba37e55f2cc09160af638a8d39
Reviewed-by: Michael Brüning <michael.bruning@theqtcompany.com>
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Added a couple of nullpointer checks in image document cleanup.
Task-number: QTBUG-42890
Change-Id: I533a517ed7402428afc3ae470398a64ed0b6a2a8
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
|
|\ \ \
| |/ /
| | |
| | |
| | |
| | |
| | | |
Conflicts:
.qmake.conf
Change-Id: Ib823c8eeb271b380811827aae1ac0bd50ea5a59b
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Add a specific setting to disable remote access for local URLs and also
enforce that on data-URLs loaded owned by local URLs.
Change-Id: Ied8ec141eb1c28775644fce184a4759a79e1d177
Task-number: QTBUG-45556
Reviewed-by: Dmitry Shachnev <mitya57@gmail.com>
Reviewed-by: Michael Brüning <michael.bruning@theqtcompany.com>
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
https://bugs.webkit.org/show_bug.cgi?id=137762.
Patch by Said Abou-Hallawa <sabouhallawa@apple.com> on 2014-10-22
Reviewed by Daniel Bates.
Source/WebCore:
SVG images have unique security rules that prevent them from loading any external
resources. This patch enforces these rules in CachedResourceLoader::canRequest for
all non-data-uri resources.
The fix and the tests are ported but modified a little from the chromium fix:
http://src.chromium.org/viewvc/blink?view=rev&rev=176084
Test: http/tests/security/svg-image-with-cached-remote-image.html
http/tests/security/svg-image-with-css-cross-domain.html
For the SVG image, prevent loading any external sub-resource except for data urls.
* loader/cache/CachedResourceLoader.cpp:
(WebCore::CachedResourceLoader::canRequest):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@175074 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Change-Id: Iec5014e81e25c37cc6754d7cc73645b17994974f
Reviewed-by: Michael Brüning <michael.bruning@theqtcompany.com>
|
|\ \
| |/
| |
| | |
Change-Id: I0d9dd63455dc12402afd56cc1bd97d0cc97c24e8
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Issue is that the TextureMapperGL creates a GraphicsContext3D using
the current OpenGL context. This is done by storing a reference on
QOpenGLContext::currentContext() inside
GraphicsContext3DPrivate::GraphicsContext3DPrivate(). When exiting the
browser, Qt releases QOpenGLContext before the release of the
GraphicsContext3D in webkit which leads to a crash when destoying
GraphicsContext3DPrivate.
Task-number: QTBUG-45481
Change-Id: I2b9d7b1a96fbbe8517ea323d45ef3922ada208a3
Reviewed-by: Julien Brianceau <jbriance@cisco.com>
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
We already block too large elements from getting a composited layer with
backing store, but in the case where an already composited layer is
resized above the size limit, it may have bad size until relayed out.
This patch blocks the backing store of such a graphics layer from being
instantiated.
Change-Id: I03c1688c00c1f96fdaf7ef251ee285c05c072730
Task-number: QTBUG-44523
Reviewed-by: Florian Bruhin <qt-project.org@the-compiler.org>
Reviewed-by: Michael Brüning <michael.bruning@theqtcompany.com>
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Ported from http://trac.webkit.org/changeset/181565 by beidson@apple.com.
Upstream patch by Sam Weinig, reviewed by Brady Eidson.
* loader/icon/IconController.cpp:
(WebCore::IconController::startLoader): Bail early here if the page is using an ephemeral session.
(WebCore::IconController::continueLoadWithDecision): Instead of here.
Change-Id: I263bb6122606caa3488d641b127dd377012ee424
Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
|
| |
| |
| |
| |
| |
| |
| | |
Do not try to cleanup on an already destroyed OpenGL context.
Change-Id: I56d2a07700b1f69100be7fc9537dd0bc69449a0c
Reviewed-by: Michael Brüning <michael.bruning@theqtcompany.com>
|
|\ \
| |/
| |
| | |
Change-Id: I824a984e8552340c2f9f631ac951f2c9eaa90b74
|