Timeline

Apr 25, 2013:

11:50 PM Changeset in webkit [149169] by zandobersek@gmail.com

• 3 edits in trunk/Source/WebKit2

[GTK] Plugin process broken due to a missing symbol
​https://bugs.webkit.org/show_bug.cgi?id=114901

Reviewed by Gustavo Noronha Silva.

Work around the missing symbols in the WebKitPluginProcess by specifying the required libraries
multiple times in the program's LDADD list. Libtool fights fiercely to avoid duplicate static library
references among the linker arguments (even if using the --preserve-dup-deps flag) so the required libraries
are specified in various ways.

The unresolved symbols in object files are no longer ignored as this can lead to further trouble (for instance
this patch also adds a few missing source files to the list of WebKitPluginProcess sources). The Freetype and
gamepad dependencies' libraries are also added to the binary's LDADD list.

• GNUmakefile.am:
• GNUmakefile.list.am: Add a few missing files to the build.

11:23 PM Changeset in webkit [149168] by mihnea@adobe.com

• 4 edits
  2 adds in trunk

[CSS Regions] Hit testing is broken for absolutely positioned regions that have overflow: hidden
​https://bugs.webkit.org/show_bug.cgi?id=113874

Reviewed by David Hyatt.

Source/WebCore:

Test: fast/regions/hit-test-abspos-overflow-region.html

When a region is an out-of-flow positioned object with an overflow clip, we need
to make sure that hit testing works also for cases other than foreground (content)
hit testing. This patch moves the previous hit testing code, that delegated foreground
hit testing to the region's flow thread hit testing, into the hitTestContent method,
now that RenderRegion is RenderBlock based.

• rendering/RenderRegion.cpp:

(WebCore::RenderRegion::hitTestContents):

• rendering/RenderRegion.h:

(RenderRegion):

LayoutTests:

Add test for hit-testing of border of an out-of-flow region with overflow: hidden.

• fast/regions/hit-test-abspos-overflow-region-expected.txt: Added.
• fast/regions/hit-test-abspos-overflow-region.html: Added.

9:51 PM Changeset in webkit [149167] by rniwa@webkit.org

• 4 edits
  2 adds in trunk

Copy and paste can strip !important CSS rules due to a bug in mergeStyleFromRules
​https://bugs.webkit.org/show_bug.cgi?id=115217

Reviewed by Darin Adler.

Source/WebCore:

The bug was caused by mergeStyleFromRules overriding "important" style rules with "unimportant" inline styles.
Fixed the bug by using addParsedProperty, which respects !important, in MutableStylePropertySet's
mergeAndOverrideOnConflict, which was only used in editing code. Now that we've fixed this function, we can use
it in ViewportStyleResolver::addViewportRule as well.

Test: editing/pasteboard/copy-paste-with-important-rules.html

• css/StylePropertySet.cpp:

(WebCore::MutableStylePropertySet::mergeAndOverrideOnConflict): Fixed to respect !important.

• css/ViewportStyleResolver.cpp:

(WebCore::ViewportStyleResolver::addViewportRule): Use mergeAndOverrideOnConflict now that the code is identical.

LayoutTests:

Added a regression test.

• editing/pasteboard/copy-paste-with-important-rules-expected.txt: Added.
• editing/pasteboard/copy-paste-with-important-rules.html: Added.

9:50 PM Changeset in webkit [149166] by commit-queue@webkit.org

• 3 edits in trunk/Tools

Remove support for Chromium from build/update scripts
​https://bugs.webkit.org/show_bug.cgi?id=115218

Patch by Seokju Kwon <Seokju Kwon> on 2013-04-25
Reviewed by Darin Adler.

• Scripts/build-webkit:

(writeCongrats):

• Scripts/update-webkit:

7:47 PM Changeset in webkit [149165] by mhahnenberg@apple.com

• 4 edits in trunk/Source/JavaScriptCore

SuperRegion doesn't call deallocate() on its PageReservation
​https://bugs.webkit.org/show_bug.cgi?id=115208

Reviewed by Geoffrey Garen.

It should. This doesn't cause us to leak physical memory, but it does cause us to leak virtual
address space (and probably mach ports), which is also bad :-( FixedVMPoolExecutableAllocator
also has this bug, but it doesn't matter much because there's only one instance of that class
throughout the entire lifetime of the process, whereas each VM has its own SuperRegion.

• heap/SuperRegion.cpp:

(JSC::SuperRegion::~SuperRegion):

• heap/SuperRegion.h:

(SuperRegion):

• jit/ExecutableAllocatorFixedVMPool.cpp:

(FixedVMPoolExecutableAllocator):
(JSC::FixedVMPoolExecutableAllocator::~FixedVMPoolExecutableAllocator):

7:30 PM Changeset in webkit [149164] by commit-queue@webkit.org

• 3 edits in trunk/Tools

Remove support for Chromium from api-test
​https://bugs.webkit.org/show_bug.cgi?id=115211

Patch by Seokju Kwon <Seokju Kwon> on 2013-04-25
Reviewed by Dirk Pranke.

• Scripts/build-api-tests:
• Scripts/run-api-tests:

(isSupportedPlatform):

6:38 PM Changeset in webkit [149163] by commit-queue@webkit.org

• 2 edits in trunk/Tools

Get rid of Chromium in build-dumprendertree
​https://bugs.webkit.org/show_bug.cgi?id=115212

Patch by Seokju Kwon <Seokju Kwon> on 2013-04-25
Reviewed by Dirk Pranke.

• Scripts/build-dumprendertree:

6:18 PM Changeset in webkit [149162] by fpizlo@apple.com

• 7 edits in trunk/Source/JavaScriptCore

DFG doesn't support to_jsnumber
​https://bugs.webkit.org/show_bug.cgi?id=115129

Reviewed by Geoffrey Garen.

Based on Oliver's patch. Implements to_jsnumber as Identity(Number:@thingy), and then does
an optimization in Fixup to turn Identity(Number:) into Identity(Int32:) if the predictions
tell us to. Identity is later turned into Phantom.

Also fixed BackPropMask, which appeared to have NodeDoesNotExit included in it. That's
wrong; NodeDoesNotExit is not a backward propagation property.

Also fixed Identity to be marked as CanExit (i.e. not NodeDoesNotExit).

This more than doubles the FPS on ammo.

• dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::parseBlock):

• dfg/DFGCapabilities.h:

(JSC::DFG::canCompileOpcode):

• dfg/DFGFixupPhase.cpp:

(JSC::DFG::FixupPhase::fixupNode):
(FixupPhase):
(JSC::DFG::FixupPhase::observeUseKindOnNode):
(JSC::DFG::FixupPhase::observeUseKindOnEdge):

• dfg/DFGNodeFlags.h:

(DFG):

• dfg/DFGNodeType.h:

(DFG):

• dfg/DFGPredictionPropagationPhase.cpp:

(JSC::DFG::PredictionPropagationPhase::propagate):

5:50 PM Changeset in webkit [149161] by mrowe@apple.com

• 4 edits in trunk/Source

Versioning.

5:49 PM Changeset in webkit [149160] by mrowe@apple.com

• 1 copy in tags/Safari-537.39

New tag.

5:41 PM Changeset in webkit [149159] by oliver@apple.com

• 21 edits
  12 adds in trunk

Add support for Math.imul
​https://bugs.webkit.org/show_bug.cgi?id=115143

Reviewed by Filip Pizlo.

Source/JavaScriptCore:

Add support for Math.imul, a thunk generator for Math.imul,
and an intrinsic.

Fairly self explanatory set of changes, DFG intrinsics simply
leverages the existing ValueToInt32 nodes.

• create_hash_table:
• dfg/DFGAbstractState.cpp:

(JSC::DFG::AbstractState::executeEffects):

• dfg/DFGBackwardsPropagationPhase.cpp:

(JSC::DFG::BackwardsPropagationPhase::propagate):

• dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::handleIntrinsic):

• dfg/DFGCSEPhase.cpp:

(JSC::DFG::CSEPhase::performNodeCSE):

• dfg/DFGFixupPhase.cpp:

(JSC::DFG::FixupPhase::fixupNode):

• dfg/DFGNodeType.h:

(DFG):

• dfg/DFGPredictionPropagationPhase.cpp:

(JSC::DFG::PredictionPropagationPhase::propagate):

• dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compileArithIMul):

• dfg/DFGSpeculativeJIT.h:

(SpeculativeJIT):

• dfg/DFGSpeculativeJIT32_64.cpp:

(JSC::DFG::SpeculativeJIT::compile):

• dfg/DFGSpeculativeJIT64.cpp:

(JSC::DFG::SpeculativeJIT::compile):

• jit/ThunkGenerators.cpp:

(JSC::imulThunkGenerator):
(JSC):

• jit/ThunkGenerators.h:

(JSC):

• runtime/Intrinsic.h:
• runtime/MathObject.cpp:

(JSC):
(JSC::mathProtoFuncIMul):

• runtime/VM.cpp:

(JSC::thunkGeneratorForIntrinsic):

LayoutTests:

Add a bunch of tests for Math.imul

• fast/js/Object-getOwnPropertyNames-expected.txt:
• fast/js/imul-expected.txt: Added.
• fast/js/imul.html: Added.
• fast/js/regress/imul-double-only-expected.txt: Added.
• fast/js/regress/imul-double-only.html: Added.
• fast/js/regress/imul-int-only-expected.txt: Added.
• fast/js/regress/imul-int-only.html: Added.
• fast/js/regress/imul-mixed-expected.txt: Added.
• fast/js/regress/imul-mixed.html: Added.
• fast/js/regress/script-tests/imul-double-only.js: Added.

(f):

• fast/js/regress/script-tests/imul-int-only.js: Added.

(f):

• fast/js/regress/script-tests/imul-mixed.js: Added.

(f):

• fast/js/script-tests/Object-getOwnPropertyNames.js:
• fast/js/script-tests/imul.js: Added.

(testIMul):

5:40 PM Changeset in webkit [149158] by fpizlo@apple.com

• 15 edits in trunk/Source

Unreviewed, roll out ​http://trac.webkit.org/changeset/148999
It broke ​http://kripken.github.io/ammo.js/examples/new/ammo.html

Source/JavaScriptCore:

• JavaScriptCore.order:
• bytecompiler/BytecodeGenerator.cpp:

(JSC::BytecodeGenerator::emitNewArray):
(JSC::BytecodeGenerator::emitThrowReferenceError):
(JSC::BytecodeGenerator::emitReadOnlyExceptionIfNeeded):

• bytecompiler/BytecodeGenerator.h:

(JSC::BytecodeGenerator::shouldEmitProfileHooks):
(BytecodeGenerator):

• bytecompiler/NodesCodegen.cpp:

(JSC):
(JSC::NullNode::emitBytecode):
(JSC::BooleanNode::emitBytecode):
(JSC::NumberNode::emitBytecode):
(JSC::StringNode::emitBytecode):
(JSC::IfNode::emitBytecode):
(JSC::IfElseNode::emitBytecode):

• parser/ASTBuilder.h:

(JSC::ASTBuilder::createIfStatement):
(ASTBuilder):

• parser/NodeConstructors.h:

(JSC):
(JSC::NullNode::NullNode):
(JSC::BooleanNode::BooleanNode):
(JSC::NumberNode::NumberNode):
(JSC::StringNode::StringNode):
(JSC::IfNode::IfNode):
(JSC::IfElseNode::IfElseNode):

• parser/Nodes.h:

(JSC::ExpressionNode::isPure):
(JSC::ExpressionNode::isSubtract):
(StatementNode):
(NullNode):
(JSC::NullNode::isNull):
(BooleanNode):
(JSC::BooleanNode::isPure):
(NumberNode):
(JSC::NumberNode::value):
(JSC::NumberNode::isPure):
(StringNode):
(JSC::StringNode::isPure):
(JSC::StringNode::isString):
(BinaryOpNode):
(IfNode):
(JSC):
(IfElseNode):
(ContinueNode):
(BreakNode):

• parser/Parser.cpp:

(JSC::::parseIfStatement):

• parser/ResultType.h:

(ResultType):

• runtime/JSCJSValueInlines.h:

(JSC::JSValue::pureToBoolean):

• runtime/JSCell.h:

(JSCell):

• runtime/JSCellInlines.h:

(JSC):

Source/WTF:

• wtf/TriState.h:

5:33 PM Changeset in webkit [149157] by akling@apple.com

• 4 edits in trunk/Source/WebCore

StylePropertySet::getPropertyShorthand() should return a String.
<​http://webkit.org/b/115213>

Reviewed by Anders Carlsson.

Return a String directly from getPropertyShorthand() instead of forcing clients
to call getPropertyNameString().

• css/PropertySetCSSStyleDeclaration.cpp:

(WebCore::PropertySetCSSStyleDeclaration::getPropertyShorthand):

• css/StylePropertySet.cpp:

(WebCore::StylePropertySet::getPropertyShorthand):

• css/StylePropertySet.h:

(StylePropertySet):

5:27 PM Changeset in webkit [149156] by Chris Fleizach

• 13 edits in trunk

WEB SPEECH: language support does not work as expected
​https://bugs.webkit.org/show_bug.cgi?id=115119

Reviewed by Alexey Proskuryakov.

Source/WebCore:

Make the Mac platform synthesizer have access to all the voices installed on the system.

• WebCore.exp.in:
• platform/mac/PlatformSpeechSynthesizerMac.mm:

(WebCore::PlatformSpeechSynthesizer::initializeVoiceList):

• platform/mac/WebCoreSystemInterface.h:
• platform/mac/WebCoreSystemInterface.mm:

Source/WebKit/mac:

• WebCoreSupport/WebSystemInterface.mm:

(InitWebCoreSystemInterface):

Source/WebKit2:

• WebProcess/WebCoreSupport/mac/WebSystemInterface.mm:

(InitWebCoreSystemInterface):

WebKitLibraries:

• WebKitSystemInterface.h:

4:33 PM Changeset in webkit [149155] by Chris Fleizach

• 11 edits
  2 adds in trunk

Source/WebCore: <meter> element not exposed to accessibility
​https://bugs.webkit.org/show_bug.cgi?id=109023
rdar://problem/13658964

Reviewed by Tim Horton.

Makes the <meter> element appear in the AX tree by reusing the
AccessibilityProgressIndicator element to handle either progress or meter
elements.

Test: accessibility/meter-element.html

• accessibility/AXObjectCache.cpp:

(WebCore::createFromRenderer):
(WebCore::AXObjectCache::getOrCreate):

• accessibility/AccessibilityNodeObject.cpp:

(WebCore::AccessibilityNodeObject::canHaveChildren):
(WebCore::AccessibilityNodeObject::visibleText):

• accessibility/AccessibilityProgressIndicator.cpp:

(WebCore::AccessibilityProgressIndicator::AccessibilityProgressIndicator):
(WebCore::AccessibilityProgressIndicator::create):
(WebCore::AccessibilityProgressIndicator::valueForRange):
(WebCore::AccessibilityProgressIndicator::maxValueForRange):
(WebCore::AccessibilityProgressIndicator::minValueForRange):
(WebCore::AccessibilityProgressIndicator::progressElement):
(WebCore::AccessibilityProgressIndicator::meterElement):

• accessibility/AccessibilityProgressIndicator.h:

(AccessibilityProgressIndicator):

• accessibility/AccessibilityRenderObject.cpp:

(WebCore::AccessibilityRenderObject::firstChild):
(WebCore::AccessibilityRenderObject::lastChild):

These methods are modified to handle when there is no rendered children, but there are Node
children. The node children are desired when calculating text within a node, for example.

LayoutTests: <meter> element not exposed to accessibility
​https://bugs.webkit.org/show_bug.cgi?id=109023

Reviewed by Tim Horton.

• accessibility/meter-element-expected.txt: Added.
• accessibility/meter-element.html: Added.
• platform/mac/accessibility/role-subrole-roledescription-expected.txt:
• platform/mac/accessibility/role-subrole-roledescription.html:

4:10 PM Changeset in webkit [149154] by fpizlo@apple.com

• 3 edits in trunk/Source/JavaScriptCore

PreciseJumpTargets should treat loop_hint as a jump target
​https://bugs.webkit.org/show_bug.cgi?id=115209

Reviewed by Mark Hahnenberg.

I didn't add a test but I turned this into a release assertion. Running Octane is enough
to trigger it.

• bytecode/PreciseJumpTargets.cpp:

(JSC::computePreciseJumpTargets):

• dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::parseBlock):

4:06 PM Changeset in webkit [149153] by commit-queue@webkit.org

• 5 edits in trunk/Source/WebCore

[CSS Shaders] Remove the meshType from the CustomFilterOperation
​https://bugs.webkit.org/show_bug.cgi?id=102529

Patch by Qiankun Miao <​qiankun.miao@intel.com> on 2013-04-25
Reviewed by Dean Jackson.

No new tests, no new functionality.

MeshType can be accessed from program, so remove the m_meshType
related code in CustomFilterOperation.

• css/StyleResolver.cpp:

(WebCore::StyleResolver::createCustomFilterOperationWithInlineSyntax):

• platform/graphics/filters/CustomFilterOperation.cpp:

(WebCore::CustomFilterOperation::CustomFilterOperation):
(WebCore::CustomFilterOperation::blend):

• platform/graphics/filters/CustomFilterOperation.h:

(WebCore::CustomFilterOperation::create):
(WebCore::CustomFilterOperation::meshType):
(CustomFilterOperation):
(WebCore::CustomFilterOperation::operator==):

• platform/graphics/texmap/coordinated/CoordinatedCustomFilterOperation.h:

(WebCore::CoordinatedCustomFilterOperation::CoordinatedCustomFilterOperation):

3:52 PM Changeset in webkit [149152] by commit-queue@webkit.org

• 10 edits
  9 adds in trunk

Source/JavaScriptCore: Fix problems with processing negative zero on DFG.
​https://bugs.webkit.org/show_bug.cgi?id=113862

Patch by Roman Zhuykov <​zhroma@ispras.ru> on 2013-04-25
Reviewed by Filip Pizlo.

Fix NodeNeedsNegZero flag propagation in BackwardPropagationPhase.
Function arithNodeFlags should not mask NodeNeedsNegZero flag for ArithNegate and DoubleAsInt32
nodes and this flag should be always used to decide where we need to generate nezative-zero checks.
Remove unnecessary negative-zero checks from integer ArithDiv on ARM.
Also remove such checks from integer ArithMod on ARM and X86, and make them always to
check not only "modulo_result == 0" but also "dividend < 0".
Generate faster code for case when ArithMod operation divisor is constant power of 2 on ARMv7
in the same way as on ARMv7s, and add negative-zero checks into this code when needed.
Change speculationCheck ExitKind from Overflow to NegativeZero where applicable.

This shows 30% speedup of math-spectral-norm, and 5% speedup
on SunSpider overall on ARMv7 Linux.

• assembler/MacroAssemblerARM.h:

(JSC::MacroAssemblerARM::branchConvertDoubleToInt32):

• assembler/MacroAssemblerARMv7.h:

(JSC::MacroAssemblerARMv7::branchConvertDoubleToInt32):

• assembler/MacroAssemblerMIPS.h:

(JSC::MacroAssemblerMIPS::branchConvertDoubleToInt32):

• assembler/MacroAssemblerSH4.h:

(JSC::MacroAssemblerSH4::branchConvertDoubleToInt32):

• assembler/MacroAssemblerX86Common.h:

(JSC::MacroAssemblerX86Common::branchConvertDoubleToInt32):

