⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Timeline



Jul 17, 2011:

11:30 PM Changeset in webkit [91173] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebKit2

[GTK] Fix selection of elements in a multi select list and remove an unnecessary include in WorkQueue
​https://bugs.webkit.org/show_bug.cgi?id=64666

Patch by Amruth Raj <​amruthraj@motorola.com> on 2011-07-17
Reviewed by Martin Robinson.

  • Platform/gtk/WorkQueueGtk.cpp: Remove an include which is no longer required
  • Shared/gtk/WebEventFactory.cpp:

(WebKit::modifiersForEvent): Fix an incorrect ASSERT statement
(WebKit::WebEventFactory::createWebMouseEvent): Call the function to determine the modifier keys

11:24 PM Changeset in webkit [91172] by rniwa@webkit.org
  • 5 edits in trunk/LayoutTests

Chromium rebaselines after r91158.

Add failing expectations for tests added in r91125. The failures are tracked by the bug 64675.

Also remove failing expectations for some editing tests tracked by the Chromium issue 89477
because they are now passing.

  • platform/chromium-linux/editing/inserting/5058163-1-expected.png:
  • platform/chromium-win/editing/inserting/5058163-1-expected.png:
  • platform/chromium-win/editing/inserting/5058163-1-expected.txt:
  • platform/chromium/test_expectations.txt:
7:58 PM Changeset in webkit [91171] by ryuan.choi@samsung.com
  • 2 edits in trunk/Source/JavaScriptCore

[EFL] Refactor scheduleDispatchFunctionsOnMainThread to fix crash.
​https://bugs.webkit.org/show_bug.cgi?id=64337

Replace ecore_timer_add to Ecore_Pipe.
This is needed because ecore_timer should not be called in a child thread,
but in the main thread.

Reviewed by Antonio Gomes.

  • wtf/efl/MainThreadEfl.cpp:

(WTF::pipeObject):
(WTF::monitorDispatchFunctions):
(WTF::initializeMainThreadPlatform):
(WTF::scheduleDispatchFunctionsOnMainThread):

6:59 PM Changeset in webkit [91170] by macpherson@chromium.org
  • 4 edits in trunk/Source/WebCore

Implement CSSPropertyCursor in CSSStyleApplyProperty
​https://bugs.webkit.org/show_bug.cgi?id=64432

Reviewed by Dimitri Glazkov.

No new tests / refectoring only.

  • css/CSSStyleApplyProperty.cpp:

Add handler for CSSPropertyCursor.

  • css/CSSStyleSelector.cpp:

Remove current handler of CSSPropertyCursor.

  • css/CSSStyleSelector.h:

Make styleImage and cachedOrPendingFromValue public.

6:47 PM Changeset in webkit [91169] by commit-queue@webkit.org
  • 8 edits in trunk/Source/WebKit2

Patch by Lukasz Slachciak <​l.slachciak@samsung.com> on 2011-07-17
Reviewed by Martin Robinson.

[GTK] [WK2] Fix for getting editor client commands.
​https://bugs.webkit.org/show_bug.cgi?id=63081

Editor client commands intepretation was incorrect. It was based on the NativeWebKeyboardEvent only.
In fact EventHandler is generating interpreted events - keypress and keydown. These event types
are now passed from UIProcess to WebProcess so KeyBindingTranslator can correctly find editor commands.
Also build break for Debug build was fixed.

  • UIProcess/API/gtk/PageClientImpl.cpp: KeyboardEvent type is used for KeyBindingTranslator.

(WebKit::PageClientImpl::getEditorCommandsForKeyEvent): getEditorCommandsForKeyEvent now has additional
parameter describing KeyboardEvent type.

  • UIProcess/API/gtk/PageClientImpl.h:
  • UIProcess/PageClient.h:
  • UIProcess/WebPageProxy.h:
  • UIProcess/WebPageProxy.messages.in:
  • UIProcess/gtk/WebPageProxyGtk.cpp:

(WebKit::WebPageProxy::getEditorCommandsForKeyEvent): KeyboardEvent type passed to PageClient.

  • WebProcess/WebCoreSupport/gtk/WebEditorClientGtk.cpp:

(WebKit::WebEditorClient::getEditorCommandsForKeyEvent): Sync message send with KeyboardEvent type.

