summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Added changes filev5.4.05.4.0Allan Sandfeld Jensen2014-12-021-0/+56
| | | | | | Change-Id: Ia8e0ad300b4651746a7741d0482587cbcd2f891c Reviewed-by: Florian Bruhin <qt-project.org@the-compiler.org> Reviewed-by: Simon Hausmann <simon.hausmann@digia.com>
* Fix disabling of aliasing for all gcc alike compilersSimon Hausmann2014-11-291-1/+2
| | | | | | | | | | Don't rely on the makespec to contain g++ in its name, instead use QMAKE_COMPILER, which gets added to CONFIG. Fixes crashes on QNX. Change-Id: If8d2a6a7aa9998b311b2ca137af2fa236adecb00 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
* Clear a stray ref on layers when purging GL resourcesv5.4.0-rc1Jocelyn Turcotte2014-11-141-0/+4
| | | | | | | | | | | | Fix a regression of 3df57f7e550a19cb4450ed3896aa51ddb38c2ed2 where a reference to TextureMapperBackingStore would be kept in the TextureMapperLayers after purgeGLResources(). This would crash later on if the window would get destroyed before the CoordinatedGraphicsScene that own those layers. Task-number: QTBUG-40734 Change-Id: Id1a596c0f0ba9afd1e163f83c38dda939f521314 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
* Do not force a direction before we know itAllan Sandfeld Jensen2014-11-131-7/+9
| | | | | | | | | | | | | | | WebCore does not calculate the text direction when calculating widths the best we can do here is let the shaper guess the direction. Forcing it to the default LTR breaks HarfbuzzNG on Arabic scripts. Since the default HarfBuzz engine changed in 5.4, this was a regression from 5.3. See also https://bugs.webkit.org/show_bug.cgi?id=101440 Task-number: QTBUG-41450 Change-Id: Ia324c8e5ebd1744ce2dfaea94d5046ef1f77b483 Reviewed-by: Eskil Abrahamsen Blomfeldt <eskil.abrahamsen-blomfeldt@theqtcompany.com>
* Use fixed executable allocator on x64Allan Sandfeld Jensen2014-11-072-10/+2
| | | | | | | | | | | Since the x86-64 assembler generator depends on jumps being expressable as 32bit offsets we need to use the fixed size executable allocator and not the on demand allocator. This patch removes its exclusion from being used on Windows. Task-number: QTBUG-42417 Change-Id: Ie6b127e9b4291869566fd99cc7efe05b566c8e66 Reviewed-by: Jocelyn Turcotte <jocelyn.turcotte@digia.com>
* Fix selections after leave eventAllan Sandfeld Jensen2014-11-063-1/+12
| | | | | | | | | | | | We were sending a bad fake mouse event on leave events that caused WebCore to stop tracking mousepresses correctly. Since we continue to receive mouse-events after leave if a mouse button is down, we don't need the fake mouse event to unset hover states and can skip, and can skip sending it. Task-number: QTBUG-41419 Change-Id: I8d50c440f7556f7e34bb0c05248577583c2550b4 Reviewed-by: Jocelyn Turcotte <jocelyn.turcotte@digia.com>
* Fixed upside down drawing of accelerated 2d canvasSamuel Rødal2014-11-041-2/+10
| | | | | | | | | | | The convention used by the Qt OpenGL paint engine's internal drawTexture() function didn't match the texture layout of the accelerated 2D canvas. We need to adjust the coordinates according to whether flipped painting is in effect. Task-number: QTBUG-42376 Change-Id: I787a2deb5f26df532f2e7c475754f0fd2189132c Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
* Fixed self-drawing of 2D canvas with accelerated 2D canvas enabled.Samuel Rødal2014-11-043-7/+34
| | | | | | | | | | | | | | | The canvas 2D API allows drawing the canvas onto itself, however with accelerated 2D canvas enabled the canvas is implemented in terms of rendering to a texture using framebuffer objects. OpenGL does not allow having the same texture be both a source and a destination in the same rendering operation, so the results are undefined. Instead, we need to blit the texture into a temporary framebuffer object, before using that as the source of the final drawing operation. Task-number: QTBUG-42275 Change-Id: I8114814dab2c8c6ca90b8d35a37b383b79923cb3 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
* Fixed crash in QGraphicsWebView when clicking on linksSamuel Rødal2014-11-031-3/+1
| | | | | | | | | | | The mouse event passed to this function is allocated on the stack, so we can't delete it. Also, it doesn't make sense to allocate a new one which immediately gets leaked. The intent must have been to use the ev parameter as a return parameter - that's what it looks like on the caller side. Change-Id: I023cd7895a278eada4ad4acd233741aff837fed6 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
* Fix build of leveldb with old Qt versionsAllan Sandfeld Jensen2014-11-032-4/+10
| | | | | | | | Applies a missing patch to leveldb that fixed building with older Qt versions. Change-Id: Ie894127d265140cbb8524e271e5779c92ab8e393 Reviewed-by: Michael Bruning <michael.bruning@digia.com>
* Fix merge errorAllan Sandfeld Jensen2014-11-031-2/+2
| | | | | | | | | | The change 4a39158368f6184c8d56e1d65866ba99f83297e0 merged a part of the diff for prepend to append. This patch moves the wrongly merged line. Task-number: QTBUG-40284 Change-Id: I32d1b2f8e730aca38964d3675d3dec94512e0026 Reviewed-by: Julien Brianceau <jbriance@cisco.com> Reviewed-by: Michael Bruning <michael.bruning@digia.com>
* Merge "Merge remote-tracking branch 'origin/5.3' into 5.4" into refs/staging/5.4Allan Sandfeld Jensen2014-10-311-0/+1
|\
| * Merge remote-tracking branch 'origin/5.3' into 5.4Oswald Buddenhagen2014-10-311-0/+1
| |\ | | | | | | | | | Change-Id: I11f35f205d2ded597d54650a948bec3e1e52b419
| | * Fix assert after makerope boundary check5.3Lucas Forschler2014-10-311-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The browser demo asserts on qt-project.org after the makerope security fix. Port of http://trac.webkit.org/changeset/169397 Change-Id: I19c98737735d9b17133742752928d4ce6349de13 Reviewed-by: Michael Bruning <michael.bruning@digia.com>
* | | avoid creation of empty qmltypes for qtquick1Fawzi Mohamed2014-10-311-7/+8
| | | | | | | | | | | | | | | | | | Change-Id: Ic28043a94151d3878219e5808075ce6ae663c316 Reviewed-by: Michael Bruning <michael.bruning@digia.com> Reviewed-by: Allan Sandfeld Jensen <allan.jensen@theqtcompany.com>
* | | Add mappings from new Qt key values to key identifiersAllan Sandfeld Jensen2014-10-311-4/+14
|/ / | | | | | | | | Change-Id: Iab57bbe15c6fd4d33dfe6e36cd0bd01ca6644841 Reviewed-by: Michael Bruning <michael.bruning@digia.com>
* | Do not send focus event to missing plugins on OS XAllan Sandfeld Jensen2014-10-301-0/+3
| | | | | | | | | | | | | | | | | | Fix a crash when trying to focus a pluginview that doesn't have a loaded plugin. Task-number: QTBUG-40273 Change-Id: I0875cab81a3484094fc705d59df80b87ca83a9ff Reviewed-by: Michael Bruning <michael.bruning@digia.com>
* | Do not use unaligned access when disabledAllan Sandfeld Jensen2014-10-303-9/+13
| | | | | | | | | | | | | | | | | | | | Use the GCC feature defines to check for support for ARMv6 style unaligned access support. This fixes configurations where it was disabled and the ARMv6-M processors where it is not supported. Change-Id: I00c9bb19ba582b64ecb9eaa6da9887ba0df8fd3c Reviewed-by: Julien Brianceau <jbriance@cisco.com> Reviewed-by: Jocelyn Turcotte <jocelyn.turcotte@digia.com>
* | Fix preloading of video when using QtMultimedia backend.Michael Brüning2014-10-302-5/+15
| | | | | | | | | | | | | | | | Based on a patch by Yoann Lopes (yoann.lopes@theqtcompany.com). Task-number: QTBUG-37381 Change-Id: I8647c578399a592f8feef0d4028187afd3726b2d Reviewed-by: Yoann Lopes <yoann.lopes@theqtcompany.com>
* | LLInt is not built with ARM softfp ABIAllan Sandfeld Jensen2014-10-281-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | In fixing QTBUG-35681 we disabled LLInt when hard fp is not set. This is however too strong a condition as we only need to be allowed to use floating point instructions, not use the hard floating point ABI. LLInt is also disabled on Android because it currently doesn't build. Task-number: QTBUG-35681 Change-Id: Ibe109b0f8fe6acaec90d42305c6151a440141ea9 Reviewed-by: Julien Brianceau <jbriance@cisco.com> Reviewed-by: Michael Bruning <michael.bruning@digia.com>
* | Use INLINE_ARM_FUNCTION in OFFLINE_ASM_GLOBAL_LABELAllan Sandfeld Jensen2014-10-241-9/+1
| | | | | | | | | | | | | | | | | | | | We can get rid of an ifdef in LowLevelInterpreter by reusing logic from InlineAsm.h. This also fixes ARM_TRADITIONAL when forced on when thumb2 is available to the compiler. Change-Id: Iac5ad22a4a41757fa03c3cbc2e8f68e44d085bae Reviewed-by: Julien Brianceau <jbriance@cisco.com> Reviewed-by: Michael Bruning <michael.bruning@digia.com>
* | Do not use floating point instructions in float-abi=softAllan Sandfeld Jensen2014-10-231-0/+4
| | | | | | | | | | | | | | | | | | Pretend VFP doesn't exist when the ABI forbids floating point instructions. Task-number: QTBUG-38371 Change-Id: Id0552201e6bf98812198c0028e2c85107e967a81 Reviewed-by: Michael Bruning <michael.bruning@digia.com>
* | Do not enumerate plugins when plugins are disabledJan Kundrát2014-10-231-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Even if plugins were disabled through QWebSettings::PluginsEnabled, WebKit would still enumerate the on-disk NPAPI plugins and attempt to load them. This caused all sorts of havoc especially when loading plugins linked to Qt4. Even if that wasn't the case, plugins should *not* be loaded unless there's a compelling reason to do so. Loading plugins needlessly takes time and it might even have security implications -- it sounds like a good practice to not execute random code, even though it comes from trusted locations, when that code is not needed. After this change, the code still passes the tst_qwebpage tests which are related to plugin loading. As it turns out, the "QtWebKit application plugins" (application/x-qt-plugin and application/x-qt-styled-widget) do not use this code path at all, they instead go through a path like this one: #0 PluginPage::createPlugin ... #1 QWebPagePrivate::createPlugin ... #2 WebCore::FrameLoaderClientQt::createPlugin ... #3 WebCore::SubframeLoader::loadPlugin ... #4 WebCore::SubframeLoader::requestPlugin ... #5 WebCore::SubframeLoader::requestObject ... #6 WebCore::HTMLObjectElement::updateWidget ... #7 WebCore::HTMLPlugInImageElement::updateWidgetIfNecessary ... #8 WebCore::HTMLPlugInImageElement::updateWidgetCallback ... #9 WebCore::ContainerNode::dispatchPostAttachCallbacks ... #10 WebCore::ContainerNode::resumePostAttachCallbacks ... #11 WebCore::PostAttachCallbackDisabler::~PostAttachCallbackDisabler ... #12 WebCore::Document::recalcStyle ... #13 WebCore::Document::updateStyleIfNeeded ... #14 WebCore::Document::finishedParsing ... #15 WebCore::HTMLConstructionSite::finishedParsing ... #16 WebCore::HTMLTreeBuilder::finished ... #17 WebCore::HTMLDocumentParser::end ... #18 WebCore::HTMLDocumentParser::attemptToRunDeferredScriptsAndEnd ... #19 WebCore::HTMLDocumentParser::prepareToStopParsing ... #20 WebCore::HTMLDocumentParser::attemptToEnd ... #21 WebCore::HTMLDocumentParser::finish ... #22 WebCore::DocumentWriter::end ... #23 WebCore::DocumentLoader::finishedLoading ... Task-number: QTBUG-36349 Change-Id: Id8f37fb16bb295781c22f439fd54a65486deef8e Reviewed-by: Allan Sandfeld Jensen <allan.jensen@digia.com>
* | Disable the build on Windows RT / Windows PhoneSimon Hausmann2014-10-231-0/+1
| | | | | | | | | | | | Change-Id: I594737c60ed2d4c08de662b8ca534e5abbbffd60 Reviewed-by: Maurice Kalinowski <maurice.kalinowski@digia.com> Reviewed-by: Michael Bruning <michael.bruning@digia.com>
* | Do not schedule navigation to invalid URLAllan Sandfeld Jensen2014-10-221-1/+1
| | | | | | | | | | | | | | | | | | | | Fix an assert triggered by navigation to a fragment URL of an unset URL. The code was assuming the string given was a valid encoded URL, so we shouldn't continue if the completed URL is invalid. Task-number: QTBUG-38546 Change-Id: I377b09814150ae9926de95efe3f3e2e750145c4a Reviewed-by: Michael Bruning <michael.bruning@digia.com>
* | [Windows] Also check for QApplication::style when animating buttons.Michael Brüning2014-10-211-1/+1
| | | | | | | | | | | | | | | | | | | | The fallback style for a custom style sheet is QApplication::style(), which defaults to QWindowsVistaStyle on most (if not all) windows systems. Task-number: QTBUG-39400 Change-Id: Ib05571033730ed26a39f89d8e9fe948c4b280e64 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@digia.com>
* | Pop pack pragma on x86 QNXAllan Sandfeld Jensen2014-10-211-3/+3
| | | | | | | | | | | | | | A recent patch fixing QNX on x86 forgot to also pop the pragma it set. Change-Id: I9834c0b57a4409e8e80c29899e6f3989607fe22f Reviewed-by: Jocelyn Turcotte <jocelyn.turcotte@digia.com>
* | Remove QtScript includesAllan Sandfeld Jensen2014-10-161-2/+1
| | | | | | | | | | | | | | Remove QtScript includes that haven't been usedin years. Change-Id: Iea2b61a6ad2ea187055daff783ce9f0280477767 Reviewed-by: Michael Bruning <michael.bruning@digia.com>
* | Fix Qt 5.3 backward compatible buildAllan Sandfeld Jensen2014-10-164-6/+29
| | | | | | | | | | | | | | | | | | | | The accelerated canvas and dynamic GL support used Qt 5.4 specific features. These can be avoided by disabling one webgl extension and being less efficient in accelerated canvas, thereby making it possible to still build QtWebKit with Qt 5.3. Change-Id: Ic64f7f5ac6b836b314331c4d509e4d64cb0dcb7c Reviewed-by: Michael Bruning <michael.bruning@digia.com>
* | Blacklist libkpartspluginSune Vuorela2014-10-161-1/+2
| | | | | | | | | | | | | | | | KPartsPlugin is built using qt4, and loading that into a Qt5 QtWebkit makes stuff crash. Change-Id: I47d01dfe3b215192b012911013b352ea5b5b1530 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@digia.com>
* | StylePendingImage needs to correctly manage the CSSValue pointer lifetimeDarin Adler2014-10-157-8/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://bugs.webkit.org/show_bug.cgi?id=125468 Reviewed by Andreas Kling. Source/WebCore: Test: fast/css/pending-image-crash.xhtml Disconnect the reference counted StylePendingImage from the CSSValue that owns it when it's not needed any more, otherwise we could end up using a pointer that might no longer be valid. * css/CSSCursorImageValue.cpp: (WebCore::CSSCursorImageValue::detachPendingImage): Added. Calls detachFromCSSValue on the current image if it is a StylePendingImage. (WebCore::CSSCursorImageValue::~CSSCursorImageValue): Call detachPendingImage. (WebCore::CSSCursorImageValue::cachedImage): Call detachPendingImage before changing m_image to a new value. (WebCore::CSSCursorImageValue::clearCachedImage): Ditto. * css/CSSCursorImageValue.h: Added detachPendingImage. * css/CSSImageSetValue.cpp: (WebCore::CSSImageSetValue::detachPendingImage): Added. Calls detachFromCSSValue on the current image set if it is a StylePendingImage. (WebCore::CSSImageSetValue::~CSSImageSetValue): Call detachPendingImage. (WebCore::CSSImageSetValue::cachedImageSet): Call detachPendingImage before changing m_imageSet to a new value. * css/CSSImageSetValue.h: Added detachPendingImage. * css/CSSImageValue.cpp: (WebCore::CSSImageValue::detachPendingImage): Added. Calls detachFromCSSValue on the current image if it is a StylePendingImage. (WebCore::CSSImageValue::~CSSImageValue): Call detachPendingImage. (WebCore::CSSImageValue::cachedImage): Call detachPendingImage before changing m_image to a new value. * css/CSSImageValue.h: Added detachPendingImage. * rendering/style/StylePendingImage.h: (WebCore::StylePendingImage::cssImageValue): Added a null check. (WebCore::StylePendingImage::cssImageGeneratorValue): Added a null check. (WebCore::StylePendingImage::cssCursorImageValue): Added a null check. (WebCore::StylePendingImage::cssImageSetValue): Added a null check. (WebCore::StylePendingImage::detachFromCSSValue): Added. Sets m_value to null since the style is no longer using this StylePendingImage. (WebCore::StylePendingImage::data): Changed to use the "this" pointer since all we need is some arbitrary pointer uniquely identifying the image. Before loading the image, we have no suitable weak identifier, so it suffices to use the unique pointer to each StylePendingImage object. This function is used only in a limited way; it would be nice to find a way to make the code less strange long term. Change-Id: Iab8a5d033c5cdf9a488ac18d8f233dee48c5f3e7 git-svn-id: http://svn.webkit.org/repository/webkit/trunk@160479 268f45cc-cd09-0410-ab3c-d52691b4dbfc Reviewed-by: Michael Bruning <michael.bruning@digia.com>
* | Fix android compile.BogDan Vatra2014-10-1510-5/+28
| | | | | | | | | | | | Change-Id: I9848c2b384e6fa85a5224ce0691456105d2fe8a1 Reviewed-by: BogDan Vatra <bogdan@kde.org> Reviewed-by: Allan Sandfeld Jensen <allan.jensen@digia.com>
* | Fix LLInt vector offsets on Windows 64Allan Sandfeld Jensen2014-10-153-1/+13
| | | | | | | | | | | | | | | | Derived classes on Windows x64 are 64bit aligned placing the m_size property of Vector slightly differently than on other platforms. Change-Id: I186de5ea200abfbdd3bdf7502f3f427cb6890f3c Reviewed-by: Jocelyn Turcotte <jocelyn.turcotte@digia.com>
* | Crash in WebCore::BidiResolver<WebCore::InlineIterator, ↵Ryosuke Niwa2014-10-141-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | WebCore::BidiRun>::createBidiRunsForLine https://bugs.webkit.org/show_bug.cgi?id=122776 Reviewed by Darin Adler. Merge https://chromium.googlesource.com/chromium/blink/+/aca89bc4d984705a1f94b623dae0ab03e239a248 Fix modification of whitespace endpoints to not assume it's operating on RenderTexts During line layout, we use midpoints to identify RenderObjects, or parts of RenderObjects, that don't need InlineBoxes, usually because of collapsed whitespace. Prior to actually creating BidiRuns (the precursor to InlineBoxes), we use checkMidpoints to fix up our lineMidpointState to handle the case where we start ignoring spaces in our line, but don't stop until somewhere on the following line. Previously, this function assumed that the final midpoint (called an endpoint) was a RenderText, but this assumption is wrong if we have a beginning midpoint created by shouldSkipWhitespaceAfterStartObject (which handles inlines and list markers) and no endpoint on that line. In that case, we'd instead adjust the position backwards on the beginning midpoint, which would cause us to fail to create an InlineBox for the inline or list marker. In the new test added, this would actually trigger a crash due to an assumption when visually re-ordering BidiRuns that a non-empty line would actually contain at least one such run. Test: fast/text/whitespace/whitespace-and-margin-wrap-after-list-marker-crash.html * rendering/RenderBlockLineLayout.cpp: (WebCore::checkMidpoints): Change-Id: I894bd9da4250dc2c79a55a8014e54df7dcbec719 git-svn-id: http://svn.webkit.org/repository/webkit/trunk@157436 268f45cc-cd09-0410-ab3c-d52691b4dbfc Reviewed-by: Michael Bruning <michael.bruning@digia.com>
* | Merge "Merge remote-tracking branch 'origin/5.3' into 5.4" into refs/staging/5.4Allan Sandfeld Jensen2014-10-1410-23/+100
|\ \
| * \ Merge remote-tracking branch 'origin/5.3' into 5.4Frederik Gladhorn2014-10-1410-23/+100
| |\ \ | | |/ | | | | | | Change-Id: I509f0440296df39aece8133382aacc43a4e05f99
| | * Add missing branchAdd32 implementations to fix sh4 and mips buildsJulien Brianceau2014-09-302-0/+29
| | | | | | | | | | | | | | | | | | | | | sh4 implementation has been taken from WebKit r173222. Change-Id: I1c188e88e9fd8772f97cac94d7ee31945b538b71 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@digia.com>
| | * [ARM] Unreviewed build fix after r167336.Csaba Osztrogonác2014-09-261-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | * assembler/MacroAssemblerARM.h: (JSC::MacroAssemblerARM::branchAdd32): Change-Id: I919df8301e69cca630f475afcc0c4311d713f10e git-svn-id: http://svn.webkit.org/repository/webkit/trunk@167565 268f45cc-cd09-0410-ab3c-d52691b4dbfc Reviewed-by: Jocelyn Turcotte <jocelyn.turcotte@digia.com>
| | * Unreviewed, ARMv7 build fix after r167336.Filip Pizlo2014-09-261-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | * assembler/MacroAssemblerARMv7.h: (JSC::MacroAssemblerARMv7::branchAdd32): Change-Id: Ia16f8dce00ce660cf5f7b9eb1c3496e08cbdb603 git-svn-id: http://svn.webkit.org/repository/webkit/trunk@167354 268f45cc-cd09-0410-ab3c-d52691b4dbfc Reviewed-by: Jocelyn Turcotte <jocelyn.turcotte@digia.com>
| | * compileMakeRope does not emit necessary bounds checksFilip Pizlo2014-09-266-23/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://bugs.webkit.org/show_bug.cgi?id=130684 <rdar://problem/16398388> Reviewed by Oliver Hunt. Add string length bounds checks in a bunch of places. We should never allow a string to have a length greater than 2^31-1 because it's not clear that the language has semantics for it and because there is code that assumes that this cannot happen. Also add a bunch of tests to that effect to cover the various ways in which this was previously allowed to happen. * dfg/DFGOperations.cpp: * dfg/DFGSpeculativeJIT.cpp: (JSC::DFG::SpeculativeJIT::compileMakeRope): * ftl/FTLLowerDFGToLLVM.cpp: (JSC::FTL::LowerDFGToLLVM::compileMakeRope): * runtime/JSString.cpp: (JSC::JSRopeString::RopeBuilder::expand): * runtime/JSString.h: (JSC::JSString::create): (JSC::JSRopeString::RopeBuilder::append): (JSC::JSRopeString::RopeBuilder::release): (JSC::JSRopeString::append): * runtime/Operations.h: (JSC::jsString): (JSC::jsStringFromRegisterArray): (JSC::jsStringFromArguments): * runtime/StringPrototype.cpp: (JSC::stringProtoFuncIndexOf): (JSC::stringProtoFuncSlice): (JSC::stringProtoFuncSubstring): (JSC::stringProtoFuncToLowerCase): * tests/stress/make-large-string-jit-strcat.js: Added. (foo): * tests/stress/make-large-string-jit.js: Added. (foo): * tests/stress/make-large-string-strcat.js: Added. * tests/stress/make-large-string.js: Added. Change-Id: If01dd2a2d2daa3d209eddf0213d2b391e94f54a0 git-svn-id: http://svn.webkit.org/repository/webkit/trunk@167336 268f45cc-cd09-0410-ab3c-d52691b4dbfc Reviewed-by: Jocelyn Turcotte <jocelyn.turcotte@digia.com>
* | | Crash in HTMLMediaElement::contextDestroyedRyosuke Niwa2014-10-141-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://bugs.webkit.org/show_bug.cgi?id=123963 Reviewed by Eric Carlson. Merge https://chromium.googlesource.com/chromium/blink/+/177999cdb34b707465670f0feff723922939f278 * html/HTMLMediaElement.cpp: (WebCore::HTMLMediaElement::~HTMLMediaElement): Change-Id: If730e758410f18ae86aca5a92cb7c48b12b8c7ae git-svn-id: http://svn.webkit.org/repository/webkit/trunk@158870 268f45cc-cd09-0410-ab3c-d52691b4dbfc Reviewed-by: Michael Bruning <michael.bruning@digia.com>
* | | Get rid of old harmattan workaround hurting Qt on ARMAllan Sandfeld Jensen2014-10-141-12/+2
| | | | | | | | | | | | | | | Change-Id: Ieecee7c05e78117e8c9fc335f60970e2144ede81 Reviewed-by: Michael Bruning <michael.bruning@digia.com>
* | | Check for ICU at configure time the same way we add it laterAllan Sandfeld Jensen2014-10-141-0/+2
|/ / | | | | | | | | | | | | | | | | Use pkg-config to get the right ICU library when searching for it at config time. This matches how it is build with later. Task-number: QTBUG-41750 Change-Id: I134ecdeb523eab225ef66f4dcc14ae4e25ea4eca Reviewed-by: Simon Hausmann <simon.hausmann@digia.com>
* | Do not use JIT on big endian architecturesAllan Sandfeld Jensen2014-10-131-0/+8
| | | | | | | | | | | | | | | | | | | | | | All the JIT implementations targets the little-endian versions of ARM, MIPS and SH4 respectively. So in case anyone builds on those platforms we should not enable JIT. Task-number: QTBUG-41896 Change-Id: I5be167511e2c3eac6c63e603b82da84fb6b29eaa Reviewed-by: Julien Brianceau <jbriance@cisco.com> Reviewed-by: Michael Bruning <michael.bruning@digia.com>
* | Fix CLoop on big-endian machinesAllan Sandfeld Jensen2014-10-132-11/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Callee and ScopeChain are JSObject pointers and saved in the 64bit structure for JS registers. On a 32bit big-endian machine the actual pointer is offset by the PayloadOffset of 4 bytes, but on little-endian and 64bit architectures there is no offset. This patch fixes four places the payloadOffset was not correctly added on 32bit big-endian, and six places it was added on 64big big-endian when it shouldn't. Task-number: QTBUG-41896 Change-Id: I46b474bee9822b8040d1b7b2e8f31ce42e0adefe Reviewed-by: Jocelyn Turcotte <jocelyn.turcotte@digia.com>
* | Fix crash when a network-request is aborted while forwarding dataAllan Sandfeld Jensen2014-10-131-1/+4
| | | | | | | | | | | | | | | | | | | | Since we forward data block-wise a network request may be aborted while we are still forwarding received data. In that case we should stop the forwarding. Task-number: QTBUG-41877 Change-Id: Ia34bbdcacdc4eb70a2e814d3b4bada9844abbb83 Reviewed-by: Michael Bruning <michael.bruning@digia.com>
* | Load flags used by Qt modules when compile testing for a moduleAllan Sandfeld Jensen2014-10-133-0/+6
| | | | | | | | | | | | | | | | | | In order to pick-up the include path and library path passed to Qt configure, the configure tests needs to load the Qt build config. Change-Id: I881d5e87a08bf8dab5058802dc80ca58d0eed5b1 Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com>
* | Do not let TexureMapper create backing stores that are too largeAllan Sandfeld Jensen2014-10-133-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | TextureMapper without Coordinated Graphics will preallocate all tiles for a backing store. This makes it impossible to handle very large backing stores as they will cause memory exhaution. This patch disables composited layers with more pixels than 8192 squared, unless the ChromeClient has allowed large accelerated layers. Task-number: QTBUG-41404 Change-Id: Id71628da139f886aceb00732ae59dac2600ef863 Reviewed-by: Jocelyn Turcotte <jocelyn.turcotte@digia.com>
* | Support building QtTestBrowser without QtTestSupportv5.4.0-beta1Allan Sandfeld Jensen2014-10-103-6/+14
| | | | | | | | | | | | | | | | Use the same flags for QtTestBrowser that MiniBrowser uses to build without webkit's QtTestSupport. Change-Id: I0b31b1253b5014393a472286ffc6674efc64e753 Reviewed-by: Michael Bruning <michael.bruning@digia.com>
* | Fix uninitialized accessAllan Sandfeld Jensen2014-10-101-0/+8
| | | | | | | | | | | | | | | | | | | | When an identifier is made from an empty string, the constructor still tries accessing the first character to populate a cache. This leads to access of uninitialized data, and wrong data in the cache. This causes no wrong behavior though except unuseful data in the cache. Change-Id: Ice9f10b08306799b160f8b95b76bd056f29d228d Reviewed-by: Michael Bruning <michael.bruning@digia.com>