• dfg/DFGBackwardsPropagationPhase.cpp:

(JSC::DFG::BackwardsPropagationPhase::isNotNegZero):
(JSC::DFG::BackwardsPropagationPhase::isNotPosZero):
(JSC::DFG::BackwardsPropagationPhase::propagate):

• dfg/DFGNode.h:

(JSC::DFG::Node::arithNodeFlags):

• dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compileDoubleAsInt32):
(JSC::DFG::SpeculativeJIT::compileSoftModulo):
(JSC::DFG::SpeculativeJIT::compileArithNegate):

LayoutTests: Arithmetic operations with negative zero should be optimized correclty.
​https://bugs.webkit.org/show_bug.cgi?id=113862

Patch by Roman Zhuykov <​zhroma@ispras.ru> on 2013-04-25
Reviewed by Filip Pizlo.

• fast/js/regress/negative-zero-divide-expected.txt: Added.
• fast/js/regress/negative-zero-divide.html: Added.
• fast/js/regress/negative-zero-modulo-expected.txt: Added.
• fast/js/regress/negative-zero-modulo.html: Added.
• fast/js/regress/negative-zero-negate-expected.txt: Added.
• fast/js/regress/negative-zero-negate.html: Added.
• fast/js/regress/script-tests/negative-zero-divide.js: Added.

(foo):

• fast/js/regress/script-tests/negative-zero-modulo.js: Added.

(foo):

• fast/js/regress/script-tests/negative-zero-negate.js: Added.

(foo):

3:51 PM Changeset in webkit [149151] by tkent@chromium.org

• 7 edits in trunk

Ignore invalid regular expressions for input[pattern].
​https://bugs.webkit.org/show_bug.cgi?id=115204

Reviewed by Darin Adler.

Source/WebCore:

According to the specification, we should not proceed regular expression
matching if a pattern attribute value is an invalid regular
expression. We had a bug that invalid expressions such as
pattern=")foo(" made RegularExpression objects successfully.

​http://www.whatwg.org/specs/web-apps/current-work/multipage/common-input-element-attributes.html#the-pattern-attribute

If an input element has a pattern attribute specified, and the
attribute's value, when compiled as a JavaScript regular expression with
the global, ignoreCase, and multiline flags disabled (see ECMA262
Edition 5, sections 15.10.7.2 through 15.10.7.4), compiles successfully,
then the resulting regular expression is the element's compiled pattern
regular expression. If the element has no such attribute, or if the
value doesn't compile successfully, then the element has no compiled
pattern regular expression.

This imports a part of Blink r148951.

Tests: Update fast/forms/ValidityState-patternMismatch.html

• html/BaseTextInputType.cpp:

(WebCore::BaseTextInputType::patternMismatch):
Check correctness of pattern attribute value before wrapping with parentheses.

• platform/text/RegularExpression.cpp:

(WebCore::RegularExpression::isValid): Added.

• platform/text/RegularExpression.h:

(RegularExpression): Declare isValid.

LayoutTests:

• fast/forms/ValidityState-patternMismatch-expected.txt:
• fast/forms/ValidityState-patternMismatch.html:

3:50 PM Changeset in webkit [149150] by oliver@apple.com

• 2 edits in trunk/Source/WTF

Fix 32bit build

3:37 PM Changeset in webkit [149149] by Simon Fraser

• 2 edits in trunk/Source/WebKit/mac

Build fix for smart compliers.

Reviewed by Tim Horton.

[NSDictionary dictionaryWithObjectsAndKeys:] returns an NSDictionary,
not a NSMutableDictionary.

• WebCoreSupport/WebEditorClient.mm:

(WebEditorClient::updateSpellingUIWithGrammarString):

3:25 PM Changeset in webkit [149148] by ap@apple.com

• 8 edits in trunk/Tools

Pass relatedPage when creating a page in WebKitTestRunner
​https://bugs.webkit.org/show_bug.cgi?id=115206

Reviewed by Geoffrey Garen.

To make sure that window.open() opens in the same process even when using multiple processes.

• WebKitTestRunner/PlatformWebView.h:
• WebKitTestRunner/TestController.cpp: (WTR::TestController::createOtherPage): (WTR::TestController::createWebViewWithOptions):
• WebKitTestRunner/efl/PlatformWebViewEfl.cpp: (WTR::PlatformWebView::PlatformWebView):
• WebKitTestRunner/gtk/PlatformWebViewGtk.cpp: (WTR::PlatformWebView::PlatformWebView):
• WebKitTestRunner/mac/PlatformWebViewMac.mm: (-[TestRunnerWKView initWithFrame:contextRef:pageGroupRef:relatedToPage:useTiledDrawing:]): (WTR::PlatformWebView::PlatformWebView):
• WebKitTestRunner/qt/PlatformWebViewQt.cpp: (WTR::PlatformWebView::PlatformWebView):
• WebKitTestRunner/win/PlatformWebViewWin.cpp: (WTR::PlatformWebView::PlatformWebView):

3:23 PM Changeset in webkit [149147] by andersca@apple.com

• 5 edits in trunk/Source/WebKit2

Rename DecidePolicyForResponse message to DecidePolicyForResponseSync
​https://bugs.webkit.org/show_bug.cgi?id=115207

Reviewed by Beth Dakin.

In preparation for adding a new async DecidePolicyForResponse message,
rename the current one to DecidePolicyForResponseSync. Also split out the async part
into a separate WebPageProxy::decidePolicyForResponse member function.

• UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::WebPageProxy):
(WebKit::WebPageProxy::receivedPolicyDecision):
(WebKit::WebPageProxy::decidePolicyForResponse):
(WebKit::WebPageProxy::decidePolicyForResponseSync):

• UIProcess/WebPageProxy.h:

(WebPageProxy):

• UIProcess/WebPageProxy.messages.in:
• WebProcess/WebCoreSupport/WebFrameLoaderClient.cpp:

(WebKit::WebFrameLoaderClient::dispatchDecidePolicyForResponse):

3:17 PM Changeset in webkit [149146] by oliver@apple.com

• 4 edits in trunk/Source

Stack guards are too conservative
​https://bugs.webkit.org/show_bug.cgi?id=115147

Reviewed by Mark Hahnenberg.

Source/JavaScriptCore:

Increase stack guard to closer to old size.

• interpreter/Interpreter.cpp:

(JSC::Interpreter::StackPolicy::StackPolicy):

Source/WTF:

Use getrlimit on darwin to get the stack size for the main thread.

• wtf/StackBounds.cpp:

(WTF::StackBounds::initialize):

2:39 PM Changeset in webkit [149145] by commit-queue@webkit.org

• 2 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Selection overlay on non-composited iframes are incorrectly positioned.
​https://bugs.webkit.org/show_bug.cgi?id=115197

Patch by Andrew Lo <​anlo@rim.com> on 2013-04-25
Reviewed by Rob Buis.

When drawing the selection overlay, the rects to
paint when selecting text on non-composited sub-frames
need to be adjusted by the frame position.

• WebKitSupport/SelectionOverlay.cpp:

(BlackBerry::WebKit::SelectionOverlay::paintContents):

2:29 PM Changeset in webkit [149144] by Antoine Quint

• 2 edits in trunk/Source/WebCore

Glyphs may fail to render when using SVG font
​https://bugs.webkit.org/show_bug.cgi?id=115193

Reviewed by Simon Fraser.

Calling SimpleFontData::applyTransforms() when the font used is
an SVG font makes little sense since Core Text doesn’t know or
understand SVG fonts and would be passed some other, unrelated
platform font.

• platform/graphics/SimpleFontData.h:

(WebCore::SimpleFontData::applyTransforms):

2:21 PM Changeset in webkit [149143] by rniwa@webkit.org

• 2 edits in trunk/Tools

Fix autocompletion for Benjamin. The Bugzilla email address must come first.

• Scripts/webkitpy/common/config/contributors.json:

2:02 PM Changeset in webkit [149142] by jer.noble@apple.com

• 3 edits in trunk/Source/WebCore

Media elements shouldn't resume playback when a page is restored from the back/forward cache if the WKView isn't in a window
​https://bugs.webkit.org/show_bug.cgi?id=115191

Reviewed by Eric Carlson.

If a page is suspended, then resumed when its WebView or WKView has been removed from a window,
the page's media elements will unpause. Check whether media is allowed to start during resume()
via Page::canMediaStart() and if not, register for mediaCanStart notifications. Then, in
mediaCanStart() if the media is force-paused, unpause it.

• html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::suspend):
(WebCore::HTMLMediaElement::resume):
(WebCore::HTMLMediaElement::mediaCanStart):

• html/HTMLMediaElement.h:

(WebCore::HTMLMediaElement::pageConsentRequiredForResume):

2:01 PM Changeset in webkit [149141] by ap@apple.com

• 2 edits in trunk/LayoutTests

Update a comment, mentioning a newly filed bug.

• platform/wk2/TestExpectations:

1:37 PM Changeset in webkit [149140] by akling@apple.com

• 7 edits in trunk/Source

Remove ENABLE(PARSED_STYLE_SHEET_CACHING) and make it always-on.

Rubber-stamped by Anders Koivisto.

Source/WebCore:

• PlatformWinCE.cmake:
• accessibility/AccessibilityObject.h:
• html/HTMLLinkElement.cpp:

(WebCore::HTMLLinkElement::setCSSStyleSheet):

Source/WebKit:

• PlatformWinCE.cmake:

Source/WebKit/blackberry:

• WebCoreSupport/AboutDataEnableFeatures.in:

Source/WTF:

• wtf/FeatureDefines.h:

1:26 PM Changeset in webkit [149139] by Patrick Gansterer

• 5 edits in trunk/Source

Remove OS(WINCE) from AccessibilityObject.h
​https://bugs.webkit.org/show_bug.cgi?id=115192

Reviewed by Andreas Kling.

Use the same files like the the other PLATFORM(WIN) ports use.

Source/WebCore:

• PlatformWinCE.cmake:
• accessibility/AccessibilityObject.h:

(AccessibilityObject):

Source/WebKit:

• PlatformWinCE.cmake:

1:11 PM Changeset in webkit [149138] by commit-queue@webkit.org

• 2 edits in trunk/Source/WebCore

Add definition of GlyphBufferGlyph for BlackBerry
​https://bugs.webkit.org/show_bug.cgi?id=115183

Patch by Alberto Garcia <​agarcia@igalia.com> on 2013-04-25
Reviewed by Xan Lopez.

BlackBerry uses an unsigned int.

• platform/graphics/GlyphBuffer.h:

(WebCore):

1:02 PM Changeset in webkit [149137] by commit-queue@webkit.org

• 5 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Make scroll position adjustment work with pages with fixed position elements.
​https://bugs.webkit.org/show_bug.cgi?id=115178

Patch by Iris Wu <​shuwu@blackberry.com> on 2013-04-25
Reviewed by Rob Buis.

PR 308796

Currently the position WebPage::adjustDocumentScrollPosition adjusts is the top
left point of the viewport.
On the page with fixed position elements, we want it to adjust the position beneath
the fixed elements so it can be always visible.

The basic idea is:

1. Detect if there are fixed position elements before going through ProximityDetector.
2. If the fixed element exists, calculate its the size and the actual visible position

beneath it.

3. Pass the position to ProximityDetector. Then according to the new position we get,

calculate the top left position of the viewport (final scroll position).

• Api/WebPage.cpp:

(BlackBerry::WebKit::WebPage::fixedElementSizeDelta):
(WebKit):

• Api/WebPage.h:
• Api/WebPageCompositor.cpp:

(BlackBerry::WebKit::WebPageCompositorPrivate::findFixedElementRect):
(WebKit):

• Api/WebPageCompositor_p.h:

(WebPageCompositorPrivate):

1:00 PM Changeset in webkit [149136] by oliver@apple.com

• 2 edits in trunk/Source/JavaScriptCore

Stack guards are too conservative
​https://bugs.webkit.org/show_bug.cgi?id=115147

Reviewed by Geoffrey Garen.

Reduce the limits and simplify the decision making.

• interpreter/Interpreter.cpp:

(JSC::Interpreter::StackPolicy::StackPolicy):

12:56 PM Changeset in webkit [149135] by commit-queue@webkit.org

• 2 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Enable balanced page group load deferrer behaviour.
​https://bugs.webkit.org/show_bug.cgi?id=115189

Patch by Mike Lattanzio <​mlattanzio@blackberry.com> on 2013-04-25
Reviewed by Rob Buis.

Prevent a possible deadlock by enabling balanced deferrers.
Internally reviewed by: Joe Mason
PR 329986

• Api/WebPage.cpp:

(BlackBerry::WebKit::WebPagePrivate::init):

12:54 PM Changeset in webkit [149134] by commit-queue@webkit.org

• 2 edits in trunk/Source/JavaScriptCore

JSC: Fix interpreter misbehavior in builds with JIT disabled
​https://bugs.webkit.org/show_bug.cgi?id=115190

Patch by Nick Diego Yamane <​nick.yamane@openbossa.org> on 2013-04-25
Reviewed by Oliver Hunt.