2:02 AM Changeset in webkit [91168] by commit-queue@webkit.org
  • 2 edits in trunk/Source/JavaScriptCore

DFG JIT operationCompareEqual does not inline JSValue::equalSlowCaseInline.
​https://bugs.webkit.org/show_bug.cgi?id=64637

Patch by Filip Pizlo <​fpizlo@apple.com> on 2011-07-17
Reviewed by Gavin Barraclough.

  • dfg/DFGOperations.cpp:
1:57 AM Changeset in webkit [91167] by commit-queue@webkit.org
  • 8 edits
    2 adds in trunk/Tools

test-webkitpy failing with Python 2.5
​https://bugs.webkit.org/show_bug.cgi?id=64594

Patch by Philippe Normand <​pnormand@igalia.com> on 2011-07-17
Reviewed by Eric Seidel.

Updated simplejson to version 2.1.6 that fixes an issue when
dumping slashes. Also use cgi.parse_qs instead of the urlparse
version to be compatible with Python2.5. Verified the fix with
test-webkitpy on python 2.5 and python 2.6.

  • Scripts/webkitpy/thirdparty/simplejson/README.txt:
  • Scripts/webkitpy/thirdparty/simplejson/init.py:
  • Scripts/webkitpy/thirdparty/simplejson/_speedups.c:

(json_PyOS_string_to_double):
(_convertPyInt_AsSsize_t):
(_convertPyInt_FromSsize_t):
(ascii_escape_char):
(ascii_escape_unicode):
(ascii_escape_str):
(raise_errmsg):
(join_list_unicode):
(join_list_string):
(_build_rval_index_tuple):
(scanstring_str):
(scanstring_unicode):
(py_scanstring):
(py_encode_basestring_ascii):
(scanner_dealloc):
(scanner_traverse):
(scanner_clear):
(_parse_object_str):
(_parse_object_unicode):
(_parse_array_str):
(_parse_array_unicode):
(_parse_constant):
(_match_number_str):
(_match_number_unicode):
(scan_once_str):
(scan_once_unicode):
(scanner_call):
(scanner_new):
(scanner_init):
(encoder_new):
(encoder_init):
(encoder_call):
(_encoded_const):
(encoder_encode_float):
(encoder_encode_string):
(_steal_list_append):
(encoder_listencode_obj):
(encoder_listencode_dict):
(encoder_listencode_list):
(encoder_dealloc):
(encoder_traverse):
(encoder_clear):
(init_speedups):

  • Scripts/webkitpy/thirdparty/simplejson/decoder.py:
  • Scripts/webkitpy/thirdparty/simplejson/encoder.py:
  • Scripts/webkitpy/thirdparty/simplejson/ordered_dict.py: Added.
  • Scripts/webkitpy/thirdparty/simplejson/scanner.py:
  • Scripts/webkitpy/thirdparty/simplejson/tool.py: Added.
  • Scripts/webkitpy/tool/servers/reflectionhandler.py:
12:38 AM Changeset in webkit [91166] by Dimitri Glazkov
  • 3 edits in trunk/Tools

Rename ModifierMatcher to SpecificityCalculator.
​https://bugs.webkit.org/show_bug.cgi?id=64660

It's a little longer than before, but it is much clearer.

Reviewed by Adam Barth.

  • Scripts/webkitpy/layout_tests/models/test_expectations.py: Renamed and adjusted

all callsites, also rewrote the comments.

  • Scripts/webkitpy/layout_tests/models/test_expectations_unittest.py: Adjusted callsites

and renamed tests.

Jul 16, 2011:

10:40 PM Changeset in webkit [91165] by cevans@google.com
  • 1 edit
    3 copies in branches/chromium/782

Merge 91152
BUG=89520
Review URL: ​http://codereview.chromium.org/7395022

10:04 PM Changeset in webkit [91164] by barraclough@apple.com
  • 9 edits in trunk

​https://bugs.webkit.org/show_bug.cgi?id=64657
Converted this value not preserved when accessed via direct eval.

Reviewed by Oliver Hunt.

Source/JavaScriptCore:

Upon entry into a non-strict function, primitive this values should be boxed as Object types
(or substituted with the global object) - which is done by op_convert_this. However we only
do so where this is used lexically within the function (we omit the conversion op if not).
The problem comes if a direct eval (running within the function's scope) accesses the this
value.

We are safe in the case of a single eval, since the this object will be converted within
callEval, however the converted value is not preserved, and a new wrapper object is allocated
each time eval is invoked. This is inefficient and incorrect, since any changes to the wrapper
object will be lost between eval statements.

  • bytecompiler/BytecodeGenerator.cpp:

(JSC::BytecodeGenerator::BytecodeGenerator):

  • If a function uses eval, we always need to convert this.
  • interpreter/Interpreter.cpp:

(JSC::Interpreter::execute):

  • Don't convert primitive values here - this is too late!

(JSC::Interpreter::privateExecute):

  • Changed op_convert_this to call new isPrimitive method.
  • jit/JITStubs.cpp:

(JSC::DEFINE_STUB_FUNCTION):

  • Changed op_convert_this to call new isPrimitive method.
  • runtime/JSCell.h:

(JSC::JSCell::JSValue::isPrimitive):

  • Added JSValue::isPrimitive.
  • runtime/JSValue.h:
    • Added JSValue::isPrimitive.

LayoutTests:

Added test case.

  • fast/js/read-modify-eval-expected.txt:
  • fast/js/script-tests/read-modify-eval.js:

(primitiveThisTest):

9:56 PM Changeset in webkit [91163] by dbates@webkit.org
  • 3 edits in trunk/Source/WebKit2

2011-07-16 Daniel Bates <​dbates@webkit.org>

Fix Win Cairo Debug build after <​http://trac.webkit.org/changeset/91085>.

Add stub implementation of scale factor overloaded ShareableBitmap::paint() for
Cairo WebKit2 port after its usage in changeset <​http://trac.webkit.org/changeset/91085>.

Also add stub implementation for GTK.

We'll need to actually implement this. See <​https://bugs.webkit.org/show_bug.cgi?id=64664>
and <​https://bugs.webkit.org/show_bug.cgi?id=64665> for Cairo and GTK, respectively.

  • Shared/cairo/ShareableBitmapCairo.cpp: (WebKit::ShareableBitmap::paint): Added scale-factor variant.
  • Shared/gtk/ShareableBitmapGtk.cpp: (WebKit::ShareableBitmap::paint): Added scale-factor variant.
9:21 PM Changeset in webkit [91162] by dbates@webkit.org
  • 2 edits in trunk/Source/WebKit2

2011-07-16 Daniel Bates <​dbates@webkit.org>

Fix Qt Linux Release build after changeset <​http://trac.webkit.org/changeset/91085>
(​https://bugs.webkit.org/show_bug.cgi?id=64611).

Stub out scale factor-variant implementation of ShareableBitmap::paint().
We'll need to implement this. See <​https://bugs.webkit.org/show_bug.cgi?id=64663>
for more details.

  • Shared/qt/ShareableBitmapQt.cpp: (WebKit::ShareableBitmap::paint): Added scale-factor variant.
7:34 PM Changeset in webkit [91161] by Simon Fraser
  • 16 edits in trunk/Source

2011-07-16 Simon Fraser <Simon Fraser>

Add code to attempt to align compositing layers to pixel boundaries when page scale changes
​https://bugs.webkit.org/show_bug.cgi?id=64658

Reviewed by Dan Bernstein.

Implemented a new behavior on GraphicsLayerCA which attempts to keep layers pixel
aligned as page scale changes.

This requires denoting which layer has the page scale on it (it is assumed to be
already aligned), via setAppliesPageScale(). We also now pass a scale and offset
down through the GraphicsLayer commits, which are used to map layer bounds to display
coordinates for rounding.

  • platform/graphics/GraphicsLayer.h: New flags and getters/setters for pixel alignment, and the layer that is the applier of the scale. (WebCore::GraphicsLayer::setMaintainsPixelAlignment): (WebCore::GraphicsLayer::maintainsPixelAlignment): (WebCore::GraphicsLayer::setAppliesPageScale): (WebCore::GraphicsLayer::appliesPageScale):
  • platform/graphics/GraphicsLayer.cpp: (WebCore::GraphicsLayer::GraphicsLayer): Initialize the new flags. (WebCore::GraphicsLayer::backingScaleFactor): Utility method that calls the client if there is one. (WebCore::GraphicsLayer::pageScaleFactor): Ditto.
  • platform/graphics/GraphicsLayerClient.h: (WebCore::GraphicsLayerClient::backingScaleFactor): Provide default implementation. (WebCore::GraphicsLayerClient::pageScaleFactor): Ditto.
  • platform/graphics/ca/GraphicsLayerCA.h: Pass scale and base-relative offset around to methods that need to update geometry during commit. Some new methods related to pixel alignment.
  • platform/graphics/ca/GraphicsLayerCA.cpp: (WebCore::GraphicsLayerCA::computePositionRelativeToBase): For single-layer commits, we have to compute the base-relative offset, and whether we're under the layer applying the scale by walking up the tree.

(WebCore::GraphicsLayerCA::syncCompositingStateForThisLayerOnly): Call
computePositionRelativeToBase() to get scaling parameters for the target layer.

(WebCore::GraphicsLayerCA::recursiveCommitChanges): Pass flags related to scaling.
positionRelativeToBase is an offset relative to the ancestor layer known to be pixel
aligned. affectedByPageScale is true for layers which are descendants of the layer
which applies the scale.

(WebCore::GraphicsLayerCA::commitLayerChangesBeforeSublayers): Pass scale and offset.
(WebCore::GraphicsLayerCA::updateGeometry): Call computePixelAlignment() to get
pixel-aligned geometry. This only needs to be applied to layers which render content,
so re-organize the code to use the unchanged layer geometry for the structural layer,
but to apply the adjusted geometry to the m_layer.

(WebCore::GraphicsLayerCA::updateStructuralLayer): Pass scale and offset.
(WebCore::GraphicsLayerCA::ensureStructuralLayer): Ditto.
(WebCore::GraphicsLayerCA::updateLayerDrawsContent): Ditto.
(WebCore::GraphicsLayerCA::updateContentsScale): Ditto.
(WebCore::GraphicsLayerCA::requiresTiledLayer): Ditto.
(WebCore::GraphicsLayerCA::swapFromOrToTiledLayer): Ditto.
(WebCore::GraphicsLayerCA::setKeepPixelAligned): When set, dirties flags
for properties that need to be recomputed.
(WebCore::GraphicsLayerCA::noteChangesForScaleSensitiveProperties):
(WebCore::isIntegral):
(WebCore::GraphicsLayerCA::computePixelAlignment): Map our bounds to screen
scale, round out to an integral rect, then map it back, adding a fudge factor
so that CA backing stores don't end up being smaller than we expect because
of integer truncation.

  • rendering/RenderLayerBacking.cpp: (WebCore::RenderLayerBacking::createGraphicsLayer): Turn on pixel alignment if the compositor says so. (WebCore::RenderLayerBacking::createPrimaryGraphicsLayer): Set the RenderView's layer as the one applying the scale, for the main frame.
  • rendering/RenderLayerCompositor.h: New method.
  • rendering/RenderLayerCompositor.cpp: (WebCore::RenderLayerCompositor::keepLayersPixelAligned): Master switch for pixel alignment. Default to on. (WebCore::RenderLayerCompositor::ensureRootLayer): Give the two root-ish layers a client so they can ask for page scale if necessary.

2011-07-16 Simon Fraser <Simon Fraser>

Add code to attempt to align compositing layers to pixel boundaries when page scale changes
​https://bugs.webkit.org/show_bug.cgi?id=64658

Reviewed by Dan Bernstein.

Removed implementation of pageScaleFactor() now that GraphicsLayerClient has
a default implementation.

  • WebProcess/WebPage/ca/LayerTreeHostCA.cpp:
  • WebProcess/WebPage/ca/LayerTreeHostCA.h:

2011-07-16 Simon Fraser <Simon Fraser>

Add code to attempt to align compositing layers to pixel boundaries when page scale changes
​https://bugs.webkit.org/show_bug.cgi?id=64658

Reviewed by Dan Bernstein.

Removed implementations of backingScaleFactor() and pageScaleFactor() now
that GraphicsLayerClient has default implementations.

  • WebView.h:

2011-07-16 Simon Fraser <Simon Fraser>

Add code to attempt to align compositing layers to pixel boundaries when page scale changes
​https://bugs.webkit.org/show_bug.cgi?id=64658

Reviewed by Dan Bernstein.

Removed implementations of backingScaleFactor() and pageScaleFactor() now
that GraphicsLayerClient has default implementations.

  • src/PageOverlay.cpp:
6:47 PM Changeset in webkit [91160] by commit-queue@webkit.org
  • 4 edits in trunk/Source/JavaScriptCore

DFG JIT compare/branch code emits is-integer tests even when a value is
definitely not an integer.
​https://bugs.webkit.org/show_bug.cgi?id=64654

Patch by Filip Pizlo <​fpizlo@apple.com> on 2011-07-16
Reviewed by Gavin Barraclough.

Added the isKnownNotInteger() method, which returns true if a node is
definitely not an integer and will always fail any is-integer test. Then
modified the compare and branch code to use this method; if it returns
true then is-int tests are omitted and the compiler always emits a slow
call.

  • dfg/DFGJITCodeGenerator.cpp:

(JSC::DFG::JITCodeGenerator::isKnownNotInteger):
(JSC::DFG::JITCodeGenerator::nonSpeculativePeepholeBranch):
(JSC::DFG::JITCodeGenerator::nonSpeculativeNonPeepholeCompare):
(JSC::DFG::JITCodeGenerator::nonSpeculativeCompare):

  • dfg/DFGJITCodeGenerator.h:
  • dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compare):

5:46 PM Changeset in webkit [91159] by dbates@webkit.org
  • 3 edits in trunk/Source/WebKit2

2011-07-16 Daniel Bates <​dbates@webkit.org>

Attempt to fix the Qt Linux Release build after changeset <​http://trac.webkit.org/changeset/91097>
(​https://bugs.webkit.org/show_bug.cgi?id=64615).

  • UIProcess/qt/QtWebPageProxy.cpp: (QtWebPageProxy::setCursorHiddenUntilMouseMoves): Added stub method.
  • UIProcess/qt/QtWebPageProxy.h:
5:33 PM Changeset in webkit [91158] by enrica@apple.com
  • 4 edits
    2 adds in trunk

REGRESSION: Pressing return in a particular document sends the cursor to the end of the document.
​https://bugs.webkit.org/show_bug.cgi?id=64140
<rdar://problem/9737491>

Source/WebCore:

Inserting a paragraph separator at a break element, inside an inline, generates redundant markup
and moves the selection at the end of the inserted block.
The fix consists in detecting that the insertion point is a break element and simply inserting another
break element. This solves the problem of the wrong final selection and also produces less redundant
markup.

Reviewed by Simon Fraser.

Test: editing/inserting/insert-paragraph-separator-at-break.html

  • editing/InsertParagraphSeparatorCommand.cpp:

(WebCore::InsertParagraphSeparatorCommand::doApply): Modified logic for insertion at a break
element.

LayoutTests:

Added new test to cover this case and updated results for one test that now
produces simpler markup.

Reviewed by Simon Fraser.

  • editing/inserting/insert-paragraph-separator-at-break-expected.txt: Added.
  • editing/inserting/insert-paragraph-separator-at-break.html: Added.
  • platform/mac/editing/inserting/5058163-1-expected.txt:
4:29 PM Changeset in webkit [91157] by psolanki@apple.com
  • 2 edits in trunk/Source/WebCore

SubresourceLoader::didReceiveDataArray can crash when calling m_client->didReceiveData()
​https://bugs.webkit.org/show_bug.cgi?id=64656
<rdar://problem/9754425>

Reviewed by Alexey Proskuryakov.

  • loader/cf/SubresourceLoaderCF.cpp:

(WebCore::SubresourceLoader::didReceiveDataArray): Check for m_client before calling
m_client->didReceiveData().

4:14 PM Changeset in webkit [91156] by commit-queue@webkit.org
  • 3 edits in trunk/Source/JavaScriptCore

DFG speculative JIT has dead code for slow calls for branches.
​https://bugs.webkit.org/show_bug.cgi?id=64653

Patch by Filip Pizlo <​fpizlo@apple.com> on 2011-07-16
Reviewed by Gavin Barraclough.

Removed SpeculativeJIT::compilePeepHoleCall.

  • dfg/DFGSpeculativeJIT.cpp:
  • dfg/DFGSpeculativeJIT.h:
4:01 PM Changeset in webkit [91155] by commit-queue@webkit.org
  • 3 edits in trunk/Source/WebCore

[Qt] Compilation fails on disabling TextureMapper
​https://bugs.webkit.org/show_bug.cgi?id=64408

Fix compilation error due to API name change in GraphicsLayer

Patch by Dinu Jacob <​dinu.jacob@nokia.com> on 2011-07-16
Reviewed by Noam Rosenthal.

  • platform/graphics/qt/GraphicsLayerQt.cpp:

(WebCore::GraphicsLayerQt::setContentsToBackgroundColor):

  • platform/graphics/qt/GraphicsLayerQt.h:
3:59 PM Changeset in webkit [91154] by psolanki@apple.com
  • 13 edits in trunk

Part of ​https://bugs.webkit.org/show_bug.cgi?id=63674
Get webkit to compile with USE(CFNETWORK) enabled on Mac

Reviewed by David Kilzer.

New WebKitSystemInterface functions for CFNetwork-based loader.

../../git/opensource/WebKit.git/Source/WebCore:

  • WebCore.exp.in:
  • platform/mac/WebCoreSystemInterface.h:
  • platform/mac/WebCoreSystemInterface.mm:

../../git/opensource/WebKit.git/Source/WebKit/mac:

  • WebCoreSupport/WebSystemInterface.mm:

(InitWebCoreSystemInterface):

../../git/opensource/WebKit.git/Source/WebKit2:

  • WebProcess/WebCoreSupport/mac/WebSystemInterface.mm:

(InitWebCoreSystemInterface):

../../git/opensource/WebKit.git/WebKitLibraries:

  • WebKitSystemInterface.h:
  • libWebKitSystemInterfaceLeopard.a:
  • libWebKitSystemInterfaceLion.a:
  • libWebKitSystemInterfaceSnowLeopard.a:
3:39 PM Changeset in webkit [91153] by commit-queue@webkit.org
  • 2 edits in trunk/Source/WebCore

Work towards determining the cause of frequent crashes due to null frame below
ApplicationCacheHost::scheduleLoadFallbackResourceFromApplicationCache.
​https://bugs.webkit.org/show_bug.cgi?id=62764

Patch by Jessie Berlin <​jberlin@apple.com> on 2011-07-16
Reviewed by Darin Adler.

No loads should be going on for a DocumentLoader that has been detached from its frame, but
that appears to be what is happening in this bug.

Add assertions with the hope that someone will run into this while debugging and thereby find
a reproducible case.

  • loader/ResourceLoader.cpp:

(WebCore::ResourceLoader::willSendRequest):
(WebCore::ResourceLoader::didSendData):
(WebCore::ResourceLoader::didReceiveResponse):
(WebCore::ResourceLoader::didReceiveData):
(WebCore::ResourceLoader::didFinishLoading):
(WebCore::ResourceLoader::didFail):
(WebCore::ResourceLoader::wasBlocked):
(WebCore::ResourceLoader::cannotShowURL):
(WebCore::ResourceLoader::shouldUseCredentialStorage):
(WebCore::ResourceLoader::willCacheResponse):

1:12 PM Changeset in webkit [91152] by commit-queue@webkit.org
  • 3 edits
    3 adds in trunk

DOMWindow::open performs a security check on a wrong window
​https://bugs.webkit.org/show_bug.cgi?id=64651

Patch by Sergey Glazunov <​serg.glazunov@gmail.com> on 2011-07-16
Reviewed by Adam Barth.

Source/WebCore:

Test: http/tests/security/xss-DENIED-window-open-parent.html

  • page/DOMWindow.cpp:

(WebCore::DOMWindow::open):

LayoutTests:

  • http/tests/security/resources/xss-DENIED-window-open-parent-attacker.html: Added.
  • http/tests/security/xss-DENIED-window-open-parent-expected.txt: Added.
  • http/tests/security/xss-DENIED-window-open-parent.html: Added.
12:07 PM Changeset in webkit [91151] by weinig@apple.com
  • 103 edits in trunk

Attribute selectors don't handle glob namespaces (e.g. *|E)
​https://bugs.webkit.org/show_bug.cgi?id=64567

Reviewed by Anders Carlsson.

Source/WebCore:

Change checking of attribute selectors to match CSS 3 Selectors in
the presence of namespaces. This entails changing the algorithm to
check each attribute on an element as a match for the attribute selector
and taking into account a prefix of *.

  • css/CSSStyleSelector.cpp:

(WebCore::attributeQualifiedNameMatches):
Checks if the a particular attributes qualified name matches
the attribute selector, taking into account a prefix of *.

(WebCore::attributeValueMatches):
Checks if a particular attribute on an element, matches
the attribute selector.

(WebCore::anyAttributeMatches):
Top level attribute check. Checks if any of an elements attributes
match the attribute selector.

(WebCore::CSSStyleSelector::SelectorChecker::checkOneSelector):
Factor out attribute checking code into the above functions.

LayoutTests:

Update results for correct handling of a * prefix in an attribute selector.
Fixes 50 tests.

  • platform/mac/css3/selectors3/xhtml/css3-modsel-104-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-104-expected.txt:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-105-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-105-expected.txt:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-106-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-106-expected.txt:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-107-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-107-expected.txt:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-108-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-108-expected.txt:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-109-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-109-expected.txt:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-110-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-110-expected.txt:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-130-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-130-expected.txt:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-130b-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-130b-expected.txt:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-131-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-131-expected.txt:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-131b-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-131b-expected.txt:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-132-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-132-expected.txt:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-132b-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-132b-expected.txt:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-133-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-133-expected.txt:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-133b-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-133b-expected.txt:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-134-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-134-expected.txt:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-134b-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-134b-expected.txt:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-135-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-135-expected.txt:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-135b-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-135b-expected.txt:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-136-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-136-expected.txt:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-136b-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-136b-expected.txt:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-173a-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-173a-expected.txt:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-173b-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-173b-expected.txt:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-174a-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-174a-expected.txt:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-174b-expected.png:
  • platform/mac/css3/selectors3/xhtml/css3-modsel-174b-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-104-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-104-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-105-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-105-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-106-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-106-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-107-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-107-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-108-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-108-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-109-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-109-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-110-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-110-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-130-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-130-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-130b-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-130b-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-131-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-131-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-131b-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-131b-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-132-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-132-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-132b-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-132b-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-133-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-133-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-133b-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-133b-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-134-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-134-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-134b-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-134b-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-135-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-135-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-135b-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-135b-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-136-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-136-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-136b-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-136b-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-173a-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-173a-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-173b-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-173b-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-174a-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-174a-expected.txt:
  • platform/mac/css3/selectors3/xml/css3-modsel-174b-expected.png:
  • platform/mac/css3/selectors3/xml/css3-modsel-174b-expected.txt:
12:06 PM Changeset in webkit [91150] by cevans@google.com
  • 3 edits
    2 copies in branches/chromium/782

Merge 91005
BUG=88730
Review URL: ​http://codereview.chromium.org/7396010

5:37 AM Changeset in webkit [91149] by jhoneycutt@apple.com
  • 2 edits in trunk/Source/WebCore

WinCE build fix. Unreviewed.

  • dom/SelectElement.cpp:
12:33 AM Changeset in webkit [91148] by commit-queue@webkit.org
  • 3 edits
    3 adds in trunk

Patch by Kulanthaivel Palanichamy <​kulanthaivel@codeaurora.org> on 2011-07-16
Reviewed by Nikolas Zimmermann.

SVG animation API crashes on SVGAnimateTransform
​https://bugs.webkit.org/show_bug.cgi?id=64104

Source/WebCore:

This patch ensures the update in AnimatedTransform list in
SVGAnimateTransformElement.cpp is in sync with its wrapper list.

Test: svg/animations/svgtransform-animation-discrete.html

  • svg/SVGAnimateTransformElement.cpp:

(WebCore::animatedTransformListFor):
(WebCore::SVGAnimateTransformElement::resetToBaseValue):
(WebCore::SVGAnimateTransformElement::calculateAnimatedValue):
(WebCore::SVGAnimateTransformElement::applyResultsToTarget):

LayoutTests:

Added the test to cover assertion in AnimatedTransform list where its
value and wrapper length mismatch occurs.

  • svg/animations/script-tests/svgtransform-animation-discrete.js: Added.

(sample1):
(sample2):
(executeTest):

  • svg/animations/svgtransform-animation-discrete-expected.txt: Added.
  • svg/animations/svgtransform-animation-discrete.html: Added.
Note: See TracTimeline for information about the timeline view.