Timeline
Aug 12, 2013:
- 11:38 PM Changeset in webkit [153982] by
-
- 1 edit2 adds in trunk/LayoutTests
Fix null dereference in HTMLAnchorElement::sendPings when frame is not attached to a page
https://bugs.webkit.org/show_bug.cgi?id=119716
Reviewed by Darin Adler.
Add a regression test for r153975.
- fast/html/detach-frame-while-clicking-on-anchor-with-ping-expected.txt: Added.
- fast/html/detach-frame-while-clicking-on-anchor-with-ping.html: Added.
- 11:26 PM Changeset in webkit [153981] by
-
- 3 edits2 adds in trunk
ASSERTION FAILED: type() == Percent in WebCore::Length::percent
https://bugs.webkit.org/show_bug.cgi?id=116715
Reviewed by Darin Adler.
Source/WebCore:
Allow the Calculated type in the Length::percent method that returns the Length's
float value. This reflects the changes in r110148 that adjusted the Length::isPercent
method to return true for Length objects of either Percent or Calculated type.
Test: fast/css/calculated-length-as-percent-crash.html
- platform/Length.h:
(WebCore::Length::percent): Assert that Length::isPercent is true when Length::percent
is called, ensuring the Length's type is either Percent or Calculated.
LayoutTests:
Add the test case, as produced by the Fuzzinator, that was causing assertions
in Length::percent due to Length objects of the Calculated type not being treated
as holding a proper percent value in that method, despite Length::isPercent
returning true for Length objects of either Percent or the Calculated type.
- fast/css/calculated-length-as-percent-crash-expected.txt: Added.
- fast/css/calculated-length-as-percent-crash.html: Added.
- 11:26 PM Changeset in webkit [153980] by
-
- 2 edits in trunk/Tools
extract-localizable-strings output is not suited for parsing by Xcode
https://bugs.webkit.org/show_bug.cgi?id=119728
Reviewed by Darin Adler.
Added spaces where needed after “path:line:” to allow Xcode to recognize the file reference,
removed “ERROR” since messages that refer to a line in a file are shown as error by default,
added “warning: ” in messages that aren’t errors (i.e. don’t prevent the script from exiting
with 0).
- Scripts/extract-localizable-strings:
- 11:16 PM Changeset in webkit [153979] by
-
- 2 edits in trunk
[Autotools] Adjust the help string for the configure's --enable-optimizations flag
https://bugs.webkit.org/show_bug.cgi?id=119682
Reviewed by Martin Robinson.
- Source/autotools/ReadCommandLineArguments.m4: Remove the 'GCC only' part of the help
string for the --enable-optimizations flag. This is not true and can lead to confusion
since the optimizations can be applied when compiling with Clang as well.
- 10:58 PM Changeset in webkit [153978] by
-
- 18 edits in trunk/Source/WebCore
Move Clipboard::declareAndWriteDragImage to DragController
https://bugs.webkit.org/show_bug.cgi?id=117683
Reviewed by Brent Fulgham.
- dom/Clipboard.h: Removed declareAndWriteDragImage.
- page/DragController.cpp:
(WebCore::selectImageBeforeDragging): Half of the old prepareClipboardForImageDrag
is now refactored into this function.
(WebCore::DragController::startDrag): Where we called prepareClipboardForImageDrag,
instead call selectImageBeforeDragging and declareAndWriteDragImage. This is needed
because declareAndWriteDragImage needs to be a member function, for Mac at least.
- page/DragController.h: Added declareAndWriteDragImage.
- page/efl/DragControllerEfl.cpp:
(WebCore::DragController::declareAndWriteDragImage): Added empty placeholder.
It would be better to delete this file rather than having these placeholders.
I don't think EFL actually has drag support, and it makes work to have to keep
source files up to date that are really unused.
- page/gtk/DragControllerGtk.cpp:
(WebCore::DragController::declareAndWriteDragImage): Added. Calls Pasteboard::writeImage.
- page/mac/DragControllerMac.mm:
(WebCore::DragController::declareAndWriteDragImage): Added. Calls to DragClient, which
does the work.
- page/qt/DragControllerQt.cpp:
(WebCore::DragController::declareAndWriteDragImage): Added. Calls Pasteboard::writeImage.
- page/win/DragControllerWin.cpp:
(WebCore::DragController::declareAndWriteDragImage): Added. Uses Pasteboard functions to
do the work.
- platform/Clipboard.h: Changed the pasteboard function to return a non-const reference.
For now, it's not clear what const will mean for pasteboards, and it's currently not
possible to write to a const pasteboard, which we need to do in the new
DragController::declareAndWriteDragImage functions.
- platform/Pasteboard.h: Added a new writeMarkup function, implemented only for Windows
at this time.
- platform/efl/ClipboardEfl.cpp: Deleted declareAndWriteDragImage.
- platform/gtk/ClipboardGtk.cpp: Ditto.
- platform/ios/ClipboardIOS.h: Ditto.
- platform/ios/ClipboardIOS.mm: Ditto.
- platform/mac/ClipboardMac.mm: Ditto.
- platform/qt/ClipboardQt.cpp: Ditto.
- platform/win/ClipboardWin.cpp: Ditto.
- platform/win/PasteboardWin.cpp:
(WebCore::Pasteboard::writePlainTextToDataObject): Took out a pointless line of code.
(WebCore::Pasteboard::writeMarkup): Added. Currently implemented only for Windows.
This factors out a significant piece of DragController::declareAndWriteDragImage,
and will probably be handy for other platforms later.
- 10:55 PM Changeset in webkit [153977] by
-
- 5 edits in trunk
[EFL][WK1] URL printing code in DumpRenderTree doesn't match WTR or Mac DRT
https://bugs.webkit.org/show_bug.cgi?id=119585
Reviewed by Gyuyoung Kim.
Source/WebKit/efl:
- WebCoreSupport/DumpRenderTreeSupportEfl.cpp:
(DumpRenderTreeSupportEfl::provisionalURL):
Implemented to pass provisionalURL of main frame to DumpRenderTreeChrome.
- WebCoreSupport/DumpRenderTreeSupportEfl.h:
Tools:
- DumpRenderTree/efl/DumpRenderTreeChrome.cpp:
(pathSuitableForTestResult):
Return path string that's relative to main frame URL, or just file name if
the resource is not in the same directory subtree.
Referenced WebKitTestRunner code in r153852.
(descriptionSuitableForTestResult):
(DumpRenderTreeChrome::onFrameRedirectRequested):
(DumpRenderTreeChrome::onNewResourceRequest): Followed changes of r153903.
- 10:53 PM Changeset in webkit [153976] by
-
- 3 edits in trunk/Tools
extract-localizable-strings doesn't know how to verify an existing strings file
https://bugs.webkit.org/show_bug.cgi?id=119704
Reviewed by Darin Adler.
- Scripts/extract-localizable-strings: Added an --exceptions option for specifying the
exceptions file, changing the way “no exceptions file” is expressed from passing “-” as the
exceptions file name to omitting the --exceptions option. Added a --skip option, changing
the way directories to skip are specified from prefixing the directory with a “-” to
specifying it with --skip.
Finally, added a --verify option. When specified, the strings file is not written to, but
instead read in and checked for consistency with the comments, keys and values derived from
the source files and the exceptions file. Missing keys, unused keys, mismatching values and
mismatching comments are reported as errors.
- Scripts/update-webkit-localizable-strings: Updated for the new way of specifying
directories to skip and no exceptions file.
- 10:48 PM Changeset in webkit [153975] by
-
- 2 edits in trunk/Source/WebCore
Fix NULL de-refernce in HTMLAnchorElement::sendPings when settings doesn't exist
https://bugs.webkit.org/show_bug.cgi?id=119716
Reviewed by Darin Adler.
Merge https://chromium.googlesource.com/chromium/blink/+/d5783da353ab783e9994b8fbecd91880be5192a1
No new tests since the test in the Blink change doesn't reproduce crash on WebKit.
- html/HTMLAnchorElement.cpp:
(WebCore::HTMLAnchorElement::sendPings):
- 10:47 PM Changeset in webkit [153974] by
-
- 2 edits in trunk/Source/WebCore
Use the correct preprocessor in shouldRepaintFixedBackgroundsOnScroll()
https://bugs.webkit.org/show_bug.cgi?id=119710
Reviewed by Darin Adler.
No behavior change, no new tests needed.
- rendering/RenderObject.cpp: Remove unused parameter warning.
(WebCore::shouldRepaintFixedBackgroundsOnScroll):
- 8:44 PM Changeset in webkit [153973] by
-
- 8 edits in trunk
http/tests/security/sandboxed-iframe-invalid.html is flaky on Mac
https://bugs.webkit.org/show_bug.cgi?id=85522
Reviewed by Andy Estes.
Source/WebCore:
Drive-by FIXMEs and trivial fixes for WebCore issues I noticed while working
on this bug. No behavior change expected.
- loader/FrameLoader.cpp:
(WebCore::FrameLoader::submitForm):
(WebCore::FrameLoader::findFrameForNavigation):
- loader/NavigationScheduler.cpp:
(WebCore::ScheduledFormSubmission::fire):
Tools:
- DumpRenderTree/mac/UIDelegate.mm:
(-[UIDelegate webView:addMessageToConsole:withSource:]): Don't print console messages
when already done with a test. We already had this check in all UI delegate functions,
except for -webView:addMessageToConsole:withSource: somehow.
LayoutTests:
- http/tests/security/sandboxed-iframe-form-top-expected.txt: Console logging got
improved while this test was marked as skipped or flaky, updating the results.
- platform/mac/TestExpectations: Removed sandboxed-iframe-form-top.html, this test
no longer leaks output into later ones. The test was erroneously marked as flaky
instead of skipped, but now it's good.
- 8:34 PM Changeset in webkit [153972] by
-
- 3 edits in trunk/Source/WebCore
Fix variable name |end| -> |size| in EventTarget::fireEventListeners
https://bugs.webkit.org/show_bug.cgi?id=119715
Reviewed by Benjamin Poulain.
Merge https://chromium.googlesource.com/chromium/blink/+/f58b017539c48059bb2b88d18ee0ee3d14decb01
and also fix variable names in FiringEventIterator.
- dom/EventTarget.cpp:
(WebCore::EventTarget::removeEventListener):
(WebCore::EventTarget::fireEventListeners):
(WebCore::EventTarget::removeAllEventListeners):
- dom/EventTarget.h:
(WebCore::FiringEventIterator::FiringEventIterator):
- 6:35 PM Changeset in webkit [153971] by
-
- 1 edit33 adds in trunk/LayoutTests
[EFL] Unreviewed gardening.
Add a missing result after r152911.
- platform/efl/fast/multicol/pagination/BottomToTop-bt-expected.png: Added.
- platform/efl/fast/multicol/pagination/BottomToTop-bt-expected.txt: Added.
- platform/efl/fast/multicol/pagination/BottomToTop-lr-expected.png: Added.
- platform/efl/fast/multicol/pagination/BottomToTop-lr-expected.txt: Added.
- platform/efl/fast/multicol/pagination/BottomToTop-rl-expected.png: Added.
- platform/efl/fast/multicol/pagination/BottomToTop-rl-expected.txt: Added.
- platform/efl/fast/multicol/pagination/BottomToTop-tb-expected.png: Added.
- platform/efl/fast/multicol/pagination/BottomToTop-tb-expected.txt: Added.
- platform/efl/fast/multicol/pagination/LeftToRight-bt-expected.png: Added.
- platform/efl/fast/multicol/pagination/LeftToRight-bt-expected.txt: Added.
- platform/efl/fast/multicol/pagination/LeftToRight-lr-expected.png: Added.
- platform/efl/fast/multicol/pagination/LeftToRight-lr-expected.txt: Added.
- platform/efl/fast/multicol/pagination/LeftToRight-rl-expected.png: Added.
- platform/efl/fast/multicol/pagination/LeftToRight-rl-expected.txt: Added.
- platform/efl/fast/multicol/pagination/LeftToRight-tb-expected.png: Added.
- platform/efl/fast/multicol/pagination/LeftToRight-tb-expected.txt: Added.
- platform/efl/fast/multicol/pagination/RightToLeft-bt-expected.png: Added.
- platform/efl/fast/multicol/pagination/RightToLeft-bt-expected.txt: Added.
- platform/efl/fast/multicol/pagination/RightToLeft-lr-expected.png: Added.
- platform/efl/fast/multicol/pagination/RightToLeft-lr-expected.txt: Added.
- platform/efl/fast/multicol/pagination/RightToLeft-rl-expected.png: Added.
- platform/efl/fast/multicol/pagination/RightToLeft-rl-expected.txt: Added.
- platform/efl/fast/multicol/pagination/RightToLeft-tb-expected.png: Added.
- platform/efl/fast/multicol/pagination/RightToLeft-tb-expected.txt: Added.
- platform/efl/fast/multicol/pagination/TopToBottom-bt-expected.png: Added.
- platform/efl/fast/multicol/pagination/TopToBottom-bt-expected.txt: Added.
- platform/efl/fast/multicol/pagination/TopToBottom-lr-expected.png: Added.
- platform/efl/fast/multicol/pagination/TopToBottom-lr-expected.txt: Added.
- platform/efl/fast/multicol/pagination/TopToBottom-rl-expected.png: Added.
- platform/efl/fast/multicol/pagination/TopToBottom-rl-expected.txt: Added.
- platform/efl/fast/multicol/pagination/TopToBottom-tb-expected.png: Added.
- platform/efl/fast/multicol/pagination/TopToBottom-tb-expected.txt: Added.
- 6:12 PM Changeset in webkit [153970] by
-
- 8 edits in trunk/Source/WebCore
Encapsulate access to documentNamedItemMap and windowNamedItemMap
https://bugs.webkit.org/show_bug.cgi?id=119701
Reviewed by Darin Adler.
Encapsulate m_documentNamedItem and m_windowNamedItem by adding various methods on HTMLDocument.
This will help us doing more refactorings in the future.
- bindings/js/JSDOMWindowCustom.cpp:
(WebCore::namedItemGetter):
(WebCore::JSDOMWindow::getOwnPropertySlot):
(WebCore::JSDOMWindow::getOwnPropertySlotByIndex):
(WebCore::JSDOMWindow::getOwnPropertyDescriptor):
- bindings/js/JSHTMLDocumentCustom.cpp:
(WebCore::JSHTMLDocument::canGetItemsForName):
(WebCore::JSHTMLDocument::nameGetter):
- dom/Element.cpp:
(WebCore::Element::updateNameForDocument):
(WebCore::Element::updateIdForDocument):
- html/HTMLDocument.cpp:
(WebCore::HTMLDocument::addDocumentNamedItem): Added.
(WebCore::HTMLDocument::removeDocumentNamedItem): Added.
(WebCore::HTMLDocument::addWindowNamedItem): Added.
(WebCore::HTMLDocument::removeWindowNamedItem): Added.
- html/HTMLDocument.h:
(WebCore::HTMLDocument::documentNamedItem): Added.
(WebCore::HTMLDocument::hasDocumentNamedItem): Added.
(WebCore::HTMLDocument::documentNamedItemContainsMultipleElements): Added.
(WebCore::HTMLDocument::windowNamedItem): Added.
(WebCore::HTMLDocument::hasWindowNamedItem): Added.
(WebCore::HTMLDocument::windowNamedItemContainsMultipleElements): Added.
- html/HTMLImageElement.cpp:
(WebCore::HTMLImageElement::parseAttribute):
- html/HTMLObjectElement.cpp:
(WebCore::HTMLObjectElement::updateDocNamedItem):
- 6:10 PM Changeset in webkit [153969] by
-
- 2 edits in trunk/Source/WebKit2
Fixed an off-by-(sizeof(char*) - 1) bug.
Reviewed by Sam Weinig.
- Shared/EntryPointUtilities/mac/XPCService/XPCServiceMain.Development.mm:
(WebKit::XPCServiceEventHandler): Allocate a sufficiently large buffer.
- 5:35 PM Changeset in webkit [153968] by
-
- 2 edits in trunk/Source/WebCore
[CMake] Use the correct list of files when compiling ANGLESupport on Windows
https://bugs.webkit.org/show_bug.cgi?id=119439
Reviewed by Gyuyoung Kim.
- CMakeLists.txt: Use ossource_win.cpp instead of ossource_posix.cpp on Windows platforms.
- 5:28 PM Changeset in webkit [153967] by
-
- 6 edits in trunk/Source/JavaScriptCore
Remove CodeBlock's notion of adding identifiers entirely
https://bugs.webkit.org/show_bug.cgi?id=119708
Reviewed by Geoffrey Garen.
Remove addAdditionalIdentifier entirely, including the bogus assertion.
Move the addition of identifiers to DFGPlan::reallyAdd
- bytecode/CodeBlock.h:
- dfg/DFGDesiredIdentifiers.cpp:
(JSC::DFG::DesiredIdentifiers::reallyAdd):
- dfg/DFGDesiredIdentifiers.h:
- dfg/DFGPlan.cpp:
(JSC::DFG::Plan::reallyAdd):
(JSC::DFG::Plan::finalize):
- dfg/DFGPlan.h:
- 5:20 PM Changeset in webkit [153966] by
-
- 3 edits4 deletes in trunk/LayoutTests
[EFL] Unreviewed gardening.
- platform/efl/TestExpectations: Added pasteboard after r152185.
- platform/efl/editing/deleting/delete-br-002-expected.txt: Updated after r152478.
- platform/efl/editing/execCommand/outdent-blockquote-test1-expected.txt:
Removed platform specific expectations after r152494.
- platform/efl/editing/execCommand/outdent-blockquote-test2-expected.txt: Ditto.
- platform/efl/editing/execCommand/outdent-blockquote-test3-expected.txt: Ditto.
- platform/efl/editing/execCommand/outdent-blockquote-test4-expected.txt: Ditto.
- 4:56 PM Changeset in webkit [153965] by
-
- 2 edits in trunk/Source/JavaScriptCore
Build fix
- 4:42 PM Changeset in webkit [153964] by
-
- 2 edits in trunk/Tools
https://bugs.webkit.org/show_bug.cgi?id=119693
[Mac] security/contentSecurityPolicy/connect-src-websocket-blocked.html is flaky
because of connect-src-websocket-allowed.html
Reviewed by Darin Adler.
Starting a load of about:blank and immediately stopping it cancels outstanding
resource loads, but the page has to transition to committed state to have timers
or WebSockets stopped too.
- DumpRenderTree/mac/DumpRenderTree.mm: (runTest): Use a synchronous load to
ensure that transition to committed state happens before we start the next test.
- 4:41 PM Changeset in webkit [153963] by
-
- 3 edits in trunk/Source/JavaScriptCore
Move additionalIdentifiers into DFGCommonData as only the optimising JITs use them
https://bugs.webkit.org/show_bug.cgi?id=119705
Reviewed by Geoffrey Garen.
Relatively trivial refactoring
- bytecode/CodeBlock.h:
(JSC::CodeBlock::numberOfAdditionalIdentifiers):
(JSC::CodeBlock::addAdditionalIdentifier):
(JSC::CodeBlock::identifier):
(JSC::CodeBlock::numberOfIdentifiers):
- dfg/DFGCommonData.h:
- 3:39 PM Changeset in webkit [153962] by
-
- 10 edits in trunk/Source/JavaScriptCore
Stop making unnecessary copy of CodeBlock Identifier Vector
https://bugs.webkit.org/show_bug.cgi?id=119702
Reviewed by Michael Saboff.
Make CodeBlock simply use a separate Vector for additional Identifiers
and use the UnlinkedCodeBlock for the initial set of identifiers.
- bytecode/CodeBlock.cpp:
(JSC::CodeBlock::printGetByIdOp):
(JSC::dumpStructure):
(JSC::dumpChain):
(JSC::CodeBlock::printGetByIdCacheStatus):
(JSC::CodeBlock::printPutByIdOp):
(JSC::CodeBlock::dumpBytecode):
(JSC::CodeBlock::CodeBlock):
(JSC::CodeBlock::shrinkToFit):
- bytecode/CodeBlock.h:
(JSC::CodeBlock::numberOfIdentifiers):
(JSC::CodeBlock::numberOfAdditionalIdentifiers):
(JSC::CodeBlock::addAdditionalIdentifier):
(JSC::CodeBlock::identifier):
- dfg/DFGDesiredIdentifiers.cpp:
(JSC::DFG::DesiredIdentifiers::reallyAdd):
- jit/JIT.h:
- jit/JITOpcodes.cpp:
(JSC::JIT::emitSlow_op_get_arguments_length):
- jit/JITPropertyAccess.cpp:
(JSC::JIT::emit_op_get_by_id):
(JSC::JIT::compileGetByIdHotPath):
(JSC::JIT::emitSlow_op_get_by_id):
(JSC::JIT::compileGetByIdSlowCase):
(JSC::JIT::emitSlow_op_put_by_id):
- jit/JITPropertyAccess32_64.cpp:
(JSC::JIT::emit_op_get_by_id):
(JSC::JIT::compileGetByIdHotPath):
(JSC::JIT::compileGetByIdSlowCase):
- jit/JITStubs.cpp:
(JSC::DEFINE_STUB_FUNCTION):
- llint/LLIntSlowPaths.cpp:
(JSC::LLInt::LLINT_SLOW_PATH_DECL):
- 3:05 PM Changeset in webkit [153961] by
-
- 4 edits in trunk/Source/WebCore
[cairo] memory corruption with putImageData and accelerated canvas.
https://bugs.webkit.org/show_bug.cgi?id=118621
Reviewed by Martin Robinson.
When we have an accelerated imageBuffer, and we putByteArray with a
non zero destPoint, we create temporary surface of sourceSize. Then,
we write to image data with an offset. This results in an out of bound
write.
In order to avoid that, we use a 0 offset when writing data.
In addition, we also create image with the minimal needed size in
getImageData, and also use a 0 offset. This decrease data transfer
between cpu and gpu
Instead of the map/unmap mechanism, perform an explicit surface copy
when needed. This change also removes an unneeded glReadPixels at the
start of putByteArray
Add two parameters to copyRectFromOneSurfaceToAnother: destination
offset, and cairo drawing operator.
No new tests. Accelerated canvas is not enabled for testing yet.
- platform/graphics/cairo/CairoUtilities.cpp:
(WebCore::copyRectFromOneSurfaceToAnother):
- platform/graphics/cairo/CairoUtilities.h:
- platform/graphics/cairo/ImageBufferCairo.cpp:
(WebCore::copySurfaceToImageAndAdjustRect):
(WebCore::getImageData):
(WebCore::ImageBuffer::putByteArray):
- 2:55 PM Changeset in webkit [153960] by
-
- 3 edits in trunk/LayoutTests
Layout Test fast/overflow/scrollbar-click-retains-focus.html is failing on Apple Mac WK1 and GTK.
https://bugs.webkit.org/show_bug.cgi?id=104104
Add back test expectation for Mac since it's still failing on ML WK1 bots.
- platform/mac/TestExpectations:
- platform/mac-wk2/TestExpectations:
- 2:50 PM Changeset in webkit [153959] by
-
- 6 edits in trunk/Source
Use const AtomicStringImpl* in DocumentOrderedMap
https://bugs.webkit.org/show_bug.cgi?id=119700
Reviewed by Benjamin Poulain.
Use const AtomicStringImpl* in DocumentOrderedMap. It caught at least one bug that HTMLNameCollection'
subclasses' nodeMatches were taking const AtomicString& instead of AtomicStringImpl*. This was forcing
DocumentOrderedMap::getElementByWindowNamedItem and DocumentOrderedMap::getElementByDocumentNamedItem
to create temporary AtomicStrings.
- dom/DocumentOrderedMap.cpp:
(WebCore::keyMatchesId):
(WebCore::keyMatchesName):
(WebCore::keyMatchesMapName):
(WebCore::keyMatchesLowercasedMapName):
(WebCore::keyMatchesLabelForAttribute):
(WebCore::keyMatchesWindowNamedItem):
(WebCore::keyMatchesDocumentNamedItem):
(WebCore::DocumentOrderedMap::add):
(WebCore::DocumentOrderedMap::remove):
(WebCore::DocumentOrderedMap::get):
(WebCore::DocumentOrderedMap::getElementById):
(WebCore::DocumentOrderedMap::getElementByName):
(WebCore::DocumentOrderedMap::getElementByMapName):
(WebCore::DocumentOrderedMap::getElementByLowercasedMapName):
(WebCore::DocumentOrderedMap::getElementByLabelForAttribute):
(WebCore::DocumentOrderedMap::getElementByWindowNamedItem):
(WebCore::DocumentOrderedMap::getElementByDocumentNamedItem):
(WebCore::DocumentOrderedMap::getAllElementsById):
- dom/DocumentOrderedMap.h:
(WebCore::DocumentOrderedMap::containsSingle):
(WebCore::DocumentOrderedMap::contains):
(WebCore::DocumentOrderedMap::containsMultiple):
- html/HTMLNameCollection.cpp:
(WebCore::WindowNameCollection::nodeMatches):
(WebCore::DocumentNameCollection::nodeMatches):
- html/HTMLNameCollection.h:
(WebCore::WindowNameCollection::nodeMatches):
(WebCore::DocumentNameCollection::nodeMatches):
- 2:37 PM Changeset in webkit [153958] by
-
- 6 edits2 adds in trunk
The LayoutMilestones didLayout callback should only fire for the main frame
https://bugs.webkit.org/show_bug.cgi?id=119457
Reviewed by Sam Weinig.
Source/WebCore:
This patch makes it so that FrameLoader::didLayout(LayoutMilestones) is only ever
called for the main frame. Technically this will change existing API for
didFirstVisuallyNonEmptyLayoutInFrame and didFirstLayoutInFrame. Clients used to
be able to register for those callbacks for any frame they wanted, however, now
they will only ever get the callback when it applies to the main frame. We think
that this is acceptable since we don’t know of any clients that ever used those
APIs for a non-main frame. This change also makes nothing but sense for the
didLayout(LayoutMilestones) callback, since that is a Page-level concept, so it
should only fire for the main frame.
Only call didLayout for the main frame.
- dom/Document.cpp:
(WebCore::Document::setVisualUpdatesAllowed):
Assert that this is the main frame.
- loader/FrameLoader.cpp:
(WebCore::FrameLoader::didLayout):
Only call didLayout for the main frame.
- page/FrameView.cpp:
(WebCore::FrameView::performPostLayoutTasks):
Tools:
Now that the didLayout callback only fires for the main frame, we want to make
sure it will fire even when all of the content is in a subframe.
- TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
- TestWebKitAPI/Tests/WebKit2/LayoutMilestonesWithAllContentInFrame.cpp: Added.
(TestWebKitAPI::didLayout):
(TestWebKitAPI::TEST):
- TestWebKitAPI/Tests/WebKit2/all-content-in-one-iframe.html: Added.
- 2:02 PM Changeset in webkit [153957] by
-
- 2 edits in branches/safari-537-branch/Source/WebCore
Merged r153917. <rdar://problem/14002715>
- 1:54 PM Changeset in webkit [153956] by
-
- 16 edits in trunk
Remove didNewFirstVisuallyNonEmptyLayout from WebKit2
https://bugs.webkit.org/show_bug.cgi?id=119658
Reviewed by Darin Adler.
Source/WebKit2:
Removing this function entirely will change some functionality in the Nightlies,
but we think that is an acceptable difference for the Nightlies. If is does cause
a problem though, we can roll out this patch and deprecate the function instead.
- UIProcess/API/C/WKPage.h:
- UIProcess/WebLoaderClient.cpp:
- UIProcess/WebLoaderClient.h:
- UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::initializeLoaderClient):
- UIProcess/WebPageProxy.h:
- UIProcess/WebPageProxy.messages.in:
- WebProcess/InjectedBundle/API/c/WKBundlePage.h:
- WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.cpp:
- WebProcess/InjectedBundle/InjectedBundlePageLoaderClient.h:
- WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:
(WebKit::WebFrameLoaderClient::dispatchDidLayout):
- WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::initializeInjectedBundleLoaderClient):
Tools:
- MiniBrowser/mac/WK2BrowserWindowController.m:
(-[WK2BrowserWindowController awakeFromNib]):
- WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp:
(WTR::InjectedBundlePage::InjectedBundlePage):
- WebKitTestRunner/TestController.cpp:
(WTR::TestController::createWebViewWithOptions):
- 12:47 PM Changeset in webkit [153955] by
-
- 15 edits2 adds in trunk
Define DOM_KEY_LOCATION_* constants on KeyboardEvent
https://bugs.webkit.org/show_bug.cgi?id=119341
Reviewed by Ryosuke Niwa.
Source/WebCore:
Add support for the DOM_KEY_LOCATION_* constants on KeyboardEvent as per
the specification:
http://www.w3.org/TR/DOM-Level-3-Events/#events-KeyboardEvent
Those constants are already supported by IE10, Firefox 22 and since
recently Blink.
Note that the following constants are not supported / exposed yet:
DOM_KEY_LOCATION_MOBILE and DOM_KEY_LOCATION_JOYSTICK.
Test: fast/events/keyboardevent-location-constants.html
- dom/KeyboardEvent.cpp:
(WebCore::keyLocationCode):
(WebCore::KeyboardEvent::KeyboardEvent):
- dom/KeyboardEvent.h:
- dom/KeyboardEvent.idl:
LayoutTests:
Make use of the new DOM_KEY_LOCATION_* constants in the KeyboardEvent
test cases instead of hardcoding their values.
- fast/events/constructors/keyboard-event-constructor-expected.txt:
- fast/events/constructors/keyboard-event-constructor.html:
- fast/events/js-keyboard-event-creation-expected.txt:
- fast/events/js-keyboard-event-creation.html:
- fast/events/keyboardevent-location-constants-expected.txt: Added.
- fast/events/keyboardevent-location-constants.html: Added.
- fast/events/keydown-leftright-keys-expected.txt:
- fast/events/keydown-leftright-keys.html:
- fast/events/keydown-numpad-keys-expected.txt:
- fast/events/script-tests/keydown-numpad-keys.js:
(testKeyEventWithLocation):
- platform/efl/fast/events/js-keyboard-event-creation-expected.txt:
- platform/gtk/fast/events/js-keyboard-event-creation-expected.txt:
- 12:41 PM Changeset in webkit [153954] by
-
- 2 edits in trunk/Tools
Remove glu's libtest rule from checker.py
https://bugs.webkit.org/show_bug.cgi?id=119674
Reviewed by Darin Adler.
- Scripts/webkitpy/style/checker.py: It is no longer used after r148032.
- 12:24 PM Changeset in webkit [153953] by
-
- 3 edits in trunk/LayoutTests
Multiple CSP tests complain about undefined strings in echo-object-data.pl and echo-script-src.pl
https://bugs.webkit.org/show_bug.cgi?id=119692
Reviewed by Darin Adler.
- http/tests/security/contentSecurityPolicy/resources/echo-object-data.pl:
- http/tests/security/contentSecurityPolicy/resources/echo-script-src.pl:
Check whether optional CGI parameters are provided.
- 12:13 PM Changeset in webkit [153952] by
-
- 11 edits in trunk/Source/WebCore
dataLog dumpers for WebCore's basic geometry types
https://bugs.webkit.org/show_bug.cgi?id=119636
Reviewed by Filip Pizlo.
- platform/graphics/FloatPoint.cpp:
(WebCore::FloatPoint::dump):
- platform/graphics/FloatPoint.h:
- platform/graphics/FloatRect.cpp:
(WebCore::FloatRect::dump):
- platform/graphics/FloatRect.h:
- platform/graphics/FloatSize.cpp:
(WebCore::FloatSize::dump):
- platform/graphics/FloatSize.h:
- platform/graphics/IntPoint.h:
(WebCore::IntPoint::dump):
- platform/graphics/IntRect.cpp:
(WebCore::IntRect::dump):
- platform/graphics/IntRect.h:
- platform/graphics/IntSize.h:
(WebCore::IntSize::dump):
- 11:41 AM Changeset in webkit [153951] by
-
- 2 edits in trunk/Source/WTF
Broken build with build-webkit --no-webgl
https://bugs.webkit.org/show_bug.cgi?id=119272
Patch by Ragner Magalhaes <ranger.n@samsung.com> on 2013-08-12
Reviewed by Alexandru Chiculita.
CSS_SHADERS requires WEBGL enabled.
Add error message to dependency failure.
- wtf/FeatureDefines.h:
- 10:47 AM Changeset in webkit [153950] by
-
- 2 edits3 adds in trunk/Source/WTF
[iOS] isMainThread implementation with USE(WEB_THREAD)
https://bugs.webkit.org/show_bug.cgi?id=119644
Reviewed by Benjamin Poulain.
On iOS isMainThread() means that the current thread is either the
MainThread or WebThread and the current thread has the WebThreadLock.
- wtf/ios/WebCoreThread.cpp: Added.
- wtf/ios/WebCoreThread.h: Added.
- wtf/mac/MainThreadMac.mm:
(WTF::isMainThread):
- 10:32 AM Changeset in webkit [153949] by
-
- 2 edits in trunk/Source/WebKit2
[Qt] [WebKit2] Remove duplicate RAM Size calculation and reuse WTF::ramSize()
https://bugs.webkit.org/show_bug.cgi?id=119663
Patch by Arunprasad Rajkumar <arurajku@cisco.com> on 2013-08-12
Reviewed by Sam Weinig.
- WebProcess/qt/WebProcessQt.cpp:
(WebKit::WebProcess::platformSetCacheModel):
- 10:23 AM Changeset in webkit [153948] by
-
- 3 edits2 adds in trunk
Shadows don't support viewport units
https://bugs.webkit.org/show_bug.cgi?id=119649
Reviewed by Darin Adler.
Source/WebCore:
We don't yet support viewport units in shadows, so
fail parsing if we see one. The bug to fix this
completely is: https://webkit.org/b/119650
Test: fast/css/shadow-viewport-units.html
- css/CSSParser.cpp:
(WebCore::CSSParser::parseShadow): Fail if we get one of
vh, vw, vmin, vmax.
LayoutTests:
Test that exercises a viewport unit in text-shadow to make sure we don't parse it.
- fast/css/shadow-viewport-units-expected.txt: Added.
- fast/css/shadow-viewport-units.html: Added.
- 10:16 AM Changeset in webkit [153947] by
-
- 4 edits in trunk/Source
[GTK] Don't load GAIL when using GTK 3.2 or greater
https://bugs.webkit.org/show_bug.cgi?id=72735
Patch by Anton Obzhirov <Anton Obzhirov> on 2013-08-12
Reviewed by Carlos Garcia Campos.
Source/WebKit/gtk:
- tests/testatk.c:
(initializeTestingFramework):
(main):
Source/WebKit2:
- UIProcess/API/gtk/tests/AccessibilityTestServer.cpp:
(main):
- 9:52 AM Changeset in webkit [153946] by
-
- 2 edits in trunk/Source/WebCore
[Curl] Cookie is not set when url string is unicode.
https://bugs.webkit.org/show_bug.cgi?id=119545
Patch by peavo@outlook.com <peavo@outlook.com> on 2013-08-12
Reviewed by Brent Fulgham.
When the method setCookiesFromDOM is called with an unicode url parameter, the cookie is not correctly set.
We need to convert the cookie string to 8 bit, before passing it to the Curl api function.
- platform/network/curl/CookieJarCurl.cpp:
(WebCore::setCookiesFromDOM): Convert cookie string to 8 bit, if needed.
- 9:49 AM Changeset in webkit [153945] by
-
- 2 edits in trunk/Source/WebCore
Revert r153912 because it broke some track tests.
Rubber-stamped by Antti Koivisto.
- platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:
(WebCore::MediaPlayerPrivateAVFoundationObjC::tracksChanged):
(WebCore::MediaPlayerPrivateAVFoundationObjC::languageOfPrimaryAudioTrack):
- 9:36 AM Changeset in webkit [153944] by
-
- 3 edits in trunk/Source/WebCore
[Curl] Crash when starting second download.
https://bugs.webkit.org/show_bug.cgi?id=119620
Patch by peavo@outlook.com <peavo@outlook.com> on 2013-08-12
Reviewed by Brent Fulgham.
When a second download is started, a crash sometimes occurs in Curl.
This happens because the Curl easy handle is deleted and removed from the Curl multi handle twice.
This is solved by adding an array to keep track of active downloads to avoid double deletion.
- platform/network/curl/CurlDownload.cpp:
(WebCore::CurlDownloadManager::CurlDownloadManager): Removed active download count member.
(WebCore::CurlDownloadManager::getActiveDownloadCount): Use active download array to return active download count.
(WebCore::CurlDownloadManager::updateHandleList): Remove Curl handles before adding, in case Curl reuses handles.
(WebCore::CurlDownloadManager::addToCurl): Add Curl handle to active download array.
(WebCore::CurlDownloadManager::removeFromCurl): Check if Curl handle is in active download array to avoid double deletion.
(WebCore::CurlDownloadManager::downloadThread): Removed active download count member.
- platform/network/curl/CurlDownload.h: Added array of active downloads member, and removed download count member.
- 9:32 AM Changeset in webkit [153943] by
-
- 2 edits in trunk/Source/WebKit
[Win] TestWebKitAPI fails to link.
https://bugs.webkit.org/show_bug.cgi?id=119619
Patch by peavo@outlook.com <peavo@outlook.com> on 2013-08-12
Reviewed by Brent Fulgham.
- WebKit.vcxproj/WebKitExportGenerator/WebKitExports.def.in: Export needed symbol.
- 8:52 AM Changeset in webkit [153942] by
-
- 17 edits in trunk/Source/WebCore
Add ElementTraversal::next/previousSibling
https://bugs.webkit.org/show_bug.cgi?id=119681
Reviewed by Andreas Kling.
Add nextSibling/previousSibling/lastWithin to ElementTraversal namespace.
- Use them to replace internal use of Element::next/previousElementSibling DOM API for traversal.
- Replace some Node::nextSibling() usage with ElementTraversal::nextSibling() for tighter code.
- Replace ElementTraversal::nextSkippingChildren where it is used to traverse over immediate children only.
- css/SelectorChecker.cpp:
(WebCore::isFirstChildElement):
(WebCore::isLastChildElement):
(WebCore::isFirstOfType):
(WebCore::isLastOfType):
(WebCore::countElementsBefore):
(WebCore::countElementsOfTypeBefore):
(WebCore::countElementsAfter):
(WebCore::countElementsOfTypeAfter):
- dom/Element.cpp:
(WebCore::Element::lastElementChild):
(WebCore::Element::previousElementSibling):
(WebCore::Element::nextElementSibling):
- dom/Element.h:
- dom/ElementTraversal.h:
(WebCore::ElementTraversal::lastElementWithinTemplate):
(WebCore::ElementTraversal::lastWithin):
(WebCore::ElementTraversal::traverseNextSiblingElementTemplate):
(WebCore::ElementTraversal::nextSibling):
(WebCore::ElementTraversal::traversePreviousSiblingElementTemplate):
(WebCore::ElementTraversal::previousSibling):
- editing/ApplyStyleCommand.cpp:
(WebCore::isSpanWithoutAttributesOrUnstyledStyleSpan):
(WebCore::ApplyStyleCommand::cleanupUnstyledAppleStyleSpans):
- editing/CompositeEditCommand.cpp:
(WebCore::CompositeEditCommand::breakOutOfEmptyListItem):
- editing/IndentOutdentCommand.cpp:
(WebCore::IndentOutdentCommand::tryIndentingAsListItem):
- editing/InsertListCommand.cpp:
(WebCore::InsertListCommand::mergeWithNeighboringLists):
- editing/markup.cpp:
(WebCore::createContextualFragment):
- html/HTMLCollection.cpp:
(WebCore::firstMatchingChildElement):
(WebCore::nextMatchingSiblingElement):
(WebCore::HTMLCollection::traverseNextElement):
(WebCore::HTMLCollection::traverseForwardToOffset):
- html/HTMLFieldSetElement.cpp:
(WebCore::HTMLFieldSetElement::childrenChanged):
(WebCore::HTMLFieldSetElement::legend):
- html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::finishParsingChildren):
- html/HTMLObjectElement.cpp:
(WebCore::HTMLObjectElement::containsJavaApplet):
- rendering/FilterEffectRenderer.cpp:
(WebCore::FilterEffectRenderer::buildReferenceFilter):
- style/StyleResolveTree.cpp:
(WebCore::Style::resolveTree):
- svg/SVGStyledElement.cpp:
(WebCore::SVGStyledElement::title):
- 5:18 AM Changeset in webkit [153941] by
-
- 7 edits in trunk
[Qt] Add Support for canvas blend modes
https://bugs.webkit.org/show_bug.cgi?id=100072
Reviewed by Jocelyn Turcotte.
Source/WebCore:
Implement basic support for the blend-modes supported by QPainter.
Covered by existing tests.
- platform/graphics/qt/GraphicsContextQt.cpp:
(WebCore::toQtCompositionMode):
(WebCore::GraphicsContext::setPlatformCompositeOperation):
- platform/graphics/qt/ImageBufferQt.cpp:
(WebCore::ImageBuffer::draw):
- platform/graphics/qt/ImageQt.cpp:
(WebCore::BitmapImage::draw):
- platform/graphics/qt/StillImageQt.cpp:
(WebCore::StillImage::draw):
LayoutTests:
Unskip tests that now passes.
- platform/qt/TestExpectations:
- 4:40 AM Changeset in webkit [153940] by
-
- 2 edits in trunk/Source/WebCore
Buildfix for !ENABLE(SVG). Fix warning by adding guard after r153901:
void WebCore::addString(WebCore::FeatureSet&, const char*)'
defined but not used [-Werror=unused-function]
Patch by Gabor Abraham <abrhm@inf.u-szeged.hu> on 2013-08-12
Reviewed by Csaba Osztrogonác.
- dom/DOMImplementation.cpp:
- 4:21 AM Changeset in webkit [153939] by
-
- 34 edits1 add in trunk/Source/WebCore
Move ElementTraversal to ElementTraversal.h
https://bugs.webkit.org/show_bug.cgi?id=119678
Rubber-stamped by Andreas Kling.
Move ElementTraversal namespace from NodeTraversal.h to a correctly named file.
- GNUmakefile.list.am:
- Target.pri:
- WebCore.vcxproj/WebCore.vcxproj:
- WebCore.xcodeproj/project.pbxproj:
- accessibility/AccessibilityRenderObject.cpp:
- css/StyleInvalidationAnalysis.cpp:
- dom/Document.cpp:
- dom/DocumentOrderedMap.cpp:
- dom/Element.cpp:
- dom/ElementTraversal.h: Added.
(WebCore::ElementTraversal::firstElementWithinTemplate):
(WebCore::ElementTraversal::firstWithin):
(WebCore::ElementTraversal::traverseNextElementTemplate):
(WebCore::ElementTraversal::next):
(WebCore::ElementTraversal::traverseNextElementSkippingChildrenTemplate):
(WebCore::ElementTraversal::nextSkippingChildren):
(WebCore::ElementTraversal::previousIncludingPseudo):
(WebCore::ElementTraversal::nextIncludingPseudo):
(WebCore::ElementTraversal::nextIncludingPseudoSkippingChildren):
(WebCore::ElementTraversal::pseudoAwarePreviousSibling):
- dom/Node.cpp:
- dom/NodeTraversal.h:
- dom/SelectorQuery.cpp:
- dom/TreeScope.cpp:
- dom/VisitedLinkState.cpp:
- editing/FrameSelection.cpp:
- editing/markup.cpp:
- html/HTMLCollection.cpp:
- html/HTMLFieldSetElement.cpp:
- html/HTMLFormElement.cpp:
- html/HTMLLabelElement.cpp:
- html/HTMLLegendElement.cpp:
- html/HTMLMapElement.cpp:
- html/HTMLObjectElement.cpp:
- html/HTMLSelectElement.cpp:
- html/shadow/ContentDistributor.cpp:
- page/FocusController.cpp:
- rendering/RenderCounter.cpp:
- rendering/RenderListItem.cpp:
- svg/SVGSVGElement.cpp:
- svg/SVGStyledElement.cpp:
- svg/SVGUseElement.cpp:
- svg/animation/SMILTimeContainer.cpp:
- svg/graphics/SVGImage.cpp:
- 3:25 AM Changeset in webkit [153938] by
-
- 4 edits in trunk/Source/WebCore
Move some Document recalcStyle code to StyleResolveTree
https://bugs.webkit.org/show_bug.cgi?id=119676
Reviewed by Andreas Kling.
Move the code that resolves the document and child styles to Style::resolveTree(Document*).
- dom/Document.cpp:
(WebCore::Document::recalcStyle):
- style/StyleResolveTree.cpp:
(WebCore::Style::resolveTree):
- style/StyleResolveTree.h:
- 3:22 AM Changeset in webkit [153937] by
-
- 2 edits in trunk/Source/WebCore
Resource leak related to gstreamer and videos
https://bugs.webkit.org/show_bug.cgi?id=109350
Reviewed by Eric Carlson.
Since active DOM object are only stopped once and can not be restarted, we can
unreference the media player there, instead of waiting for garbage collection.
If the r152778 is rolled back, this also breaks a circular reference between
GStreamer and WebKit which prevented garbage collecting from triggering.
- html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::stop):
- 2:38 AM Changeset in webkit [153936] by
-
- 2 edits in trunk/Source/WebCore
[Qt] Unreviewed buildfix for webaudio.
- Target.pri: Add missing files
- 2:17 AM Changeset in webkit [153935] by
-
- 2 edits in trunk/Source/WebKit/win
More Windows build juice.
- WebView.cpp:
(WebView::notifyPreferencesChanged):
- 1:55 AM Changeset in webkit [153934] by
-
- 3 edits in trunk/Source/WebCore
Some build juice for the Windows bots after r153926 + r153927.
- html/shadow/MediaControlElements.cpp:
- platform/graphics/win/MediaPlayerPrivateQuickTimeVisualContext.cpp:
(WebCore::MediaPlayerPrivateQuickTimeVisualContext::supportsFullscreen):
(WebCore::MediaPlayerPrivateQuickTimeVisualContext::setUpCookiesForQuickTime):
- 1:20 AM Changeset in webkit [153933] by
-
- 12 edits13 adds in trunk/LayoutTests
[Qt] Unreviewed gardening. Rebaselining after r153903.
- platform/qt-5.0-wk1/fast/loader/main-document-url-for-non-http-loads-expected.txt: Added.
- platform/qt-5.0-wk1/http/tests/cache/iframe-304-crash-expected.txt: Added.
- platform/qt-5.0-wk1/http/tests/loading/307-after-303-after-post-expected.txt: Added.
- platform/qt-5.0-wk1/http/tests/loading/redirect-methods-expected.txt: Added.
- platform/qt-5.0-wk1/http/tests/misc/favicon-loads-with-images-disabled-expected.txt: Added.
- platform/qt-5.0-wk1/http/tests/security/XFrameOptions/x-frame-options-allowall-expected.txt: Added.
- platform/qt-5.0-wk1/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-parent-same-origin-allow-expected.txt: Added.
- platform/qt-5.0-wk1/http/tests/security/XFrameOptions/x-frame-options-invalid-expected.txt: Added.
- platform/qt-5.0-wk1/http/tests/security/XFrameOptions/x-frame-options-multiple-headers-sameorigin-allow-expected.txt: Added.
- platform/qt-5.0-wk1/http/tests/security/XFrameOptions/x-frame-options-parent-same-origin-allow-expected.txt: Added.
- platform/qt-5.0-wk2/compositing/geometry/preserve-3d-switching-expected.txt:
- platform/qt-5.0-wk2/http/tests/misc/will-send-request-returns-null-on-redirect-expected.txt:
- platform/qt-5.0-wk2/http/tests/misc/window-dot-stop-expected.txt:
- platform/qt-5.0-wk2/http/tests/security/XFrameOptions/x-frame-options-deny-expected.txt:
- platform/qt-5.0-wk2/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-expected.txt:
- platform/qt-5.0-wk2/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-in-body-expected.txt:
- platform/qt-5.0-wk2/http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-parent-same-origin-deny-expected.txt:
- platform/qt-5.0-wk2/http/tests/security/XFrameOptions/x-frame-options-multiple-headers-conflict-expected.txt:
- platform/qt-5.0-wk2/http/tests/security/XFrameOptions/x-frame-options-multiple-headers-sameorigin-deny-expected.txt:
- platform/qt-5.0-wk2/http/tests/security/XFrameOptions/x-frame-options-parent-same-origin-deny-expected.txt:
- platform/qt-5.0-wk2/loader/go-back-cached-main-resource-expected.txt:
- 12:57 AM Changeset in webkit [153932] by
-
- 3 edits in trunk/LayoutTests
Update spellcheck-attribute.html description.
https://bugs.webkit.org/show_bug.cgi?id=119486
Reviewed by Ryosuke Niwa.
r70512 r72782 added new fields to spellcheck-attribute.html to verify
spellcheck attribute.
Update the test to be more descriptive by dumping additional information
whether the misspelled word should be marked or not.
- editing/spelling/spellcheck-attribute-expected.txt:
- editing/spelling/spellcheck-attribute.html:
- 12:12 AM Changeset in webkit [153931] by
-
- 1 copy in releases/WebKitGTK/webkit-2.1.4
Tagging the WebKitGTK+ 2.1.4 release
Aug 11, 2013:
- 4:15 PM Changeset in webkit [153930] by
-
- 2 edits in trunk/Source/WebCore
Move RenderMathMLSpace.h/cpp to the right group in the Xcode project.
Reviewed by Dean Jackson.
- WebCore.xcodeproj/project.pbxproj:
- 4:13 PM Changeset in webkit [153929] by
-
- 2 edits in trunk/Source/WebInspectorUI
Update localizedStrings - I think I checked in an unmerged version
in the previous commit.
- Localizations/en.lproj/localizedStrings.js:
- 4:10 PM Changeset in webkit [153928] by
-
- 6 edits4 adds in trunk/Source/WebInspectorUI
Bootstrap canvas profiler
https://bugs.webkit.org/show_bug.cgi?id=119652
<rdar://problem/14703665>
Reviewed by Joseph Pecoraro.
Add enough to get an empty Canvas profile recording
and displayed.
- Localizations/en.lproj/localizedStrings.js: Add canvas strings.
- UserInterface/CanvasProfileObject.js: Added. Mostly copied from CSS Selector Profiles.
- UserInterface/CanvasProfileType.js: Added. Ditto.
- UserInterface/CanvasDataGridNode.js: Added. Ditto.
- UserInterface/CanvasProfileView.js: Added. Ditto.
- UserInterface/ContentView.js:
(WebInspector.ContentView): Add support for the new View type.
(WebInspector.ContentView.isViewable):
- UserInterface/InstrumentSidebarPanel.js:
(WebInspector.InstrumentSidebarPanel): Add support for canvas profiles.
(WebInspector.InstrumentSidebarPanel.prototype._recordProfileGlyphMousedOver):
(WebInspector.InstrumentSidebarPanel.prototype._recordProfileGlyphMousedDown):
(WebInspector.InstrumentSidebarPanel.prototype._recordProfileGlyphClicked):
(WebInspector.InstrumentSidebarPanel.prototype._profileTypeWasSelected):
(WebInspector.InstrumentSidebarPanel.prototype._profilesCleared):
- UserInterface/Main.html: Load the new files.
- UserInterface/ProfileManager.js: New methods and support for canvas profiles.
(WebInspector.ProfileManager):
(WebInspector.ProfileManager.prototype.initialize):
(WebInspector.ProfileManager.prototype.isProfilingCanvas):
(WebInspector.ProfileManager.prototype.startProfilingCanvas):
(WebInspector.ProfileManager.prototype.stopProfilingCanvas):
(WebInspector.ProfileManager.prototype._checkForInterruptions):
(WebInspector.ProfileManager.prototype._attemptToResumeProfiling):
- 1:30 PM Changeset in webkit [153927] by
-
- 78 edits in trunk/Source
Make Page::settings() return a reference.
<http://webkit.org/b/119662>
Reviewed by Anders Carlsson.
...and remove some pointless null checks that were exposed by doing this.
- 1:26 PM Changeset in webkit [153926] by
-
- 119 edits in trunk/Source
Make some things that return never-null pointers return references instead.
<http://webkit.org/b/119660>
Reviewed by Antti Koivisto.
Source/WebCore:
These functions never return null anyway, so remove any ambiguity by letting
them return references instead of pointers.
- Node::ensureRareData()
- Element::ensureUniqueElementData()
- Element::ensureShadow()
- Element::ensureUserAgentShadowRoot()
- StyledElement::ensureMutableInlineStyle()
- NodeRareData::ensureNodeLists()
- NodeRareData::ensureMutationObserverData()
- EventTarget::ensureEventTargetData()
- Document::ensureStyleResolver()
- Document::selectorQueryCache()
- Document::mediaQueryMatcher()
- FlowThreadController::ensureRenderFlowThreadWithName()
- HTMLTrackElement::ensureTrack()
- SVGElement::ensureSVGRareData()
- AnimationControllerPrivate::ensureCompositeAnimation()
Source/WebKit/qt:
- Api/qwebelement.cpp:
(QWebElement::styleProperty):
- 1:11 PM Changeset in webkit [153925] by
-
- 19 edits1 delete in trunk/Source/WebCore
Cleanup MediaQueryListListener
https://bugs.webkit.org/show_bug.cgi?id=119664
Reviewed by Andreas Kling.
Make MediaQueryListListener a proper WebIDL callback.
- CMakeLists.txt:
- DerivedSources.cpp:
- DerivedSources.make:
- DerivedSources.pri:
- GNUmakefile.list.am:
- Target.pri:
- WebCore.vcxproj/WebCore.vcxproj:
- WebCore.vcxproj/WebCore.vcxproj.filters:
- WebCore.xcodeproj/project.pbxproj:
Update project files.
- bindings/scripts/CodeGeneratorJS.pm:
(GenerateCallbackHeader):
(GenerateCallbackImplementation):
(JSValueToNative):
- bindings/scripts/IDLAttributes.txt:
Add support for CallbackNeedsOperatorEqual, which adds an operator==.
- css/MediaAllInOne.cpp:
Remove MediaQueryListListener.cpp.
- css/MediaQueryListListener.cpp:
Removed.
- css/MediaQueryListListener.h:
(WebCore::MediaQueryListListener::~MediaQueryListListener):
(WebCore::MediaQueryListListener::MediaQueryListListener):
- css/MediaQueryListListener.idl:
Convert to a proper WebIDL callback.
- css/MediaQueryMatcher.cpp:
(WebCore::MediaQueryMatcher::Listener::evaluate):
(WebCore::MediaQueryMatcher::styleResolverChanged):
- css/MediaQueryMatcher.h:
Remove all traces of the ScriptState.
- 10:41 AM Changeset in webkit [153924] by
-
- 4 edits in trunk
Unreviewed. Update NEWS and Versions.m4 for 2.1.4 release.
.:
- Source/autotools/Versions.m4: Update version numbers.
Source/WebKit/gtk:
- NEWS: Update release notes.
- 10:40 AM Changeset in webkit [153923] by
-
- 2 edits in trunk/Tools
Unreviewed. Change my primary mail address.
- Scripts/webkitpy/common/config/contributors.json:
- 8:16 AM Changeset in webkit [153922] by
-
- 4 edits in trunk/Source
Unreviewed. Fix make distcheck.
Source/WebCore:
- GNUmakefile.list.am: Remove unexistent header file.
Source/WTF:
- GNUmakefile.list.am: Add missing header file.
Aug 10, 2013:
- 8:45 AM Changeset in webkit [153921] by
-
- 5 edits in trunk/Source/WebCore
[GObject] Wrap KeyboardEvent
https://bugs.webkit.org/show_bug.cgi?id=119651
Reviewed by Christophe Dumez.
This exposes KeyboardEvent to our GObject API. It is required for handling
keyboard events, such as keydown, keypress, and so on.
- bindings/gobject/GNUmakefile.am:
- bindings/gobject/WebKitDOMPrivate.cpp:
(WebKit::wrap): make sure we create a WebKitDOMKeyboardEvent for a DOM event that
is a KeyboardEvent.
- bindings/scripts/CodeGeneratorGObject.pm:
(GetBaseClass): make WebKitDOMEvent be WebKitDOMKeyboardEvent's parent, like we do
for WebKitDOMMouseEvent.
- dom/KeyboardEvent.idl: generate a single initialization method.
Aug 9, 2013:
- 8:29 PM Changeset in webkit [153920] by
-
- 2 edits in trunk/Source/WebCore
Crash in com.apple.WebKit.WebContent at com.apple.MediaToolbox.
https://bugs.webkit.org/show_bug.cgi?id=119645
Reviewed by Eric Carlson.
Work around crash caused by <rdar://problem/14688471> by releasing
m_request as soon as keys have been added to it.
- Modules/encryptedmedia/CDMPrivateAVFoundation.mm:
(WebCore::CDMSessionAVFoundation::update):
- 6:43 PM Changeset in webkit [153919] by
-
- 5 edits in branches/safari-537-branch/Source
Versioning.
- 6:41 PM Changeset in webkit [153918] by
-
- 1 copy in tags/Safari-537.56
New Tag.
- 5:36 PM Changeset in webkit [153917] by
-
- 2 edits in trunk/Source/WebCore
Artifacts seen sometimes with layer backed WebKit1 plug-in
https://bugs.webkit.org/show_bug.cgi?id=119647
<rdar://problem/14002715>
Reviewed by Beth Dakin.
Don't try to paint layer backed views - they will be painted by Core Animation.
- platform/mac/WidgetMac.mm:
(WebCore::Widget::paint):
- 5:07 PM Changeset in webkit [153916] by
-
- 1 edit3 adds in trunk/LayoutTests
Merge an encoding test from Blink
https://bugs.webkit.org/show_bug.cgi?id=116943
Original patch by Adam Barth.
- fast/encoding/meta-overrules-auto-expected.txt: Added.
- fast/encoding/meta-overrules-auto.html: Added.
- fast/encoding/resources/meta-input-encoding.html: Added.
- 5:01 PM Changeset in webkit [153915] by
-
- 36 edits5 deletes in trunk/LayoutTests
Tests that dump delegate calls are flaky because of cached redirects
https://bugs.webkit.org/show_bug.cgi?id=119641
Reviewed by Brady Eidson.
Made redirect.php emit Cache-Control: no store as a workaround for bug 77538.
Removed several reimplementations of it, and changed the test to use the
central copy.
- http/tests/permissionclient/resources/redir.php: Removed.
- http/tests/resources/redirect.php:
- http/tests/security/beforeload-iframe-server-redirect-expected.txt:
- http/tests/security/beforeload-iframe-server-redirect.html:
- http/tests/security/cannot-read-cssrules-redirect.html:
- http/tests/security/canvas-remote-read-redirect-to-remote-image.html:
- http/tests/security/canvas-remote-read-remote-image-redirect.html:
- http/tests/security/contentSecurityPolicy/frame-src-redirect-blocked.html:
- http/tests/security/contentSecurityPolicy/resources/multiple-iframe-test.js:
- http/tests/security/contentSecurityPolicy/resources/redir.php: Removed.
- http/tests/security/contentSecurityPolicy/script-src-redirect-expected.txt:
- http/tests/security/contentSecurityPolicy/script-src-redirect.html:
- http/tests/security/cross-origin-css.html:
- http/tests/security/cross-origin-script-window-onerror-redirected.html:
- http/tests/security/mixedContent/redirect-http-to-https-iframe-in-main-frame-expected.txt:
- http/tests/security/mixedContent/redirect-http-to-https-script-in-iframe-expected.txt:
- http/tests/security/mixedContent/redirect-https-to-http-iframe-in-main-frame-expected.txt:
- http/tests/security/mixedContent/resources/frame-with-redirect-http-to-https-frame.html:
- http/tests/security/mixedContent/resources/frame-with-redirect-http-to-https-script.html:
- http/tests/security/mixedContent/resources/frame-with-redirect-https-to-http-frame.html:
- http/tests/security/mixedContent/resources/frame-with-redirect-https-to-http-script.html:
- http/tests/security/redirect-BLOCKED-to-localURL.html:
- http/tests/security/resources/cross-origin-xsl-redirect.xml:
- http/tests/security/resources/redir.php: Removed.
- http/tests/security/resources/xsl-using-document-redirect.xsl:
- http/tests/security/resources/xsl-using-external-entity-redirect.xsl:
- http/tests/security/stylesheet-href-redirect-expected.txt:
- http/tests/security/stylesheet-href-redirect.html:
- http/tests/security/xssAuditor/block-does-not-leak-referrer.html:
- http/tests/security/xssAuditor/resources/redir.php: Removed.
- http/tests/security/xssAuditor/script-tag-open-redirect.html:
- http/tests/security/xssAuditor/script-tag-redirect.html:
- http/tests/xmlhttprequest/redirect-cross-origin-2.html:
- http/tests/xmlhttprequest/redirect-cross-origin-post-sync.html:
- http/tests/xmlhttprequest/redirect-cross-origin-post.html:
- http/tests/xmlhttprequest/redirect-cross-origin-sync-double.html:
- http/tests/xmlhttprequest/redirect-cross-origin-sync.html:
- http/tests/xmlhttprequest/redirect-cross-origin.html:
- http/tests/xmlhttprequest/resources/redirect.php: Removed.
- http/tests/xmlhttprequest/xmlhttprequest-unsafe-redirect.html:
- 4:30 PM Changeset in webkit [153914] by
-
- 3 edits2 adds in trunk
ASSERTION FAILED: stroke->opacity != other->stroke->opacity in 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.
- 4:25 PM Changeset in webkit [153913] by
-
- 8 edits2 adds in trunk
REGRESSION (r142755): window.open creates an invisible window when width and height are 0
https://bugs.webkit.org/show_bug.cgi?id=119633
Reviewed by Darin Adler.
Source/WebCore:
Test: fast/dom/Window/open-zero-size-as-default.html
Relying on each WebKit to refuse setting size to zero was fragile - because this
required each one to have the check, and because by the time the client was called,
the initially zero size was not necessarily zero.
- loader/FrameLoader.cpp: (WebCore::createWindow): When sizes are zero, keep the
size the window was created with, because that's the default one by definition.
- page/DOMWindow.cpp: (WebCore::DOMWindow::adjustWindowRect): It's too late to
check for zero size now, it's been mangled to adjust for the difference between
window and viewport size.
Source/WebKit/efl:
- WebCoreSupport/ChromeClientEfl.cpp: (WebCore::ChromeClientEfl::setWindowRect):
Once again, the passed rect cannot be empty.
Source/WebKit/gtk:
- WebCoreSupport/ChromeClientGtk.cpp: (WebKit::ChromeClient::setWindowRect):
Once again, the passed rect cannot be empty.
LayoutTests:
- fast/dom/Window/open-zero-size-as-default-expected.txt: Added.
- fast/dom/Window/open-zero-size-as-default.html: Added.
- 4:09 PM Changeset in webkit [153912] by
-
- 2 edits in trunk/Source/WebCore
[Mac] some track language tags are not recognized
https://bugs.webkit.org/show_bug.cgi?id=119643
Reviewed by Dean Jackson.
- platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:
(WebCore::MediaPlayerPrivateAVFoundationObjC::tracksChanged): Always call languageOfPrimaryAudioTrack(),
a track may have changed so we may have a new language.
(WebCore::MediaPlayerPrivateAVFoundationObjC::languageOfPrimaryAudioTrack): Call [AVAssetTrack languageCode]
if [AVAssetTrack extendedLanguageTag] returns NULL in case the media file has an old
QuickTime language code.
- 4:00 PM Changeset in webkit [153911] by
-
- 10 edits1 copy in branches/safari-537-branch
Merged r153907. <rdar://problem/14281275>
- 3:56 PM Changeset in webkit [153910] by
-
- 2 edits in trunk/LayoutTests
http/tests/navigation/back-twice-without-commit.html times out
https://bugs.webkit.org/show_bug.cgi?id=82980
- platform/mac/TestExpectations: Marked as such again, reverting r153842.
- 3:46 PM Changeset in webkit [153909] by
-
- 2 edits in trunk/Source/WebCore
Captions menu needs to reset text-align
https://bugs.webkit.org/show_bug.cgi?id=119637
Reviewed by Eric Carlson.
Reset the text-align property on the menu so that it
doesn't get the inherited value. The full fix for all
controls is http://webkit.org/b/116100.
- css/mediaControlsQuickTime.css:
(video::-webkit-media-controls-closed-captions-container):
- 2:12 PM Changeset in webkit [153908] by
-
- 2 edits in trunk/Tools
[GTK] Misses mod_ssl and php in the list of packages required for tests for yum
https://bugs.webkit.org/show_bug.cgi?id=115751
Patch by Gustavo Noronha Silva <gustavo.noronha@collabora.com> on 2013-08-09
Reviewed by Martin Robinson.
- gtk/install-dependencies: add mod_ssl and php to the packages installed for tests
with yum.
- 12:58 PM Changeset in webkit [153907] by
-
- 10 edits1 add in trunk
AX: Not able to use arrow keys to read text in a WK2 app
https://bugs.webkit.org/show_bug.cgi?id=119605
-and corresponding-
<rdar://problem/14281275>
Reviewed by Darin Adler.
Source/WebCore:
This code from defaultKeyboardEventHandler() is the magic that makes this AX
functionality work. This patch moves that into a separate function so that we can
call it from WK2.
- WebCore.exp.in:
- page/EventHandler.cpp:
(WebCore::EventHandler::handleKeyboardSelectionMovementForAccessibility):
(WebCore::EventHandler::defaultKeyboardEventHandler):
- page/EventHandler.h:
Source/WebKit2:
Make all of these scrolling functions actually return the WebCore bools that
indicate whether or not scrolling happened.
- WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::scroll):
(WebKit::WebPage::logicalScroll):
(WebKit::WebPage::scrollBy):
- WebProcess/WebPage/WebPage.h:
- WebProcess/WebPage/mac/WebPageMac.mm:
(WebKit::WebPage::executeKeypressCommandsInternal):
This function now gives accessibility a chance to handle the event too. And it
also actually tracks whether or not the event was handled by scrolling instead of
assuming that it was and universally returning true.
(WebKit::WebPage::performNonEditingBehaviorForSelector):
Tools:
New test verifies that didNotHandleKeyEvent will be called when there is nothing
to scroll.
- TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
- TestWebKitAPI/Tests/WebKit2/DidNotHandleKeyDown.cpp: Added.
(TestWebKitAPI::didFinishLoadForFrame):
(TestWebKitAPI::didNotHandleKeyEventCallback):
(TestWebKitAPI::TEST):
- 12:44 PM Changeset in webkit [153906] by
-
- 15 edits4 adds in trunk/Source/WebCore
Move static StyleResolver functions to Style namespace
https://bugs.webkit.org/show_bug.cgi?id=119629
Reviewed by Sam Weinig and Andreas Kling.
- Move static font size computation functions to StyleFontSizeFunctions.h/.cpp
- Move StyleResolver::styleForDocument to Style::resolveForDocument
- WebCore.xcodeproj/project.pbxproj:
- css/DeprecatedStyleBuilder.cpp:
(WebCore::ApplyPropertyFontFamily::applyInitialValue):
(WebCore::ApplyPropertyFontFamily::applyValue):
(WebCore::ApplyPropertyFontSize::applyInitialValue):
(WebCore::ApplyPropertyFontSize::applyValue):
- css/StyleResolver.cpp:
(WebCore::StyleResolver::adjustRenderStyle):
(WebCore::checkForOrientationChange):
(WebCore::StyleResolver::applyProperty):
(WebCore::StyleResolver::checkForGenericFamilyChange):
(WebCore::StyleResolver::initializeFontStyle):
(WebCore::StyleResolver::setFontSize):
- css/StyleResolver.h:
- dom/Document.cpp:
(WebCore::Document::recalcStyle):
- editing/EditingStyle.cpp:
(WebCore::legacyFontSizeFromCSSValue):
- html/HTMLLinkElement.cpp:
(WebCore::HTMLLinkElement::process):
- rendering/style/RenderStyle.cpp:
(WebCore::RenderStyle::getFontAndGlyphOrientation):
Moved from StyleResolver.cpp so it can be used by Style::resolveForDocument().
This depends on RenderStyle only so it is a natural member function.
- rendering/style/RenderStyle.h:
- rendering/svg/RenderSVGInlineText.cpp:
(WebCore::RenderSVGInlineText::computeNewScaledFontForStyle):
- style/StyleFontSizeFunctions.cpp: Copied from Source/WebCore/css/StyleResolver.cpp.
(WebCore::Style::computedFontSizeFromSpecifiedSize):
(WebCore::Style::computedFontSizeFromSpecifiedSizeForSVGInlineText):
(WebCore::Style::fontSizeForKeyword):
(WebCore::Style::legacyFontSizeForPixelSize):
- style/StyleFontSizeFunctions.h: Added.
- style/StyleResolveForDocument.cpp: Added.
(WebCore::Style::resolveForDocument):
- style/StyleResolveForDocument.h: Added.
- 11:56 AM Changeset in webkit [153905] by
-
- 26 edits in trunk
Add support for KeyboardEvent.location attribute
https://bugs.webkit.org/show_bug.cgi?id=119326
Reviewed by Benjamin Poulain.
Source/WebCore:
Add support for KeyboardEvent.location attribute as per the latest specification:
http://dev.w3.org/2006/webapi/DOM-Level-3-Events/html/DOM3-Events.html#events-KeyboardEvent
IE10, Firefox 22 and recently Blink all support KeyboardEvent.location already.
The deprecated 'keyLocation' attribute was not removed to maintain backward
compatibility.
No new tests, covered by existing tests.
- bindings/objc/PublicDOMInterfaces.h:
- bindings/scripts/CodeGeneratorJS.pm:
(GenerateConstructorDefinition):
- dom/KeyboardEvent.cpp:
(WebCore::KeyboardEventInit::KeyboardEventInit):
(WebCore::KeyboardEvent::KeyboardEvent):
(WebCore::KeyboardEvent::initKeyboardEvent):
- dom/KeyboardEvent.h:
(WebCore::KeyboardEvent::location):
- dom/KeyboardEvent.idl:
Source/WebKit/win:
Add new location attribute to DOMKeyboardEvent.
- DOMEventsClasses.cpp:
(DOMKeyboardEvent::location):
- DOMEventsClasses.h:
- Interfaces/DOMEvents.idl:
Tools:
Rename the deprecated 'keyLocation' to 'location' in Mac's DRT code.
- DumpRenderTree/mac/EventSendingController.mm:
(-[EventSendingController keyDown:withModifiers:withLocation:]):
(-[EventSendingController keyDownWrapper:withModifiers:withLocation:]):
(-[EventSendingController scheduleAsynchronousKeyDown:withModifiers:withLocation:]):
(-[EventSendingController handleEvent:]):
(-[EventSendingController fireKeyboardEventsToElement:]):
LayoutTests:
Rename KeyboardEvent.keyLocation to KeyboardEvent.location in the
tests and add test cases to make sure that the deprecated
KeyboardEvent.keyLocation still works.
- fast/events/arrow-keys-on-body-expected.txt:
- fast/events/arrow-keys-on-body.html:
- fast/events/constructors/keyboard-event-constructor-expected.txt:
- fast/events/constructors/keyboard-event-constructor.html:
- fast/events/init-events-expected.txt:
- fast/events/js-keyboard-event-creation.html:
- fast/events/keydown-leftright-keys-expected.txt:
- fast/events/keydown-leftright-keys.html:
- fast/events/keydown-numpad-keys-expected.txt:
- fast/events/script-tests/init-events.js:
- fast/events/script-tests/keydown-numpad-keys.js:
(recordKeyEvent):
(testKeyEventWithLocation):
- platform/mac/fast/events/objc-keyboard-event-creation.html:
- platform/win/fast/events/keyLocation-numpad.html:
- 11:54 AM Changeset in webkit [153904] by
-
- 8 edits in trunk
Make atob() throw an InvalidCharacterError on excess padding characters
https://bugs.webkit.org/show_bug.cgi?id=118898
Reviewed by Darin Adler.
Source/WebCore:
According to the latest specification, window.atob() should throw an
InvalidCharacterError on excess padding characters:
http://www.whatwg.org/specs/web-apps/current-work/multipage/webappapis.html#dom-windowbase64-atob
This behavior is consistent with both Firefox, IE10 and recently Blink.
No new tests, already covered by:
fast/dom/Window/atob-btoa.html
- page/DOMWindow.cpp:
(WebCore::DOMWindow::atob):
Source/WTF:
Add a Base64FailOnInvalidCharacterOrExcessPadding value to the Base64DecodePolicy
enumeration so that the caller can request strict padding validation in addition
to strict character validation. This is needed so that we can use
WTF::base64Decode() for window.atob().
- wtf/text/Base64.cpp:
(WTF::base64DecodeInternal):
- wtf/text/Base64.h:
LayoutTests:
Update and rebaseline fast/dom/Window/atob-btoa.html now that window.atob()'s
behavior has changed and now that we throw an InvalidCharacterError on excess
pad characters.
- fast/dom/Window/atob-btoa-expected.txt:
- fast/dom/Window/atob-btoa.html:
- 11:40 AM Changeset in webkit [153903] by
-
- 44 edits in trunk
<unknown> is often dumped instead of path in test output
https://bugs.webkit.org/show_bug.cgi?id=119590
Reviewed by Anders Carlsson.
Tools:
There were two reasons for <unknown>:
- When delegate logging is enabled via a testRunner call, that obviously happens
after main resource load starts, so we don't record its URL.
- Authentication delegate logging relied on URL recording, but it's only turned on
when enabling resource loading delegate logging, not auth one.
URL recording was made conditional in <http://trac.webkit.org/r19316>, which was
landed without posted rationale and unreviewed. Just undoing that change now.
- DumpRenderTree/mac/ResourceLoadDelegate.mm:
(-[ResourceLoadDelegate webView:identifierForInitialRequest:fromDataSource:]):
- WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp:
(WTR::InjectedBundlePage::didInitiateLoadForResource):
LayoutTests:
Updating test results.
- fast/loader/file-protocol-fragment-expected.txt:
- fast/loader/main-document-url-for-non-http-loads-expected.txt:
- http/tests/cache/iframe-304-crash-expected.txt:
- http/tests/cache/willsendrequest-returns-null-for-memory-cache-load-expected.txt:
- http/tests/loading/307-after-303-after-post-expected.txt:
- http/tests/loading/authentication-after-redirect-stores-wrong-credentials/authentication-after-redirect-stores-wrong-credentials-expected.txt:
- http/tests/loading/basic-auth-resend-wrong-credentials-expected.txt:
- http/tests/loading/basic-credentials-sent-automatically-expected.txt:
- http/tests/loading/redirect-methods-expected.txt:
- http/tests/misc/401-alternative-content-expected.txt:
- http/tests/misc/authentication-redirect-1/authentication-sent-to-redirect-cross-origin-expected.txt:
- http/tests/misc/authentication-redirect-2/authentication-sent-to-redirect-same-origin-expected.txt:
- http/tests/misc/authentication-redirect-4/authentication-sent-to-redirect-same-origin-url-expected.txt:
- http/tests/misc/favicon-loads-with-images-disabled-expected.txt:
- http/tests/misc/link-rel-icon-beforeload-expected.txt:
- http/tests/misc/will-send-request-returns-null-on-redirect-expected.txt:
- http/tests/misc/window-dot-stop-expected.txt:
- http/tests/security/401-logout/401-logout-expected.txt:
- http/tests/security/XFrameOptions/x-frame-options-allowall-expected.txt:
- http/tests/security/XFrameOptions/x-frame-options-deny-expected.txt:
- http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-expected.txt:
- http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-in-body-expected.txt:
- http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-parent-same-origin-allow-expected.txt:
- http/tests/security/XFrameOptions/x-frame-options-deny-meta-tag-parent-same-origin-deny-expected.txt:
- http/tests/security/XFrameOptions/x-frame-options-invalid-expected.txt:
- http/tests/security/XFrameOptions/x-frame-options-multiple-headers-conflict-expected.txt:
- http/tests/security/XFrameOptions/x-frame-options-multiple-headers-sameorigin-allow-expected.txt:
- http/tests/security/XFrameOptions/x-frame-options-multiple-headers-sameorigin-deny-expected.txt:
- http/tests/security/XFrameOptions/x-frame-options-parent-same-origin-allow-expected.txt:
- http/tests/security/XFrameOptions/x-frame-options-parent-same-origin-deny-expected.txt:
- http/tests/xmlhttprequest/failed-auth-expected.txt:
- http/tests/xmlhttprequest/remember-bad-password-expected.txt:
- loader/go-back-cached-main-resource-expected.txt:
- platform/efl-wk2/loader/go-back-cached-main-resource-expected.txt:
- platform/mac/fast/images/support-broken-image-delegate-expected.txt:
- platform/mac/security/block-test-expected.txt:
- platform/mac/security/block-test-no-port-expected.txt:
- platform/mac/webarchive/loading/cache-expired-subresource-expected.txt:
- webarchive/loading/test-loading-archive-expected.txt:
- webarchive/loading/test-loading-archive-subresource-null-mimetype-expected.txt:
- 10:35 AM Changeset in webkit [153902] by
-
- 45 edits in trunk/LayoutTests
Modify the existing canvas blending test suite to test both
accelerated and non accelerated modes.
https://bugs.webkit.org/show_bug.cgi?id=119628
Patch by Mihai Tica <mitica@adobe.com> on 2013-08-09
Reviewed by Dirk Schulze.
- fast/canvas/canvas-blending-clipping-expected.txt:
- fast/canvas/canvas-blending-clipping.html:
- fast/canvas/canvas-blending-color-over-color-expected.txt:
- fast/canvas/canvas-blending-color-over-color.html:
- fast/canvas/canvas-blending-color-over-gradient-expected.txt:
- fast/canvas/canvas-blending-color-over-gradient.html:
- fast/canvas/canvas-blending-color-over-image-expected.txt:
- fast/canvas/canvas-blending-color-over-image.html:
- fast/canvas/canvas-blending-color-over-pattern-expected.txt:
- fast/canvas/canvas-blending-color-over-pattern.html:
- fast/canvas/canvas-blending-fill-style-expected.txt:
- fast/canvas/canvas-blending-fill-style.html:
- fast/canvas/canvas-blending-global-alpha-expected.txt:
- fast/canvas/canvas-blending-global-alpha.html:
- fast/canvas/canvas-blending-gradient-over-color-expected.txt:
- fast/canvas/canvas-blending-gradient-over-color.html:
- fast/canvas/canvas-blending-gradient-over-gradient-expected.txt:
- fast/canvas/canvas-blending-gradient-over-gradient.html:
- fast/canvas/canvas-blending-gradient-over-image-expected.txt:
- fast/canvas/canvas-blending-gradient-over-image.html:
- fast/canvas/canvas-blending-gradient-over-pattern-expected.txt:
- fast/canvas/canvas-blending-gradient-over-pattern.html:
- fast/canvas/canvas-blending-image-over-color-expected.txt:
- fast/canvas/canvas-blending-image-over-color.html:
- fast/canvas/canvas-blending-image-over-gradient-expected.txt:
- fast/canvas/canvas-blending-image-over-gradient.html:
- fast/canvas/canvas-blending-image-over-image-expected.txt:
- fast/canvas/canvas-blending-image-over-image.html:
- fast/canvas/canvas-blending-image-over-pattern-expected.txt:
- fast/canvas/canvas-blending-image-over-pattern.html:
- fast/canvas/canvas-blending-pattern-over-color-expected.txt:
- fast/canvas/canvas-blending-pattern-over-color.html:
- fast/canvas/canvas-blending-pattern-over-gradient-expected.txt:
- fast/canvas/canvas-blending-pattern-over-gradient.html:
- fast/canvas/canvas-blending-pattern-over-image-expected.txt:
- fast/canvas/canvas-blending-pattern-over-image.html:
- fast/canvas/canvas-blending-pattern-over-pattern-expected.txt:
- fast/canvas/canvas-blending-pattern-over-pattern.html:
- fast/canvas/canvas-blending-shadow-expected.txt:
- fast/canvas/canvas-blending-shadow.html:
- fast/canvas/canvas-blending-text-expected.txt:
- fast/canvas/canvas-blending-text.html:
- fast/canvas/canvas-blending-transforms-expected.txt:
- fast/canvas/canvas-blending-transforms.html:
- 10:31 AM Changeset in webkit [153901] by
-
- 44 edits in trunk
Make DOMImplementation::hasFeature() behave according to specification
https://bugs.webkit.org/show_bug.cgi?id=119329
Reviewed by Ryosuke Niwa.
Source/WebCore:
Make DOMImplementation::hasFeature() behave according to the latest specification:
http://dom.spec.whatwg.org/#dom-domimplementation-hasfeature
Specifically, the function should now return true unconditionally for any non-SVG feature.
This behavior is consistent with Firefox and since recently Blink.
hasFeature() originally would report whether the user agent claimed to support a
given DOM feature, but experience proved it was not nearly as reliable or granular
as simply checking whether the desired interfaces, attributes, or methods existed.
As such, it should no longer be used, but continues to exist (and simply returns
true) so that old pages don't stop working.
No new tests, covered by existing tests.
- dom/DOMImplementation.cpp:
(WebCore::isSupportedSVG10Feature):
(WebCore::isSupportedSVG11Feature):
(WebCore::DOMImplementation::hasFeature):
LayoutTests:
Rebaseline several test cases now that DOMImplementation::hasFeature() returns
true unconditionally for non-SVG features.
Also use a valid svg prefix for unknown features in SVG tests so that they are
properly recognized as SVG features and so that hasFeature() returns false as
expected.
- dom/xhtml/level3/core/hasFeature01-expected.txt:
- dom/xhtml/level3/core/hasFeature02-expected.txt:
- dom/xhtml/level3/core/hasFeature03-expected.txt:
- dom/xhtml/level3/core/hasFeature04-expected.txt:
- fast/dom/features-expected.txt:
- platform/efl/svg/W3C-SVG-1.1-SE/types-dom-06-f-expected.txt:
- platform/efl/svg/W3C-SVG-1.1/struct-dom-03-b-expected.txt:
- platform/gtk/svg/W3C-SVG-1.1-SE/types-dom-06-f-expected.txt:
- platform/gtk/svg/W3C-SVG-1.1/struct-dom-03-b-expected.txt:
- platform/mac/svg/W3C-SVG-1.1-SE/types-dom-06-f-expected.txt:
- platform/mac/svg/W3C-SVG-1.1/struct-dom-03-b-expected.txt:
- platform/qt-mac/svg/W3C-SVG-1.1-SE/types-dom-06-f-expected.txt:
- platform/qt/svg/W3C-SVG-1.1-SE/types-dom-06-f-expected.txt:
- svg/W3C-SVG-1.1/struct-dom-03-b-expected.txt:
- svg/custom/outer-svg-unknown-feature.svg:
- svg/dynamic-updates/script-tests/SVGCircleElement-dom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGCircleElement-svgdom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGEllipseElement-dom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGEllipseElement-svgdom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGForeignObjectElement-dom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGForeignObjectElement-svgdom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGGElement-dom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGGElement-svgdom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGImageElement-dom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGImageElement-svgdom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGLineElement-dom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGLineElement-svgdom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGPathElement-dom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGPathElement-svgdom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGPolygonElement-dom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGPolygonElement-svgdom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGPolylineElement-dom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGPolylineElement-svgdom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGRectElement-dom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGRectElement-svgdom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGSVGElement-dom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGSVGElement-svgdom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGTextElement-dom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGTextElement-svgdom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGUseElement-dom-requiredFeatures.js:
(repaintTest):
- svg/dynamic-updates/script-tests/SVGUseElement-svgdom-requiredFeatures.js:
(repaintTest):
- 10:25 AM Changeset in webkit [153900] by
-
- 3 edits in releases/WebKitGTK/webkit-2.0/Source/WebCore
Merge 153749 - webkit-gtk fails to build with bison-3.0
https://bugs.webkit.org/show_bug.cgi?id=119373
Reviewed by Gustavo Noronha Silva.
Source/ThirdParty/ANGLE:
- GNUmakefile.am: Stop relying on generated glslang_tab.(cpp|h) and glslang.cpp sources as Bison 3.0
produces non-compilable source code. Instead, compile these pregenerated files as they're included in
the tree. Originally, the ANGLE source tree provides these files as generated by Bison 2.7, but they're
at the moment regenerated with Bison 2.3 when the third-party ANGLE source is being updated.
Source/WebCore:
- GNUmakefile.am: Remove rules for generating ANGLE's glslang source files with Bison.
Conflicts:
Source/ThirdParty/ANGLE/GNUmakefile.am
Source/WebCore/GNUmakefile.am
- 10:24 AM WebKitGTK/2.0.x edited by
- (diff)
- 10:09 AM Changeset in webkit [153899] by
-
- 2 edits in branches/safari-537-branch/Source/WebCore
Merged r153809. <rdar://problem/14669029>
- 10:00 AM Changeset in webkit [153898] by
-
- 3 edits in trunk/Source/WebCore
[GTK] [EFL] Enable tiled shadow blur for the inset shadows.
https://bugs.webkit.org/show_bug.cgi?id=119078
Reviewed by Martin Robinson.
Already covered in the tests.
- platform/graphics/GraphicsContext.cpp:
- platform/graphics/cairo/GraphicsContextCairo.cpp:
(WebCore::fillCurrentCairoPath): Added to allow fill the path
without the shadow.
(WebCore::shadowAndFillCurrentCairoPath):
(WebCore::GraphicsContext::fillRectWithRoundedHole): Added cairo
implementation.
- 9:53 AM Changeset in webkit [153897] by
-
- 7 edits in trunk/LayoutTests
move-by-word-visually-multi-space.html takes 21s and spends most of its time doing spelling and grammar checking
https://bugs.webkit.org/show_bug.cgi?id=119613
Reviewed by Darin Adler.
editing/selection tests are very slow, and much of the time is spent
doing spelling and grammar checking (which is not what these tests are
testing). So call internals.setContinuousSpellCheckingEnabled(false)
to turn that off.
- editing/selection/caret-at-bidi-boundary.html:
- editing/selection/move-by-character-brute-force.html:
- editing/selection/move-left-right.html:
- editing/selection/programmatic-selection-on-mac-is-directionless.html:
- editing/selection/resources/extend-selection.js:
- editing/selection/resources/move-by-word-visually.js:
(runTest):
- 9:07 AM Changeset in webkit [153896] by
-
- 3 edits in trunk/Source/WebKit/blackberry
[BlackBerry] Viewport allows scrolling when it shouldn't after rotation to landscape
https://bugs.webkit.org/show_bug.cgi?id=119595
Patch by Jacky Jiang <zhajiang@blackberry.com> on 2013-08-09
Reviewed by Rob Buis.
Internally reviewed by Konrad Piascik, Max Feil, Arvid Nilsson and John Griggs.
JIRA 466384
Initial scales of portrait mode and landscape mode can be different due
to different viewport size and contents size. In this case, we restored
to the intial scale of portrait mode instead of the initial scale of current
landscape mode which zoomed in the WebPage a little bit and made the WebPage
scallable as the scale is larger than the initial scale of the current
landscape mode WebPage.
To fix that, we should restore to the current initial scale instead if
the WebPage was at initial scale before fullscreen.
Meanwhile, move the FIXME comments to the right place.
- Api/WebPage.cpp:
(BlackBerry::WebKit::WebPagePrivate::WebPagePrivate):
(BlackBerry::WebKit::WebPagePrivate::setViewportSize):
(BlackBerry::WebKit::WebPagePrivate::enterFullScreenForElement):
- Api/WebPage_p.h:
- 8:28 AM Changeset in webkit [153895] by
-
- 4 edits in trunk/Source/WebCore
[BlackBerry] Don't release the canvas Drawable, flush it directly instead.
https://bugs.webkit.org/show_bug.cgi?id=119589
https://jira.bbqnx.net/browse/BRWSR-12531
JIRA 444857
Patch by Jakob Petsovits <jpetsovits@blackberry.com> on 2013-08-09
Reviewed by George Staikos.
The previous code here used releaseBufferDrawable() to
flush painted ImageBuffer contents, but then proceeded
to use that released Drawable for further painting
without ever locking a new one.
In an implementation that actually releases the Drawable
rather than keeping it around and just flushing it,
this will wreak havoc as bad as black screens or crashes.
As a solution that doesn't require the ImageBuffer object
to swap out its m_canvas member for a new Drawable, we can
access the PlatformGraphicsContext::flush() method directly
to achieve the same effect as was previously intended by
calling releaseBufferDrawable(). All it needs is letting
the canvas layer know about the Drawable so it can call
that flush() method by itself.
No new tests, verified by not crashing with an aforementioned
implementation of releaseBufferDrawable().
- platform/graphics/blackberry/CanvasLayerWebKitThread.cpp:
(WebCore::CanvasLayerCompositingThreadClient::CanvasLayerCompositingThreadClient):
(WebCore::CanvasLayerCompositingThreadClient::commitPendingTextureUploads):
(WebCore::CanvasLayerWebKitThread::CanvasLayerWebKitThread):
- platform/graphics/blackberry/CanvasLayerWebKitThread.h:
(WebCore::CanvasLayerWebKitThread::create):
- platform/graphics/blackberry/ImageBufferBlackBerry.cpp:
(WebCore::ImageBuffer::ImageBuffer):
(WebCore::ImageBuffer::~ImageBuffer):
- 8:26 AM Changeset in webkit [153894] by
-
- 25 edits42 deletes in trunk/Source/WebInspectorUI
Switch the generated images off of PDFs and onto SVGs now that bug 119492 is fixed.
https://bugs.webkit.org/show_bug.cgi?id=118677
Reviewed by Joseph Pecoraro.
- UserInterface/ApplicationCacheDetailsSidebarPanel.js:
(WebInspector.ApplicationCacheDetailsSidebarPanel):
- UserInterface/CSSStyleDeclarationSection.js:
(WebInspector.CSSStyleDeclarationSection):
- UserInterface/CSSStyleDetailsSidebarPanel.js:
(WebInspector.CSSStyleDetailsSidebarPanel):
- UserInterface/ContentBrowser.js:
(WebInspector.ContentBrowser):
- UserInterface/DOMNodeDetailsSidebarPanel.js:
(WebInspector.DOMNodeDetailsSidebarPanel):
- UserInterface/DOMTreeContentView.js:
(WebInspector.DOMTreeContentView):
- UserInterface/DataGrid.js:
(WebInspector.DataGrid.prototype._generateSortIndicatorImagesIfNeeded):
- UserInterface/DebuggerSidebarPanel.js:
(WebInspector.DebuggerSidebarPanel):
- UserInterface/DetailsSection.js:
(WebInspector.DetailsSection.prototype._generateDisclosureTrianglesIfNeeded):
- UserInterface/FindBanner.js:
(WebInspector.FindBanner.prototype._generateButtonsGlyphsIfNeeded):
- UserInterface/Images/BackArrow.pdf: Removed.
- UserInterface/Images/BottomUpTree.pdf: Removed.
- UserInterface/Images/Breakpoints.pdf: Removed.
- UserInterface/Images/Close.pdf: Removed.
- UserInterface/Images/Crosshair.pdf: Removed.
- UserInterface/Images/DisclosureTriangleSmallClosed.pdf: Removed.
- UserInterface/Images/DisclosureTriangleSmallOpen.pdf: Removed.
- UserInterface/Images/DisclosureTriangleTinyClosed.pdf: Removed.
- UserInterface/Images/DisclosureTriangleTinyOpen.pdf: Removed.
- UserInterface/Images/DockBottom.pdf: Removed.
- UserInterface/Images/DockRight.pdf: Removed.
- UserInterface/Images/ForwardArrow.pdf: Removed.
- UserInterface/Images/GoToArrow.pdf: Removed.
- UserInterface/Images/LayerBorders.pdf: Removed.
- UserInterface/Images/LessColumns.pdf: Removed.
- UserInterface/Images/Locked.pdf: Removed.
- UserInterface/Images/MoreColumns.pdf: Removed.
- UserInterface/Images/NavigationItemAngleBrackets.pdf: Removed.
- UserInterface/Images/NavigationItemBrushAndRuler.pdf: Removed.
- UserInterface/Images/NavigationItemBug.pdf: Removed.
- UserInterface/Images/NavigationItemCurleyBraces.pdf: Removed.
- UserInterface/Images/NavigationItemFile.pdf: Removed.
- UserInterface/Images/NavigationItemLayers.pdf: Removed.
- UserInterface/Images/NavigationItemLog.pdf: Removed.
- UserInterface/Images/NavigationItemMagnifyingGlass.pdf: Removed.
- UserInterface/Images/NavigationItemStopwatch.pdf: Removed.
- UserInterface/Images/NavigationItemStorage.pdf: Removed.
- UserInterface/Images/NavigationItemTrash.pdf: Removed.
- UserInterface/Images/NavigationItemVariable.pdf: Removed.
- UserInterface/Images/Pause.pdf: Removed.
- UserInterface/Images/Percent.pdf: Removed.
- UserInterface/Images/Plus.pdf: Removed.
- UserInterface/Images/Resume.pdf: Removed.
- UserInterface/Images/ShadowDOM.pdf: Removed.
- UserInterface/Images/SortIndicatorDownArrow.pdf: Removed.
- UserInterface/Images/SortIndicatorUpArrow.pdf: Removed.
- UserInterface/Images/SplitToggleDown.pdf: Removed.
- UserInterface/Images/SplitToggleUp.pdf: Removed.
- UserInterface/Images/StepInto.pdf: Removed.
- UserInterface/Images/StepOut.pdf: Removed.
- UserInterface/Images/StepOver.pdf: Removed.
- UserInterface/Images/Undock.pdf: Removed.
- UserInterface/InstrumentSidebarPanel.js:
(WebInspector.InstrumentSidebarPanel):
- UserInterface/JavaScriptProfileView.js:
(WebInspector.JavaScriptProfileView):
- UserInterface/LayerTreeSidebarPanel.js:
(WebInspector.LayerTreeSidebarPanel):
- UserInterface/LogContentView.js:
(WebInspector.LogContentView):
- UserInterface/Main.js:
(WebInspector.contentLoaded):
(WebInspector._updateDockNavigationItems):
(WebInspector._generateDisclosureTriangleImages):
- UserInterface/NavigationSidebarPanel.js:
(WebInspector.NavigationSidebarPanel.prototype._generateDisclosureTrianglesIfNeeded):
- UserInterface/ProfileView.js:
(WebInspector.ProfileView):
- UserInterface/ResourceDetailsSidebarPanel.js:
(WebInspector.ResourceDetailsSidebarPanel):
- UserInterface/ResourceSidebarPanel.js:
(WebInspector.ResourceSidebarPanel):
- UserInterface/ScopeChainDetailsSidebarPanel.js:
(WebInspector.ScopeChainDetailsSidebarPanel):
- UserInterface/ScriptContentView.js:
(WebInspector.ScriptContentView):
- UserInterface/TextContentView.js:
(WebInspector.TextContentView):
- UserInterface/TextResourceContentView.js:
(WebInspector.TextResourceContentView):
- UserInterface/TimelinesContentView.js:
- 8:00 AM Changeset in webkit [153893] by
-
- 2 edits in trunk/Source/WebCore
[Qt][WK2] Composited and transformed links asserts in tap highlight
https://bugs.webkit.org/show_bug.cgi?id=94345
Reviewed by Jocelyn Turcotte.
Use a mapping that does not assert when hitting transforms. We still skip
transforms that produces non-rectangular areas.
- page/GestureTapHighlighter.cpp:
- 7:41 AM Changeset in webkit [153892] by
-
- 2 edits in trunk/Source/WebCore
Buildfix. Fix warning after r153887:
control reaches end of non-void function [-Werror=return-type]
Patch by Gabor Abraham <abrhm@inf.u-szeged.hu> on 2013-08-09
Reviewed by Csaba Osztrogonác.
- svg/SVGAnimateElement.cpp:
(WebCore::SVGAnimateElement::animatedPropertyTypeSupportsAddition):
- 7:15 AM Changeset in webkit [153891] by
-
- 7 edits in trunk
Text decorations should be listed as non-inheritable
https://bugs.webkit.org/show_bug.cgi?id=119608
Reviewed by Andreas Kling.
Source/WebCore:
Both "text-decoration" specs (CSS 2.1, CSS 3) and "text-decoration-line"
spec (CSS 3) states these properties are not inherited (w/ exceptions,
which are currently handled by -in-effect property), however those were
listed as inherited for no reason - thus preserving behavior as shown by
layout test results.
Backported from Blink: https://codereview.chromium.org/20751008
Updated getComputedStyle tests for both 'text-decoration' and
'text-decoration-line' properties to check that both are not implicitly
inherited.
- css/CSSProperty.cpp:
(WebCore::CSSProperty::isInheritedProperty):
LayoutTests:
Added test to verify that 'text-decoration' and 'text-decoration-line'
properties are not implicitly inherited.
- fast/css/getComputedStyle/getComputedStyle-text-decoration-expected.txt:
- fast/css/getComputedStyle/script-tests/getComputedStyle-text-decoration.js:
- fast/css3-text/css3-text-decoration/getComputedStyle/getComputedStyle-text-decoration-line-expected.txt:
- fast/css3-text/css3-text-decoration/getComputedStyle/script-tests/getComputedStyle-text-decoration-line.js:
- 7:14 AM Changeset in webkit [153890] by
-
- 2 edits in trunk/Source/WebCore
I wish we had a Windows EWS. :(
- css/ElementRuleCollector.cpp:
(WebCore::ElementRuleCollector::doCollectMatchingRulesForList): No range-based for today.
- 7:05 AM Changeset in webkit [153889] by
-
- 6 edits2 adds in trunk/LayoutTests
[Qt] Unreviewed gardening. Rebaselinnig some new tests.
- platform/qt-5.0-wk1/TestExpectations:
- platform/qt-5.0-wk2/compositing/overflow-trumps-transform-style-expected.txt:
- platform/qt-5.0-wk2/fast/repaint/caret-jump-between-nodes-expected.txt: Added.
- platform/qt/compositing/geometry/preserve-3d-switching-expected.txt:
- platform/qt/fast/forms/search-rtl-expected.png:
- platform/qt/fast/forms/search-rtl-expected.txt:
- platform/qt/fast/repaint/caret-jump-between-nodes-expected.txt: Added.
- 6:26 AM Changeset in webkit [153888] by
-
- 2 edits in trunk/Source/ThirdParty/ANGLE
[Qt] Buidfix after r153664. Adding ANGLE_WEBKIT_WIN definition.
https://bugs.webkit.org/show_bug.cgi?id=119577
Reviewed by Allan Sandfeld Jensen.
- Target.pri:
- 6:23 AM Changeset in webkit [153887] by
-
- 4 edits4 adds in trunk
ASSERT_NOT_REACHED() touched in WebCore::SVGAnimatedStringAnimator::addAnimatedTypes
https://bugs.webkit.org/show_bug.cgi?id=113224
Source/WebCore:
Reviewed by Darin Adler.
Do not perform from-by animations for non-additive attributes.
Test: svg/animations/non-additive-type-from-by-animation.html
- svg/SVGAnimateElement.cpp:
(WebCore::SVGAnimateElement::calculateFromAndByValues):
(WebCore::SVGAnimateElement::animatedPropertyTypeSupportsAddition):
(WebCore::SVGAnimateElement::isAdditive):
- svg/SVGAnimateElement.h:
LayoutTests:
Reviewed by Dirk Schulze.
Add a test for animating non-additive attributes in from-by animations
- svg/animations/non-additive-type-from-by-animation-expected.txt: Added.
- svg/animations/non-additive-type-from-by-animation.html: Added.
- svg/animations/resources/non-additive-type-from-by-animation.svg: Added.
- svg/animations/script-tests/non-additive-type-from-by-animation.js: Added.
(sample):
(executeTest):
- 6:05 AM Changeset in webkit [153886] by
-
- 3 edits2 adds in trunk
ASSERT(m_type == toSVGTransform.type())
https://bugs.webkit.org/show_bug.cgi?id=113217
Reviewed by Darin Adler.
Source/WebCore:
For to animations clear the transform list while animating and reset to it after the animation is done.
This matches Firefox and Opera 12 behavior.
Tests: svg/animations/animateTransform-to-scale-expected.svg
svg/animations/animateTransform-to-scale.svg
- svg/SVGAnimatedTransformList.cpp:
(WebCore::SVGAnimatedTransformListAnimator::calculateAnimatedValue):
LayoutTests:
Add reftest for additive to animation where from and to transform types differ.
- svg/animations/animateTransform-to-scale-expected.svg: Added.
- svg/animations/animateTransform-to-scale.svg: Added.
- 5:48 AM WebKitGTK/2.0.x edited by
- Add r153749 to the list of proposed merges for 2.0.5. (diff)
- 5:46 AM Changeset in webkit [153885] by
-
- 4 edits in trunk
[Automake] Clean up OpenGL graphics configuration sections
https://bugs.webkit.org/show_bug.cgi?id=119554
Reviewed by Martin Robinson.
Clean up sections in the Automake configuration process that are configuring the OpenGL graphics
features and dependencies. Remove unnecessary variable assignments, merge similar code paths,
clarify a couple of comments and error messages and fix a few whitespace problems.
- Source/autotools/FindDependencies.m4:
- Source/autotools/ReadCommandLineArguments.m4:
- Source/autotools/SetupAutomake.m4:
- 5:11 AM Changeset in webkit [153884] by
-
- 7 edits in trunk/Source
[Qt] QtWebKit should allow sending domain specific keycode to HTML applications
https://bugs.webkit.org/show_bug.cgi?id=118445
Patch by Arunprasad Rajkumar <arurajku@cisco.com> on 2013-08-09
Reviewed by Allan Sandfeld Jensen.
Source/WebCore:
No new test required, it is covered by QtWebKit API test.
This fix allows sending QKeyEvent::nativeVirtualKey as the DOM KeyboardEvent keyCode value instead
of Windows virtual key code. It can be enabled/disabled using "_q_useNativeVirtualKeyAsDOMKey"
dynamic property present in QWebPage.
- platform/PlatformKeyboardEvent.h:
(WebCore::PlatformKeyboardEvent::PlatformKeyboardEvent):
- platform/qt/PlatformKeyboardEventQt.cpp:
(WebCore::PlatformKeyboardEvent::PlatformKeyboardEvent):
(WebCore::PlatformKeyboardEvent::disambiguateKeyDownEvent):
Source/WebKit/qt:
Added API test.
- WebCoreSupport/QWebPageAdapter.cpp:
(QWebPageAdapter::QWebPageAdapter):
(QWebPageAdapter::dynamicPropertyChangeEvent):
(QWebPageAdapter::handleKeyEvent):
- WebCoreSupport/QWebPageAdapter.h:
- tests/qwebpage/tst_qwebpage.cpp:
(tst_QWebPage::domainSpecificKeyEvent):
- 5:08 AM Changeset in webkit [153883] by
-
- 3 edits2 adds in trunk
Missing NULL check in ApplyStyleCommand::applyInlineStyleToNodeRange()
https://bugs.webkit.org/show_bug.cgi?id=119570
Reviewed by Ryosuke Niwa.
Source/WebCore:
The m_mutableStyleSet of EditingStyle can be NULL in ApplyStyleCommand::applyInlineStyleToNodeRange but
this scenario was not checked earlier.
Test: editing/execCommand/null-style-crash.html
- css/StylePropertySet.cpp:
(WebCore::MutableStylePropertySet::mergeAndOverrideOnConflict):
- css/StylePropertySet.h:
- css/ViewportStyleResolver.cpp:
(WebCore::ViewportStyleResolver::addViewportRule):
- editing/ApplyStyleCommand.cpp:
(WebCore::ApplyStyleCommand::applyInlineStyleToNodeRange):
- editing/EditingStyle.cpp:
(WebCore::styleFromMatchedRulesForElement):
(WebCore::EditingStyle::mergeStyleFromRules):
(WebCore::EditingStyle::mergeStyleFromRulesForSerialization):
LayoutTests:
- editing/execCommand/null-style-crash-expected.txt: Added.
- editing/execCommand/null-style-crash.html: Added.
- 3:53 AM WebKitGTK/WebKit2Roadmap edited by
- (diff)
- 3:52 AM WebKitGTK/WebKit2Roadmap edited by
- (diff)
- 3:48 AM Changeset in webkit [153882] by
-
- 11 edits6 adds in trunk/Source/WebKit2
[GTK] [WebKit2] Add an 'authenticate' signal to WebKitWebView
https://bugs.webkit.org/show_bug.cgi?id=99352
Patch by Brian Holt <brian.holt@samsung.com> on 2013-08-09
Reviewed by Carlos Garcia Campos.
Add a new public class to the API, WebKitAuthenticationRequest, to be
emitted along with a new WebKitWebView::authenticate signal to
let client applications to provide their own authentication
when the user is challenged with HTTP authentication. The
credential is exposed through a new class WebKitCredential.
- GNUmakefile.list.am:
- UIProcess/API/gtk/WebKitAuthenticationDialog.cpp:
(okButtonClicked): Use new API for authentication.
(cancelButtonClicked): Ditto.
(webkitAuthenticationDialogInitialize):Get
authenticationChallenge from request object.
(webkitAuthenticationDialogNew): Ditto.
- UIProcess/API/gtk/WebKitAuthenticationDialog.h:
- UIProcess/API/gtk/WebKitAuthenticationRequest.cpp: Added new
class representing a HTTP authentication request.
(webkitAuthenticationRequestDispose):
(webkit_authentication_request_class_init):
(webkitAuthenticationRequestCreate):
(webkitAuthenticationRequestGetAuthenticationChallenge):
(webkit_authentication_request_can_save_credentials):
(webkit_authentication_request_get_proposed_credential):
(webkit_authentication_request_get_host):
(webkit_authentication_request_get_port):
(webkit_authentication_request_get_realm):
(webkit_authentication_request_get_scheme):
(webkit_authentication_request_is_for_proxy):
(webkit_authentication_request_is_retry):
(webkit_authentication_request_authenticate):
(webkit_authentication_request_cancel):
- UIProcess/API/gtk/WebKitAuthenticationRequest.h: Added.
- UIProcess/API/gtk/WebKitAuthenticationRequestPrivate.h: Added.
- UIProcess/API/gtk/WebKitCredential.cpp: Added new class to
support credentials for HTTP authentication.
(_WebKitCredential::_WebKitCredential):
(webkitCredentialCreate):
(webkitCredentialGetCredential):
(webkit_credential_new):
(webkit_credential_copy):
(webkit_credential_free):
(webkit_credential_get_username):
(webkit_credential_get_password):
(webkit_credential_has_password):
(webkit_credential_get_persistence):
- UIProcess/API/gtk/WebKitCredential.h: Added.
- UIProcess/API/gtk/WebKitCredentialPrivate.h: Added.
- UIProcess/API/gtk/WebKitWebView.cpp: Added authenticate signal.
(webkitWebViewAuthenticate): Default signal handler for authenticate.
(webkit_web_view_class_init): Ditto.
(webkitWebViewHandleAuthenticationChallenge): Create an
authentication request and emit it with authenticate signal.
- UIProcess/API/gtk/WebKitWebView.h: Replaced reserved function
pointer slot with for WebViewClass with function pointer to
authenticate signal handler.
- UIProcess/API/gtk/docs/webkit2gtk-docs.sgml: Added new section for
the authentication request.
- UIProcess/API/gtk/docs/webkit2gtk-sections.txt: Ditto.
- UIProcess/API/gtk/docs/webkit2gtk.types: Added authentication
request and credential types.
- UIProcess/API/gtk/tests/TestWebKitWebView.cpp: Created a new
unit test for HTTP authentication. A libsoup server callback
handler is installed that creates soup messages with appropriate
headers and body to test HTTP authentication.
(testWebViewAuthenticationRequest): Test the authentication
request and credential API.
(testWebViewAuthenticationCancel): Test cancelled authentication.
(testWebViewAuthenticationFailure): Test repeated authentication failures.
(testWebViewAuthenticationNoCredential): Test continue without
credentials.
(testWebViewAuthenticationStorage): Test credential storage.
(testWebViewAuthenticationSuccess): Test successful authentication.
(beforeAll): Added tests.
- UIProcess/API/gtk/webkit2.h: Added WebKitAuthenticationRequest and
WebKitCredential headers.
- 3:15 AM Changeset in webkit [153881] by
-
- 2 edits in trunk/Source/WebCore
ElementRuleCollector: Use range-based for syntax in rule collection loop.
<http://webkit.org/b/119617>
<rdar://problem/14695311>
Reviewed by Antti Koivisto.
- css/ElementRuleCollector.cpp:
(WebCore::ElementRuleCollector::doCollectMatchingRulesForList):
- 2:58 AM Changeset in webkit [153880] by
-
- 27 edits in trunk/Source
StyleRule*::properties() should return const references.
<http://webkit.org/b/119596>
Reviewed by Antti Koivisto.
Source/WebCore:
Make StyleRule::properties() & friends return const-references instead of
pointers. This exposed some pointless null checking, since rules should
always have properties.
All rule objects that contain properties now require a StylePropertySet at
the time of construction instead having you call setProperties() later.
The only rules that could ever have null properties were the default-inserted
animation keyframe rules for 0% and 100% that we'd conjure up if the author
didn't provide them. Now they just have an empty property set instead.
- css/CSSFontFaceRule.cpp:
(WebCore::CSSFontFaceRule::cssText):
- css/CSSFontSelector.cpp:
(WebCore::CSSFontSelector::addFontFaceRule):
- css/CSSPageRule.cpp:
(WebCore::CSSPageRule::cssText):
- css/CSSParser.cpp:
(WebCore::CSSParser::createFilterRule):
(WebCore::CSSParser::createStyleRule):
(WebCore::CSSParser::createFontFaceRule):
(WebCore::CSSParser::rewriteSpecifiers):
- css/CSSStyleRule.cpp:
(WebCore::CSSStyleRule::cssText):
- css/ElementRuleCollector.cpp:
(WebCore::leftToRightDeclaration):
(WebCore::rightToLeftDeclaration):
(WebCore::ElementRuleCollector::addElementStyleProperties):
(WebCore::ElementRuleCollector::doCollectMatchingRulesForList):
- css/PageRuleCollector.cpp:
(WebCore::PageRuleCollector::matchPageRulesForList):
- css/StylePropertySet.cpp:
(WebCore::MutableStylePropertySet::mergeAndOverrideOnConflict):
- css/StylePropertySet.h:
- css/StyleResolver.cpp:
(WebCore::StyleResolver::MatchResult::addMatchedProperties):
(WebCore::StyleResolver::styleForKeyframe):
(WebCore::StyleResolver::keyframeStylesForAnimation):
- css/StyleResolver.h:
- css/StyleRule.cpp:
(WebCore::StyleRule::StyleRule):
(WebCore::StyleRule::create):
(WebCore::StyleRulePage::StyleRulePage):
(WebCore::StyleRuleFontFace::StyleRuleFontFace):
(WebCore::StyleRuleViewport::StyleRuleViewport):
(WebCore::StyleRuleViewport::mutableProperties):
(WebCore::StyleRuleFilter::StyleRuleFilter):
- css/StyleRule.h:
(WebCore::StyleRule::create):
(WebCore::StyleRule::properties):
(WebCore::StyleRuleFontFace::create):
(WebCore::StyleRuleFontFace::properties):
(WebCore::StyleRulePage::create):
(WebCore::StyleRulePage::properties):
(WebCore::StyleRuleViewport::create):
(WebCore::StyleRuleViewport::properties):
(WebCore::StyleRuleFilter::create):
(WebCore::StyleRuleFilter::properties):
- css/StyleSheetContents.cpp:
(WebCore::StyleSheetContents::addSubresourceStyleURLs):
(WebCore::childRulesHaveFailedOrCanceledSubresources):
- css/WebKitCSSFilterRule.cpp:
(WebCore::WebKitCSSFilterRule::cssText):
- css/WebKitCSSKeyframeRule.cpp:
(WebCore::StyleKeyframe::StyleKeyframe):
(WebCore::StyleKeyframe::mutableProperties):
- css/WebKitCSSKeyframeRule.h:
(WebCore::StyleKeyframe::create):
(WebCore::StyleKeyframe::properties):
- css/WebKitCSSViewportRule.cpp:
(WebCore::WebKitCSSViewportRule::cssText):
- editing/ApplyStyleCommand.cpp:
(WebCore::ApplyStyleCommand::applyInlineStyleToNodeRange):
- editing/EditingStyle.cpp:
(WebCore::EditingStyle::mergeStyleFromRules):
(WebCore::EditingStyle::mergeStyleFromRulesForSerialization):
- inspector/InspectorStyleSheet.cpp:
(WebCore::InspectorStyleSheet::revalidateStyle):
- page/PageSerializer.cpp:
(WebCore::PageSerializer::retrieveResourcesForRule):
- svg/SVGFontFaceElement.cpp:
(WebCore::SVGFontFaceElement::SVGFontFaceElement):
(WebCore::SVGFontFaceElement::fontFamily):
(WebCore::SVGFontFaceElement::rebuildFontFace):
Source/WebKit/qt:
- Api/qwebelement.cpp:
(QWebElement::styleProperty):
- 2:12 AM Changeset in webkit [153879] by
-
- 3 edits in trunk/Source/WebKit2
PDFPlugin is disabled by default in nightlies on Mac accidentally
https://bugs.webkit.org/show_bug.cgi?id=119615
Rubber-stamped by Andreas Kling.
Enable PDFPlugin by default on Mac at the WebKit2 level.
- Shared/WebPreferencesStore.h:
- UIProcess/API/C/WKPreferencesPrivate.h:
- 2:09 AM WebKitGTK/Releasing edited by
- Added suggestions for "after release" (diff)
- 2:04 AM Changeset in webkit [153878] by
-
- 2 edits in trunk/Source/WebCore
[Qt] Fix build (broken by changeset r153826).
https://bugs.webkit.org/show_bug.cgi?id=119616
Patch by Julien Brianceau <jbrianceau@nds.com> on 2013-08-09
Reviewed by Andreas Kling.
- xml/parser/XMLDocumentParserQt.cpp:
(WebCore::XMLDocumentParser::XMLDocumentParser):
- 2:03 AM WebKitGTK/StartHacking edited by
- Minor change (diff)
- 2:01 AM WebKitGTK/StartHacking edited by
- Added branches and dependencies (diff)
- 12:53 AM WebKitGTK/StartHacking edited by
- Minor typo (diff)
Aug 8, 2013:
- 11:44 PM Changeset in webkit [153877] by
-
- 11 edits2 adds in trunk
navigator.plugins has plugins in it when plugins are disabled
https://bugs.webkit.org/show_bug.cgi?id=119607
<rdar://problem/14678030>
Reviewed by Anders Carlsson.
Test: plugins/navigator-plugins-disabled.html
- page/Settings.cpp:
(WebCore::Settings::setPluginsEnabled):
Refresh the plugin database when plugins are enabled or disabled.
This is necessary to ensure that navigator.plugins will have the
correct set of plugins after dynamically changing whether plugins
are enabled, which makes the test for this patch possible (but it
makes sense in the browser as well).
- UIProcess/WebProcessProxy.cpp:
(WebKit::WebProcessProxy::getPlugins):
Return a list of all plugins, and a list of only application plugins.
- UIProcess/WebProcessProxy.h:
- UIProcess/WebProcessProxy.messages.in:
- WebProcess/WebCoreSupport/WebPlatformStrategies.h:
- WebProcess/WebCoreSupport/WebPlatformStrategies.cpp:
(WebKit::WebPlatformStrategies::getPluginInfo):
Add applicationPlugin parameter to getPlugins, and storage for the returned list.
(WebKit::WebPlatformStrategies::populatePluginCache):
If plugins are disabled for the given page, give WebCore the list that
only includes application plugins. Otherwise, give it the whole list
as we are doing currently.
- WebCoreSupport/WebPlatformStrategies.mm:
(WebPlatformStrategies::getPluginInfo):
If plugins are disabled for the given page, give WebCore an empty list of
plugins. If WebKit1 had any application plugins, we'd return just them
instead.
Add a test that ensures that navigator.plugins only contains
application plugins when plugins are disabled.
- plugins/navigator-plugins-disabled-expected.txt: Added.
- plugins/navigator-plugins-disabled.html: Added.
- 11:39 PM Changeset in webkit [153876] by
-
- 9 edits1 move7 adds5 deletes in trunk
Allow SVG images to be drawn into canvas without tainting.
https://bugs.webkit.org/show_bug.cgi?id=119492
Reviewed by Darin Adler.
Source/WebCore:
Tests: svg/as-image/svg-canvas-not-tainted.html
svg/as-image/svg-canvas-link-not-colored.html
svg/as-image/svg-canvas-xhtml-tainted.html
- html/HTMLAnchorElement.cpp:
(WebCore::HTMLAnchorElement::parseAttribute): Call shouldProhibitLinks.
(WebCore::shouldProhibitLinks): Added.
- html/HTMLAnchorElement.h:
- html/HTMLImageElement.cpp:
(WebCore::HTMLImageElement::parseAttribute): Call shouldProhibitLinks.
- rendering/svg/RenderSVGRoot.cpp:
(WebCore::RenderSVGRoot::isEmbeddedThroughSVGImage): Use isInSVGImage.
- svg/SVGAElement.cpp:
(WebCore::SVGAElement::svgAttributeChanged): Call shouldProhibitLinks.
- svg/graphics/SVGImage.cpp:
(WebCore::SVGImage::hasSingleSecurityOrigin): Added.
(WebCore::isInSVGImage): Added.
- svg/graphics/SVGImage.h:
LayoutTests:
- fast/canvas/svg-taint-expected.txt: Removed.
- fast/canvas/svg-taint.html: Removed. Obsolete.
- http/tests/security/canvas-remote-read-data-url-svg-image-expected.txt: Removed.
- http/tests/security/canvas-remote-read-data-url-svg-image.html: Removed. Obsolete.
- http/tests/security/canvas-remote-read-svg-image-expected.txt: Removed.
- http/tests/security/canvas-remote-read-svg-image.html: Removed. Obsolete.
- svg/as-image/resources/link-xhtml-svg.svg: Added.
- svg/as-image/resources/link-xhtml.svg: Added.
- svg/as-image/resources/link.svg: Added.
- svg/as-image/svg-canvas-link-not-colored-expected.txt: Added.
- svg/as-image/svg-canvas-link-not-colored.html: Added.
- svg/as-image/svg-canvas-not-tainted-expected.txt: Added.
- svg/as-image/svg-canvas-not-tainted.html: Added.
- svg/as-image/svg-canvas-xhtml-tainted-expected.txt: Added.
- svg/as-image/svg-canvas-xhtml-tainted.html: Added.
- 7:40 PM Changeset in webkit [153875] by
-
- 2 edits in trunk/Source/WebCore
Fix ASSERT(m_contentsSolidColor.isValid()) assertion on iOS at nytimes.com
https://bugs.webkit.org/show_bug.cgi?id=119603
Reviewed by Dean Jackson.
On iOS it's possible for a GraphicsLayer to transition between having
a solid background color, and containing media. This happens on nytimes.com
when playing video.
In this case, GraphicsLayerCA::updateContentsColorLayer() would assert
because m_contentsLayer had been already set up to contain media, but
it would be trying to update the color.
Fix by bailing from all the updateContents* functions if there is no
contents layer, or that layer exists for a different purpose.
- platform/graphics/ca/GraphicsLayerCA.cpp:
(WebCore::GraphicsLayerCA::updateContentsMediaLayer):
(WebCore::GraphicsLayerCA::updateContentsCanvasLayer):
(WebCore::GraphicsLayerCA::updateContentsColorLayer):
- 7:12 PM Changeset in webkit [153874] by
-
- 5 edits2 adds in trunk/Source/ThirdParty/ANGLE
Link to correct DirectX libraries for Win64 ANGLE build.
https://bugs.webkit.org/show_bug.cgi?id=119600
Reviewed by Brent Fulgham.
- ANGLE.vcxproj/DirectX32.props: Added with x86 DirectX library directory.
- ANGLE.vcxproj/DirectX64.props: Added with x64 DirectX library directory.
- ANGLE.vcxproj/libEGL.vcxproj:
- ANGLE.vcxproj/libGLESv2.vcxproj:
Use DirectX32.props and DirectX64.props.
- ANGLE.vcxproj/libEGLCommon.props:
- ANGLE.vcxproj/libGLESv2Common.props:
Removed DirectX library directory.
- 5:49 PM Changeset in webkit [153873] by
-
- 3 edits4 deletes in trunk
Unreviewed, rolling out r153847.
http://trac.webkit.org/changeset/153847
https://bugs.webkit.org/show_bug.cgi?id=119609
caused new SVG animation test failures (Requested by smfr on
#webkit).
Source/WebCore:
- svg/SVGAnimateElement.cpp:
(WebCore::SVGAnimateElement::calculateFromAndByValues):
LayoutTests:
- svg/animations/non-additive-type-from-by-animation-expected.txt: Removed.
- svg/animations/non-additive-type-from-by-animation.html: Removed.
- svg/animations/resources/non-additive-type-from-by-animation.svg: Removed.
- svg/animations/script-tests/non-additive-type-from-by-animation.js: Removed.
- 5:47 PM Changeset in webkit [153872] by
-
- 6 edits in trunk/Source/WebCore
Kill updateStyleForAllDocuments()
https://bugs.webkit.org/show_bug.cgi?id=118093
Reviewed by Geoff Garen.
We should never have to eagerly update style, especially for all documents
that the process knows about. The WebKit model is to update style when
required for layout etc, not to update it eagerly.
This is historical code whose origins are lost in the mists of KHTML.
r42384 makes it possible to do away with this eager updating, and
r55568 removed some calls. Discussion in bug 32580 suggests that
form control code still need this, but tests still pass without it.
- bindings/js/JSCallbackData.cpp:
(WebCore::JSCallbackData::invokeCallback):
- bindings/js/JSCustomXPathNSResolver.cpp:
(WebCore::JSCustomXPathNSResolver::lookupNamespaceURI):
- dom/Document.cpp:
(WebCore::Document::scheduleStyleRecalc):
(WebCore::Document::unscheduleStyleRecalc):
- dom/Document.h:
- dom/Element.cpp:
(WebCore::Element::setActive):
- 5:40 PM Changeset in webkit [153871] by
-
- 3 edits in trunk/LayoutTests
Unreviewed gardening; update test after addKey() was renamed to update().
- media/encrypted-media/encrypted-media-v2-events-expected.txt:
- media/encrypted-media/encrypted-media-v2-events.html:
- 5:16 PM Changeset in webkit [153870] by
-
- 17 edits in branches/safari-537-branch
Merged r153867. <rdar://problem/14691925>
- 5:14 PM Changeset in webkit [153869] by
-
- 17 edits in tags/Safari-537.55.1
Merged r153867. <rdar://problem/14691925>
- 5:11 PM Changeset in webkit [153868] by
-
- 5 edits2 adds in trunk/Websites/webkit.org
Fix webkit.org content overflowing their content box
Patch by Benjamin Poulain <bpoulain@apple.com> on 2013-08-08
Rubberstamped by Simon Fraser.
More fixes to have webkit.org show properly on iPad and iPhone.
- blog/wp-content/themes/webkit/header.php:
Add the correct viewport width for the blog.
- building/debug.html:
- building/launch-debugger-vs2010-small.png: Added.
- building/set-debugging-properties-vs2010-small.png: Added.
Fix the images to make them fit in the page.
- coding/adding-features.html:
Using <pre> for the example of email just make it looks like our code examples.
Change the source to use <blockquote> so that the text can reflow to the content box.
- css/main.css:
(.code, pre, tt, code):
Scroll the content when it overflows the box for code and pre.
- 5:07 PM Changeset in webkit [153867] by
-
- 17 edits in trunk
[EME] MediaKey APIs should be prefixed.
https://bugs.webkit.org/show_bug.cgi?id=119606
Reviewed by Anders Carlsson.
Source/WebCore:
No new tests; updated exsisting tests with new names.
Prefix all new classes with "WebKit" and new methods on existing unprefixed
classes with "webkit"
- Modules/encryptedmedia/MediaKeyMessageEvent.idl: MediaKeyMessageEvent -> WebKitMediaKeyMessageEvent
- Modules/encryptedmedia/MediaKeySession.idl: MediaKeySession -> WebKitMediaKeySession
- Modules/encryptedmedia/MediaKeys.idl: MediaKeys -> WebKitMediaKeys
- html/HTMLMediaElement.idl: keys -> webkitKeys, setMediaKeys -> webkitSetMediaKeys
- html/MediaKeyError.idl: MediaKeyError -> WebKitMediaKeyError
LayoutTests:
- media/encrypted-media/encrypted-media-constants-expected.txt:
- media/encrypted-media/encrypted-media-constants.html:
- media/encrypted-media/encrypted-media-events-expected.txt:
- media/encrypted-media/encrypted-media-events.html:
- media/encrypted-media/encrypted-media-is-type-supported-expected.txt:
- media/encrypted-media/encrypted-media-is-type-supported.html:
- media/encrypted-media/encrypted-media-v2-events-expected.txt:
- media/encrypted-media/encrypted-media-v2-events.html:
- media/encrypted-media/encrypted-media-v2-syntax-expected.txt:
- media/encrypted-media/encrypted-media-v2-syntax.html:
- 5:06 PM Changeset in webkit [153866] by
-
- 4 edits in branches/safari-537-branch
Merged r153851. <rdar://problem/14688700>
- 5:04 PM Changeset in webkit [153865] by
-
- 13 edits2 copies in branches/safari-537-branch
Merged r153838. <rdar://problem/14688350>
- 5:02 PM Changeset in webkit [153864] by
-
- 2 edits in branches/safari-537-branch/Source/WebKit2
Merged r153856. <rdar://problem/14267843>
- 4:57 PM Changeset in webkit [153863] by
-
- 5 edits in branches/safari-537-branch/Source/WebCore
Merged r153810. <rdar://problem/14669010>
- 4:47 PM Changeset in webkit [153862] by
-
- 4 edits in tags/Safari-537.55.1
Merged r153851. <rdar://problem/14688700>
- 4:41 PM Changeset in webkit [153861] by
-
- 13 edits2 copies in tags/Safari-537.55.1
Merged r153838. <rdar://problem/14688350>
- 4:28 PM Changeset in webkit [153860] by
-
- 2 edits in trunk/Source/WebInspectorUI
Remove use of prefixed transition event
https://bugs.webkit.org/show_bug.cgi?id=119598
Reviewed by Joseph Pecoraro.
"webkitTransitionEnd" -> "transitionend".
- UserInterface/Popover.js:
(WebInspector.Popover):
(WebInspector.Popover.prototype.handleEvent):
- 4:28 PM Changeset in webkit [153859] by
-
- 4 edits in trunk
Windows build fixes.
Rubberstamped by Brent Fulgham.
Tools:
- DumpRenderTree/DumpRenderTree.vcxproj/DumpRenderTree/DumpRenderTreeLauncher.vcxproj:
Removed /SAFESEH link option which doesn't work with x64.
WebKitLibraries:
- win/tools/vsprops/debug_wincairo.props: Removed MinimalRebuild option preventing parallel compiling.
- 4:25 PM Changeset in webkit [153858] by
-
- 5 edits in tags/Safari-537.55.1/Source
Versioning.
- 4:19 PM Changeset in webkit [153857] by
-
- 1 copy in tags/Safari-537.55.1
New Tag.
- 4:19 PM Changeset in webkit [153856] by
-
- 2 edits in trunk/Source/WebKit2
Disable suppression of plugin process for Java
https://bugs.webkit.org/show_bug.cgi?id=119604
Reviewed by Sam Weinig.
Workaround, the Java plugin doesn't like this.
- PluginProcess/mac/PluginProcessMac.mm:
(WebKit::PluginProcess::platformInitializeProcess):
- +1 active task count on the java plugin, to inhibit supression.
- 4:12 PM Changeset in webkit [153855] by
-
- 4 edits in trunk/LayoutTests
Unreviewed, remove TestExpectations for removed microdata tests.
- platform/mac/TestExpectations:
- platform/qt/TestExpectations:
- platform/win/TestExpectations:
- 3:51 PM Changeset in webkit [153854] by
-
- 2 edits in trunk/Tools
Add iOS support to run-javascriptcore-tests
https://bugs.webkit.org/show_bug.cgi?id=119602
Reviewed by Joseph Pecoraro.
Make run-javascriptcore-tests work for iOS WebKit.
- Scripts/run-javascriptcore-tests:
- 3:46 PM Changeset in webkit [153853] by
-
- 2 edits in trunk/Websites/webkit.org
Give a smaller viewport to webkit.org on devices
Rubberstamped by Simon Fraser.
The CSS of WebKit.org layout the content on 615px + a right margin of
215px + a left margin of 20px. The total layout width is 850px.
The standard viewport size on mobile devices is 980px. This leaves a gap
on the right side of the content when opening the size on iPhone/iPad.
This patch changes the layout width to be 860 to display more content on devices
while keeping a little margin.
- header.inc:
- 3:38 PM Changeset in webkit [153852] by
-
- 10 edits1 delete in trunk
[WK2] loader/go-back-cached-main-resource.html fails
https://bugs.webkit.org/show_bug.cgi?id=116491
Reviewed by Tim Horton.
- DumpRenderTree/efl/DumpRenderTreeChrome.cpp:
- DumpRenderTree/gtk/DumpRenderTree.cpp:
- DumpRenderTree/qt/DumpRenderTreeQt.cpp:
- DumpRenderTree/win/DumpRenderTree.cpp: Added FIXMEs about making path printing normalization more compatible.
- DumpRenderTree/mac/ResourceLoadDelegate.mm: (-[NSURL _drt_descriptionSuitableForTestResult]): Return last path component in cases where we used to return a full path, which is never desirable. Added a null check to basePath to prevent potentially getting an Objective C exception.
- WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp: (WTR::pathSuitableForTestResult): Changed to use path for main test window, not for a currently loading one (which is different for tests that use multiple windows). Changed to return "(null)" for empty URLs to match WK1 output. Changed to return last path component as last fallback. (WTR::dumpRequestDescriptionSuitableForTestResult): We no longer need to pass main frame URL here, and it was potentially a wrong frame. (WTR::dumpResponseDescriptionSuitableForTestResult): Ditto. (WTR::InjectedBundlePage::willPerformClientRedirectForFrame): Ditto. (WTR::InjectedBundlePage::didInitiateLoadForResource): Ditto. (WTR::InjectedBundlePage::willSendRequestForFrame): Ditto. (WTR::InjectedBundlePage::didReceiveResponseForResource): Ditto.
- 3:29 PM Changeset in webkit [153851] by
-
- 4 edits in trunk
[EME] setMediaKeys function as defined in the EME specification does not work
https://bugs.webkit.org/show_bug.cgi?id=119597
Reviewed by Anders Carlsson.
Source/WebCore:
No new tests; updated media/encrypted-media/encrypted-media-v2-syntax.html
Rename the mediaKeys attribute 'keys' and make readonly, and add an explicit
setMediaKeys() method in HTMLMediaElement's IDL.
- html/HTMLMediaElement.h:
(WebCore::HTMLMediaElement::keys):
- html/HTMLMediaElement.idl:
LayoutTests:
- media/encrypted-media/encrypted-media-v2-syntax-expected.txt:
- media/encrypted-media/encrypted-media-v2-syntax.html:
- 3:01 PM Changeset in webkit [153850] by
-
- 3 edits in trunk/Source/WebCore
Hashing SecurityOrigin's can lead to trouble if you mutate them :(
https://bugs.webkit.org/show_bug.cgi?id=119533
<rdar://problem/12978338>
Reviewed by Andreas Kling.
We are getting into trouble in the following circumstance:
- You have a HashMap<RefPtr<SecurityOrigin>, Value> map.
- You add security origin A (http, www.webkit.org, 80) to the map.
- You mutate security origin A by domain relaxation, so that it has the domain webkit.org.
- You add security origin B (http, www.webkit.org, 80) to the map.
- You mutate security origin B by domain relaxation, so that it has the domain webkit.org.
You now have two identical keys in the map.
- Add few more items to the map causing a rehash.
- When you try to add A and B back into the map, you will have a collision, because they now equal each other.
We should probably stop using SecurityOrigins as keys in HashMaps (and move to using a non-mutable SecurityOriginTuple,
or something), but for now, we can just only use the scheme / host / port part for equality, which is what all the users
really want.
- page/SecurityOriginHash.h:
(WebCore::SecurityOriginHash::equal):
Switch to using isSameSchemeHostPort() for SecurityOriginHash::equal().
- 2:37 PM Changeset in webkit [153849] by
-
- 3 edits in branches/safari-537-branch/Source/WebCore
Merged r153845. <rdar://problem/14688769>
- 2:33 PM Changeset in webkit [153848] by
-
- 2 edits in trunk/Source/WebKit2
Add logging to the primary plugin detection
https://bugs.webkit.org/show_bug.cgi?id=119594
Reviewed by Tim Horton.
Add some simple logging to our primary plug-in detection.
- WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::determinePrimarySnapshottedPlugIn):
- 2:19 PM Changeset in webkit [153847] by
-
- 3 edits4 adds in trunk
ASSERT_NOT_REACHED() touched in WebCore::SVGAnimatedStringAnimator::addAnimatedTypes
https://bugs.webkit.org/show_bug.cgi?id=113224
Reviewed by Dirk Schulze.
Source/WebCore:
Do not perform from-by animations for non-additive attributes.
Test: svg/animations/non-additive-type-from-by-animation.html
- svg/SVGAnimateElement.cpp:
(WebCore::SVGAnimateElement::calculateFromAndByValues):
LayoutTests:
Add a test for animating non-additive attributes in from-by animations
- svg/animations/non-additive-type-from-by-animation-expected.txt: Added.
- svg/animations/non-additive-type-from-by-animation.html: Added.
- svg/animations/resources/non-additive-type-from-by-animation.svg: Added.
- svg/animations/script-tests/non-additive-type-from-by-animation.js: Added.
(sample):
(executeTest):
- 2:17 PM Changeset in webkit [153846] by
-
- 2 edits in trunk/LayoutTests
fullscreen/exit-full-screen-iframe.html is failing
https://bugs.webkit.org/show_bug.cgi?id=90708
Unreviewed gardening.
- platform/mac/TestExpectations: Mark test as flakey, it doesn't always pass.
- 2:12 PM Changeset in webkit [153845] by
-
- 3 edits in trunk/Source/WebCore
[Windows] Improper Caption Button Displayed to Users
https://bugs.webkit.org/show_bug.cgi?id=119593
Reviewed by Eric Carlson.
Remove the custom drawing code for the Windows port, and
have it render using the CSS/SVG markup like we do on OS X.
- rendering/RenderThemeWin.cpp: Remove drawing code.
- rendering/RenderThemeWin.h: Remove declaration.
- 2:05 PM Changeset in webkit [153844] by
-
- 2 edits in trunk/LayoutTests
fast/events/scroll-div-with-prevent-default-in-subframe.html is failing
https://bugs.webkit.org/show_bug.cgi?id=82747
Unreviewed gardening.
- platform/mac/TestExpectations: Mark test as flakey, it doesn't always pass.
- 1:19 PM Changeset in webkit [153843] by
-
- 6 edits in trunk/Source/WebCore
Work towards WebGL on AppleWin port.
https://bugs.webkit.org/show_bug.cgi?id=119562
Reviewed by Brent Fulgham.
- WebCore.vcxproj/WebCore.vcxproj:
Build GLContext.cpp and GraphicsContext3DPrivate.cpp on AppleWin port.
- WebCore.vcxproj/WebCore.vcxproj.filters:
Moved GLContext and GraphicsContext3DPrivate from Cairo filter to graphics.
- platform/graphics/GraphicsContext3D.h:
Added m_webGLLayer for AppleWin port.
- platform/graphics/GraphicsContext3DPrivate.cpp:
Protected inclusion of PlatformContextCairo.h in USE(CAIRO) macro for non-cairo ports.
- platform/graphics/win/GraphicsContext3DWin.cpp:
Removed stub GraphicsContext3DPrivate implementation in favor of GraphicsContext3DPrivate.cpp.
(WebCore::GraphicsContext3D::GraphicsContext3D):
Added GraphicsContext3DPrivate and PlatformCALayer initialization.
(WebCore::GraphicsContext3D::makeContextCurrent):
(WebCore::GraphicsContext3D::platformGraphicsContext3D):
Added implementation based on Cairo ports' to be refactored soon.
(WebCore::GraphicsContext3D::platformLayer):
Return PlatformCALayer's PlatformLayer instead of 0.
- 12:53 PM Changeset in webkit [153842] by
-
- 2 edits in trunk/LayoutTests
http/tests/navigation/back-twice-without-commit.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=82980
Unreviewed gardening.
- platform/mac/TestExpectations: Unskip passing test.
- 12:48 PM Changeset in webkit [153841] by
-
- 2 edits in trunk/LayoutTests
Layout Test fast/overflow/scrollbar-click-retains-focus.html is failing on Apple Mac WK1 and GTK.
https://bugs.webkit.org/show_bug.cgi?id=104104
Unreviewed gardening.
- platform/mac/TestExpectations: Unskip passing test.
- 12:22 PM Changeset in webkit [153840] by
-
- 5 edits in trunk/Source/WebInspectorUI
Unprefix use of transitions in Web Inspector code
https://bugs.webkit.org/show_bug.cgi?id=119588
Reviewed by Tim Hatcher.
Replace "-webkit-transition" with "transition".
- UserInterface/CSSCompletions.js:
(WebInspector.CSSCompletions.requestCSSNameCompletions.collectPropertyNameForCodeMirror):
- UserInterface/FindBanner.css:
(.find-banner):
(.find-banner.showing):
(.supports-find-banner.no-find-banner-transition):
(.supports-find-banner):
(.supports-find-banner.showing-find-banner):
- UserInterface/NetworkTimeline.css:
(.network-graph-side:hover .network-graph-label):
- UserInterface/Popover.css:
(.popover.fade-out):
- 12:22 PM Changeset in webkit [153839] by
-
- 2 edits in trunk/Source/WebKit2
WKBundleFrameCopyProvisionalURL crashes when there is no provisional loader
https://bugs.webkit.org/show_bug.cgi?id=119587
Reviewed by Darin Adler.
- WebProcess/WebPage/WebFrame.cpp: (WebKit::WebFrame::provisionalURL): Added a null check, like we already have in WebFrame::url().
- 12:17 PM Changeset in webkit [153838] by
-
- 13 edits2 adds in trunk
[EME] Implement MediaKeys.isTypeSupported()
https://bugs.webkit.org/show_bug.cgi?id=119586
Reviewed by Eric Carlson.
Source/WebCore:
Test: media/encrypted-media/encrypted-media-is-type-supported.html
The EME spec has removed the extra keySystem parameter from
HTMLMediaElement.canPlayType() in favor of a new isTypeSupported()
method on MediaKeys.
To implement this method, the CDMFactory needs to know whether the
registered CDM supports the specified mime type. CDMs must register
this new factory method, which requires changes to CDMPrivateAVFoundation
and MockCDM.
Add the new MediaKeys.isTypeSupported() method:
- Modules/encryptedmedia/MediaKeys.cpp:
(WebCore::MediaKeys::isTypeSupported):
- Modules/encryptedmedia/MediaKeys.h:
- Modules/encryptedmedia/MediaKeys.idl:
Add the new factory CDMSupportsKeySystemAndMimeType method:
- Modules/encryptedmedia/CDM.cpp:
(WebCore::CDMFactory::CDMFactory): Now takes a third parameter.
(WebCore::installedCDMFactories): When adding the AVFoundation CDM, pass its new method.
(WebCore::CDM::registerCDMFactory): Now takes a third parameter.
(WebCore::CDM::keySystemSupportsMimeType): Added.
- Modules/encryptedmedia/CDM.h:
- WebCore.exp.in:
Register this new method with the factory for all concrete CDMs:
- Modules/encryptedmedia/CDMPrivateAVFoundation.h:
- Modules/encryptedmedia/CDMPrivateAVFoundation.mm:
(WebCore::CDMPrivateAVFoundation::supportsKeySystem):
(WebCore::CDMPrivateAVFoundation::supportsKeySystemAndMimeType):
- testing/Internals.cpp:
(WebCore::Internals::initializeMockCDM):
- testing/MockCDM.cpp:
(WebCore::MockCDM::supportsKeySystem):
(WebCore::MockCDM::supportsKeySystemAndMimeType):
- testing/MockCDM.h:
LayoutTests:
- media/encrypted-media/encrypted-media-is-type-supported-expected.txt: Added.
- media/encrypted-media/encrypted-media-is-type-supported.html: Added.
- 11:47 AM Changeset in webkit [153837] by
-
- 2 edits in trunk/LayoutTests
fast/events/scroll-div-with-prevent-default-in-subframe.html is failing
https://bugs.webkit.org/show_bug.cgi?id=82747
Unreviewed gardening.
- platform/mac/TestExpectations: Unskip passing test.
- 11:40 AM Changeset in webkit [153836] by
-
- 2 edits in trunk/LayoutTests
REGRESSION(r130839): fast/canvas/webgl/arraybuffer-transfer-of-control.html is failing
https://bugs.webkit.org/show_bug.cgi?id=99096
Unreviewed gardening.
- platform/mac/TestExpectations: Unskip passing test.
- 11:38 AM Changeset in webkit [153835] by
-
- 3 edits2 adds in trunk
Crash in WTF::RefPtr<WebCore::SpaceSplitStringData>::operator UnspecifiedBoolType
https://bugs.webkit.org/show_bug.cgi?id=118839
Reviewed by Darin Adler.
Source/WebCore:
Take into account that elementData() can be null in classAttributeChanged.
Test: svg/animations/classAttributeSettingCrash.html
- dom/Element.cpp:
(WebCore::Element::classAttributeChanged):
LayoutTests:
Add testcase that showed up to cause a crash by the fuzzer.
- svg/animations/classAttributeSettingCrash-expected.txt: Added.
- svg/animations/classAttributeSettingCrash.html: Added.
- 11:35 AM Changeset in webkit [153834] by
-
- 2 edits in trunk/LayoutTests
fullscreen/exit-full-screen-iframe.html is failing
https://bugs.webkit.org/show_bug.cgi?id=90708
Unreviewed gardening.
- platform/mac/TestExpectations: Unskip passing test.
- 11:17 AM Changeset in webkit [153833] by
-
- 6 edits in trunk/LayoutTests
svg/custom/use-href-update-crash.svg failing on certain platforms
https://bugs.webkit.org/show_bug.cgi?id=104054
Reviewed by Darin Adler.
Fix mistake in use-href-update-crash.svg which caused the gc() not found problem. Update the test expectations
that had this marked as failure, it passes after the test fix.
- platform/gtk/TestExpectations:
- platform/mac/TestExpectations:
- platform/qt/TestExpectations:
- platform/win/TestExpectations:
- svg/custom/use-href-update-crash.svg:
- 11:13 AM Changeset in webkit [153832] by
-
- 2 edits in trunk/Source/WebCore
REGRESSION(r147019): Page has extra space
https://bugs.webkit.org/show_bug.cgi?id=119424
Reviewed by David Hyatt.
Clear the overflow rect before recomputing it in table sections.
No new tests: the reduction uses jquery and I was unable to simulate the reduction without it.
- rendering/RenderTableSection.cpp:
(WebCore::RenderTableSection::layoutRows):
(WebCore::RenderTableSection::computeOverflowFromCells):
- 10:47 AM Changeset in webkit [153831] by
-
- 2 edits in branches/safari-537-branch/Source/JavaScriptCore
Merge 153819 for <rdar://problem/14680524>
- 10:41 AM Changeset in webkit [153830] by
-
- 5 edits in trunk
WTR::pathSuitableForTestResult should behave the same as _drt_descriptionSuitableForTestResult so we can unskip tests.
https://bugs.webkit.org/show_bug.cgi?id=116125
Reviewed by Tim Horton. Tweaked and re-landed by Alexey Proskuryakov.
- WebKitTestRunner/InjectedBundle/InjectedBundlePage.cpp: (WTR::pathSuitableForTestResult): Changed string operations to match _drt_descriptionSuitableForTestResult. (WTR::dumpRequestDescriptionSuitableForTestResult): (WTR::dumpResponseDescriptionSuitableForTestResult): (WTR::InjectedBundlePage::willPerformClientRedirectForFrame): (WTR::InjectedBundlePage::didInitiateLoadForResource): (WTR::InjectedBundlePage::willSendRequestForFrame): (WTR::InjectedBundlePage::didReceiveResponseForResource): Pass the main frame's URL to pathSuitableForTestResult.
- WebKitTestRunner/InjectedBundle/InjectedBundlePage.h: Added WKBundlePageRef to willPerformClientRedirectForFrame.
- 10:41 AM Changeset in webkit [153829] by
-
- 8 edits in trunk/Source/WebCore
Inserting multiple rules into an empty style sheet should avoid style recalc if possible.
<http://webkit.org/b/119568>
Reviewed by Antti Koivisto.
As a follow-up to <http://webkit.org/b/119475>, where I added a hack for inserting
a single rule into an empty style sheet, this broadens the optimization to support
any number of rules.
This optimizes the scenario where a style sheet is added to the DOM and then populated
rule-by-rule via CSSOM insertRule()/addRule() calls. Previously we'd do a full style
recalc for this case, but now we'll treat it the same as a full sheet added at once.
- css/CSSStyleSheet.h:
- css/CSSStyleSheet.cpp:
(WebCore::CSSStyleSheet::willMutateRules):
Made willMutateRules() return whether the style sheet contents were cloned by the
copy-on-write mechanism.
- dom/Document.h:
(WebCore::CSSStyleSheet::didMutateRules):
(WebCore::CSSStyleSheet::insertRule):
Replaced the InsertionIntoEmptySheet mutation type by a general RuleInsertion.
The mutation callback checks if we're inserting into a rule that's not (yet) part
of the document's active sheet set. In that case, we defer doing the style sheet
until all the insertions are done (or something forces us to style+layout.)
Note that this optimization only happens if the style sheet had a single client.
Shared style sheets that just got cloned before mutation may have pointers into
them from the Document's StyleResolver, so we're forced to do an immediate sheet
update in that case.
(WebCore::CSSStyleSheet::RuleMutationScope::RuleMutationScope):
(WebCore::CSSStyleSheet::RuleMutationScope::~RuleMutationScope):
Moved these out-of-line.
(WebCore::CSSStyleSheet::didMutateRuleFromCSSStyleDeclaration):
- css/PropertySetCSSStyleDeclaration.cpp:
(WebCore::StyleRuleCSSStyleDeclaration::didMutate):
Made a separate mutation callback for CSSStyleDeclaration since its needs are
so simple compared to the mutation callback from CSSStyleSheet. Seems better
than adding yet another mode to the enum.
- dom/Document.cpp:
(WebCore::Document::Document):
(WebCore::Document::recalcStyle):
(WebCore::Document::styleResolverChanged):
(WebCore::Document::optimizedStyleSheetUpdateTimerFired):
(WebCore::Document::scheduleOptimizedStyleSheetUpdate):
Added mechanism to defer doing a RecalcStyleIfNeeded.
(WebCore::Document::updateStyleIfNeeded):
Synchronize the optimized style sheet update if there's one scheduled.
This ensures that stuff like layout-dependent property access won't operate
on stale style.
- dom/DocumentStyleSheetCollection.h:
(WebCore::DocumentStyleSheetCollection::pendingUpdateType):
(WebCore::DocumentStyleSheetCollection::setPendingUpdateType):
(WebCore::DocumentStyleSheetCollection::flushPendingUpdates):
- dom/DocumentStyleSheetCollection.cpp:
(WebCore::DocumentStyleSheetCollection::DocumentStyleSheetCollection):
(WebCore::DocumentStyleSheetCollection::updateActiveStyleSheets):
Have DSSC track the kind of style sheet update it needs to do (instead of just
a boolean "needs update.") This is used by Document::recalcStyle() to make sure
the right kind of update happens if there's one scheduled.
(WebCore::DocumentStyleSheetCollection::activeStyleSheetsContains):
Added helper to check if a CSSStyleSheet is part of the active set.
- 10:38 AM Changeset in webkit [153828] by
-
- 2 edits in trunk/Source/WebCore
[WK1] NPStream::headers not initialized
https://bugs.webkit.org/show_bug.cgi?id=119574
Reviewed by Alexey Proskuryakov.
Initialize the header field. According to user feedback that should solve some
cases of random crashes when loading plugins on Mac and Windows.
- plugins/PluginStream.cpp:
(WebCore::PluginStream::PluginStream):
- 10:27 AM Changeset in webkit [153827] by
-
- 1 edit in trunk/Source/WebCore/ChangeLog
Oops, didn't mean to implicate Mark Lam in the ChangeLog there.
- 10:25 AM Changeset in webkit [153826] by
-
- 30 edits in trunk/Source
Element: Modernize attribute storage accessor functions.
<http://webkit.org/b/119578>
Source/WebCore:
Patch by Mark Lam <mark.lam@apple.com> on 2013-08-08
Reviewed by Antti Koivisto.
Rename these functions to more WebKit-style names.
- const Attribute* attributeItem(index) => const Attribute& attributeAt(index)
- const Attribute* getAttributeItem(name) => const Attribute* findAttributeByName(name)
- unsigned getAttributeItemIndex(name) => unsigned findAttributeIndexByName(name)
Note that attributeAt() now returns a reference instead of a pointer. That's right.
- css/SelectorChecker.cpp:
(WebCore::attributeValueMatches):
(WebCore::anyAttributeMatches):
- css/SelectorChecker.h:
(WebCore::SelectorChecker::checkExactAttribute):
- dom/Attr.cpp:
(WebCore::Attr::elementAttribute):
- dom/DatasetDOMStringMap.cpp:
(WebCore::DatasetDOMStringMap::getNames):
(WebCore::DatasetDOMStringMap::item):
(WebCore::DatasetDOMStringMap::contains):
- dom/Element.cpp:
(WebCore::Element::detachAttribute):
(WebCore::Element::removeAttribute):
(WebCore::Element::getAttribute):
(WebCore::Element::setAttribute):
(WebCore::Element::setSynchronizedLazyAttribute):
(WebCore::Element::setAttributeInternal):
(WebCore::Element::setAttributeNode):
(WebCore::Element::removeAttributeNode):
(WebCore::Element::removeAttributeInternal):
(WebCore::Element::getAttributeNode):
(WebCore::Element::getAttributeNodeNS):
(WebCore::Element::hasAttribute):
(WebCore::Element::hasAttributeNS):
(WebCore::Element::computeInheritedLanguage):
(WebCore::Element::normalizeAttributes):
(WebCore::Element::getURLAttribute):
(WebCore::Element::getNonEmptyURLAttribute):
(WebCore::Element::detachAllAttrNodesFromElement):
(WebCore::Element::cloneAttributesFromElement):
(WebCore::ElementData::isEquivalent):
(WebCore::ElementData::findAttributeIndexByNameSlowCase):
(WebCore::ElementData::findAttributeIndexByNameForAttributeNode):
(WebCore::UniqueElementData::findAttributeByName):
(WebCore::UniqueElementData::attributeAt):
- dom/Element.h:
(WebCore::Element::findAttributeIndexByName):
(WebCore::Element::fastHasAttribute):
(WebCore::Element::fastGetAttribute):
(WebCore::Element::attributeAt):
(WebCore::Element::findAttributeByName):
(WebCore::ElementData::findAttributeByName):
(WebCore::ElementData::findAttributeIndexByName):
(WebCore::ElementData::attributeAt):
- dom/NamedNodeMap.cpp:
(WebCore::NamedNodeMap::removeNamedItem):
(WebCore::NamedNodeMap::removeNamedItemNS):
(WebCore::NamedNodeMap::item):
- dom/Node.cpp:
(WebCore::Node::dumpStatistics):
(WebCore::Node::isDefaultNamespace):
(WebCore::Node::lookupNamespaceURI):
(WebCore::Node::lookupNamespacePrefix):
(WebCore::Node::compareDocumentPosition):
- dom/StyledElement.cpp:
(WebCore::StyledElement::makePresentationAttributeCacheKey):
(WebCore::StyledElement::rebuildPresentationAttributeStyle):
- editing/MarkupAccumulator.cpp:
(WebCore::MarkupAccumulator::appendElement):
- editing/markup.cpp:
(WebCore::completeURLs):
(WebCore::StyledMarkupAccumulator::appendElement):
- html/HTMLEmbedElement.cpp:
(WebCore::HTMLEmbedElement::parametersForPlugin):
- html/HTMLInputElement.cpp:
(WebCore::HTMLInputElement::updateType):
- html/HTMLObjectElement.cpp:
(WebCore::HTMLObjectElement::parametersForPlugin):
- html/HTMLSelectElement.cpp:
(WebCore::HTMLSelectElement::parseAttribute):
- html/parser/HTMLConstructionSite.cpp:
(WebCore::HTMLConstructionSite::mergeAttributesFromTokenIntoElement):
- inspector/DOMPatchSupport.cpp:
(WebCore::DOMPatchSupport::innerPatchNode):
(WebCore::DOMPatchSupport::createDigest):
- inspector/InspectorDOMAgent.cpp:
(WebCore::InspectorDOMAgent::setAttributesAsText):
(WebCore::InspectorDOMAgent::performSearch):
(WebCore::InspectorDOMAgent::buildArrayForElementAttributes):
- page/PageSerializer.cpp:
(WebCore::isCharsetSpecifyingNode):
- svg/SVGStyledElement.cpp:
(WebCore::SVGStyledElement::getPresentationAttribute):
- xml/XPathFunctions.cpp:
(WebCore::XPath::FunLang::evaluate):
- xml/XPathNodeSet.cpp:
(WebCore::XPath::NodeSet::traversalSort):
- xml/XPathStep.cpp:
(WebCore::XPath::Step::nodesInAxis):
- xml/parser/XMLDocumentParserLibxml2.cpp:
(WebCore::XMLDocumentParser::XMLDocumentParser):
- xml/parser/XMLDocumentParserQt.cpp:
(WebCore::XMLDocumentParser::XMLDocumentParser):
Source/WebKit/blackberry:
Reviewed by Antti Koivisto.
- Api/WebPage.cpp:
(BlackBerry::WebKit::WebPagePrivate::webContext):
Source/WebKit/qt:
Reviewed by Antti Koivisto.
- Api/qwebelement.cpp:
(QWebElement::attributeNames):
- 9:57 AM Changeset in webkit [153825] by
-
- 6 edits in trunk/Source
Restoring use of StackIterator instead of Interpreter::getStacktrace().
https://bugs.webkit.org/show_bug.cgi?id=119575.
Reviewed by Oliver Hunt.
Source/JavaScriptCore:
- interpreter/Interpreter.h:
- Made getStackTrace() private.
- interpreter/StackIterator.cpp:
(JSC::StackIterator::StackIterator):
(JSC::StackIterator::numberOfFrames):
- Computes the number of frames by iterating through the whole stack from the starting frame. The iterator will save its current frame position before counting the frames, and then restoring it after the counting.
(JSC::StackIterator::gotoFrameAtIndex):
(JSC::StackIterator::gotoNextFrame):
(JSC::StackIterator::resetIterator):
- Points the iterator to the starting frame.
- interpreter/StackIteratorPrivate.h:
Source/WebCore:
No new tests.
- bindings/js/ScriptCallStackFactory.cpp:
(WebCore::createScriptCallStack):
- 9:55 AM Changeset in webkit [153824] by
-
- 12 edits5 copies34 adds in trunk
SunSpider: Move the 3d-morph fix from version 1.0 to version 1.0.1
https://bugs.webkit.org/show_bug.cgi?id=117993
Patch by Cosmin Truta <ctruta@blackberry.com> on 2013-08-08
Reviewed by George Staikos.
PerformanceTests/SunSpider:
Bump the SunSpider version to 1.0.1.
Unroll r149548 in order to restore version 1.0 to its original,
and add the 3d-morph fix to version 1.0.1 instead.
- hosted/sunspider.html: Updated to version 1.0.1.
- hosted/versions.html: Updated to version 1.0.1.
- make-hosted: Added support for version 1.0.1.
- resources/driver-TEMPLATE.html: Updated to version 1.0.1.
- resources/results-TEMPLATE.html: Updated to version 1.0.1.
- sunspider: Updated to version 1.0.1.
- sunspider-compare-results: Updated to version 1.0.1.
- tests/sunspider-1.0/3d-morph.js: Rolled back to the original version.
- tests/sunspider-1.0.1/LIST: Added.
- tests/sunspider-1.0.1/3d-cube.js: Added.
- tests/sunspider-1.0.1/3d-morph.js: Added.
- tests/sunspider-1.0.1/3d-raytrace.js: Added.
- tests/sunspider-1.0.1/access-binary-trees.js: Added.
- tests/sunspider-1.0.1/access-fannkuch.js: Added.
- tests/sunspider-1.0.1/access-nbody.js: Added.
- tests/sunspider-1.0.1/access-nsieve.js: Added.
- tests/sunspider-1.0.1/bitops-3bit-bits-in-byte.js: Added.
- tests/sunspider-1.0.1/bitops-bits-in-byte.js: Added.
- tests/sunspider-1.0.1/bitops-bitwise-and.js: Added.
- tests/sunspider-1.0.1/bitops-nsieve-bits.js: Added.
- tests/sunspider-1.0.1/controlflow-recursive.js: Added.
- tests/sunspider-1.0.1/crypto-aes.js: Added.
- tests/sunspider-1.0.1/crypto-md5.js: Added.
- tests/sunspider-1.0.1/crypto-sha1.js: Added.
- tests/sunspider-1.0.1/date-format-tofte.js: Added.
- tests/sunspider-1.0.1/date-format-xparb.js: Added.
- tests/sunspider-1.0.1/math-cordic.js: Added.
- tests/sunspider-1.0.1/math-partial-sums.js: Added.
- tests/sunspider-1.0.1/math-spectral-norm.js: Added.
- tests/sunspider-1.0.1/regexp-dna.js: Added.
- tests/sunspider-1.0.1/string-base64.js: Added.
- tests/sunspider-1.0.1/string-fasta.js: Added.
- tests/sunspider-1.0.1/string-tagcloud.js: Added.
- tests/sunspider-1.0.1/string-unpack-code.js: Added.
- tests/sunspider-1.0.1/string-validate-input.js: Added.
Websites/webkit.org:
- perf/sunspider-1.0.1/json2.js: Added.
- perf/sunspider-1.0.1/sunspider-1.0.1/driver.html: Added.
- perf/sunspider-1.0.1/sunspider-1.0.1/results.html: Added.
- perf/sunspider-1.0.1/sunspider-1.0.1/sunspider-test-contents.js: Added.
- perf/sunspider-1.0.1/sunspider-1.0.1/sunspider-test-prefix.js: Added.
- perf/sunspider-1.0.1/sunspider-analyze-results.js: Added.
- perf/sunspider-1.0.1/sunspider-compare-results.js: Added.
- perf/sunspider-1.0.1/sunspider.css: Added.
- perf/sunspider-1.0.1/sunspider.html: Added.
- perf/sunspider/sunspider.html: Updated to version 1.0.1.
- perf/sunspider/versions.html: Updated to version 1.0.1.
- 9:38 AM Changeset in webkit [153823] by
-
- 4 edits in trunk/Source/JavaScriptCore
Moved ErrorConstructor and NativeErrorConstructor helper functions into
the Interpreter class.
https://bugs.webkit.org/show_bug.cgi?id=119576.
Reviewed by Oliver Hunt.
This change is needed to prepare for making Interpreter::getStackTrace()
private. It does not change the behavior of the code, only the lexical
scoping.
- interpreter/Interpreter.h:
- Added helper functions for ErrorConstructor and NativeErrorConstructor.
- runtime/ErrorConstructor.cpp:
(JSC::Interpreter::constructWithErrorConstructor):
(JSC::ErrorConstructor::getConstructData):
(JSC::Interpreter::callErrorConstructor):
(JSC::ErrorConstructor::getCallData):
- Don't want ErrorConstructor to call Interpreter::getStackTrace() directly. So, we moved the helper functions into the Interpreter class.
- runtime/NativeErrorConstructor.cpp:
(JSC::Interpreter::constructWithNativeErrorConstructor):
(JSC::NativeErrorConstructor::getConstructData):
(JSC::Interpreter::callNativeErrorConstructor):
(JSC::NativeErrorConstructor::getCallData):
- Don't want NativeErrorConstructor to call Interpreter::getStackTrace() directly. So, we moved the helper functions into the Interpreter class.
- 7:49 AM Changeset in webkit [153822] by
-
- 5 edits2 adds in trunk
REGRESSION (r121551) Incorrect handling of invalid media query list.
https://bugs.webkit.org/show_bug.cgi?id=119164
Reviewed by Antti Koivisto.
Differentiate between syntactically invalid media queries and media queries with empty expressions.
Introduce a dedicated function for valid, but empty expression queries, so that passing NULL MediaQuerySet
object to CSSParser::createMediaRule() can indicate invalid media query.
Source/WebCore:
Test: fast/media/invalid-media-query-list.html
- css/CSSGrammar.y.in:
- css/CSSParser.cpp:
(WebCore::CSSParser::createMediaRule):
(WebCore::CSSParser::createEmptyMediaRule):
- css/CSSParser.h:
LayoutTests:
- fast/media/invalid-media-query-list-expected.html: Added.
- fast/media/invalid-media-query-list.html: Added.
- 6:30 AM Changeset in webkit [153821] by
-
- 25 edits3 adds in trunk/LayoutTests
[Qt] Unreviewed rebaselining after r153647.
- platform/qt/fast/css/input-search-padding-expected.png:
- platform/qt/fast/css/input-search-padding-expected.txt:
- platform/qt/fast/css/text-overflow-input-expected.png:
- platform/qt/fast/css/text-overflow-input-expected.txt:
- platform/qt/fast/forms/box-shadow-override-expected.png:
- platform/qt/fast/forms/box-shadow-override-expected.txt:
- platform/qt/fast/forms/control-restrict-line-height-expected.png:
- platform/qt/fast/forms/control-restrict-line-height-expected.txt:
- platform/qt/fast/forms/placeholder-position-expected.png:
- platform/qt/fast/forms/placeholder-position-expected.txt:
- platform/qt/fast/forms/placeholder-pseudo-style-expected.png:
- platform/qt/fast/forms/placeholder-pseudo-style-expected.txt:
- platform/qt/fast/forms/search-cancel-button-style-sharing-expected.png:
- platform/qt/fast/forms/search-cancel-button-style-sharing-expected.txt:
- platform/qt/fast/forms/search-display-none-cancel-button-expected.png:
- platform/qt/fast/forms/search-display-none-cancel-button-expected.txt:
- platform/qt/fast/forms/search-rtl-expected.png:
- platform/qt/fast/forms/search-rtl-expected.txt:
- platform/qt/fast/forms/search-styled-expected.png:
- platform/qt/fast/forms/search-styled-expected.txt:
- platform/qt/fast/forms/search-vertical-alignment-expected.png:
- platform/qt/fast/forms/search-vertical-alignment-expected.txt:
- platform/qt/fast/forms/search/search-size-with-decorations-expected.png: Added.
- platform/qt/fast/forms/search/search-size-with-decorations-expected.txt: Added.
- platform/qt/fast/repaint/search-field-cancel-expected.png:
- platform/qt/fast/repaint/search-field-cancel-expected.txt:
- 4:00 AM Changeset in webkit [153820] by
-
- 4 edits in trunk/Source/WebCore
Introduce toSVGMPathElement(), and use it
https://bugs.webkit.org/show_bug.cgi?id=119566
Reviewed by Andreas Kling.
As a step to change static_cast with toSVGXXX, static_cast<SVGMPathElement*> can
be changed with toSVGMPathElement().
- svg/SVGAnimateMotionElement.cpp:
(WebCore::SVGAnimateMotionElement::updateAnimationPath):
- svg/SVGMPathElement.h:
(WebCore::toSVGMPathElement):
- svg/SVGPathElement.cpp:
(WebCore::SVGPathElement::invalidateMPathDependencies):
- 3:25 AM Changeset in webkit [153819] by
-
- 2 edits in trunk/Source/JavaScriptCore
<rdar://problem/14680524> REGRESSION(153806): Crash @ yahoo.com when WebKit is built with a .order file
Unreviewed.
Ensure llint symbols are in source order.
- JavaScriptCore.order:
- 2:54 AM Changeset in webkit [153818] by
-
- 2 edits in trunk/LayoutTests
[Qt] Skip more tests depending on canvas blending.
https://bugs.webkit.org/show_bug.cgi?id=100072
Unreviewed gardening.
- platform/qt/TestExpectations:
- 12:58 AM Changeset in webkit [153817] by
-
- 4 edits in trunk/Source/WebInspectorUI
Web Inspector: Use granular DOMStorage modification events to avoid complete DataGrid update.
https://bugs.webkit.org/show_bug.cgi?id=119541
Reviewed by Joseph Pecoraro.
InspectorDOMStorageAgent sends more granular events about the storage modifications.
Using these would avoid the complete rebuilding of the DataGrid.
- UserInterface/DOMStorageContentView.js:
(WebInspector.DOMStorageContentView.prototype.itemsCleared):
(WebInspector.DOMStorageContentView.prototype.itemRemoved):
(WebInspector.DOMStorageContentView.prototype.itemAdded):
(WebInspector.DOMStorageContentView.prototype.itemUpdated):
- UserInterface/DOMStorageObserver.js:
(WebInspector.DOMStorageObserver.prototype.domStorageItemsCleared):
(WebInspector.DOMStorageObserver.prototype.domStorageItemRemoved):
(WebInspector.DOMStorageObserver.prototype.domStorageItemAdded):
(WebInspector.DOMStorageObserver.prototype.domStorageItemUpdated):
- UserInterface/StorageManager.js:
(WebInspector.StorageManager.prototype.domStorageItemsCleared):
(WebInspector.StorageManager.prototype.domStorageItemRemoved):
(WebInspector.StorageManager.prototype.domStorageItemAdded):
(WebInspector.StorageManager.prototype.domStorageItemUpdated):
- 12:18 AM Changeset in webkit [153816] by
-
- 10 edits in trunk/Source/WebCore
Move more style recalc code to StyleResolveTree.cpp
https://bugs.webkit.org/show_bug.cgi?id=119553
Reviewed by Andreas Kling.
- css/StyleResolver.cpp:
- css/StyleResolver.h:
Remove now unused StyleResolver::styleForText.
- dom/ElementShadow.cpp:
- dom/ElementShadow.h:
Remove ElementShadow::recalcStyle.
- dom/ShadowRoot.cpp:
(WebCore::ShadowRoot::setResetStyleInheritance):
Use Style::resolveTree instead of ElementShadow::recalcStyle.
- dom/ShadowRoot.h:
Remove ShadowRoot::recalcStyle.
- dom/Text.cpp:
- dom/Text.h:
Remove Text::recalcTextStyle.
- style/StyleResolveTree.cpp:
(WebCore::Style::updateTextStyle):
Moved from Text::recalcTextStyle. Speed up by just setting the parent style on common case.
(WebCore::Style::resolveShadowTree):
Moved from ShadowRoot::recalcStyle.
Simplified to cover only cases that can occur with the current shadow DOM support.
(WebCore::Style::resolveTree):
- 12:16 AM Changeset in webkit [153815] by
-
- 3 edits2 adds in trunk
REGRESSION(r139282): Old caret sometimes gets "stuck" (not repainted) in contenteditable elements.
<http://webkit.org/b/119520>
<rdar://problem/14658929>
Reviewed by Simon Fraser.
Source/WebCore:
When computing a new caret rect, don't forget to repaint the old rect if the caret jumped
from one node to another.
We were being a little too clever, assuming that oldRect==newRect meant that no repaint was
necessary, but moving from (0,0) in one node to (0,0) in another would cause a ghost caret
in the old node.
Test: fast/repaint/caret-jump-between-nodes.html
- editing/FrameSelection.cpp:
(WebCore::FrameSelection::recomputeCaretRect):
Fall through all the way down to caret repaints if the selection's new anchor node is not
the same as m_previousCaretNode.
LayoutTests:
- fast/repaint/caret-jump-between-nodes-expected.txt: Added.
- fast/repaint/caret-jump-between-nodes.html: Added.