Commit ​http://trac.webkit.org/changeset/147858 modified
some details on how JS stack traces are built. The method
"getLineNumberForCallFrame", renamed in that changeset to
"getBytecodeOffsetForCallFrame" is always returning `0' when
JIT is disabled

How to reproduce:

• Build webkit with JIT disabled
• Open MiniBrowser, for example, with ​http://google.com
• In a debug build, WebProcess will hit the following ASSERT: Source/JavaScriptCore/bytecode/UnlinkedCodeBlock.cpp:279 ASSERT(low);

• interpreter/Interpreter.cpp:

(JSC::getBytecodeOffsetForCallFrame):

12:50 PM Changeset in webkit [149133] by rniwa@webkit.org

• 2 edits in trunk/Tools

Build fix. Forgot to merge args.

• Scripts/webkitpy/tool/bot/irc_command.py:

(Hi.execute):

12:44 PM Changeset in webkit [149132] by rniwa@webkit.org

• 2 edits in trunk/Tools

webkitbot should recognize its own nickname in greetings.
​https://bugs.webkit.org/show_bug.cgi?id=115196

Reviewed by Andreas Kling.

Make webkitbot recognize other forms of its nick. Also be tolerant of spaces between the nick and !.

• Scripts/webkitpy/tool/bot/irc_command.py:

(Hi.execute):

12:31 PM Changeset in webkit [149131] by Joseph Pecoraro

• 21 edits
  2 adds
  2 deletes in trunk

ScriptExecutionContext log exception should include a column number
​https://bugs.webkit.org/show_bug.cgi?id=114315

Reviewed by Oliver Hunt.

Source/WebCore:

Test: inspector/console/console-exception-stack-traces.html

• bindings/js/ScriptCallStackFactory.h:
• bindings/js/ScriptCallStackFactory.cpp:

(WebCore::createScriptCallStackFromException):
Generate a ScriptCallStack from an exception. Use the vm.exceptionStack
if available, and fallback to the exception object where needed.

• bindings/js/JSDOMBinding.cpp:

(WebCore::reportException):
Always include a non-empty call stack with exceptions.
Where not provided, fallback to the exception object.

• dom/Document.h:
• dom/Document.cpp:

(WebCore::Document::logExceptionToConsole):

• dom/ScriptExecutionContext.h:
• dom/ScriptExecutionContext.cpp:

(WebCore::ScriptExecutionContext::PendingException::PendingException):
(ScriptExecutionContext::PendingException):
(WebCore::ScriptExecutionContext::reportException):

• workers/DefaultSharedWorkerRepository.cpp:
• workers/SharedWorkerContext.cpp:

(WebCore::SharedWorkerContext::logExceptionToConsole):

• workers/SharedWorkerContext.h:
• workers/WorkerContext.cpp:

(WebCore::WorkerContext::logExceptionToConsole):

• workers/WorkerContext.h:
• workers/WorkerMessagingProxy.cpp:

(WebCore::WorkerExceptionTask::performTask):
Plumb columnNumber through as needed.

LayoutTests:

• inspector/console/console-exception-stack-traces-expected.txt: Added.
• inspector/console/console-exception-stack-traces.html: Added.

Verifies that exceptions have stack traces and match a console.trace stack.

• http/tests/inspector-enabled/console-exception-while-no-inspector-expected.txt: Removed.
• http/tests/inspector-enabled/console-exception-while-no-inspector.html: Removed.

We now save exception backtraces even if the inspector is not open.

• fast/dom/javascript-url-exception-isolation-expected.txt
• fast/events/remove-target-with-shadow-in-drag-expected.txt
• fast/events/set-attribute-listener-window-onerror-crash-expected.txt
• inspector-protocol/media-query-listener-exception-expected.txt:
• inspector/console/console-uncaught-exception-expected.txt:
• inspector/console/console-uncaught-exception-in-eval-expected.txt:

Update results that now include backtraces or different data. Most better, some worse.

12:02 PM WebKitIDL edited by Christophe Dumez

[Callback] only applies to parameters, not interfaces (diff)

11:59 AM Changeset in webkit [149130] by oliver@apple.com

• 4 edits in trunk/Source/JavaScriptCore

Make checkSyntax take a VM instead of an ExecState

RS=Tim

11:57 AM WebKitIDL edited by Christophe Dumez

Update [Callback] extended attribute doc due to r149113 (diff)

11:50 AM Changeset in webkit [149129] by Gregg Tavares

• 5 edits in trunk/Source/WebCore

Fix build files so WebGLDebugXXX extensions can be used if enabled.
​https://bugs.webkit.org/show_bug.cgi?id=113976

Reviewed by Dean Jackson.

No new tests as no new functionality.

• DerivedSources.make:
• GNUmakefile.list.am:
• WebCore.xcodeproj/project.pbxproj:
• bindings/js/JSWebGLRenderingContextCustom.cpp:

11:35 AM Changeset in webkit [149128] by msaboff@apple.com

• 2 edits in trunk/Source/JavaScriptCore

32 Bit: Crash due to RegExpTest nodes not setting result type to Boolean
​https://bugs.webkit.org/show_bug.cgi?id=115188

Reviewed by Geoffrey Garen.

Changed the RegExpTest node to set the AbstractValue to boolean, since that
what it is.

• dfg/DFGAbstractState.cpp:

(JSC::DFG::AbstractState::executeEffects):

11:33 AM Changeset in webkit [149127] by rniwa@webkit.org

• 2 edits in trunk/Source/WebCore

cloneChildNodes looks for deleteButtonController in each level of recursion
​https://bugs.webkit.org/show_bug.cgi?id=115146

Reviewed by Andreas Kling.

Obtain the delete button controller upfront, and shallow copy descendents of each child
so that we don't look for the delete button controller inside cloneNode called on each child.

Performance Tests: DOM/CloneNodes.html

• dom/ContainerNode.cpp:

(WebCore::cloneChildNodesAvoidingDeleteButon): Extracted.
(WebCore::ContainerNode::cloneChildNodes):

11:32 AM Changeset in webkit [149126] by rniwa@webkit.org

• 8 edits in trunk

HTMLOptionsCollection's namedItem and name getter should return the first item
​https://bugs.webkit.org/show_bug.cgi?id=115150

Reviewed by Andreas Kling.

Source/WebCore:

Following the resolution in ​http://lists.whatwg.org/pipermail/whatwg-whatwg.org/2012-December/038355.html,
the spefication has been updated to only return the first item when there are multiple items of the same name
in HTMLOptionsCollection; this new behavior matches that of Firefox and Opera (Presto).

Implement this new behavior to remove the custom binding code and use the fast path in namedItem and name
getter of HTMLOptionsCollection. (Obtaining all items for a given name is expensive!).

Tests: fast/dom/HTMLSelectElement/named-options.html

fast/dom/html-collections-named-getter.html

• bindings/js/JSHTMLOptionsCollectionCustom.cpp:

(WebCore): Removed the custom bindings for name getter and namedItem.

• html/HTMLOptionsCollection.idl:

LayoutTests:

Changed the expectations of the tests.

• fast/dom/HTMLSelectElement/named-options-expected.txt:
• fast/dom/HTMLSelectElement/script-tests/named-options.js:
• fast/dom/html-collections-named-getter-expected.txt:
• fast/dom/html-collections-named-getter.html:

11:31 AM Changeset in webkit [149125] by Joseph Pecoraro

• 59 edits
  1 move
  3 adds
  1 delete in trunk

Web Inspector: ConsoleMessage should include line and column number where possible
​https://bugs.webkit.org/show_bug.cgi?id=114929

Source/WebCore:

• adds "m_column" to WebCore::ConsoleMessage
• adds "column" to Console.ConsoleMessage in the inspector protocol
• set the column number for console.* functions (Console.cpp)
• set the column number for XSLT errors (XSLTProcessor)
• plumb columnNumber everywhere else it is needed, set it to 0 and file bugs for all cases missing columnNumber that could provide it.

Reviewed by Timothy Hatcher.

Test: inspector/console/console-url-line-column.html

inspector/console/console-messages-stack-traces.html

• inspector/ConsoleMessage.h:
• inspector/ConsoleMessage.cpp:

(WebCore::ConsoleMessage::ConsoleMessage):
(WebCore::ConsoleMessage::autogenerateMetadata):
(WebCore::ConsoleMessage::addToFrontend):
(WebCore::ConsoleMessage::isEqual):
Add m_column and set it where appropriate.

• inspector/Inspector.json:

Add column property to Console.ConsoleMessage.

• page/Console.cpp:

(WebCore::internalAddMessage):
(WebCore::Console::profile):
Set columnNumber like lineNumber from the last stack frame.
(WebCore::Console::groupEnd):
Line and column are unused in this message type, set both to 0.

• xml/XSLTProcessorLibxslt.cpp:

(WebCore::XSLTProcessor::parseErrorFunc):

• xml/XSLTProcessorQt.cpp:

(WebCore::XSLTMessageHandler::handleMessage):
Add real column numbers, the XSLT handlers already had it available.

• bindings/js/JSCustomXPathNSResolver.cpp:

(WebCore::JSCustomXPathNSResolver::lookupNamespaceURI):

• css/CSSParser.cpp:

(WebCore::CSSParser::logError):

• dom/ScriptExecutionContext.h:
• dom/ScriptExecutionContext.cpp:

(WebCore::ScriptExecutionContext::addConsoleMessage):

• dom/Document.cpp:

(WebCore::Document::logExceptionToConsole):
(WebCore::Document::addMessage):

• dom/Document.h:
• inspector/InspectorConsoleAgent.cpp:

(WebCore::InspectorConsoleAgent::addMessageToConsole):
(WebCore::InspectorConsoleAgent::stopTiming):
(WebCore::InspectorConsoleAgent::didFinishXHRLoading):
(WebCore::InspectorConsoleAgent::didReceiveResponse):
(WebCore::InspectorConsoleAgent::didFailLoading):

• inspector/InspectorConsoleAgent.h:
• inspector/InspectorConsoleInstrumentation.h:

(WebCore::InspectorInstrumentation::addMessageToConsole):
(WebCore::InspectorInstrumentation::addStartProfilingMessageToConsole):

• inspector/InspectorInstrumentation.cpp:

(WebCore::InspectorInstrumentation::addMessageToConsoleImpl):
(WebCore::InspectorInstrumentation::addStartProfilingMessageToConsoleImpl):
(WebCore::InspectorInstrumentation::addProfileImpl):

• inspector/InspectorInstrumentation.h:

(InspectorInstrumentation):

• inspector/InspectorProfilerAgent.cpp:

(WebCore::InspectorProfilerAgent::addProfile):
(WebCore::InspectorProfilerAgent::addProfileFinishedMessageToConsole):
(WebCore::InspectorProfilerAgent::addStartProfilingMessageToConsole):
(WebCore::InspectorProfilerAgent::start):
(WebCore::InspectorProfilerAgent::stop):

• inspector/InspectorProfilerAgent.h:

(InspectorProfilerAgent):

• loader/EmptyClients.h:

(WebCore::EmptyChromeClient::addMessageToConsole):

• page/ChromeClient.h:

(WebCore::ChromeClient::addMessageToConsole):

• page/ContentSecurityPolicy.cpp:

(WebCore::ContentSecurityPolicy::logToConsole):

• page/PageConsole.cpp:

(WebCore::PageConsole::addMessage):

• page/PageConsole.h:
• workers/DefaultSharedWorkerRepository.cpp:

(SharedWorkerProxy):
(WebCore::postExceptionTask):
(WebCore::SharedWorkerProxy::postExceptionToWorkerObject):
(WebCore::postConsoleMessageTask):
(WebCore::SharedWorkerProxy::postConsoleMessageToWorkerObject):

• workers/SharedWorkerContext.cpp:

(WebCore::SharedWorkerContext::logExceptionToConsole):

• workers/WorkerContext.cpp:

(WebCore::WorkerContext::logExceptionToConsole):
(WebCore::WorkerContext::addConsoleMessage):
(WebCore::WorkerContext::addMessage):
(WebCore::WorkerContext::addMessageToWorkerConsole):

• workers/WorkerContext.h:
• workers/WorkerMessagingProxy.cpp:

(WebCore::WorkerExceptionTask::create):
(WebCore::WorkerExceptionTask::WorkerExceptionTask):
(WebCore::WorkerExceptionTask::performTask):
(WorkerExceptionTask):
(WebCore::WorkerMessagingProxy::postExceptionToWorkerObject):
(WebCore::postConsoleMessageTask):
(WebCore::WorkerMessagingProxy::postConsoleMessageToWorkerObject):

• workers/WorkerMessagingProxy.h:

(WorkerMessagingProxy):

• workers/WorkerReportingProxy.h:

(WorkerReportingProxy):
Plumb columnNumber through where appropriate. File bugs where missing.

Source/WebKit/blackberry:

Reviewed by Timothy Hatcher.

• Api/DumpRenderTreeClient.h:
• Api/WebPageClient.h:
• WebCoreSupport/ChromeClientBlackBerry.cpp:

(WebCore::ChromeClientBlackBerry::addMessageToConsole):

• WebCoreSupport/ChromeClientBlackBerry.h:

(ChromeClientBlackBerry):

Source/WebKit/efl:

Reviewed by Timothy Hatcher.

• WebCoreSupport/ChromeClientEfl.cpp:

(WebCore::ChromeClientEfl::addMessageToConsole):

• WebCoreSupport/ChromeClientEfl.h:

(ChromeClientEfl):

Source/WebKit/gtk:

Reviewed by Timothy Hatcher.

• WebCoreSupport/ChromeClientGtk.cpp:

(WebKit::ChromeClient::addMessageToConsole):

• WebCoreSupport/ChromeClientGtk.h:

(ChromeClient):

Source/WebKit/mac:

Reviewed by Timothy Hatcher.

• WebCoreSupport/WebChromeClient.h:
• WebCoreSupport/WebChromeClient.mm:

(WebChromeClient::addMessageToConsole):

Source/WebKit/qt:

Reviewed by Timothy Hatcher.

• WebCoreSupport/ChromeClientQt.cpp:

(WebCore::ChromeClientQt::addMessageToConsole):

• WebCoreSupport/ChromeClientQt.h:

Source/WebKit/win:

Reviewed by Timothy Hatcher.

• WebCoreSupport/WebChromeClient.cpp:

(WebChromeClient::addMessageToConsole):

• WebCoreSupport/WebChromeClient.h:

Source/WebKit/wince:

Reviewed by Timothy Hatcher.

• WebCoreSupport/ChromeClientWinCE.cpp:

(WebKit::ChromeClientWinCE::addMessageToConsole):

• WebCoreSupport/ChromeClientWinCE.h:

Source/WebKit2:

Reviewed by Timothy Hatcher.

• WebProcess/WebCoreSupport/WebChromeClient.cpp:

(WebKit::WebChromeClient::addMessageToConsole):

• WebProcess/WebCoreSupport/WebChromeClient.h:

LayoutTests:

Update a test that was outputting the url and line number of console
method calls, to output the column number as well. Added a test that
verifies that console API messages have stack traces if appropriate.

Reviewed by Timothy Hatcher.

• inspector/console/console-url-and-line-expected.txt: Removed.
• inspector/console/console-url-line-column-expected.txt: Added.
• inspector/console/console-url-line-column.html: Renamed from LayoutTests/inspector/console/console-url-and-line.html.
• inspector/console/console-messages-stack-traces-expected.txt: Added.
• inspector/console/console-messages-stack-traces.html: Added.

11:29 AM Changeset in webkit [149124] by commit-queue@webkit.org

• 2 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Get rid of return in void method
​https://bugs.webkit.org/show_bug.cgi?id=115186

Patch by Konrad Piascik <​kpiascik@blackberry.com> on 2013-04-25
Reviewed by Rob Buis.

• Api/WebPage.cpp:

(BlackBerry::WebKit::WebPage::setForcedTextEncoding):

11:19 AM Changeset in webkit [149123] by allan.jensen@digia.com

• 5 edits in trunk/Source

[Qt] Animated opacity does not trigger accelerated compositing
​https://bugs.webkit.org/show_bug.cgi?id=115107

Reviewed by Simon Fraser.

Source/WebCore:

Add extra option to also trigger compositing on animated opacity.

• page/ChromeClient.h:
• rendering/RenderLayerCompositor.cpp:

(WebCore::RenderLayerCompositor::requiresCompositingForAnimation):

Source/WebKit/qt:

We should trigger compositing on animated opacity as it is always faster on Qt.

• WebCoreSupport/ChromeClientQt.cpp:

VideoTrigger removed because we have already disabled AC for video in qwebsettings.

11:09 AM Changeset in webkit [149122] by Alexandru Chiculita

• 1 edit in trunk/Tools/Scripts/webkitpy/common/config/contributors.json

Unreviewed. Adding myself to the reviwers list.

10:59 AM Changeset in webkit [149121] by beidson@apple.com

• 3 edits in trunk/Source/WebKit2

Thread safety issues in WKCustomProtocol.
<rdar://problem/13247304> and ​https://bugs.webkit.org/show_bug.cgi?id=115185

Reviewed by Alexey Proskuryakov.

• Shared/Network/CustomProtocols/CustomProtocolManager.h:

• Shared/Network/CustomProtocols/mac/CustomProtocolManagerMac.mm:

(WebKit::CustomProtocolManager::addCustomProtocol): Protect m_customProtocolMap with its mutex.
(WebKit::CustomProtocolManager::removeCustomProtocol): Ditto.
(WebKit::CustomProtocolManager::didFailWithError):
(WebKit::CustomProtocolManager::didLoadData):
(WebKit::CustomProtocolManager::didReceiveResponse):
(WebKit::CustomProtocolManager::didFinishLoading):
(WebKit::CustomProtocolManager::protocolForID): Protect m_customProtocolMap with its mutex, and return

a RetainPtr instead of a raw pointer.

10:48 AM Changeset in webkit [149120] by commit-queue@webkit.org

• 4 edits
  3 adds in trunk

Sec-WebSocket-Extensions header field must not appear more than once in an HTTP response.
​https://bugs.webkit.org/show_bug.cgi?id=115128

Patch by Lamarque V. Souza <​Lamarque.Souza@basyskom.com> on 2013-04-25
Reviewed by Alexey Proskuryakov.

Source/WebCore:

According to WebSocket specification Sec-WebSocket-Extensions header field
must not appear more than once in an HTTP response. It is ok if it appears
more than once in client request. Also add a check for invalid UTF-8
characters when parsing Sec-WebSocket-Extensions quoted string.

Test: http/tests/websocket/tests/hybi/handshake-fail-by-more-extensions-header.html

• Modules/websockets/WebSocketExtensionParser.cpp:

(WebCore::WebSocketExtensionParser::consumeQuotedString): Return false if there is
invalid character in the quoted string being parsed.

• Modules/websockets/WebSocketHandshake.cpp:

(WebCore::WebSocketHandshake::readHTTPHeaders): Check if Sec-WebSocket-Extensions
appears more than once in header response. Abort handshake if it does.

LayoutTests:

• http/tests/websocket/tests/hybi/handshake-fail-by-more-extensions-header-expected.txt: Added.
• http/tests/websocket/tests/hybi/handshake-fail-by-more-extensions-header.html: Added.
• http/tests/websocket/tests/hybi/handshake-fail-by-more-extensions-header_wsh.py: Added.

10:31 AM Changeset in webkit [149119] by jer.noble@apple.com

• 3 edits in trunk/Source/WebKit2

Further build fixes: correct two more misnames of WebProcessShim.dyld.

• WebProcess/EntryPoint/mac/LegacyProcess/WebContentProcessMain.mm:

(WebKit::WebContentProcessMainDelegate::doPreInitializationWork):

• WebProcess/EntryPoint/mac/XPCService/WebContentService/Info.plist:

10:12 AM Changeset in webkit [149118] by Martin Robinson

• 2 edits in trunk/Source/WebKit2

Make gobject-introspection warnings non-fatal
​https://bugs.webkit.org/show_bug.cgi?id=115184

Reviewed by Carlos Garcia Campos.

• GNUmakefile.am: Make it so that gobject-introspection errors do not fail the buld

until we get to a point where gobject-introspection gives consistent warnings.

9:46 AM Changeset in webkit [149117] by Carlos Garcia Campos

• 14 edits
  4 adds in trunk/Source/WebKit2

[GTK] Add WebKitWebViewGroup to WebKit2 GTK+ API
​https://bugs.webkit.org/show_bug.cgi?id=111265

Reviewed by Anders Carlsson.

In the current API the settings are always shared by all web
views, because they are always created on the default page
group. This makes impossible to have different settings on
different web views. Expose the page group in the API,
so that users can create web views on a group different than the
default one to have their own settings.

• GNUmakefile.list.am: Add new files to compilation.
• UIProcess/API/gtk/WebKitSettings.cpp:

(webkitSettingsGetPreferences): Helper private method to get the
WebPreferences wrapped by a WebKitSettings object.

• UIProcess/API/gtk/WebKitSettingsPrivate.h:
• UIProcess/API/gtk/WebKitWebContext.cpp:

(_WebKitWebContextPrivate): Add a default WebKitWebViewGroup.
(webkitWebContextCreatePageForWebView): Create the WebPageProxy
with the WebPageGroup of the given WebKitWebViewGroup.
(webkitWebContextGetDefaultWebViewGroup): Helper private method to
get the default WebKitWebViewGroup.

• UIProcess/API/gtk/WebKitWebContextPrivate.h:
• UIProcess/API/gtk/WebKitWebView.cpp:

(_WebKitWebViewPrivate): Add a WebKitWebViewGroup in case the view
is created with a group.
(webkitWebViewUpdateSettings): Keep a reference of the group
settings to be able to disconnect the signals when the settings
object is replaced in the view group and connect to the signals
for the custom settings.
(webkitWebViewSettingsChanged): Disconnect the signals for the
previous settings and set the new settings.
(webkitWebViewDisconnectSettingsChangedSignalHandler): Disconnect
notify::settings signal of WebKitWebViewGroup.
(webkitWebViewConstructed): Pass the current WebKitWebViewGroup to
webkitWebContextCreatePageForWebView().
(webkitWebViewSetProperty): Add setter for group property.
(webkitWebViewGetProperty): Add getter for group property.
(webkitWebViewDispose): Call
webkitWebViewDisconnectSettingsChangedSignalHandler().
(webkit_web_view_class_init): Add WebKitWebView:group property.
(webkitWebViewHandleAuthenticationChallenge): Use
webkit_web_view_get_settings().
(webkit_web_view_new_with_group): Create a new web view with the
given WebKitWebViewGroup.
(webkit_web_view_get_group): Return the current WebKitWebViewGroup
or the default one if the view was not created with
webkit_web_view_new_with_group().
(webkit_web_view_set_settings): Set the settings of the current
WebKitWebViewGroup.
(webkit_web_view_get_settings): Get the settings of the current
WebKitWebViewGroup.
(webkit_web_view_set_zoom_level): Use
webkit_web_view_get_settings().
(webkit_web_view_get_zoom_level): Ditto.

• UIProcess/API/gtk/WebKitWebView.h:
• UIProcess/API/gtk/WebKitWebViewGroup.cpp: Added.

(_WebKitWebViewGroupPrivate):
(webkitWebViewGroupSetProperty):
(webkitWebViewGroupGetProperty):
(webkitWebViewGroupConstructed):
(webkit_web_view_group_class_init):
(webkitWebViewGroupAttachSettingsToPageGroup): Set the
WebPreferences of the current WebKitSettings to the WebPageGroup.
(webkitWebViewGroupCreate): Create a new WebKitWebViewGroup for
the given WebPageGroup.
(webkitWebViewGroupGetPageGroup): Helper private method to get the
WebPageGroup wrapped by the WebKitWebViewGroup.
(webkit_web_view_group_new): Create a new WebKitWebViewGroup.
(webkit_web_view_group_get_name): Return the name of a
WebKitWebViewGroup.
(webkit_web_view_group_get_settings): Get the settings of a
WebKitWebViewGroup.
(webkit_web_view_group_set_settings): Set new settings of a
WebKitWebViewGroup.

• UIProcess/API/gtk/WebKitWebViewGroup.h: Added.

(_WebKitWebViewGroup):
(_WebKitWebViewGroupClass):

• UIProcess/API/gtk/WebKitWebViewGroupPrivate.h: Added.
• UIProcess/API/gtk/docs/webkit2gtk-docs.sgml: Add

WebKitWebViewGroup section.

• UIProcess/API/gtk/docs/webkit2gtk-sections.txt: Add new symbols.
• UIProcess/API/gtk/docs/webkit2gtk.types: Add

webkit_web_view_group_get_type.

• UIProcess/API/gtk/tests/GNUmakefile.am: Add new files to

compilation.

• UIProcess/API/gtk/tests/TestWebKitWebView.cpp:

(testWebViewSettings): We can't watch settings in the default view
group anymore, because they are not released with the web view but
with the web context that is freed after the Test destructor.

• UIProcess/API/gtk/tests/TestWebKitWebViewGroup.cpp: Added.

(testWebViewGroupDefault):
(testWebViewGroupNewGroup):
(testWebViewNewWithGroup):
(testWebViewGroupSettings):
(beforeAll):
(afterAll):

• UIProcess/API/gtk/webkit2.h: Include WebKitWebViewGroup.h.

9:39 AM Changeset in webkit [149116] by commit-queue@webkit.org

• 2 edits in trunk/Source/WTF

[Windows] Compile fix.
​https://bugs.webkit.org/show_bug.cgi?id=115170

Patch by ​peavo@outlook.com <​peavo@outlook.com> on 2013-04-25
Reviewed by Filip Pizlo.

Forward declare function, instead of including intrin.h.

• wtf/Atomics.h:

9:32 AM Changeset in webkit [149115] by rakuco@webkit.org

• 2 edits in trunk/Source/WebCore

REGRESSION(r148758): Remove WTFLogAlways call from IconLoader.
​https://bugs.webkit.org/show_bug.cgi?id=115182

Reviewed by Dan Bernstein.

The call added in r148758 was apparently a leftover from some debugging
session, and it makes the Interactive/window-resize.html performance
test randomly fail on Qt and EFL due to WTFLogAlways writing to stderr.

• loader/icon/IconLoader.cpp:

(WebCore::IconLoader::notifyFinished):

9:18 AM Changeset in webkit [149114] by commit-queue@webkit.org

• 2 edits in trunk/Source/JavaScriptCore

REGRESSION(r137994): Random crashes occur with SH4 JSC.
​https://bugs.webkit.org/show_bug.cgi?id=115167.

Patch by Julien Brianceau <​jbrianceau@nds.com> on 2013-04-25
Reviewed by Oliver Hunt.

Since r137994, uncommited pages could be inside the area of memory in
parameter of the cacheFlush function. That's why we have to flush each
page separately to avoid a fail of the whole flush, if an uncommited page
is in the area.

This patch is very similar to changeset 145194 made for ARMv7 architecture,
see ​https://bugs.webkit.org/show_bug.cgi?id=111441 for further information.

• assembler/SH4Assembler.h:

(JSC::SH4Assembler::cacheFlush):

8:55 AM Changeset in webkit [149113] by Christophe Dumez

• 32 edits in trunk/Source/WebCore

Add support for Web IDL callback interfaces to the bindings generator
​https://bugs.webkit.org/show_bug.cgi?id=115179

Reviewed by Kentaro Hara.

Add support in the bindings generator for Web IDL callback interfaces:
​http://dev.w3.org/2006/webapi/WebIDL/#dfn-callback-interface

Drop support for the WebKit-specific [Callback] extended attributes
on Web IDL interfaces and update existing IDL files to use proper
callback interfaces instead.

No new tests, no behavior change.

• Modules/filesystem/EntriesCallback.idl:
• Modules/filesystem/EntryCallback.idl:
• Modules/filesystem/ErrorCallback.idl:
• Modules/filesystem/FileCallback.idl:
• Modules/filesystem/FileSystemCallback.idl:
• Modules/filesystem/FileWriterCallback.idl:
• Modules/filesystem/MetadataCallback.idl:
• Modules/geolocation/PositionCallback.idl:
• Modules/geolocation/PositionErrorCallback.idl:
• Modules/mediastream/NavigatorUserMediaErrorCallback.idl:
• Modules/mediastream/NavigatorUserMediaSuccessCallback.idl:
• Modules/mediastream/RTCErrorCallback.idl:
• Modules/mediastream/RTCSessionDescriptionCallback.idl:
• Modules/mediastream/RTCStatsCallback.idl:
• Modules/notifications/NotificationPermissionCallback.idl:
• Modules/quota/StorageErrorCallback.idl:
• Modules/quota/StorageQuotaCallback.idl:
• Modules/quota/StorageUsageCallback.idl:
• Modules/webaudio/AudioBufferCallback.idl:
• Modules/webdatabase/DatabaseCallback.idl:
• Modules/webdatabase/SQLStatementCallback.idl:
• Modules/webdatabase/SQLStatementErrorCallback.idl:
• Modules/webdatabase/SQLTransactionCallback.idl:
• Modules/webdatabase/SQLTransactionErrorCallback.idl:
• Modules/webdatabase/SQLTransactionSyncCallback.idl:
• bindings/scripts/CodeGeneratorJS.pm:

(GenerateInterface):

• bindings/scripts/IDLParser.pm:

(parseCallbackRestOrInterface):

• bindings/scripts/test/TestCallback.idl:
• dom/RequestAnimationFrameCallback.idl:
• dom/StringCallback.idl:
• html/VoidCallback.idl:

8:17 AM Changeset in webkit [149112] by mikhail.pozdnyakov@intel.com

• 2 edits in trunk/Source/WTF

Use std::move in AtomicString
​https://bugs.webkit.org/show_bug.cgi?id=115102

Reviewed by Anders Carlsson.

Use std::move in AtomicString instead of direct casting to rvalue references.

• wtf/text/AtomicString.h:

(AtomicString):
(WTF::AtomicString::AtomicString):
(WTF::AtomicString::operator=):

7:52 AM Changeset in webkit [149111] by commit-queue@webkit.org

• 8 edits
  1 copy in trunk

[GStreamer] Add audio/speex MIME type as supported
​https://bugs.webkit.org/show_bug.cgi?id=115032

Patch by Adrian Perez de Castro <Adrian Perez de Castro> on 2013-04-25
Reviewed by Philippe Normand.

Source/WebCore:

GStreamer has support for the Speex codec (​http://www.speex.org).
Speex streams may be embedded in a Ogg container, or standalone.
The case of the Ogg container is already covered by the "*/ogg"
MIME types declared as supported by the GStreamer media playing
code, but for standalone streams to work, "audio/speex" has to
be added.

With this, and the needed GStreamer plugin installed, the GTK+
launcher is able to correctly play Speex streams served with
"Content-type: audio/speex".

• platform/graphics/gstreamer/MediaPlayerPrivateGStreamer.cpp:

LayoutTests:

Speex (​http://www.speex.org) streams can be embedded into Ogg
Ogg containers, so add a check for it in the test case.

• media/media-can-play-ogg-expected.txt:
• media/media-can-play-ogg.html:
• platform/blackberry/media/media-can-play-ogg-expected.txt:
• platform/mac/media/media-can-play-ogg-expected.txt:
• platform/qt-linux/media/media-can-play-ogg-expected.txt:
• platform/win/media/media-can-play-ogg-expected.txt:

7:33 AM Changeset in webkit [149110] by akling@apple.com

• 9 edits in trunk/Source/WebCore

Add FINAL decorators to the InlineBox class hierarchy.
<​http://webkit.org/b/115177>

Reviewed by Antti Koivisto.

From Blink r148628 by <​cevans@chromium.org>:

FINAL is a macro in wtf/Compiler.h that does the correct thing if the compiler does not support "final")
The approach used is as simple as possible whilst being thorough.
So, leaf classes have FINAL applied to the whole class whereas intermediary classes have FINAL applied to relevant methods.

FINAL allows a compiler to devirtualize call sites and turn them into direct calls. As you might expect, this is perf positive:
(clang on Linux):

• line_layout.html goes from 120 runs/s -> 123 runs/2, +2.5%
• html5-full-render.html goes from 3176ms -> 3162ms, +0.4%

I have confidence that the former result is statistically significant (as the numbers are very very stable) but not the latter.

• rendering/EllipsisBox.h:
• rendering/InlineFlowBox.h:
• rendering/InlineTextBox.h:
• rendering/RootInlineBox.h:
• rendering/TrailingFloatsRootInlineBox.h:
• rendering/svg/SVGInlineFlowBox.h:
• rendering/svg/SVGInlineTextBox.h:
• rendering/svg/SVGRootInlineBox.h:

7:19 AM Changeset in webkit [149109] by commit-queue@webkit.org

• 2 edits in trunk/Tools

[GTK] Tools/gtk/install-dependencies added ragel dependency
​https://bugs.webkit.org/show_bug.cgi?id=115163

Patch by Xavier Castano <​xcastanho@igalia.com> on 2013-04-25
Reviewed by Xan Lopez.

The install-dependencies script was missing a dependency on ragel
which is expected when building HarfBuzz.

• gtk/install-dependencies: Added a dependency on ragel for both

apt and yum.

7:05 AM Changeset in webkit [149108] by commit-queue@webkit.org

• 2 edits in trunk/Tools

[GTK] Remove pango from optional jhbuild dependencies
​https://bugs.webkit.org/show_bug.cgi?id=115162

Patch by Eduardo Lima Mitev <​elima@igalia.com> on 2013-04-25
Reviewed by Martin Robinson.

Pango is not needed since revision r148293 that lowers the pango
dependency to 1.30, which is available in most distributions.

• gtk/jhbuild-optional.modules: Removes description of pango module.

6:46 AM Changeset in webkit [149107] by jer.noble@apple.com

• 5 edits in trunk/Source/WebKit2

Unreviewed build fix: Fix the name of DyldInterpose.h in
SecItemShimLibrary.mm, fix the name of WebProcessShim.dyld in
WebContentServiceEntryPoint.mm, and stop linking against libWTF.a
directly (which means removing ASSERTs from
CookieStorageShimLibrary.cpp).

• Shared/mac/CookieStorageShimLibrary.cpp:

(WebKit::shimCFHTTPCookieStorageCopyRequestHeaderFieldsForURL):
(WebKit::WebKitCookieStorageShimInitialize):

• WebKit2.xcodeproj/project.pbxproj:
• WebProcess/EntryPoint/mac/XPCService/WebContentServiceEntryPoint.mm:

(WebContentServiceInitializer):

• WebProcess/mac/SecItemShimLibrary.mm:

6:43 AM Changeset in webkit [149106] by akling@apple.com

• 4 edits
  4 adds in trunk

CSS parser: Add error recovery while parsing @-webkit-keyframes key values.
<​http://webkit.org/b/115175>

Source/WebCore:

From Blink r148714 by <​apavlov@chromium.org>:

If not a percentage, "from", or "to" value is used in a key list, the rule is erroneous,
and due to the absense of recovery, the parser skips the following, valid CSS rule.

On a related note, keyframes, whose selectors contain invalid keys, should be discarded
altogether, according to <​http://www.w3.org/TR/css3-animations/#keyframes>

Tests: animations/keyframes-invalid-keys.html

fast/css/webkit-keyframes-errors.html

• css/CSSGrammar.y.in:
• css/CSSParser.cpp:

(WebCore::CSSParser::rewriteSpecifiers):

LayoutTests:

From Blink r148714 by <​apavlov@chromium.org>.

• animations/keyframes-invalid-keys-expected.txt: Added.
• animations/keyframes-invalid-keys.html: Added.
• fast/css/webkit-keyframes-errors-expected.html: Added.
• fast/css/webkit-keyframes-errors.html: Added.

6:23 AM Changeset in webkit [149105] by Antti Koivisto

• 7 edits in trunk/Source/WebCore

REGRESSION (r147797): Animations slideshows of images on www.thesuperficial.com are slow
​https://bugs.webkit.org/show_bug.cgi?id=115172

Reviewed by Andreas Kling.

On this page ads dynamically loaded to subframes on slideshow navigation switch us to state where we throttle layer flushes.

Fix by ignoring any subframe-originated loads when determining throttling.

• loader/FrameLoader.cpp:

(WebCore::FrameLoader::loadProgressingStatusChanged):

• loader/ProgressTracker.cpp:

(WebCore::ProgressTracker::isMainLoadProgressing):

Rename, ignore subframe originated loads. Subframe loads that are initiated during the main load still count.

• loader/ProgressTracker.h:

(ProgressTracker):

• page/FrameView.cpp:

(WebCore::FrameView::updateLayerFlushThrottlingInAllFrames):

• page/FrameView.h:

(FrameView):

• rendering/RenderLayerBacking.cpp:

(WebCore::computeTileCoverage):

6:10 AM Changeset in webkit [149104] by kadam@inf.u-szeged.hu

• 2 edits in trunk/LayoutTests

[Qt] Unreviewed gardening. Skip some failing tests after r148996.

• platform/qt/TestExpectations:

6:04 AM Changeset in webkit [149103] by commit-queue@webkit.org

• 2 edits in trunk/Tools

Remove Chromium-specific cases from FeatureList.pm
​https://bugs.webkit.org/show_bug.cgi?id=115169

Patch by Seokju Kwon <Seokju Kwon> on 2013-04-25
Reviewed by Andreas Kling.

• Scripts/webkitperl/FeatureList.pm:

5:56 AM Changeset in webkit [149102] by akling@apple.com

• 4 edits
  2 adds in trunk

SVG: Fix viewBox animations on shapes with non-scaling-stroke.
<​http://webkit.org/b/115173>

Source/WebCore:

From Blink r149058 by <​pdr@chromium.org>:

Previously hasAttribute() to determine if a viewBox was set in the transform
code for non-scaling-strokes. hasAttribute() should not be used in this case,
as it will return false if the attribute is not set but the value is animating.

This patch switches to checking if the viewBox is empty instead of checking for
the presence of the attribute.

Test: svg/stroke/animated-non-scaling-stroke.html

• svg/SVGSVGElement.cpp:

(WebCore::SVGSVGElement::localCoordinateSpaceTransform):

• svg/SVGSVGElement.h:

(WebCore::SVGSVGElement::hasEmptyViewBox):

LayoutTests:

From Blink r149058 by <​pdr@chromium.org>.

• svg/stroke/animated-non-scaling-stroke-expected.html: Added.
• svg/stroke/animated-non-scaling-stroke.html: Added.

5:32 AM Changeset in webkit [149101] by akling@apple.com

• 2 edits in trunk/Source/WebCore

Remove isPluginElement hack in Document::setFocusedNode()
<​http://webkit.org/b/115171>

From Blink r148800 by <​tkent@chromium.org>:

This hack is no longer necessary because r147591 made isFocusable for plugin
elements work correctly.

• dom/Document.cpp:

(WebCore::Document::setFocusedNode):

5:13 AM Changeset in webkit [149100] by kadam@inf.u-szeged.hu

• 4 edits in trunk/LayoutTests

[Qt] Unreviewed gardening. Unskip now passing tests.

• platform/qt-5.0-wk1/TestExpectations:
• platform/qt-5.0-wk2/TestExpectations:
• platform/qt/TestExpectations:

3:49 AM Changeset in webkit [149099] by kadam@inf.u-szeged.hu

• 23 edits
  22 copies
  8 adds in trunk/LayoutTests

[Qt] Unreviewed gardening. Organize expected files after r149095.

• platform/qt-5.0-wk1/editing/deleting/delete-by-word-002-expected.txt: Copied from LayoutTests/platform/qt/editing/deleting/delete-by-word-002-expected.txt.
• platform/qt-5.0-wk1/editing/deleting/delete-cell-contents-expected.txt: Copied from LayoutTests/platform/qt/editing/deleting/delete-cell-contents-expected.txt.
• platform/qt-5.0-wk1/editing/execCommand/button-expected.txt: Copied from LayoutTests/platform/qt/editing/execCommand/button-expected.txt.
• platform/qt-5.0-wk1/editing/execCommand/createLink-expected.txt: Copied from LayoutTests/platform/qt/editing/execCommand/createLink-expected.txt.
• platform/qt-5.0-wk1/editing/execCommand/indent-pre-expected.txt: Copied from LayoutTests/platform/qt/editing/execCommand/indent-pre-expected.txt.
• platform/qt-5.0-wk1/editing/execCommand/unlink-expected.txt: Copied from LayoutTests/platform/qt/editing/execCommand/unlink-expected.txt.
• platform/qt-5.0-wk1/editing/input/emacs-ctrl-o-expected.txt: Copied from LayoutTests/platform/qt/editing/input/emacs-ctrl-o-expected.txt.
• platform/qt-5.0-wk1/editing/inserting/insert-before-link-1-expected.txt: Copied from LayoutTests/platform/qt/editing/inserting/insert-before-link-1-expected.txt.
• platform/qt-5.0-wk1/editing/pasteboard/display-block-on-spans-expected.txt: Copied from LayoutTests/platform/qt/editing/pasteboard/display-block-on-spans-expected.txt.
• platform/qt-5.0-wk1/editing/pasteboard/emacs-cntl-y-001-expected.txt: Copied from LayoutTests/platform/qt/editing/pasteboard/emacs-cntl-y-001-expected.txt.
• platform/qt-5.0-wk1/editing/pasteboard/merge-end-list-2-expected.txt: Copied from LayoutTests/platform/qt/editing/pasteboard/merge-end-list-2-expected.txt.
• platform/qt-5.0-wk1/editing/pasteboard/merge-end-table-2-expected.txt: Copied from LayoutTests/platform/qt/editing/pasteboard/merge-end-table-2-expected.txt.
• platform/qt-5.0-wk1/editing/pasteboard/page-zoom-expected.txt: Copied from LayoutTests/platform/qt/editing/pasteboard/page-zoom-expected.txt.
• platform/qt-5.0-wk1/editing/selection/extend-selection-word-expected.txt: Copied from LayoutTests/platform/qt/editing/selection/extend-selection-word-expected.txt.
• platform/qt-5.0-wk1/editing/style/apply-through-end-of-document-expected.txt: Copied from LayoutTests/platform/qt/editing/style/apply-through-end-of-document-expected.txt.
• platform/qt-5.0-wk1/editing/style/remove-underline-across-paragraph-expected.txt: Copied from LayoutTests/platform/qt/editing/style/remove-underline-across-paragraph-expected.txt.
• platform/qt-5.0-wk1/editing/style/remove-underline-across-paragraph-in-bold-expected.txt: Copied from LayoutTests/platform/qt/editing/style/remove-underline-across-paragraph-in-bold-expected.txt.
• platform/qt-5.0-wk1/editing/style/remove-underline-expected.txt: Copied from LayoutTests/platform/qt/editing/style/remove-underline-expected.txt.
• platform/qt-5.0-wk1/editing/style/remove-underline-from-stylesheet-expected.txt: Copied from LayoutTests/platform/qt/editing/style/remove-underline-from-stylesheet-expected.txt.
• platform/qt-5.0-wk1/editing/style/remove-underline-in-bold-expected.txt: Copied from LayoutTests/platform/qt/editing/style/remove-underline-in-bold-expected.txt.
• platform/qt-5.0-wk1/editing/style/unbold-in-bold-expected.txt: Copied from LayoutTests/platform/qt/editing/style/unbold-in-bold-expected.txt.
• platform/qt-5.0-wk1/editing/undo/replace-text-in-node-preserving-markers-crash-expected.txt: Copied from LayoutTests/platform/qt/editing/undo/replace-text-in-node-preserving-markers-crash-expected.txt.
• platform/qt/editing/deleting/delete-by-word-002-expected.txt:
• platform/qt/editing/deleting/delete-cell-contents-expected.txt:
• platform/qt/editing/execCommand/button-expected.txt:
• platform/qt/editing/execCommand/createLink-expected.txt:
• platform/qt/editing/execCommand/indent-pre-expected.txt:
• platform/qt/editing/execCommand/unlink-expected.txt:
• platform/qt/editing/input/emacs-ctrl-o-expected.txt:
• platform/qt/editing/inserting/insert-before-link-1-expected.txt:
• platform/qt/editing/pasteboard/display-block-on-spans-expected.txt:
• platform/qt/editing/pasteboard/emacs-cntl-y-001-expected.txt:
• platform/qt/editing/pasteboard/merge-end-list-2-expected.txt:
• platform/qt/editing/pasteboard/merge-end-table-2-expected.txt:
• platform/qt/editing/pasteboard/page-zoom-expected.txt:
• platform/qt/editing/selection/extend-selection-word-expected.txt:
• platform/qt/editing/style/apply-through-end-of-document-expected.txt:
• platform/qt/editing/style/remove-underline-across-paragraph-expected.txt:
• platform/qt/editing/style/remove-underline-across-paragraph-in-bold-expected.txt:
• platform/qt/editing/style/remove-underline-expected.txt:
• platform/qt/editing/style/remove-underline-from-stylesheet-expected.txt:
• platform/qt/editing/style/remove-underline-in-bold-expected.txt:
• platform/qt/editing/style/unbold-in-bold-expected.txt:
• platform/qt/editing/undo/replace-text-in-node-preserving-markers-crash-expected.txt:

2:51 AM Changeset in webkit [149098] by commit-queue@webkit.org

• 4 edits
  2 adds in trunk

EventSource: Synchronous loader callback not handled properly
​https://bugs.webkit.org/show_bug.cgi?id=115104

Source/WebCore:

When attempting a cross-origin request towards a non-HTTP URL, an early verification
of the protocol scheme will cause didFailAccessControlCheck to be called synchronously
before the loader has even finished being created. This special case was not handled
properly, since we tried to cancel a non-existing loader, which resulted in a crash.
In addition to checking whether a request is in flight before trying to cancel it,
this change also schedules the initial creation of the loader to happen asynchronously
when an EventSource is constructed, so that a script can register for the error event
before it is dispatched (as a result of passing a non-HTTP URL to the constructor).

Patch by Per-Erik Brodin <​per-erik.brodin@ericsson.com> on 2013-04-25
Reviewed by Alexey Proskuryakov.

Test: http/tests/eventsource/eventsource-cors-non-http.html

• page/EventSource.cpp:

(WebCore::EventSource::EventSource):
(WebCore::EventSource::create):
(WebCore::EventSource::scheduleInitialConnect):
(WebCore):
(WebCore::EventSource::scheduleReconnect):
(WebCore::EventSource::connectTimerFired):
(WebCore::EventSource::close):
(WebCore::EventSource::abortConnectionAttempt):

• page/EventSource.h:

(EventSource):

LayoutTests:

Patch by Per-Erik Brodin <​per-erik.brodin@ericsson.com> on 2013-04-25
Reviewed by Alexey Proskuryakov.

• http/tests/eventsource/eventsource-cors-non-http-expected.txt: Added.
• http/tests/eventsource/eventsource-cors-non-http.html: Added.

2:43 AM Changeset in webkit [149097] by Patrick Gansterer

• 2 edits in trunk/Source/WTF

Unreviewed WinCE build fix after r148888.

• wtf/Atomics.h: WinCE version of intrin.h is called cmnintrin.h.

2:42 AM Changeset in webkit [149096] by Patrick Gansterer

• 2 edits in trunk/Source/WebCore

Unreviewed WinCE build fix after r148545.

• page/wince/FrameWinCE.cpp:

1:01 AM Changeset in webkit [149095] by kadam@inf.u-szeged.hu

• 7 edits
  21 adds in trunk/LayoutTests

[Qt] Unreviewed gardening. Added platform specific expected files after r148987.

• platform/qt/editing/deleting/delete-by-word-002-expected.txt: Added.
• platform/qt/editing/deleting/delete-cell-contents-expected.txt: Added.
• platform/qt/editing/execCommand/button-expected.txt: Added.
• platform/qt/editing/execCommand/createLink-expected.txt: Added.
• platform/qt/editing/execCommand/indent-pre-expected.txt: Added.
• platform/qt/editing/execCommand/unlink-expected.txt: Added.
• platform/qt/editing/input/emacs-ctrl-o-expected.png:
• platform/qt/editing/input/emacs-ctrl-o-expected.txt:
• platform/qt/editing/inserting/insert-before-link-1-expected.txt: Added.
• platform/qt/editing/pasteboard/display-block-on-spans-expected.txt: Added.
• platform/qt/editing/pasteboard/emacs-cntl-y-001-expected.png:
• platform/qt/editing/pasteboard/emacs-cntl-y-001-expected.txt:
• platform/qt/editing/pasteboard/merge-end-list-2-expected.txt: Added.
• platform/qt/editing/pasteboard/merge-end-table-2-expected.txt: Added.
• platform/qt/editing/pasteboard/page-zoom-expected.txt: Added.
• platform/qt/editing/selection/extend-selection-word-expected.txt: Added.
• platform/qt/editing/selection/selection-extend-should-not-move-across-caret-on-mac-expected.txt: Added.
• platform/qt/editing/style/apply-through-end-of-document-expected.txt: Added.
• platform/qt/editing/style/make-text-writing-direction-inline-win-expected.txt: Added.
• platform/qt/editing/style/remove-underline-across-paragraph-expected.txt: Added.
• platform/qt/editing/style/remove-underline-across-paragraph-in-bold-expected.txt: Added.
• platform/qt/editing/style/remove-underline-expected.txt: Added.
• platform/qt/editing/style/remove-underline-from-stylesheet-expected.txt: Added.
• platform/qt/editing/style/remove-underline-in-bold-expected.txt: Added.
• platform/qt/editing/style/unbold-in-bold-expected.png:
• platform/qt/editing/style/unbold-in-bold-expected.txt:
• platform/qt/editing/undo/replace-text-in-node-preserving-markers-crash-expected.txt: Added.

12:50 AM Changeset in webkit [149094] by rniwa@webkit.org

• 2 edits in trunk/Source/WebKit/efl

Unreviewed, rolling out r149032.
​http://trac.webkit.org/changeset/149032
​https://bugs.webkit.org/show_bug.cgi?id=115151

EFL WK1 API test has been broken after r149032 (Requested by
gyuyoung on #webkit).

• tests/test_ewk_setting.cpp:

(TEST_F):

Apr 24, 2013:

11:42 PM Changeset in webkit [149093] by tkent@chromium.org

• 5 edits in trunk

Back references don't work in pattern attribute
​https://bugs.webkit.org/show_bug.cgi?id=105875

Reviewed by Geoffrey Garen.

Source/WebCore:

This imports a part of Blink r148951.

Tests: Update fast/forms/ValidityState-patternMismatch.html

• html/BaseTextInputType.cpp:

(WebCore::BaseTextInputType::patternMismatch):
Don't use capturing parentheses. They affected back references in
pattern attribute values.

LayoutTests:

• fast/forms/ValidityState-patternMismatch-expected.txt:
• fast/forms/ValidityState-patternMismatch.html:

10:41 PM Changeset in webkit [149092] by commit-queue@webkit.org

• 2 edits in trunk/Tools

Remove checking chromium from run-webkit-tests
​https://bugs.webkit.org/show_bug.cgi?id=115139

Patch by Seokju Kwon <Seokju Kwon> on 2013-04-24
Reviewed by Benjamin Poulain.

• Scripts/run-webkit-tests:

(useNewRunWebKitTests):

10:30 PM Changeset in webkit [149091] by timothy_horton@apple.com

• 5 edits in trunk/Source

PDFPlugin: Cursor should become an i-beam when over selectable text
​https://bugs.webkit.org/show_bug.cgi?id=115018
<rdar://problem/12645012>

Reviewed by Alexey Proskuryakov.

• WebProcess/Plugins/PDF/PDFPlugin.h:

Add updateCursor(), the UpdateCursorMode and HitTestResult enums, and
storage for the last PDF hit test result.

• WebProcess/Plugins/PDF/PDFPlugin.mm:

(WebKit::PDFPlugin::updateCursor):
Hit test the PDF. If there is a word under the cursor, use the I-beam cursor,
otherwise, use the default pointer cursor. This approximates PDFKit's behavior,
though does not match it precisely in the case of annotations.
Only send the cursor update to the UIProcess if it is a change, or if
we're explicitly asked to update.
(WebKit::PDFPlugin::handleMouseEvent):
If we're processing a mouse moved event, update the cursor if needed.
(WebKit::PDFPlugin::handleMouseEnterEvent):
If the cursor just moved over the PDFPlugin, force-update the cursor,
even if it matches the previously-set cursor (because something else
likely updated the cursor while it was not over the PDFPlugin).

• WebCore.exp.in: Export the I-beam and arrow cursors.

9:56 PM Changeset in webkit [149090] by roger_fong@apple.com

• 1 edit
  559 adds
  297 deletes in trunk/LayoutTests

Roll out AppleWin re-baselines from r149088.

7:59 PM Changeset in webkit [149089] by mark.lam@apple.com

• 11 edits in trunk/Source/JavaScriptCore

Add watchdog timer polling for the DFG.
​https://bugs.webkit.org/show_bug.cgi?id=115134.

Reviewed by Geoffrey Garen.

The strategy is to add a speculation check to the DFG generated code to
test if the watchdog timer has fired or not. If the watchdog timer has
fired, the generated code will do an OSR exit to the baseline JIT, and
let it handle servicing the watchdog timer.

If the watchdog is not enabled, this speculation check will not be
emitted.

• API/tests/testapi.c:

(currentCPUTime_callAsFunction):
(extendTerminateCallback):
(main):

• removed try/catch statements so that we can test the watchdog on the DFG.
• added JS bindings to a native currentCPUTime() function so that the timeout tests can be more accurate.
• also shortened the time values so that the tests can complete sooner.

• bytecode/ExitKind.h:
• dfg/DFGAbstractState.cpp:

(JSC::DFG::AbstractState::executeEffects):

• dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::parseBlock):

• dfg/DFGFixupPhase.cpp:

(JSC::DFG::FixupPhase::fixupNode):

• dfg/DFGNodeType.h:
• dfg/DFGPredictionPropagationPhase.cpp:

(JSC::DFG::PredictionPropagationPhase::propagate):

• dfg/DFGSpeculativeJIT32_64.cpp:

(JSC::DFG::SpeculativeJIT::compile):

• dfg/DFGSpeculativeJIT64.cpp:

(JSC::DFG::SpeculativeJIT::compile):

• runtime/Watchdog.cpp:

(JSC::Watchdog::setTimeLimit):

7:44 PM Changeset in webkit [149088] by roger_fong@apple.com

• 103 edits
  2046 adds
  1760 deletes in trunk/LayoutTests

Unreviewed optimize-baselines for mac and win.

7:09 PM Changeset in webkit [149087] by Simon Fraser

• 17 edits in trunk/Tools

Pixel tests in SVG are all broken in WK2
​https://bugs.webkit.org/show_bug.cgi?id=114218

Reviewed by Darin Adler.

Window snapshots in WebKitTestRunner on Mac are window-sized
(800x600), which is wrong for the W3C SVG tests that require a
480x360 snapshot.

Fix by resizing the window, as well as the web view in
PlatformWebView::resizeTo() so that window snapshots are correctly
sized.

Also do some drive-by cleanup of the window size constants in both
DRT and WTR to remove magic numbers, and make the code more similar.

• DumpRenderTree/TestRunner.cpp:

(TestRunner::setShouldPaintBrokenImage):

• DumpRenderTree/TestRunner.h:
• DumpRenderTree/efl/DumpRenderTree.cpp:
• DumpRenderTree/gtk/DumpRenderTree.cpp:

(webInspectorShowWindow):

• DumpRenderTree/mac/DumpRenderTree.mm:

(createWebViewAndOffscreenWindow):
(sizeWebViewForCurrentTest):

• DumpRenderTree/mac/DumpRenderTreeMac.h:
• DumpRenderTree/mac/PixelDumpSupportMac.mm:

(createPagedBitmapContext):

• DumpRenderTree/qt/DumpRenderTreeQt.cpp:

(DumpRenderTree::DumpRenderTree):
(DumpRenderTree::open):

• DumpRenderTree/qt/TestRunnerQt.cpp:
• DumpRenderTree/qt/TestRunnerQt.h:
• DumpRenderTree/win/DumpRenderTree.cpp:

(sizeWebViewForCurrentTest):

• DumpRenderTree/wx/DumpRenderTreeWx.cpp:
• WebKitTestRunner/TestController.cpp:

(WTR::TestController::resetStateToConsistentValues):

• WebKitTestRunner/TestController.h:
• WebKitTestRunner/TestInvocation.cpp:

(WTR::sizeWebViewForCurrentTest):

• WebKitTestRunner/mac/PlatformWebViewMac.mm:

(WTR::PlatformWebView::PlatformWebView):
(WTR::PlatformWebView::resizeTo):

6:32 PM Changeset in webkit [149086] by timothy_horton@apple.com

• 2 edits in trunk/Source/WebKit2

Page Overlays shouldn't waste time flushing layers if we're not going to use it
​https://bugs.webkit.org/show_bug.cgi?id=115145

Reviewed by Simon Fraser.

Don't flush the page overlay layer when it's installed - it starts out
with paintContents=false, so we don't really need a CALayer yet. We'll
flush later if need be.

• WebProcess/WebPage/mac/TiledCoreAnimationDrawingArea.mm:

(WebKit::TiledCoreAnimationDrawingArea::didInstallPageOverlay):

6:32 PM Changeset in webkit [149085] by fpizlo@apple.com

• 5 edits
  1 add in branches/dfgFourthTier/Source/JavaScriptCore

fourthTier: AbstractValue methods that deal with watchpoints should have access to Graph, so that in debug mode, Graph can track the history of watchpoint states and detect races
​https://bugs.webkit.org/show_bug.cgi?id=115084

Reviewed by Geoffrey Garen.

The idea is that as part of ​https://bugs.webkit.org/show_bug.cgi?id=115083, I'll have
Graph record the initial state of a watchpoint at the time that we decide to take
advantage of it; then I will use this to disable any watchpoint-related assertions
in debug mode. Note that this "watchpoint cache" will only be maintained in debug
mode, so there will be no release performance implications. But to do this, I need to
ensure that all of the places that reason about watchpoints have access to Graph.
For example, I'll want AbstractValue::setFuturePossibleStructure to record the state
of the watchpoint in Graph so that subsequent assertions can check if the watchpoint's
state had changed since that decision was made.

• JavaScriptCore.xcodeproj/project.pbxproj:
• dfg/DFGAbstractState.cpp:

(JSC::DFG::AbstractState::initialize):
(JSC::DFG::AbstractState::executeEffects):
(JSC::DFG::AbstractState::mergeStateAtTail):

• dfg/DFGAbstractState.h:

(JSC::DFG::AbstractState::trySetConstant):

• dfg/DFGAbstractValue.cpp: Added.

(DFG):
(JSC::DFG::AbstractValue::setMostSpecific):
(JSC::DFG::AbstractValue::set):
(JSC::DFG::AbstractValue::filter):
(JSC::DFG::AbstractValue::setFuturePossibleStructure):
(JSC::DFG::AbstractValue::filterFuturePossibleStructure):
(JSC::DFG::AbstractValue::dump):

• dfg/DFGAbstractValue.h:

(DFG):
(AbstractValue):
(JSC::DFG::AbstractValue::setType):
(JSC::DFG::AbstractValue::filterByValue):

6:27 PM Changeset in webkit [149084] by Simon Fraser

• 3 edits
  2 adds in trunk

Garbage at the top of ​http://www.technologyreview.com after scrolling
​https://bugs.webkit.org/show_bug.cgi?id=114825

Source/WebCore:

Reviewed by Tim Horton.

Garbage pixels are caused by GraphicsLayerCA setting a layer to be opaque,
but then not painting anything into it. On this page, the element is
toggled to be visibility:hidden on scrolling, but RenderLayer::backgroundIsKnownToBeOpaqueInRect()
failed to consider that as something that can cause backgrounds not to be opaque.

For the bug to happen, some subtle interactions with r142012 come into play
for the layer to remain visible, hence the slightly complex testcase.

Test: compositing/contents-opaque/visibility-hidden.html

• rendering/RenderLayer.cpp:

(WebCore::RenderLayer::backgroundIsKnownToBeOpaqueInRect):

LayoutTests:

Reviewed by Tim Horton.

Test that sets visibility:hidden on an element with some complex
configuration of layer children, and dumps the layer tree to check
that the contents are not marked as opaque.

• compositing/contents-opaque/visibility-hidden-expected.txt: Added.
• compositing/contents-opaque/visibility-hidden.html: Added.

6:27 PM Changeset in webkit [149083] by Simon Fraser

• 2 edits in trunk/Source/WebCore

Dump layer opaqueness in the Compositing log output
​https://bugs.webkit.org/show_bug.cgi?id=115132

Reviewed by Tim Horton.

It's useful to see whether we consider layers to be opaque in the Compositing
log channel output, so dump it.

• rendering/RenderLayerCompositor.cpp:

(WebCore::RenderLayerCompositor::logLayerInfo):

6:26 PM Changeset in webkit [149082] by fpizlo@apple.com

• 5 edits in trunk/Source/JavaScriptCore

Special thunks for math functions should work on ARMv7
​https://bugs.webkit.org/show_bug.cgi?id=115144

Reviewed by Gavin Barraclough and Oliver Hunt.

The only hard bit here was ensuring that we implemented the very special
"cheap C call" convention on ARMv7.

• assembler/AbstractMacroAssembler.h:

(JSC::isARMv7s):
(JSC):
(JSC::isX86):

• dfg/DFGCommon.h:
• jit/SpecializedThunkJIT.h:

(SpecializedThunkJIT):
(JSC::SpecializedThunkJIT::callDoubleToDoublePreservingReturn):

• jit/ThunkGenerators.cpp:

(JSC::floorThunkGenerator):
(JSC::ceilThunkGenerator):
(JSC::roundThunkGenerator):
(JSC::expThunkGenerator):
(JSC::logThunkGenerator):

5:51 PM Changeset in webkit [149081] by commit-queue@webkit.org

• 2 edits in trunk/Source/WebCore

Add efl to JSInspectorFrontendHost::port()
​https://bugs.webkit.org/show_bug.cgi?id=115137

Patch by Seokju Kwon <Seokju Kwon> on 2013-04-24
Reviewed by Timothy Hatcher.

No new tests, because there is no change in behavior.

• bindings/js/JSInspectorFrontendHostCustom.cpp:

(WebCore::JSInspectorFrontendHost::port):

5:44 PM Changeset in webkit [149080] by kangil.han@samsung.com

• 3 edits in trunk/Source/WebCore

Function parameter quotePair can be passed by reference
​https://bugs.webkit.org/show_bug.cgi?id=115089

Reviewed by Alexey Proskuryakov.

Minor performance patch.

• rendering/style/QuotesData.cpp:

(WebCore::QuotesData::addPair):

• rendering/style/QuotesData.h:

(QuotesData):

5:17 PM Changeset in webkit [149079] by beidson@apple.com

• 10 edits in trunk/Source/WebCore

Implementors of CachedResource subclasses should be forced to decide if encoded data can be replaced.
​https://bugs.webkit.org/show_bug.cgi?id=115140

Reviewed by Beth Dakin.

No new tests (No behavior change).

This makes mayTryReplaceEncodedData() return false in CachedResource, but overrides to true
in all CachedResource subclasses besides CachedFont (which already has an implementation).

• loader/cache/CachedCSSStyleSheet.h:
• loader/cache/CachedImage.h:
• loader/cache/CachedRawResource.h:
• loader/cache/CachedResource.h:
• loader/cache/CachedSVGDocument.h:
• loader/cache/CachedScript.h:
• loader/cache/CachedShader.h:
• loader/cache/CachedTextTrack.h:
• loader/cache/CachedXSLStyleSheet.h:

5:10 PM Changeset in webkit [149078] by commit-queue@webkit.org

• 2 edits in trunk/Source/JavaScriptCore

Misc bugfix and cleaning in sh4 base JIT.
​https://bugs.webkit.org/show_bug.cgi?id=115022.

Patch by Julien Brianceau <​jbrianceau@nds.com> on 2013-04-24
Reviewed by Oliver Hunt.

Remove unused add32() and sub32() with scratchreg parameter to avoid
confusion as this function prototype means another behaviour.
Remove unused "void push(Address)" function which seems quite buggy.

• assembler/MacroAssemblerSH4.h:

(JSC::MacroAssemblerSH4::and32): Cosmetic change.
(JSC::MacroAssemblerSH4::lshift32): Cosmetic change.
(JSC::MacroAssemblerSH4::or32): Cosmetic change.
(JSC::MacroAssemblerSH4::xor32): Cosmetic change.
(MacroAssemblerSH4):
(JSC::MacroAssemblerSH4::load32): Cosmetic change.
(JSC::MacroAssemblerSH4::load8Signed): Fix invalid offset upper limit
when using r0 register and cosmetic changes.
(JSC::MacroAssemblerSH4::load8): Reuse load8Signed to avoid duplication.
(JSC::MacroAssemblerSH4::load16): Fix invalid offset upper limit when
using r0 register, fix missing offset shift and cosmetic changes.
(JSC::MacroAssemblerSH4::store32): Cosmetic change.
(JSC::MacroAssemblerSH4::branchAdd32): Store result value before branch.

4:46 PM Changeset in webkit [149077] by jer.noble@apple.com

• 2 edits in trunk/Source/WebKit2

Further unreviewed build fix; CFNetwork.framework is the one which Lion
refuses to link against.

• WebKit2.xcodeproj/project.pbxproj:

4:32 PM Changeset in webkit [149076] by jer.noble@apple.com

• 2 edits in trunk/Source/WebKit2

Unreviewed build fix; link against CoreServices.framework instead of against
CoreFoundation.framework directly, to make the linker happy on Lion.

• WebKit2.xcodeproj/project.pbxproj:

4:31 PM Changeset in webkit [149075] by ddkilzer@apple.com

• 2 edits in trunk/LayoutTests

Fix invalid test names in jsc-test-list
<​http://webkit.org/b/115136>

Reviewed by Geoffrey Garen.

This shell script finds all the invalid tests listed in
LayoutTests/fast/js/jsc-test-list:

$ for F in cat LayoutTests/fast/js/jsc-test-list; do if [ ! -f LayoutTests/$F.html ]; then echo $F; fi; done
fast/js/dfg-double-addition-simplify-to-int.html
fast/js/dfg-mul-big-integers-with-small-integer-and-bitor
fast/js/dfg-mul-big-integers-with-small-integer-and-detect-overflow
fast/js/dfg-mul-big-integers-with-small-integer
fast/js/dfg-value-to-int32-with-side-effects

• fast/js/jsc-test-list: Fix invalid test names by removing

*.html suffix or stray "s" characters.

4:20 PM Changeset in webkit [149074] by jer.noble@apple.com

• 9 edits
  3 copies
  2 adds in trunk/Source/WebKit2

WWDC session videos don’t play at developer.apple.com
​https://bugs.webkit.org/show_bug.cgi?id=114858

Reviewed by Eric Carlson. Rubber stamped by Brady Eidson.

AVFoundation uses CFNetwork to store and retrieve cookies from the global store.
However, in the case where network access happens in the NetworkProcess, session
cookies are stored in-memory, and are not accessable in the WebProcess. Until such
a time as AVFoundation can provide an API which would allow us to provide cookies
for a specific request, we will interpose the CFNetwork method which they use to
retrieve the cookie string from the cookie store for their pending request.

Duplicate the previous SecItemShim target to a new, WebProcessShim target. This
target includes the SecItemShim functionality, but will add a new shim for cookie
retrieval:

• Shared/mac/CookieStorageShimLibrary.h: Added

(CookieStorageShimCallbacks):

• Shared/mac/CookieStorageShimLibrary.cpp: Added.

(WebKit::ShimProtector::ShimProtector): A simple stack-based counter class.
(WebKit::ShimProtector::~ShimProtector):
(WebKit::ShimProtector::count):
(WebKit::shimCFHTTPCookieStorageCopyRequestHeaderFieldsForURL): Interpose

the CFNetwork call, and pass to the registered callback.

(WebKit::WebKitCookieStorageShimInitialize): Register the callbacks.

Add a helper singleton class which will talk to the shim through the
registered callbacks:

• Shared/mac/CookieStorageShim.h: Added

(WebKit::CookieStorageShim::CookieStorageShim):

• Shared/mac/CookieStorageShim.cpp: Added.

(WebKit::webKitCookieStorageCopyRequestHeaderFieldsForURL): Pass the request

over to the NetworkProcess.

(WebKit::CookieStorageShim::shared): Simple singleton.
(WebKit::CookieStorageShim::initialize): Call the library initializer.

Initialize the shim only when the WebProcess is delegating network loading
to the NetworkProcess:

• WebProcess/WebProcess.cpp:

(WebKit::WebProcess::initializeWebProcess):

Rename the WebProcess's shim from SecItemShim -> WebProcessShim:

• UIProcess/Launcher/mac/ProcessLauncherMac.mm:

(WebKit::addDYLDEnvironmentAdditions):

• WebProcess/EntryPoint/mac/LegacyProcess/WebContentProcessMain.mm:

(WebKit::WebContentProcessMainDelegate::doPreInitializationWork):

• WebProcess/EntryPoint/mac/XPCService/WebContentService/Info.plist:
• WebProcess/EntryPoint/mac/XPCService/WebContentServiceEntryPoint.mm:

(WebContentServiceInitializer):

As the DYLD_INTERPOSE macro is used in multiple files now, put it in its
own header:

• Shared/mac/DyldInterpose.h: Added.
• PluginProcess/mac/PluginProcessShim.mm:
• WebProcess/mac/SecItemShimLibrary.mm:

Add new files to the project:

• WebKit2.xcodeproj/project.pbxproj:

3:50 PM Changeset in webkit [149073] by bw80.lee@samsung.com

• 4 edits in trunk

[EFL] Fix build error after r148963.
​https://bugs.webkit.org/show_bug.cgi?id=115126

Reviewed by Laszlo Gombos.

Generate forwarding headers for 'CoordinatedGraphics' to generate WKView.h

Source/WebKit2:

• PlatformEfl.cmake:

Tools:

• TestWebKitAPI/PlatformEfl.cmake:

3:42 PM Changeset in webkit [149072] by Patrick Gansterer

• 13 edits in trunk/Source/JavaScriptCore

[WIN] Remove pthread from Visual Studio files in JavaScriptCore
​https://bugs.webkit.org/show_bug.cgi?id=114864

Reviewed by Brent Fulgham.

• JavaScriptCore.vcproj/JavaScriptCore/JavaScriptCoreCommon.vsprops:
• JavaScriptCore.vcproj/LLIntOffsetsExtractor/LLIntOffsetsExtractorCommon.vsprops:
• JavaScriptCore.vcproj/jsc/jscCommon.vsprops:
• JavaScriptCore.vcproj/testRegExp/testRegExpCommon.vsprops:
• JavaScriptCore.vcproj/testapi/testapiCommon.vsprops:
• JavaScriptCore.vcxproj/JavaScriptCoreCommon.props:
• JavaScriptCore.vcxproj/JavaScriptCoreExportGenerator/JavaScriptCoreExportGeneratorCommon.props:
• JavaScriptCore.vcxproj/LLInt/LLIntOffsetsExtractor/LLIntOffsetsExtractorCommon.props:
• JavaScriptCore.vcxproj/jsc/jscCommon.props:
• JavaScriptCore.vcxproj/testRegExp/testRegExpCommon.props:
• JavaScriptCore.vcxproj/testapi/testapiCommon.props:
• JavaScriptCore.vcxproj/testapi/testapiCommonCFLite.props:

3:39 PM Changeset in webkit [149071] by fpizlo@apple.com

• 4 edits
  3 adds in trunk

DFG should keep the operand to create_this alive if it's emitting code for create_this
​https://bugs.webkit.org/show_bug.cgi?id=115133

Source/JavaScriptCore:

Reviewed by Mark Hahnenberg.

The DFG must model bytecode liveness, or else OSR exit is going to have a really bad time.

• dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::parseBlock):

LayoutTests:

Reviewed by Mark Hahnenberg.

This test crashes prior to my change.

• fast/js/dfg-allocation-profile-watch-point-exit-expected.txt: Added.
• fast/js/dfg-allocation-profile-watch-point-exit.html: Added.
• fast/js/jsc-test-list:
• fast/js/script-tests/dfg-allocation-profile-watch-point-exit.js: Added.

(Foo):
(foo):

3:35 PM Changeset in webkit [149070] by beidson@apple.com

• 5 edits in trunk/Source/WebCore

Once a custom font is cached to disk, it starts failing to render until the page is refreshed.
<rdar://problem/13622998> and ​https://bugs.webkit.org/show_bug.cgi?id=115131

Reviewed by Alexey Proskuryakov.

No new tests (Not a tested config, nor are disk cache issues currently testable).

• loader/cache/CachedResource.h:

(WebCore::CachedResource:: mayTryReplaceEncodedData): Allow subclasses to refuse encoded data replacement.

• loader/cache/CachedResource.cpp:

(WebCore::CachedResource::tryReplaceEncodedData): Only try if shouldTryReplaceEncodedData() is true.

• loader/cache/CachedFont.cpp:

(WebCore::CachedFont::CachedFont):
(WebCore::CachedFont::ensureCustomFontData):
(WebCore::CachedFont::mayTryReplaceEncodedData): Return false if the custom font data has ever been created.

• loader/cache/CachedFont.h:

3:22 PM Changeset in webkit [149069] by jpfau@apple.com

• 2 edits in branches/safari-536.30-branch/LayoutTests

<rdar://problem/12952167> 11G56: “AVF Info: …” logged to stdout

• platform/mac-lion/Skipped: Skipped all tests that were failing due to AVF info being printed.

3:05 PM Changeset in webkit [149068] by ap@apple.com

• 5 edits in trunk/LayoutTests

fast/js/global-constructors.html was deleted in r149001, removing it from
TextExpectations too.

• platform/mac/TestExpectations:
• platform/qt-arm/TestExpectations:
• platform/win/TestExpectations:
• platform/wincairo/TestExpectations:

2:43 PM Changeset in webkit [149067] by roger_fong@apple.com

• 13 edits in trunk

Have VS2010 WebKit solution look in WebKit_Libraries/lib32 for dependencies.

• JavaScriptCore.vcxproj/JavaScriptCoreExportGenerator/JavaScriptCoreExportGeneratorPostBuild.cmd:
• JavaScriptCore.vcxproj/JavaScriptCorePreLink.cmd:
• JavaScriptCore.vcxproj/jsc/jscPostBuild.cmd:
• JavaScriptCore.vcxproj/jsc/jscPreLink.cmd:
• JavaScriptCore.vcxproj/testRegExp/testRegExp.vcxproj.filters:
• JavaScriptCore.vcxproj/testRegExp/testRegExpPostBuild.cmd:
• JavaScriptCore.vcxproj/testRegExp/testRegExpPreLink.cmd:
• JavaScriptCore.vcxproj/testapi/testapiPreLink.cmd:
• WTF.vcxproj/build-generated-files.sh:
• win/tools/vsprops/common.props:

2:37 PM Changeset in webkit [149066] by Beth Dakin

• 12 edits in trunk/Source/WebCore

Vertical overlay scrollbar in iframes fades in and out rapidly when you scroll in
a circle
​https://bugs.webkit.org/show_bug.cgi?id=115124
-and corresponding-
<rdar://problem/13168957>

Reviewed by Anders Carlsson.

With ​http://trac.webkit.org/changeset/119834 we started calling
ScrollbarPainterController's contentAreaScrolled/contentAreaScrolledInDirection
API on a zero-delay timer instead of calling it right away. This prevented some
crashes that we saw whenever this was called during a layout. However, that delay,
combined with the particulars of contentAreaScrolledInDirection cause this bug
where sometimes the scrollbars in an iframe will fade out very noticeably when
scrolling in a circle.

This change makes it so we will only use the zero-delay timer if the
ScrollableArea is not currently handling a wheel event. If it IS handling a wheel
event, then we will send the notifications to AppKit right away. I confirmed that
this change did not reintroduce the old crashes.

Keep track of whether we are currently handling a wheel event with the new member
variable m_isHandlingWheelEvent.

• page/EventHandler.cpp:

(WebCore::EventHandler::EventHandler):
(WebCore::EventHandler::handleWheelEvent):

• page/EventHandler.h:

(WebCore::EventHandler::isHandlingWheelEvent):
(EventHandler):

To prevent layering violations, the ScrollableArea sub-classes will have to access
this information from the EventHandler.

• page/FrameView.cpp:

(WebCore::FrameView::isHandlingWheelEvent):

• page/FrameView.h:
• platform/ScrollableArea.h:

(WebCore::ScrollableArea::isHandlingWheelEvent):

• rendering/RenderLayer.cpp:

(WebCore::RenderLayer::isHandlingWheelEvent):

• rendering/RenderLayer.h:

(RenderLayer):

• rendering/RenderListBox.cpp:

(WebCore::RenderListBox::isHandlingWheelEvent):

• rendering/RenderListBox.h:

sendContentAreaScrolledSoon() can be private. Also add new function
sendContentAreaScrolled().

• platform/mac/ScrollAnimatorMac.h:

(ScrollAnimatorMac):

If the ScrollableArea is handling a wheel event, call
sendContentAreaScrolled(), otherwise call sendContentAreaScrolledSoon()

• platform/mac/ScrollAnimatorMac.mm:

(WebCore::ScrollAnimatorMac::notifyContentAreaScrolled):
(WebCore::ScrollAnimatorMac::sendContentAreaScrolled):

Re-factored to use sendContentAreaScrolled()
(WebCore::ScrollAnimatorMac::sendContentAreaScrolledTimerFired):

2:35 PM Changeset in webkit [149065] by rniwa@webkit.org

• 2 edits in trunk/Source/WebCore

Fix a merge error in r149007 (was missing a null check added in r148777).

• rendering/LogicalSelectionOffsetCaches.h:

(WebCore::containingBlockForAbsolutePosition):

2:33 PM Changeset in webkit [149064] by andersca@apple.com

• 2 edits in trunk/Source/WebKit2

Crash when trying to determine the length of an invalid message
​https://bugs.webkit.org/show_bug.cgi?id=115127
<rdar://problem/13700736>

Reviewed by Beth Dakin.

Use message instead of incomingMessage since incomingMessage will always be null.

• Platform/CoreIPC/Connection.cpp:

(CoreIPC::Connection::processIncomingMessage):

1:38 PM Changeset in webkit [149063] by commit-queue@webkit.org

• 4 edits in trunk/Source/WebCore

Battery API won't return updated battery status until client calls didChangeBatteryStatus
​https://bugs.webkit.org/show_bug.cgi?id=114891

Patch by Otto Derek Cheung <​otcheung@rim.com> on 2013-04-24
Reviewed by Rob Buis.

The BatteryStatus object in BatteryManager won't get updated until BatteryClient calls
didChangeBatteryStatus in BatteryController. Any attempts to call webkitBattery.charged()
or other get functions will always return the default values until the next battery status
change object gets passed to the controller.

We need to update the manager when we attach it to the controller, and update all
existing managers when the controller receives it's first battery status object.

• Modules/battery/BatteryController.cpp:

(WebCore::BatteryController::addListener):
(WebCore::BatteryController::updateBatteryStatus):

• Modules/battery/BatteryManager.cpp:

(WebCore::BatteryManager::didChangeBatteryStatus):
(WebCore::BatteryManager::updateBatteryStatus):
(WebCore):

• Modules/battery/BatteryManager.h:

(BatteryManager):

1:11 PM Changeset in webkit [149062] by zhajiang@rim.com

• 2 edits in trunk/Source/WebCore

Clean up unused code in ViewportArguments.h
​https://bugs.webkit.org/show_bug.cgi?id=115121

Patch by Jacky Jiang <​zhajiang@blackberry.com> on 2013-04-24.
Reviewed by Benjamin Poulain.

The target-densityDPI emulation implementation has been removed by
r147893, so clean up the remaining code as well.

• dom/ViewportArguments.h:

1:03 PM Changeset in webkit [149061] by akling@apple.com

• 5 edits in trunk/Source/WebCore

ElementData should use 'unsigned' attribute indices.
<​http://webkit.org/b/115103>

Reviewed by Antti Koivisto.

Switch to using 'unsigned' for attribute indices. This gives consistent behavior
on 32/64-bit, and the underlying storage is already limited by Vector's 32-bit capacity.

Added an ElementData::attributeNotFound constant (-1) since we can't use WTF::notFound.

• dom/Element.cpp:

(WebCore::Element::detachAttribute):
(WebCore::Element::removeAttribute):
(WebCore::Element::setAttribute):
(WebCore::Element::setSynchronizedLazyAttribute):
(WebCore::Element::setAttributeInternal):
(WebCore::Element::setAttributeNode):
(WebCore::Element::removeAttributeNode):
(WebCore::Element::removeAttributeInternal):
(WebCore::UniqueElementData::removeAttribute):
(WebCore::ElementData::getAttributeItemIndexSlowCase):

• dom/Element.h:

(ElementData):
(UniqueElementData):
(Element):
(WebCore::Element::getAttributeItemIndex):
(WebCore::Element::attributeCount):
(WebCore::ElementData::length):
(WebCore::ElementData::getAttributeItem):
(WebCore::ElementData::getAttributeItemIndex):

• dom/NamedNodeMap.cpp:

(WebCore::NamedNodeMap::removeNamedItem):
(WebCore::NamedNodeMap::removeNamedItemNS):
(WebCore::NamedNodeMap::length):

• dom/NamedNodeMap.h:

(NamedNodeMap):

1:00 PM Changeset in webkit [149060] by akling@apple.com

• 2 edits in trunk/Source/WebCore

ElementData::attributeItem() should bounds-check the index.
<​http://webkit.org/b/115076>

Reviewed by Antti Koivisto.

Switch the ASSERT_WITH_SECURITY_IMPLICATION to a RELEASE_ASSERT.

• dom/Element.h:

(WebCore::ElementData::attributeItem):

12:59 PM Changeset in webkit [149059] by akling@apple.com

• 2 edits in trunk/Source/WebCore

Setting an inline style property to "" shouldn't cause style recalc unless the property was present.
<​http://webkit.org/b/115122>

Reviewed by Antti Koivisto.

StylePropertySet::setProperty() handles empty strings by removing the property entirely.
If there was no property to remove, it shouldn't claim that it changed something, as that
will cause the CSSOM wrapper to mark the element for style recalc.

A common idiom that triggers this is excessive use of { element.style.display = ; }

• css/StylePropertySet.cpp:

(WebCore::MutableStylePropertySet::setProperty):

12:50 PM Changeset in webkit [149058] by rniwa@webkit.org

• 3 edits
  2 adds in trunk

Moving word boundaries backwards fails when there is a text node starting with an apostrophe
​https://bugs.webkit.org/show_bug.cgi?id=115070

Reviewed by Alexey Proskuryakov.

Source/WebCore:

The bug was caused by previousBoundary erroneously assuming that we don't need any more context if a word
boundary is found at the beginning of a string. For example, when "I'll" is split into two text nodes,
"I" and "'ll", there is a word boundary between "'" and "ll" in "'ll" so we need to examine the whole "I'll".

Fixed the bug by obtaining more context when the character starts exactly at offset 1 in a text node to
work around this bug. In the long term, we probably need to provide Foundation of the entire context since in
languages like Hebrew and some of European languages, there could be many accents and combining characters
between split into multiple text nodes as one variant is seen in the newly added test case.

Test: editing/selection/previous-word-boundary-across-text-nodes.html

• editing/VisibleUnits.cpp:

(WebCore::previousBoundary):

LayoutTests:

Added a test case for moving bacwkards with a word granurality across multiple text nodes.
Some test cases still fail since this patch only implements a work around.

• editing/selection/previous-word-boundary-across-text-nodes-expected.txt: Added.
• editing/selection/previous-word-boundary-across-text-nodes.html: Added.

12:46 PM Changeset in webkit [149057] by benjamin@webkit.org

• 3 edits in trunk/Source/WebCore

Do not use static string in DiagnosticLoggingKeys
​https://bugs.webkit.org/show_bug.cgi?id=115093

Patch by Benjamin Poulain <​bpoulain@apple.com> on 2013-04-24
Reviewed by Andreas Kling.

The code is not hot enough to justify keeping the memory around.
This removes 3kb from the binary on x86_64.

• page/DiagnosticLoggingKeys.cpp:

(WebCore::DiagnosticLoggingKeys::mediaLoadedKey):
(WebCore::DiagnosticLoggingKeys::mediaLoadingFailedKey):
(WebCore::DiagnosticLoggingKeys::pluginLoadedKey):
(WebCore::DiagnosticLoggingKeys::pluginLoadingFailedKey):
(WebCore::DiagnosticLoggingKeys::pageContainsPluginKey):
(WebCore::DiagnosticLoggingKeys::pageContainsAtLeastOnePluginKey):
(WebCore::DiagnosticLoggingKeys::pageContainsMediaEngineKey):
(WebCore::DiagnosticLoggingKeys::pageContainsAtLeastOneMediaEngineKey):
(WebCore::DiagnosticLoggingKeys::passKey):
(WebCore::DiagnosticLoggingKeys::failKey):
(WebCore::DiagnosticLoggingKeys::noopKey):

• page/DiagnosticLoggingKeys.h:

(DiagnosticLoggingKeys):

12:38 PM Changeset in webkit [149056] by benjamin@webkit.org

• 22 edits in trunk/Source/WebCore

Remove wxWebKit #ifdefs from WebCore/platform/graphics
​https://bugs.webkit.org/show_bug.cgi?id=115081

Reviewed by Geoffrey Garen.

• platform/graphics/BitmapImage.h:

(BitmapImage):

• platform/graphics/Color.h:

(Color):

• platform/graphics/FloatRect.h:

(FloatRect):

• platform/graphics/FloatSize.h:

(FloatSize):

• platform/graphics/FontPlatformData.h:
• platform/graphics/GlyphBuffer.h:

(WebCore):
(WebCore::GlyphBuffer::glyphAt):
(WebCore::GlyphBuffer::add):

• platform/graphics/Gradient.h:
• platform/graphics/GraphicsContext.h:

(GraphicsContext):

• platform/graphics/Image.cpp:

(WebCore::Image::drawTiled):

• platform/graphics/ImageBufferData.h:
• platform/graphics/IntPoint.h:

(IntPoint):

• platform/graphics/IntRect.h:
• platform/graphics/IntSize.h:

(IntSize):

• platform/graphics/NativeImagePtr.h:

(WebCore):

• platform/graphics/Path.h:
• platform/graphics/Pattern.h:
• platform/graphics/SimpleFontData.h:

(SimpleFontData):
(WebCore::SimpleFontData::widthForGlyph):

• platform/graphics/mac/ComplexTextController.cpp:

(WebCore::ComplexTextController::collectComplexTextRuns):

• platform/graphics/mac/ComplexTextControllerCoreText.mm:

(WebCore::ComplexTextController::collectComplexTextRunsForCharacters):

• platform/graphics/transforms/AffineTransform.h:

(AffineTransform):

• platform/graphics/transforms/TransformationMatrix.h:

(TransformationMatrix):

12:36 PM Changeset in webkit [149055] by benjamin@webkit.org

• 4 edits in trunk/Source/WebCore

Remove wxWebKit #ifdefs from WebCore/plugins
​https://bugs.webkit.org/show_bug.cgi?id=115080

Reviewed by Geoffrey Garen.

• plugins/PluginView.h:

(PluginView):

• plugins/mac/PluginViewMac.mm:

(WebCore::nativeWindowFor):
(WebCore::cgHandleFor):
(WebCore::topLevelOffsetFor):
(WebCore::PluginView::platformStart):

• plugins/win/PluginViewWin.cpp:

(windowHandleForPageClient):
(WebCore::PluginView::handleMouseEvent):
(WebCore::PluginView::platformStart):
(WebCore::PluginView::snapshot):

12:35 PM Changeset in webkit [149054] by benjamin@webkit.org

• 4 edits in trunk/Source/WebCore

Remove wxWebKit #ifdefs from WebCore/page
​https://bugs.webkit.org/show_bug.cgi?id=115079

Reviewed by Geoffrey Garen.

• page/FrameView.cpp:

(WebCore::FrameView::wheelEvent):

• page/Settings.cpp:

(WebCore):

• page/Settings.h:

(Settings):

12:34 PM Changeset in webkit [149053] by benjamin@webkit.org

• 2 edits in trunk/Source/WTF

Use post-branch store for RefCountedBase::derefBase
​https://bugs.webkit.org/show_bug.cgi?id=115078

Reviewed by Andreas Kling.

• wtf/RefCounted.h:

(WTF::RefCountedBase::derefBase): This makes the assembly easier to follow.

11:34 AM Changeset in webkit [149052] by Carlos Garcia Campos

• 5 edits in trunk

[GTK] Use stamp files for generate-forwarding-headers makefile rules
​https://bugs.webkit.org/show_bug.cgi?id=115118

Reviewed by Martin Robinson.

Source/WebKit2:

Use a stamp file for the WebKit2 forwarding generator rule. Also
fix the rule dependencies that pointed to a non existent variable.

• GNUmakefile.am:

Tools:

Use a stamp file for the TestWebKitAPI and WebKitTestRunner
forwarding generator rules and don't call it for soup since there
aren't soup directories in TestWebKitAPI and WebKitTestRunner.

• TestWebKitAPI/GNUmakefile.am:
• WebKitTestRunner/GNUmakefile.am:

11:33 AM Changeset in webkit [149051] by Chris Fleizach

• 12 edits
  2 adds in trunk

AX: WAI-ARIA landmarks no longer speak type of landmark on iOS
​https://bugs.webkit.org/show_bug.cgi?id=114547

Reviewed by David Kilzer.

Source/WebCore:

We want iOS to speak the landmark type.
I've moved the landmark role description to the base Mac class and exposed
these localized strings to iOS (the ones used to return landmark descriptions).

I've also updated the accessibilityLabel to account for landmark type, but also
handle multiple label sources more gracefully by appending commas, which changed a few
tests in minor ways.

Test: platform/iphone-simulator/accessibility/landmark-type.html

• accessibility/ios/WebAccessibilityObjectWrapperIOS.mm:

(-[WebAccessibilityObjectWrapper accessibilityLabel]):

• accessibility/mac/WebAccessibilityObjectWrapperBase.h:
• accessibility/mac/WebAccessibilityObjectWrapperBase.mm:

(-[WebAccessibilityObjectWrapperBase ariaLandmarkRoleDescription]):

• accessibility/mac/WebAccessibilityObjectWrapperMac.mm:

(-[WebAccessibilityObjectWrapper roleDescription]):

• platform/LocalizedStrings.cpp:

(WebCore):
(WebCore::AXButtonActionVerb):
(WebCore::AXRadioButtonActionVerb):
(WebCore::AXTextFieldActionVerb):
(WebCore::AXCheckedCheckBoxActionVerb):
(WebCore::AXUncheckedCheckBoxActionVerb):
(WebCore::AXLinkActionVerb):
(WebCore::AXMenuListPopupActionVerb):
(WebCore::AXMenuListActionVerb):

• platform/LocalizedStrings.h:

(WebCore):

LayoutTests:

• platform/iphone-simulator/accessibility/landmark-type-expected.txt: Added.
• platform/iphone-simulator/accessibility/landmark-type.html: Added.
• platform/iphone-simulator/accessibility/link-with-images-text-expected.txt:
• platform/iphone-simulator/accessibility/link-with-images-text.html:

Minor change to accomodate trimming of white space.

• platform/iphone-simulator/accessibility/math-expected.txt:
• platform/iphone-simulator/accessibility/math.html:

Math objects now include "math" as the landmark type.

11:16 AM Changeset in webkit [149050] by ggaren@apple.com

• 2 edits in trunk/Source/JavaScriptCore

32-bit build fix.

Unreviewed.

• dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compilePeepHoleBooleanBranch): Explicitly
truncate to 32-bit to avoid compiler warnings. It's safe to truncate
because the payload of a boolean is the low bits on both 64-bit and 32-bit.

11:06 AM Changeset in webkit [149049] by zandobersek@gmail.com

• 2 edits in trunk/Source/WebKit2

[GTK][WK2] Implement LayerTreeHostGtk::pageBackgroundTransparencyChanged
​https://bugs.webkit.org/show_bug.cgi?id=115094

Reviewed by Tim Horton.

The method was introduced in r149004 (and a stub implementation for the GTK port added in r149017).
Implement the method for the GTK port in the same manner it is implemented under the Mac port,
calling setContentsOpaque on the non-composited content layer.

• WebProcess/WebPage/gtk/LayerTreeHostGtk.cpp:

(WebKit::LayerTreeHostGtk::pageBackgroundTransparencyChanged):

10:50 AM Changeset in webkit [149048] by commit-queue@webkit.org

• 2 edits in trunk/Tools

[GTK] Update build dependencies
​https://bugs.webkit.org/show_bug.cgi?id=115111

Patch by Simon Pena <​simon.pena@samsung.com> on 2013-04-24
Reviewed by Martin Robinson.

The install-dependencies script was missing a dependency on EGL,
which is expected when building Cairo.

• gtk/install-dependencies: Added a dependency on libegl1-mesa

for both apt and yum.

10:43 AM Changeset in webkit [149047] by akling@apple.com

• 2 edits in trunk/Source/WebCore

CSS attribute selectors cause unnecessary style recalc when setting attribute to same value.
<​http://webkit.org/b/115116>
<rdar://problem/13727709>

Reviewed by Simon Fraser.

The logic that dirties the style if there's a relevant attribute selector in the document
shouldn't run if the attribute is being overwritten with an identical value.
Move this into willModifyAttribute() instead, since we need access to both the old and the new value.

This reduces unnecessary style recalculation in Mac App Store content.

• dom/Element.cpp:

(WebCore::Element::attributeChanged):
(WebCore::Element::willModifyAttribute):

10:07 AM Changeset in webkit [149046] by commit-queue@webkit.org

• 2 edits in trunk/Tools

Remove the Chromium-specific helper from run_webkit_tests.py
​https://bugs.webkit.org/show_bug.cgi?id=114996

Patch by Seokju Kwon <Seokju Kwon> on 2013-04-24
Reviewed by Dirk Pranke.

• Scripts/webkitpy/layout_tests/run_webkit_tests.py:

(parse_args):

9:12 AM Changeset in webkit [149045] by commit-queue@webkit.org

• 3 edits
  1 add in trunk/Tools

[EFL][WK2]: Fix WKViewClientWebProcessCallbacks WK2 API test
​https://bugs.webkit.org/show_bug.cgi?id=114850

Patch by Sergio Correia <Sergio Correia> on 2013-04-24
Reviewed by Andreas Kling.

Revision r148312 fixed WebPageProxy cleanup and also changed the process
termination semantics when requested by the user so that a client is not
notified of a crash anymore, since there was no crash anyway.

That change broke WKViewClientWebProcessCallbacks WK2 API test, since it
relied on being notified of a crash after calling WKPageTerminate(). As
a result of not being notified of such non-existent crash, the test would
timeout right after making the terminate call.

This patch adds an InjectedBundle to be used for simulating a crash, by
calling abort() upon receiving a "Crash" message, and thus we are able to
keep testing the crash callback.

This patch also re-enables the WKViewClientWebProcessCallbacks test, which
had been disabled in revisions r148858 and r148855, since it was failing.

• TestWebKitAPI/PlatformEfl.cmake:
• TestWebKitAPI/Tests/WebKit2/efl/WKViewClientWebProcessCallbacks.cpp:

(TestWebKitAPI::TEST):

• TestWebKitAPI/Tests/WebKit2/efl/WKViewClientWebProcessCallbacks_Bundle.cpp: Added.

(TestWebKitAPI):
(WKViewClientWebProcessCallbacksTest):
(TestWebKitAPI::WKViewClientWebProcessCallbacksTest::WKViewClientWebProcessCallbacksTest):
(TestWebKitAPI::WKViewClientWebProcessCallbacksTest::didReceiveMessage):

9:08 AM Changeset in webkit [149044] by ap@apple.com

• 2 edits in trunk/Source/WebKit2

2013-04-24 Simon Cooper <​scooper@apple.com>

<rdar://problem/13614117> Refine the Flash Player profile
​https://bugs.webkit.org/show_bug.cgi?id=115016

Reviewed by Alexey Proskuryakov.

Add some refinements to the Flash Player profile. Make a non-/ home
directory work. Fixes that allow file uploading via user selected files.

• Resources/PlugInSandboxProfiles/com.macromedia.Flash Player.plugin.sb:

8:59 AM Changeset in webkit [149043] by commit-queue@webkit.org

• 6 edits in trunk/Source/WebKit/blackberry

[BlackBerry] Do not clear focus on a node when tapping on form controls
​https://bugs.webkit.org/show_bug.cgi?id=115055

Patch by Nima Ghanavatian <​nghanavatian@blackberry.com> on 2013-04-24
Reviewed by Rob Buis.

Internally reviewed by Genevieve Mak.

PR316069
To allow for rich text editors to apply styles on an input field
or highlighted text, we must maintain focus on the current element
when tapping on form elements. Moving the code that clears this
context to trigger off TouchHold instead of TouchPress.

• WebKitSupport/TouchEventHandler.cpp:

(BlackBerry::WebKit::TouchEventHandler::doFatFingers):

8:49 AM Changeset in webkit [149042] by commit-queue@webkit.org

• 2 edits in trunk/Source/WebCore

[BlackBerry] Fixes the San Angeles demo on khronos.org
​https://bugs.webkit.org/show_bug.cgi?id=115106

Patch by Jonathan Feldstein <​jfeldstein@blackberry.com> on 2013-04-24
Reviewed by Rob Buis
Internally Reviewed by Jeremy Nicholl and Arvid Nilsson

Fixes a bug in which the currently bound vertex array object's vertex array
state was being modified as opposed to the default vertex array state.

• platform/graphics/blackberry/EGLImageLayerWebKitThread.cpp:

(EGLImageLayerWebKitThread::blitToFrontBuffer):

8:48 AM Changeset in webkit [149041] by ggaren@apple.com

• 8 edits in trunk

Filled out more cases of branch folding in the DFG
​https://bugs.webkit.org/show_bug.cgi?id=115088

Reviewed by Oliver Hunt.

Source/JavaScriptCore:

No change on the benchmarks we track, but a 3X speedup on a
microbenchmark that uses these techniques.

• dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::parseBlock): (!/=)= and (!/=)== can constant
fold all types, not just numbers, because true constants have no
side effects when type-converted at runtime.

• dfg/DFGFixupPhase.cpp:

(JSC::DFG::FixupPhase::fixupNode):

• dfg/DFGNode.h:

(JSC::DFG::Node::shouldSpeculateBoolean): Added support for fixing up
boolean uses, like we do for other types like number.

• dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compilePeepHoleBooleanBranch):
(JSC::DFG::SpeculativeJIT::compilePeepHoleBranch):
(JSC::DFG::SpeculativeJIT::compare):
(JSC::DFG::SpeculativeJIT::compileStrictEq):
(JSC::DFG::SpeculativeJIT::compileBooleanCompare): Peephole fuse
boolean compare and/or compare-branch, now that we have the types for
them.

• dfg/DFGSpeculativeJIT.h: Updated declarations.

LayoutTests:

• fast/js/regress/script-tests/branch-fold.js:

(g): Added some boolean and constant-folded cases.

8:44 AM Changeset in webkit [149040] by thakis@chromium.org

• 4 edits
  2 deletes in trunk/Source/WebCore

Fold RenderThemeMacShared into RenderThemeMac
​https://bugs.webkit.org/show_bug.cgi?id=115086

Reviewed by Anders Carlsson.

No intended functionality change.

• WebCore.xcodeproj/project.pbxproj:
• rendering/RenderThemeMac.h:

(RenderThemeMac):
(WebCore::RenderThemeMac::supportsControlTints):
(WebCore::RenderThemeMac::scrollbarControlSizeForPart):
(WebCore::RenderThemeMac::supportsSelectionForegroundColors):
(WebCore::RenderThemeMac::supportsClosedCaptioning):
(WebCore::RenderThemeMac::updateActiveState):

• rendering/RenderThemeMac.mm:

(-[WebCoreRenderThemeNotificationObserver initWithTheme:WebCore::]):
(-[WebCoreRenderThemeNotificationObserver systemColorsDidChange:]):
(-[WebCoreTextFieldCell _coreUIDrawOptionsWithFrame:inView:includeFocus:]):
(WebCore::RenderThemeMac::RenderThemeMac):
(WebCore::RenderThemeMac::~RenderThemeMac):
(WebCore::RenderThemeMac::platformActiveSelectionBackgroundColor):
(WebCore):
(WebCore::RenderThemeMac::platformInactiveSelectionBackgroundColor):
(WebCore::RenderThemeMac::platformActiveListBoxSelectionBackgroundColor):
(WebCore::RenderThemeMac::platformActiveListBoxSelectionForegroundColor):
(WebCore::RenderThemeMac::platformInactiveListBoxSelectionForegroundColor):
(WebCore::RenderThemeMac::platformFocusRingColor):
(WebCore::RenderThemeMac::platformInactiveListBoxSelectionBackgroundColor):
(WebCore::toFontWeight):
(WebCore::RenderThemeMac::systemFont):
(WebCore::convertNSColorToColor):
(WebCore::menuBackgroundColor):
(WebCore::RenderThemeMac::platformColorsDidChange):
(WebCore::RenderThemeMac::systemColor):
(WebCore::RenderThemeMac::usesTestModeFocusRingColor):
(WebCore::RenderThemeMac::isControlStyled):
(WebCore::RenderThemeMac::adjustRepaintRect):
(WebCore::RenderThemeMac::inflateRect):
(WebCore::RenderThemeMac::convertToPaintingRect):
(WebCore::RenderThemeMac::updateCheckedState):
(WebCore::RenderThemeMac::updateEnabledState):
(WebCore::RenderThemeMac::updateFocusedState):
(WebCore::RenderThemeMac::updatePressedState):
(WebCore::RenderThemeMac::controlSupportsTints):
(WebCore::RenderThemeMac::controlSizeForFont):
(WebCore::RenderThemeMac::setControlSize):
(WebCore::RenderThemeMac::sizeForFont):
(WebCore::RenderThemeMac::sizeForSystemFont):
(WebCore::RenderThemeMac::setSizeFromFont):
(WebCore::RenderThemeMac::setFontFromControlSize):
(WebCore::RenderThemeMac::controlSizeForSystemFont):
(WebCore::RenderThemeMac::paintTextField):
(WebCore::RenderThemeMac::adjustTextFieldStyle):
(WebCore::RenderThemeMac::paintCapsLockIndicator):
(WebCore::RenderThemeMac::paintTextArea):
(WebCore::RenderThemeMac::adjustTextAreaStyle):
(WebCore::RenderThemeMac::popupButtonMargins):
(WebCore::RenderThemeMac::popupButtonSizes):
(WebCore::RenderThemeMac::popupButtonPadding):
(WebCore::RenderThemeMac::paintMenuList):
(WebCore::RenderThemeMac::meterSizeForBounds):
(WebCore::RenderThemeMac::paintMeter):
(WebCore::RenderThemeMac::supportsMeter):
(WebCore::RenderThemeMac::levelIndicatorStyleFor):
(WebCore::RenderThemeMac::levelIndicatorFor):
(WebCore::RenderThemeMac::progressBarSizes):
(WebCore::RenderThemeMac::progressBarMargins):
(WebCore::RenderThemeMac::minimumProgressBarHeight):
(WebCore::RenderThemeMac::animationRepeatIntervalForProgressBar):
(WebCore::RenderThemeMac::animationDurationForProgressBar):
(WebCore::RenderThemeMac::adjustProgressBarStyle):
(WebCore::RenderThemeMac::paintProgressBar):
(WebCore::TopGradientInterpolate):
(WebCore::BottomGradientInterpolate):
(WebCore::MainGradientInterpolate):
(WebCore::TrackGradientInterpolate):
(WebCore::RenderThemeMac::paintMenuListButtonGradients):
(WebCore::RenderThemeMac::paintMenuListButton):
(WebCore::menuListButtonSizes):
(WebCore::RenderThemeMac::adjustMenuListStyle):
(WebCore::RenderThemeMac::popupInternalPaddingLeft):
(WebCore::RenderThemeMac::popupInternalPaddingRight):
(WebCore::RenderThemeMac::popupInternalPaddingTop):
(WebCore::RenderThemeMac::popupInternalPaddingBottom):
(WebCore::RenderThemeMac::adjustMenuListButtonStyle):
(WebCore::RenderThemeMac::setPopupButtonCellState):
(WebCore::RenderThemeMac::menuListSizes):
(WebCore::RenderThemeMac::minimumMenuListSize):
(WebCore::RenderThemeMac::adjustSliderTrackStyle):
(WebCore::RenderThemeMac::paintSliderTrack):
(WebCore::RenderThemeMac::adjustSliderThumbStyle):
(WebCore::RenderThemeMac::paintSliderThumb):
(WebCore::RenderThemeMac::paintSearchField):
(WebCore::RenderThemeMac::setSearchCellState):
(WebCore::RenderThemeMac::searchFieldSizes):
(WebCore::RenderThemeMac::setSearchFieldSize):
(WebCore::RenderThemeMac::adjustSearchFieldStyle):
(WebCore::RenderThemeMac::paintSearchFieldCancelButton):
(WebCore::RenderThemeMac::cancelButtonSizes):
(WebCore::RenderThemeMac::adjustSearchFieldCancelButtonStyle):
(WebCore::RenderThemeMac::resultsButtonSizes):
(WebCore::RenderThemeMac::adjustSearchFieldDecorationStyle):
(WebCore::RenderThemeMac::paintSearchFieldDecoration):
(WebCore::RenderThemeMac::adjustSearchFieldResultsDecorationStyle):
(WebCore::RenderThemeMac::paintSearchFieldResultsDecoration):
(WebCore::RenderThemeMac::adjustSearchFieldResultsButtonStyle):
(WebCore::RenderThemeMac::paintSearchFieldResultsButton):
(WebCore::RenderThemeMac::paintSnapshottedPluginOverlay):
(WebCore::RenderThemeMac::sliderTickSize):
(WebCore::RenderThemeMac::sliderTickOffsetFromTrackCenter):
(WebCore::RenderThemeMac::adjustSliderThumbSize):
(WebCore::RenderThemeMac::shouldShowPlaceholderWhenFocused):
(WebCore::RenderThemeMac::popupButton):
(WebCore::RenderThemeMac::search):
(WebCore::RenderThemeMac::searchMenuTemplate):
(WebCore::RenderThemeMac::sliderThumbHorizontal):
(WebCore::RenderThemeMac::sliderThumbVertical):
(WebCore::RenderThemeMac::textField):
(WebCore::RenderThemeMac::fileListNameForWidth):

• rendering/RenderThemeMacShared.h: Removed.
• rendering/RenderThemeMacShared.mm: Removed.

8:38 AM CoordinatedGraphicsSystem edited by allan.jensen@digia.com

(diff)

8:36 AM Changeset in webkit [149039] by commit-queue@webkit.org

• 2 edits in trunk/Source/WebCore

[GTK][AC] Visible rect doesn't update after resizing a window.
​https://bugs.webkit.org/show_bug.cgi?id=115096

Patch by ChangSeok Oh <ChangSeok Oh> on 2013-04-24
Reviewed by Gustavo Noronha Silva.

Once we set actor's a clip rect, we should update the rect when resizing the actor.

No new tests since no new functionality.

• platform/graphics/clutter/GraphicsLayerClutter.cpp:

(WebCore::GraphicsLayerClutter::setSize):

8:34 AM Changeset in webkit [149038] by commit-queue@webkit.org

• 2 edits in trunk/Source/WebCore

[GTK][AC] Fix unexpected clear of ClutterContent.
​https://bugs.webkit.org/show_bug.cgi?id=115099

Patch by ChangSeok Oh <ChangSeok Oh> on 2013-04-24
Reviewed by Gustavo Noronha Silva.

adoptGRef doesn't increase a reference count. So if a actor has a valid ClutterContent,
it would be cleared unexpectedly by smart pointer when escaping a function.

No new tests, no functionality changed.

• platform/graphics/clutter/GraphicsLayerActor.cpp:

(graphicsLayerActorUpdateTexture):

8:30 AM Changeset in webkit [149037] by akling@apple.com

• 2 edits in trunk/Source/WebKit/win

Wild build fix attempt.

• WebCoreSupport/WebChromeClient.h:

(WebChromeClient):

8:26 AM Changeset in webkit [149036] by commit-queue@webkit.org

• 2 edits in trunk/Source/WebKit2

[WebKit2] Add CoordinatedGraphics to the list of platforms in generate-forwarding-headers.pl
​https://bugs.webkit.org/show_bug.cgi?id=115098

Patch by Carlos Garcia Campos <​cgarcia@igalia.com> on 2013-04-24
Reviewed by Martin Robinson.

r148963 introduced a new WKView.h header in a directory not listed
as a platform prefix in generate-forwarding-headers.pl.

• Scripts/generate-forwarding-headers.pl: Replace unused chromium

plaform prefix with CoordinatedGraphics that is used now.

8:15 AM Changeset in webkit [149035] by jberlin@webkit.org

• 7 edits
  1 delete in trunk/Source/WebKit

Assertion failure (crash in release) attempting to delete database storage in WK1 without
creating a WebView first
​https://bugs.webkit.org/show_bug.cgi?id=115065

Reviewed by Brady Eidson.

Source/WebKit:

• WebKit.xcodeproj/project.pbxproj:

Remove WebDatabaseManagerInternal.h.

Source/WebKit/mac:

Make sure the WebPlatformStrategies is initialized before using the WebDatabaseManager.

• Storage/WebDatabaseManager.mm:

(-[WebDatabaseManager init]):
Move the logic from WebKitInitializeDatabasesIfNecessary here, and add a call to
WebPlatformStrategies::initializeIfNecessary.

• Storage/WebDatabaseManagerInternal.h: Removed.

• WebCoreSupport/WebPlatformStrategies.h:

(WebPlatformStrategies):
Change "initialize" to "initalizeIfNecessary".

• WebCoreSupport/WebPlatformStrategies.mm:

(WebPlatformStrategies::initializeIfNecessary):
Only call setPlatformStrategies once.

• WebView/WebView.mm:

(-[WebView _commonInitializationWithFrameName:groupName:]):
Updated for the name change from initialize to initializeIfNecessary.
Instead of calling WebKitInitializeDatabasesIfNecessary, call [WebDatabaseManager
sharedWebDatabaseManager].
(+[WebView _setLoadResourcesSerially:]):
Updated for the name change from initialize to initializeIfNecessary.

7:27 AM Changeset in webkit [149034] by commit-queue@webkit.org

• 2 edits in trunk/Source/WebCore

DOMFileSystemBase: fix multiple definitions in the BlackBerry port
​https://bugs.webkit.org/show_bug.cgi?id=114950

Patch by Alberto Garcia <​agarcia@igalia.com> on 2013-04-24
Reviewed by Xan Lopez.

The following methods are already defined in DOMFileSystemBlackBerry.cpp:

crackFileSystemURL()
createFileSystemURL()
isValidType()
supportsToURL()

• Modules/filesystem/DOMFileSystemBase.cpp:

(WebCore):

7:10 AM Changeset in webkit [149033] by commit-queue@webkit.org

• 3 edits in trunk/Source/WebCore

[BlackBerry] Add support for JPEG image quality during encoding.
​https://bugs.webkit.org/show_bug.cgi?id=105773

Patch by George Staikos <​gstaikos@rim.com> on 2013-04-24
Reviewed by Xan Lopez.

PR 271611
Internally reviewed by Liam Quinn.

Pass the quality flag into the encoder, with a default of 65

Covered by existing tests.

• platform/image-encoders/JPEGImageEncoder.cpp:

(WebCore::compressRGBABigEndianToJPEG):

• platform/image-encoders/JPEGImageEncoder.h:

(WebCore):

6:54 AM Changeset in webkit [149032] by commit-queue@webkit.org

• 2 edits in trunk/Source/WebKit/efl

[EFL] Add unit test cases for ewk_settings APIs
​https://bugs.webkit.org/show_bug.cgi?id=114897

Patch by Jose Lejin PJ <​jose.lejin@gmail.com> on 2013-04-24
Reviewed by Gyuyoung Kim.

Added unit tests for ewk_settings APIs.

• tests/test_ewk_setting.cpp:

(TEST_F):

6:11 AM Changeset in webkit [149031] by Csaba Osztrogonác

• 2 edits in trunk/Source/WebCore

Fix cast-align WebCore/platform/graphics/GraphicsContext3D.cpp on ARM
​https://bugs.webkit.org/show_bug.cgi?id=115036

Reviewed by Benjamin Poulain.

• platform/graphics/GraphicsContext3D.cpp:

(WebCore):

6:03 AM Changeset in webkit [149030] by Christophe Dumez

• 2 edits in trunk/LayoutTests

Unreviewed EFL gardening.

Rebaseline fast/js/constructor-length.html for EFL port after
r148997.

• platform/efl/fast/js/constructor-length-expected.txt:

5:33 AM Changeset in webkit [149029] by akling@apple.com

• 3 edits in trunk/Source/WebCore

Remove unnecessary MutableStylePropertySet casts.
<​http://webkit.org/b/115075>

Reviewed by Geoffrey Garen.

Get rid of some now-unneeded static_casts that were left from the StylePropertySet
refactoring last weekend.

• css/StylePropertySet.cpp:

(WebCore::MutableStylePropertySet::setProperty):

• editing/EditingStyle.cpp:

(WebCore::EditingStyle::triStateOfStyle):
(WebCore::EditingStyle::removeStyleFromRulesAndContext):

5:17 AM Changeset in webkit [149028] by kangil.han@samsung.com

• 2 edits in trunk/Source/WebCore

[Minor code cleanup] Remove duplicated condition check
​https://bugs.webkit.org/show_bug.cgi?id=115082

Reviewed by Andreas Kling.

areRectsPartiallyAligned() has duplicated condition check, so remove it.

• page/SpatialNavigation.cpp:

(WebCore::areRectsPartiallyAligned):

5:01 AM Changeset in webkit [149027] by Christophe Dumez

• 2 edits in trunk/Tools

[EFL][WK2] WebKitTestRunner failures due to IconDatabase assertions
​https://bugs.webkit.org/show_bug.cgi?id=115095

Reviewed by Antonio Gomes.

Use different subfolders under DumpRenderTree Temp folder for every
functionality (icon database, cache, localstorage, ...). This makes
sure there is no conflict between the functionalities.

In EFL's case, libsoup cache was causing problems for the icon
database if sharing the same folder.

• WebKitTestRunner/TestController.cpp:

(WTR::TestController::initialize):

4:21 AM Changeset in webkit [149026] by commit-queue@webkit.org

• 7 edits in trunk/Source/WebCore

[EFL] Fix build warnings caused by missing images attributes for default buttons
​https://bugs.webkit.org/show_bug.cgi?id=114955

Patch by Krzysztof Wolanski <​k.wolanski@samsung.com> on 2013-04-24
Reviewed by Gyuyoung Kim.

Added default image for each button from mediacontrol catalog.

No new tests, because there is no change in behavior.

• platform/efl/DefaultTheme/widget/mediacontrol/fullscreenbutton/fullscreen_button.edc:
• platform/efl/DefaultTheme/widget/mediacontrol/mutebutton/mute_button.edc:
• platform/efl/DefaultTheme/widget/mediacontrol/playpausebutton/playpause_button.edc:
• platform/efl/DefaultTheme/widget/mediacontrol/seekbackwardbutton/seekbackward_button.edc:
• platform/efl/DefaultTheme/widget/mediacontrol/seekforwardbutton/seekforward_button.edc:
• platform/efl/DefaultTheme/widget/mediacontrol/togglecaptionsbutton/toggle_captions_button.edc:

3:54 AM WebKitGTK/2.0.x edited by Manuel Rego Casasnovas

(diff)

3:53 AM WebKitGTK/2.0.x edited by Manuel Rego Casasnovas

(diff)

3:50 AM Changeset in webkit [149025] by commit-queue@webkit.org

• 2 edits in trunk/Source/WebKit2

Web Inspector: Add number to list from remote web inspector.
​https://bugs.webkit.org/show_bug.cgi?id=115014

Patch by Seokju Kwon <Seokju Kwon> on 2013-04-24
Reviewed by Benjamin Poulain.

• UIProcess/InspectorServer/front-end/inspectorPageIndex.html:

3:48 AM Changeset in webkit [149024] by commit-queue@webkit.org

• 23 edits in trunk/LayoutTests

[EFL] Rebaselining after r148944
​https://bugs.webkit.org/show_bug.cgi?id=115085

Unreviewed gardening.

Patch by Seokju Kwon <Seokju Kwon> on 2013-04-24

• platform/efl/TestExpectations:
• platform/efl/fast/table/giantRowspan-expected.txt:
• platform/efl/fast/table/giantRowspan2-expected.png:
• platform/efl/fast/table/giantRowspan2-expected.txt:
• platform/efl/tables/mozilla/bugs/bug133756-1-expected.png:
• platform/efl/tables/mozilla/bugs/bug133756-1-expected.txt:
• platform/efl/tables/mozilla/bugs/bug133756-2-expected.png:
• platform/efl/tables/mozilla/bugs/bug133756-2-expected.txt:
• platform/efl/tables/mozilla/bugs/bug220536-expected.png:
• platform/efl/tables/mozilla/bugs/bug220536-expected.txt:
• platform/efl/tables/mozilla/bugs/bug8858-expected.png:
• platform/efl/tables/mozilla/bugs/bug8858-expected.txt:
• platform/efl/tables/mozilla/core/bloomberg-expected.png:
• platform/efl/tables/mozilla/core/bloomberg-expected.txt:
• platform/efl/tables/mozilla/core/row_span-expected.png:
• platform/efl/tables/mozilla/core/row_span-expected.txt:
• platform/efl/tables/mozilla_expected_failures/bugs/bug131020-3-expected.png:
• platform/efl/tables/mozilla_expected_failures/bugs/bug131020-3-expected.txt:
• platform/efl/tables/mozilla_expected_failures/bugs/bug23847-expected.png:
• platform/efl/tables/mozilla_expected_failures/bugs/bug23847-expected.txt:
• platform/efl/tables/mozilla_expected_failures/bugs/bug65372-expected.png:
• platform/efl/tables/mozilla_expected_failures/bugs/bug65372-expected.txt:

2:01 AM Changeset in webkit [149023] by abucur@adobe.com

• 3 edits in trunk/LayoutTests

Fix test style issues following the commit of
​https://bugs.webkit.org/show_bug.cgi?id=114412

Unreviewed gardening.

• css3/compositing/effect-background-blend-mode-color.html:
• css3/compositing/effect-background-blend-mode-color2.html:

1:44 AM Changeset in webkit [149022] by commit-queue@webkit.org

• 2 edits in trunk/Source/WebKit2

--minimal build fails with error: Source/WebKit2/Platform/CoreIPC/ArgumentCoder.h:44:36: error: decode is not a member of WebCore::TextCheckingResult
​https://bugs.webkit.org/show_bug.cgi?id=114367

Patch by Vlad Vasilyeu <​vasvlad@gmail.com> on 2013-04-24
Reviewed by Benjamin Poulain.

Include WebCoreArgumentCoders.h to WebEditorClientEfl.cpp from
WebKit2/Shared/ for successfully building with option --minimal

• WebProcess/WebCoreSupport/efl/WebEditorClientEfl.cpp:

1:23 AM Changeset in webkit [149021] by Christophe Dumez

• 2 edits in trunk/Source/WebCore

Unreviewed build fix with gstreamer 0.10.x after r147555.

• platform/audio/gstreamer/WebKitWebAudioSourceGStreamer.cpp:

(webKitWebAudioSrcLoop):

1:11 AM Changeset in webkit [149020] by mrowe@apple.com

• 6 edits
  6 copies in trunk

Roll deps^HH^HHChangeLogs.

1:09 AM Changeset in webkit [149019] by anilsson@rim.com

• 5 edits in trunk/Source/WebCore

[BlackBerry] Remove the setContentsToColor flavor of LayerTiler::TextureJob
​https://bugs.webkit.org/show_bug.cgi?id=115092

Reviewed by Carlos Garcia Campos.

With the Skia backend, we had an optimization to detect solid color
layers. The Skia code has been purged, but these bits remained. With
BlackBerry::Platform::Graphics::PlatformGraphicsContext, only the solid
color will be drawn anyway, so we don't need to detect such layers at
the WebCore level. Fixed by removing the setContentsToColor flavor of
LayerTiler::TextureJob.

BlackBerry::WebKit::WebOverlay still uses
TextureCacheCompositingThread::textureForColor() so that one can stay
for now.

This was dead code so no new tests.

• platform/graphics/blackberry/LayerTile.cpp:
• platform/graphics/blackberry/LayerTile.h:

(LayerTile):

• platform/graphics/blackberry/LayerTiler.cpp:

(WebCore::LayerTiler::processTextureJob):
(WebCore::LayerTiler::performTileJob):

• platform/graphics/blackberry/LayerTiler.h:

(WebCore::LayerTiler::TextureJob::setContents):
(TextureJob):

12:43 AM Changeset in webkit [149018] by zandobersek@gmail.com

• 4 edits in trunk/Source

[GTK] Move image decoders, some other GStreamer, Cairo and Soup sources to libPlatform
​https://bugs.webkit.org/show_bug.cgi?id=115050

Reviewed by Martin Robinson.

Source/Platform:

• GNUmakefile.am: Add the image-decoders directory and its subdirectories to the list of header inclusion directories.

List the Cairo, GStreamer and Libsoup cppflags under the cppflags libPlatform uses for compiling its sources.

Source/WebCore:

No new tests - no new functionality.

• GNUmakefile.list.am: Move the image decoders source files and a couple of GStreamer, Cairo and Libsoup source

files under the libPlatform's list of files to build. All of these are already free of platform layer violations.

12:25 AM Changeset in webkit [149017] by Claudio Saavedra

• 3 edits in trunk/Source/WebKit2

Build break after r149004
​https://bugs.webkit.org/show_bug.cgi?id=115091

Reviewed by Csaba Osztrogonác.

• WebProcess/WebPage/gtk/LayerTreeHostGtk.cpp:

(WebKit::LayerTreeHostGtk::pageBackgroundTransparencyChanged):
(WebKit): Add dummy method for new virtual method.

• WebProcess/WebPage/gtk/LayerTreeHostGtk.h:

(LayerTreeHostGtk): Ditto.