Timeline

Oct 5, 2016:

11:54 PM Changeset in webkit [206852] by commit-queue@webkit.org

• 4 edits in trunk/Source/WebCore

Unreviewed, rolling out r206845.
​https://bugs.webkit.org/show_bug.cgi?id=162998

broke el capitan build (Requested by alexchristensen on
#webkit).

Reverted changeset:

"Add Display P3 ColorSpace"
​https://bugs.webkit.org/show_bug.cgi?id=162880
​http://trac.webkit.org/changeset/206845

11:45 PM Changeset in webkit [206851] by commit-queue@webkit.org

• 5 edits in trunk

Reuse CodeGenerator::UpdateFile in Tools CodeGenerator
​https://bugs.webkit.org/show_bug.cgi?id=162960

Patch by Youenn Fablet <​youenn@apple.com> on 2016-10-05
Reviewed by Alex Christensen.

Source/WebCore:

No change of behavior.

• bindings/scripts/CodeGenerator.pm:

(UpdateFile): Adding FIXME to enable lazy writing when feasible.

Tools:

• DumpRenderTree/Bindings/CodeGeneratorDumpRenderTree.pm:

(WriteData): Using CodeGenerator::UpdateFile to do lazy generated file update when it is made feasible.

• WebKitTestRunner/InjectedBundle/Bindings/CodeGeneratorTestRunner.pm:

(WriteData): Ditto.

11:39 PM Changeset in webkit [206850] by Michael Catanzaro

• 2 edits in trunk/Source/WebCore

[SOUP] HostTLSCertificateSet should not use SHA-1 hashes to compare certificates
​https://bugs.webkit.org/show_bug.cgi?id=162965

Reviewed by Carlos Garcia Campos.

Use SHA-256 instead.

No new tests, covered by TestSSL.

• platform/network/soup/SoupNetworkSession.cpp:

(WebCore::HostTLSCertificateSet::computeCertificateHash):

11:38 PM Changeset in webkit [206849] by Michael Catanzaro

• 2 edits in trunk/Source/WebCore

[GTK] Bump Chrome version in Chrome user agent quirk
​https://bugs.webkit.org/show_bug.cgi?id=162984

Reviewed by Carlos Garcia Campos.

• platform/gtk/UserAgentGtk.cpp:

(WebCore::buildUserAgentString):

10:51 PM Changeset in webkit [206848] by Yusuke Suzuki

• 2 edits in trunk/Source/JavaScriptCore

Unreviewed, attempt to fix CLoop build after r206846
​https://bugs.webkit.org/show_bug.cgi?id=162941

Attempt to fix CLoop build part 2. r206847 was not valid.

• jsc.cpp:

10:38 PM Changeset in webkit [206847] by Yusuke Suzuki

• 2 edits in trunk/Source/JavaScriptCore

Unreviewed, build fix after r206846
​https://bugs.webkit.org/show_bug.cgi?id=162941

DOMJIT::Patchpoint part should be guarded by ENABLE(JIT).

• jsc.cpp:

10:20 PM Changeset in webkit [206846] by Yusuke Suzuki

• 34 edits
  1 copy
  5 adds in trunk

[DOMJIT] Add initial CheckDOM and CallDOM implementations
​https://bugs.webkit.org/show_bug.cgi?id=162941

Reviewed by Filip Pizlo.

JSTests:

• stress/domjit-getter-poly.js: Added.

(shouldBe):
(access):

• stress/domjit-getter-proto.js: Added.

(shouldBe):
(access):

• stress/domjit-getter-super-poly.js: Added.

(shouldBe):
(access):

• stress/domjit-getter.js: Added.

(shouldBe):
(access):

Source/JavaScriptCore:

This patch implements a prototype of DOMJIT accelerated getter.
We add two new DFG nodes, CheckDOM and CallDOM.

CheckDOM is used to filter inappropriate |this| object for DOM getter. Its functionality
is equivalent to jsDynamicCast's Check. You can use like "CheckDOM, @1, JSNode::info()",
and this CheckDOM incurs a BadType exit if the class of the given @1 is not a subclass of
JSNode::info().

CallDOM is used to emit actual DOM operations. It takes GlobalObject and checked DOM
object. And it returns JSValue as its result.

Both CheckDOM and CallDOM can take a DOMJIT::Patchpoint. This is somewhat code snippet
generator, and is injectable to DFG and FTL. DFG and FTL set up registers correctly
according to DOMJIT::Patchpoint's requirement and invoke this patchpoint generator to emit code.
While CallDOM always requires a patchpoint, ideally CheckDOM does not require it since
isSubclassOf check can be implemented in DFG / FTL side. However, some classes have a
faster way to query isSubclassOf. For example, JSNode in WebCore introduces a special
JSType to optimize this query. CheckDOM's patchpoint gives us a chance to emit special
faster code for such a case.

By leveraging above nodes, we can construct DOMJIT accelerated getter. When DFG recognizes the
given getter call is CustomGetter and it has DOMJIT::GetterSetter information, DFG emits the above nodes.
We implemented a prototype in jsc.cpp shell as DOMJITGetter to test the functionality.

Notes about the future extensions.

1. Currently, we do not allow CallDOM to emit any function calls. This will be extended by adding addSlowPathCall functionality to DOMJIT::Patchpoint later. Interesting thing is that we need to create an abstraction over DFG slow path call and FTL slow path call!

2. CheckDOM is not handled in DFGTypeCheckHoistingPhase yet. And we have a chance to merge several CheckDOM into one. For example, given CheckDOM A and CheckDOM B to the same target. If A is subclass of B, we can merge them to CheckDOM A.

• JavaScriptCore.xcodeproj/project.pbxproj:
• b3/B3Effects.h:

(JSC::B3::Effects::forCheck):

• b3/B3Value.cpp:

(JSC::B3::Value::effects):

• bytecode/GetByIdStatus.cpp:

(JSC::GetByIdStatus::computeForStubInfoWithoutExitSiteFeedback):
(JSC::GetByIdStatus::makesCalls):
(JSC::GetByIdStatus::dump):

• bytecode/GetByIdStatus.h:

(JSC::GetByIdStatus::GetByIdStatus):
(JSC::GetByIdStatus::isCustom):
(JSC::GetByIdStatus::takesSlowPath):
(JSC::GetByIdStatus::isSimple): Deleted.

• bytecode/SpeculatedType.cpp:

(JSC::speculationFromClassInfo):

• dfg/DFGAbstractInterpreter.h:

(JSC::DFG::AbstractInterpreter::filterClassInfo):

• dfg/DFGAbstractInterpreterInlines.h:

(JSC::DFG::AbstractInterpreter<AbstractStateType>::executeEffects):
(JSC::DFG::AbstractInterpreter<AbstractStateType>::filterClassInfo):

• dfg/DFGAbstractValue.cpp:

(JSC::DFG::AbstractValue::filterClassInfo):

• dfg/DFGAbstractValue.h:
• dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::handleDOMJITGetter):
(JSC::DFG::ByteCodeParser::handleGetById):

• dfg/DFGClobberize.h:

(JSC::DFG::clobberize):

• dfg/DFGConstantFoldingPhase.cpp:

(JSC::DFG::ConstantFoldingPhase::foldConstants):

• dfg/DFGDoesGC.cpp:

(JSC::DFG::doesGC):

• dfg/DFGFixupPhase.cpp:

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

• dfg/DFGNode.h:

(JSC::DFG::Node::hasHeapPrediction):
(JSC::DFG::Node::hasDOMJIT):
(JSC::DFG::Node::domJIT):
(JSC::DFG::Node::hasClassInfo):
(JSC::DFG::Node::classInfo):
(JSC::DFG::Node::OpInfoWrapper::OpInfoWrapper):
(JSC::DFG::Node::OpInfoWrapper::operator=):

• dfg/DFGNodeType.h:
• dfg/DFGOpInfo.h:

(JSC::DFG::OpInfo::OpInfo):

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

(JSC::DFG::safeToExecute):

• dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::allocateTemporaryRegistersForPatchpoint):
(JSC::DFG::SpeculativeJIT::compileCallDOM):
(JSC::DFG::SpeculativeJIT::compileCheckDOM):

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

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

• dfg/DFGSpeculativeJIT64.cpp:

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

• dfg/DFGStructureAbstractValue.cpp:

(JSC::DFG::StructureAbstractValue::filterClassInfoSlow):
(JSC::DFG::StructureAbstractValue::isSubClassOf):

• dfg/DFGStructureAbstractValue.h:

(JSC::DFG::StructureAbstractValue::filterClassInfo):
(JSC::DFG::StructureAbstractValue::filter): Deleted.

• domjit/DOMJITPatchpointParams.h: Copied from Source/JavaScriptCore/dfg/DFGOpInfo.h.

(JSC::DOMJIT::PatchpointParams::~PatchpointParams):
(JSC::DOMJIT::PatchpointParams::size):
(JSC::DOMJIT::PatchpointParams::at):
(JSC::DOMJIT::PatchpointParams::operator[]):
(JSC::DOMJIT::PatchpointParams::gpScratch):
(JSC::DOMJIT::PatchpointParams::fpScratch):
(JSC::DOMJIT::PatchpointParams::PatchpointParams):

• domjit/DOMJITReg.h: Added.

(JSC::DOMJIT::Reg::Reg):
(JSC::DOMJIT::Reg::isGPR):
(JSC::DOMJIT::Reg::isFPR):
(JSC::DOMJIT::Reg::isJSValueRegs):
(JSC::DOMJIT::Reg::gpr):
(JSC::DOMJIT::Reg::fpr):
(JSC::DOMJIT::Reg::jsValueRegs):

• ftl/FTLCapabilities.cpp:

(JSC::FTL::canCompile):

• ftl/FTLLowerDFGToB3.cpp:

(JSC::FTL::DFG::LowerDFGToB3::compileNode):
(JSC::FTL::DFG::LowerDFGToB3::compileCheckDOM):
(JSC::FTL::DFG::LowerDFGToB3::compileCallDOM):
(JSC::FTL::DFG::LowerDFGToB3::compileUnreachable): Deleted.

• jit/AssemblyHelpers.h:
• jsc.cpp:

(WTF::DOMJITNode::DOMJITNode):
(WTF::DOMJITNode::createStructure):
(WTF::DOMJITNode::create):
(WTF::DOMJITNode::value):
(WTF::DOMJITNode::offsetOfValue):
(WTF::DOMJITGetter::DOMJITGetter):
(WTF::DOMJITGetter::createStructure):
(WTF::DOMJITGetter::create):
(WTF::DOMJITGetter::DOMJITNodeDOMJIT::DOMJITNodeDOMJIT):
(WTF::DOMJITGetter::domJITNodeGetterSetter):
(WTF::DOMJITGetter::finishCreation):
(WTF::DOMJITGetter::customGetter):
(GlobalObject::finishCreation):
(functionCreateDOMJITNodeObject):
(functionCreateDOMJITGetterObject):

Source/WTF:

• wtf/Box.h:

(WTF::Box::Box):

10:15 PM Changeset in webkit [206845] by dino@apple.com

• 4 edits in trunk/Source/WebCore

Add Display P3 ColorSpace
​https://bugs.webkit.org/show_bug.cgi?id=162880
<rdar://problem/28598564>

Reviewed by Sam Weinig.

Add a new ColorSpace for Display P3.

This will be tested once the Color class can create
objects that reference other color spaces.

• platform/graphics/ColorSpace.h:
• platform/graphics/cg/GraphicsContextCG.cpp:

(WebCore::displayP3ColorSpaceRef):

• platform/graphics/cg/GraphicsContextCG.h:

(WebCore::cachedCGColorSpace):

9:24 PM Changeset in webkit [206844] by Yusuke Suzuki

• 2 edits in trunk/Source/JavaScriptCore

[JSC] Do not construct Simple GetByIdStatus against self-custom-accessor case
​https://bugs.webkit.org/show_bug.cgi?id=162993

Reviewed by Filip Pizlo.

We accidentally created a Simple GetByIdStatus against self-custom-accessor case: the object has own custom accessor property and get_by_id hits.
If we returned such a result, the GetById will be turned to GetByOffset and it looks up incorrect thing like CustomGetterSetter object.
We do not hit this bug before since maybe there is no object that has own custom-accessor and this custom-accessor does not raise an error.
For example, "Node.prototype" has "firstChild" custom accessor. But since "Node.prototype" itself does not have Node::info(), "Node.prototype.firstChild"
access always raises an error. I guess all the custom accessors follow this pattern. This bug is uncovered when testing DOMJIT (This bug causes crash and
it can occur even if we disabled DOMJIT).

But such a assumption is not guaranteed. In this patch, we fix this by not returning Simple GetById.

• bytecode/GetByIdStatus.cpp:

(JSC::GetByIdStatus::computeFromLLInt):
(JSC::GetByIdStatus::computeForStubInfoWithoutExitSiteFeedback):
(JSC::GetByIdStatus::computeFor):

9:05 PM Changeset in webkit [206843] by Wenson Hsieh

• 30 edits
  7 adds in trunk

Introduce InputEvent bindings in preparation for the input events spec
​https://bugs.webkit.org/show_bug.cgi?id=162954

Reviewed by Ryosuke Niwa.

Source/WebCore:

Creates new bindings for InputEvent. Also introduces a runtime switch that allows Node to dispatch InputEvents
rather than generic events of type 'input'. Adds a new test verifying that when the switch is enabled, inputs
and contenteditable divs fire these new InputEvents on input. So far, only inputType is a part of the object
prototype, since we will be implementing this first. getTargetRanges() will follow once the finer details of
StaticRange are hashed out. See ​https://bugs.webkit.org/show_bug.cgi?id=162947 for more details.

Tests: fast/events/input-events-fired-when-typing.html

fast/events/ios/input-events-fired-when-typing-in-contenteditable.html

• CMakeLists.txt:
• DerivedSources.cpp:
• DerivedSources.make:
• WebCore.xcodeproj/project.pbxproj:
• dom/DOMAllInOne.cpp:
• dom/Event.cpp:

(WebCore::Event::composed):
(WebCore::Event::isInputEvent):

• dom/Event.h:
• dom/EventNames.in:
• dom/InputEvent.cpp: Added.

(WebCore::InputEvent::InputEvent):

• dom/InputEvent.h: Added.
• dom/InputEvent.idl: Added.
• dom/Node.cpp:

(WebCore::Node::dispatchInputEvent):
(WebCore::Node::defaultEventHandler):

• dom/Node.h:
• editing/Editor.cpp:

(WebCore::Editor::setBaseWritingDirection):

• html/HTMLFormControlElement.cpp:

(WebCore::HTMLFormControlElement::dispatchFormControlInputEvent):

• html/HTMLInputElement.cpp:

(WebCore::HTMLInputElement::setEditingValue):
(WebCore::HTMLInputElement::setValueFromRenderer):

• html/HTMLSelectElement.cpp:

(WebCore::HTMLSelectElement::listBoxOnChange):
(WebCore::HTMLSelectElement::dispatchChangeEventForMenuList):

• html/shadow/MediaControlElements.cpp:

(WebCore::MediaControlTimelineElement::defaultEventHandler):

• page/Settings.in:
• testing/InternalSettings.cpp:

(WebCore::InternalSettings::Backup::Backup):
(WebCore::InternalSettings::Backup::restoreTo):

• testing/InternalSettings.h:
• testing/InternalSettings.idl:

Source/WebKit2:

Adds a new runtime switch for enabling InputEvents.

• Shared/WebPreferencesDefinitions.h:
• WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::updatePreferences):

LayoutTests:

Adds a new test verifying that InputEvents are fired when the runtime switch for input events is enabled.

• fast/events/input-events-fired-when-typing-expected.txt: Added.
• fast/events/input-events-fired-when-typing.html: Added.
• fast/events/ios/input-events-fired-when-typing-in-contenteditable-expected.txt: Added.
• fast/events/ios/input-events-fired-when-typing-in-contenteditable.html: Added.
• js/dom/global-constructors-attributes-expected.txt:
• platform/gtk/js/dom/global-constructors-attributes-expected.txt:
• platform/ios-simulator/TestExpectations:
• platform/mac-wk1/js/dom/global-constructors-attributes-expected.txt:
• platform/mac-yosemite/js/dom/global-constructors-attributes-expected.txt:
• platform/mac/js/dom/global-constructors-attributes-expected.txt:

7:35 PM Changeset in webkit [206842] by achristensen@apple.com

• 5 edits in trunk

URLParser should parse IPv4 addresses as the last two pieces of an IPv6 address
​https://bugs.webkit.org/show_bug.cgi?id=162991

Reviewed by Saam Barati.

Source/WebCore:

The last 32 bits of an IPv6 address can be written in the form of an IPv4 address,
but the syntax is stricter than if it were just an IPv4 address. There must be 4 pieces,
and each piece must be decimal (no octal or hex allowed). These should serialize to
regular hex IPv6 addresses.

Covered by new API tests.

• platform/URLParser.cpp:

(WebCore::URLParser::parseIPv4Piece):
(WebCore::URLParser::parseIPv4Host):
(WebCore::URLParser::parseIPv4PieceInsideIPv6):
(WebCore::URLParser::parseIPv4AddressInsideIPv6):
(WebCore::URLParser::parseIPv6Host):
(WebCore::URLParser::parseIPv4Number): Deleted.

• platform/URLParser.h:

Tools:

• TestWebKitAPI/Tests/WebCore/URLParser.cpp:

(TestWebKitAPI::TEST_F):

5:50 PM Changeset in webkit [206841] by sbarati@apple.com

• 3 edits in trunk/Source/JavaScriptCore

PCToCodeOriginMap builder should use labelIgnoringWatchpoints() inside the DFG
​https://bugs.webkit.org/show_bug.cgi?id=162936

Reviewed by Michael Saboff.

label() may insert nops because of an InvalidationPoint. It does that
because we don't want code that comes after an InvalidationPoint that isn't
effected by the invalidation point to be overwritten if we fire the
InvalidationPoint. PCToCodeOriginMap just grabs labels to build
a mapping, it never emits code that actually jumps to those labels.
Therefore, it should never cause us to emit nops.

• dfg/DFGJITCompiler.cpp:

(JSC::DFG::JITCompiler::compile):
(JSC::DFG::JITCompiler::compileFunction):

• dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::runSlowPathGenerators):
(JSC::DFG::SpeculativeJIT::compileCurrentBlock):

5:47 PM Changeset in webkit [206840] by Gyuyoung Kim

• 2 edits in trunk/LayoutTests

[EFL] Mark timeout tests to *Timeout*

Unreviewed EFL gardening.

Timeout has been happened on some tests.

Patch by Gyuyoung Kim <​gyuyoung.kim@navercorp.com> on 2016-10-05

• platform/efl/TestExpectations:

5:40 PM Changeset in webkit [206839] by mmaxfield@apple.com

• 33 edits in trunk

Put variation fonts work behind a compile-time flag
​https://bugs.webkit.org/show_bug.cgi?id=162949

Reviewed by Simon Fraser.

Source/JavaScriptCore:

• Configurations/FeatureDefines.xcconfig:

Source/WebCore:

No new tests because there is no behavior change.

• Configurations/FeatureDefines.xcconfig:
• css/CSSComputedStyleDeclaration.cpp:

(WebCore::ComputedStyleExtractor::propertyValue):

• css/CSSFontVariationValue.cpp:
• css/CSSFontVariationValue.h:
• css/CSSPropertyNames.in:
• css/CSSValue.cpp:

(WebCore::CSSValue::equals):
(WebCore::CSSValue::cssText):
(WebCore::CSSValue::destroy):

• css/CSSValue.h:

(WebCore::CSSValue::isFontFeatureValue): Deleted.
(WebCore::CSSValue::isFontVariationValue): Deleted.

• css/StyleBuilderConverter.h:

(WebCore::StyleBuilderConverter::convertFontVariationSettings): Deleted.

• css/StyleBuilderCustom.h:

(WebCore::StyleBuilderCustom::applyInheritFontFeatureSettings): Deleted.
(WebCore::StyleBuilderCustom::applyInheritFontVariationSettings): Deleted.

• css/parser/CSSParser.cpp:

(WebCore::CSSParserContext::CSSParserContext):
(WebCore::operator==):
(WebCore::CSSParser::parseValue):
(WebCore::CSSParser::parseFontVariationSettings): Deleted.

• css/parser/CSSParserMode.h:
• page/Settings.in:
• page/animation/CSSPropertyAnimation.cpp:

(WebCore::CSSPropertyAnimationWrapperMap::CSSPropertyAnimationWrapperMap):
(WebCore::blendFunc): Deleted.

• platform/graphics/FontCache.h:

(WebCore::FontDescriptionKey::FontDescriptionKey):
(WebCore::FontDescriptionKey::operator==):
(WebCore::FontDescriptionKey::computeHash):

• platform/graphics/FontDescription.cpp:
• platform/graphics/FontDescription.h:

(WebCore::FontDescription::operator==):
(WebCore::FontDescription::setFeatureSettings): Deleted.
(WebCore::FontDescription::setVariationSettings): Deleted.

• platform/graphics/FontTaggedSettings.cpp:

(WebCore::operator<<): Deleted.

• platform/graphics/FontTaggedSettings.h:

(WebCore::FontVariationSettings::isEmpty):

• platform/graphics/cocoa/FontCacheCoreText.cpp:

(WebCore::preparePlatformFont):

• platform/graphics/cocoa/FontPlatformDataCocoa.mm:

(WebCore::cascadeToLastResortAndVariationsFontDescriptor):

• rendering/style/RenderStyle.cpp:

(WebCore::RenderStyle::setFontVariationSettings): Deleted.

• rendering/style/RenderStyle.h:

(WebCore::RenderStyle::fontVariationSettings): Deleted.

• testing/InternalSettings.cpp:

(WebCore::InternalSettings::Backup::Backup):
(WebCore::InternalSettings::Backup::restoreTo):
(WebCore::InternalSettings::variationFontsEnabled):
(WebCore::InternalSettings::setVariationFontsEnabled):

• testing/InternalSettings.h:

Source/WebKit/mac:

• Configurations/FeatureDefines.xcconfig:

Source/WebKit2:

• Configurations/FeatureDefines.xcconfig:

5:35 PM Changeset in webkit [206838] by rniwa@webkit.org

• 2 edits
  12 moves
  41 adds in trunk/LayoutTests

Import v1 custom elements tests from W3C
​https://bugs.webkit.org/show_bug.cgi?id=162987

Reviewed by Chris Dumez.

LayoutTests/imported/w3c:

Import tests in ​https://github.com/w3c/web-platform-tests/tree/master/custom-elements except v1 tests
as of eae188126a21a593715ba2626cc059144d6f0978.

• web-platform-tests/custom-elements/CustomElementRegistry-expected.txt: Added.
• web-platform-tests/custom-elements/CustomElementRegistry.html: Added.
• web-platform-tests/custom-elements/HTMLElement-constructor-expected.txt: Added.
• web-platform-tests/custom-elements/HTMLElement-constructor.html: Added.
• web-platform-tests/custom-elements/OWNERS: Added.
• web-platform-tests/custom-elements/adopted-callback-expected.txt: Added.
• web-platform-tests/custom-elements/adopted-callback.html: Added.
• web-platform-tests/custom-elements/attribute-changed-callback-expected.txt: Added.
• web-platform-tests/custom-elements/attribute-changed-callback.html: Added.
• web-platform-tests/custom-elements/connected-callbacks-expected.txt: Added.
• web-platform-tests/custom-elements/connected-callbacks.html: Added.
• web-platform-tests/custom-elements/custom-element-registry/define-expected.txt: Added.
• web-platform-tests/custom-elements/custom-element-registry/define.html: Added.
• web-platform-tests/custom-elements/custom-element-registry/w3c-import.log: Added.
• web-platform-tests/custom-elements/disconnected-callbacks-expected.txt: Added.
• web-platform-tests/custom-elements/disconnected-callbacks.html: Added.
• web-platform-tests/custom-elements/htmlconstructor/newtarget-expected.txt: Added.
• web-platform-tests/custom-elements/htmlconstructor/newtarget.html: Added.
• web-platform-tests/custom-elements/htmlconstructor/w3c-import.log: Added.
• web-platform-tests/custom-elements/reaction-timing-expected.txt: Added.
• web-platform-tests/custom-elements/reaction-timing.html: Added.
• web-platform-tests/custom-elements/reactions/Attr-expected.txt: Added.
• web-platform-tests/custom-elements/reactions/Attr.html: Added.
• web-platform-tests/custom-elements/reactions/ChildNode-expected.txt: Added.
• web-platform-tests/custom-elements/reactions/ChildNode.html: Added.
• web-platform-tests/custom-elements/reactions/DOMTokenList-expected.txt: Added.
• web-platform-tests/custom-elements/reactions/DOMTokenList.html: Added.
• web-platform-tests/custom-elements/reactions/Document-expected.txt: Added.
• web-platform-tests/custom-elements/reactions/Document.html: Added.
• web-platform-tests/custom-elements/reactions/Element-expected.txt: Added.
• web-platform-tests/custom-elements/reactions/Element.html: Added.
• web-platform-tests/custom-elements/reactions/NamedNodeMap-expected.txt: Added.
• web-platform-tests/custom-elements/reactions/NamedNodeMap.html: Added.
• web-platform-tests/custom-elements/reactions/Node-expected.txt: Added.
• web-platform-tests/custom-elements/reactions/Node.html: Added.
• web-platform-tests/custom-elements/reactions/ParentNode-expected.txt: Added.
• web-platform-tests/custom-elements/reactions/ParentNode.html: Added.
• web-platform-tests/custom-elements/reactions/Range-expected.txt: Added.
• web-platform-tests/custom-elements/reactions/Range.html: Added.
• web-platform-tests/custom-elements/reactions/resources/reactions.js: Added.
• web-platform-tests/custom-elements/reactions/resources/w3c-import.log: Added.
• web-platform-tests/custom-elements/reactions/w3c-import.log: Added.
• web-platform-tests/custom-elements/resources/custom-elements-helpers.js: Added.
• web-platform-tests/custom-elements/resources/document-types.js: Added.
• web-platform-tests/custom-elements/resources/empty-html-document.html: Added.
• web-platform-tests/custom-elements/resources/w3c-import.log: Added.
• web-platform-tests/custom-elements/w3c-import.log: Added.

LayoutTests:

Removed custom elements tests that have been upstreamed to and reimported from web-platform-tests.

• fast/custom-elements/CustomElementRegistry-expected.txt: Removed.
• fast/custom-elements/CustomElementRegistry.html: Removed.
• fast/custom-elements/HTMLElement-constructor-expected.txt: Removed.
• fast/custom-elements/HTMLElement-constructor.html: Removed.
• fast/custom-elements/adopted-callback-expected.txt: Removed.
• fast/custom-elements/adopted-callback.html: Removed.
• fast/custom-elements/attribute-changed-callback-expected.txt: Removed.
• fast/custom-elements/attribute-changed-callback.html: Removed.
• fast/custom-elements/connected-callbacks-expected.txt: Removed.
• fast/custom-elements/connected-callbacks.html: Removed.
• fast/custom-elements/disconnected-callbacks-expected.txt: Removed.
• fast/custom-elements/disconnected-callbacks.html: Removed.

5:07 PM Changeset in webkit [206837] by commit-queue@webkit.org

• 5 edits in trunk

[INTL] Implement Intl.getCanonicalLocales
​https://bugs.webkit.org/show_bug.cgi?id=162768

Patch by Andy VanWagoner <​thetalecrafter@gmail.com> on 2016-10-05
Reviewed by Benjamin Poulain.

Source/JavaScriptCore:

Implement Intl.getCanonicalLocales from ECMA 402 (3rd edition)
​http://ecma-international.org/ecma-402/3.0/index.html#sec-intl.getcanonicallocales

Reuse canonicalizeLocaleList and copy the results into a new JSArray.

• runtime/IntlObject.cpp:

(JSC::IntlObject::finishCreation):
(JSC::intlObjectFuncGetCanonicalLocales):

LayoutTests:

• js/intl-expected.txt: Added tests for Intl.getCanonicalLocales
• js/script-tests/intl.js: Added test for Intl.getCanonicalLocales

4:47 PM Changeset in webkit [206836] by msaboff@apple.com

• 2 edits in trunk/Source/JavaScriptCore

Bad ASSERT in ClonedArguments::createByCopyingFrom()
​https://bugs.webkit.org/show_bug.cgi?id=162988

Reviewed by Keith Miller.

Removed bogus assert.

• runtime/ClonedArguments.cpp:

(JSC::ClonedArguments::createByCopyingFrom):

4:19 PM Changeset in webkit [206835] by Ryan Haddad

• 2 edits in trunk/LayoutTests

Marking inspector/formatting/formatting-javascript.html as a flaky crash on mac-wk1.
​https://bugs.webkit.org/show_bug.cgi?id=162986

Unreviewed test gardening.

• platform/mac-wk1/TestExpectations:

4:12 PM Changeset in webkit [206834] by timothy_horton@apple.com

• 2 edits in trunk/Source/WebKit2

Fix the build.

• UIProcess/Cocoa/WebViewImpl.mm:

4:08 PM Changeset in webkit [206833] by timothy_horton@apple.com

• 11 edits in trunk

Avoid automatically re-taking snapshots for back-forward items that were never loaded into the view
​https://bugs.webkit.org/show_bug.cgi?id=162955
<rdar://problem/27659173>

Reviewed by Simon Fraser.

Make it possible for clients to control the snapshot for back-forward
items that are restored from session state without navigating to them,
by ensuring that we won't stomp on the snapshot that they explicitly take,
until a load occurs.

• UIProcess/WebBackForwardList.cpp:

(WebKit::WebBackForwardList::addItem):
(WebKit::WebBackForwardList::goToItem):

• UIProcess/ios/ViewGestureControllerIOS.mm:

(WebKit::ViewGestureController::beginSwipeGesture):

• UIProcess/mac/ViewGestureControllerMac.mm:

(WebKit::ViewGestureController::trackSwipeGesture):
Disambiguate explicit API-driven snapshot recording from automatic,
navigation-driven snapshot recording.

• UIProcess/Cocoa/WebViewImpl.mm:

(WebKit::WebViewImpl::saveBackForwardSnapshotForCurrentItem):
Get rid of the version of recordNavigationSnapshot() that doesn't take a
back-forward list item, and grab the current item at the one remaining caller.

• UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::recordAutomaticNavigationSnapshot):
(WebKit::WebPageProxy::recordNavigationSnapshot):
(WebKit::WebPageProxy::restoreFromSessionState):
(WebKit::WebPageProxy::didCommitLoadForFrame):

• UIProcess/WebPageProxy.h:

Rename m_suppressNavigationSnapshotting to m_suppressAutomaticNavigationSnapshotting,
and make it be only about automatic (navigation-driven) snapshots; it won't have
any impact on explicit snapshots forced by clients.

Set m_suppressAutomaticNavigationSnapshotting unconditionally when restoring
from session state, so that we won't start automatically snapshotting until
something has loaded in the view.

• TestWebKitAPI/Tests/WebKit2Cocoa/SnapshotStore.mm:

(-[SnapshotTestWKWebView init]):
(forceRepaintCallback):
(-[SnapshotTestWKWebView synchronouslyForceRepaint]):
(-[SnapshotTestWKWebView synchronouslyLoadTestPageAndForceRepaint:]):
(TEST):
(makeRedSquareView):
Add a test that restoring session state into a web view without navigating,
then explicitly snapshotting and navigating away, leaves the original snapshot alone.

Adjust the existing test, as well, to ensure that it will reliably fail
if the feature is broken. Use an explicitly added and removed red square
instead of scrolling, because we can't scroll in the restore-without-navigating case.

Stop trying to override the window scale, because it's not working (we're getting partial snapshots)
and isn't necessary; instead just multiply the expected value by the page scale.

(-[SnapshotTestWKWebView loadPageNamed:]): Deleted.

• TestWebKitAPI/mac/TestWKWebViewMac.h:
• TestWebKitAPI/mac/TestWKWebViewMac.mm:

(-[TestWKWebView synchronouslyLoadTestPageNamed:]):
Reorganize to reduce duplication.

3:30 PM Changeset in webkit [206832] by bshafiei@apple.com

• 1 copy in tags/Safari-603.1.8

New tag.

2:23 PM Changeset in webkit [206831] by Brent Fulgham

• 2 edits in trunk/Source/WebCore

Unreviewed build fix after r206830

• platform/graphics/win/FontPlatformDataWin.cpp: Add missing include

when building with Direct2D.

2:16 PM Changeset in webkit [206830] by Brent Fulgham

• 45 edits
  2 adds in trunk/Source/WebCore

[Win][Direct2D] Disable some CoreGraphics and CoreAnimation-based logic when building for D2D
​https://bugs.webkit.org/show_bug.cgi?id=162977

Reviewed by Alex Christensen.

Currently the Direct2D implementation does not support some of the accelerated compositing
features that require CoreAnimation. During this bring-up, we want to disable these routines
so that we can build and run in (mostly) unaccelerated drawing.

This patch mostly consists of adding "#if USE(CG)" and "#if USE(CA)" to files that are
specific to those implementations.

This change should have no impact on non-Direct2D ports.

No change in behavior for active ports.

• config.h: Turn of CG when building in Direct2D.
• css/CSSFontFaceSource.cpp:
• loader/cache/CachedFont.cpp:
• loader/cache/CachedSVGFont.cpp:
• page/CaptionUserPreferencesMediaAF.cpp:
• page/win/FrameCGWin.cpp:
• page/win/FrameDirect2D.cpp: Added.
• platform/graphics/Font.cpp:
• platform/graphics/FontCache.cpp:
• platform/graphics/FontPlatformData.cpp:
• platform/graphics/GraphicsContext.h:
• platform/graphics/Image.cpp:
• platform/graphics/avfoundation/cf/CDMSessionAVFoundationCF.cpp:
• platform/graphics/avfoundation/cf/MediaPlayerPrivateAVFoundationCF.cpp:
• platform/graphics/ca/GraphicsLayerCA.cpp:
• platform/graphics/ca/PlatformCALayer.cpp:
• platform/graphics/ca/TileController.cpp:
• platform/graphics/ca/TileGrid.cpp:
• platform/graphics/ca/TileGrid.h:
• platform/graphics/ca/win/CACFLayerTreeHost.cpp:
• platform/graphics/ca/win/LayerChangesFlusher.cpp:
• platform/graphics/ca/win/PlatformCAAnimationWin.cpp:
• platform/graphics/ca/win/PlatformCAAnimationWin.h:
• platform/graphics/ca/win/PlatformCAFiltersWin.cpp:
• platform/graphics/ca/win/PlatformCALayerWin.cpp:
• platform/graphics/ca/win/PlatformCALayerWinInternal.cpp:
• platform/graphics/ca/win/WKCACFViewLayerTreeHost.cpp:
• platform/graphics/ca/win/WebTiledBackingLayerWin.cpp:
• platform/graphics/cg/GradientCG.cpp:
• platform/graphics/cg/GraphicsContextCG.cpp:
• platform/graphics/cg/ImageBufferCG.cpp:
• platform/graphics/cg/ImageBufferDataCG.cpp:
• platform/graphics/cg/PDFDocumentImage.cpp:
• platform/graphics/cg/PatternCG.cpp:
• platform/graphics/cg/SubimageCacheWithTimer.cpp:
• platform/graphics/win/FontCGWin.cpp:
• platform/graphics/win/FontPlatformDataCGWin.cpp:
• platform/graphics/win/GlyphPageTreeNodeCGWin.cpp:
• platform/graphics/win/GlyphPageTreeNodeDirect2D.cpp:
• platform/graphics/win/GradientDirect2D.cpp:
• platform/graphics/win/PathDirect2D.cpp: Fix a few build problems I didn't notice

while building without Direct2D active.

• platform/graphics/win/SimpleFontDataCGWin.cpp:
• platform/graphics/win/SimpleFontDataWin.cpp:
• platform/win/DragImageCGWin.cpp:
• platform/win/DragImageDirect2D.cpp: Added.
• rendering/RenderMediaControls.cpp:

2:14 PM Changeset in webkit [206829] by timothy_horton@apple.com

• 12 edits
  2 adds in trunk

Make it possible to test ViewSnapshotStore behaviors
​https://bugs.webkit.org/show_bug.cgi?id=162983

Reviewed by Simon Fraser.

Add two pieces of SPI to make it easier to test ViewSnapshotStore:

• [WKWebView _disableBackForwardSnapshotVolatilityForTesting]

This makes it so that we don't make snapshots volatile after we
take them, so that they won't get purged while the test is running
and cause the test to be flaky.

• [WKBackForwardListItem _copySnapshotForTesting]

This returns a CGImage with the content of the given back-forward
item's current snapshot, if there is one.

• UIProcess/API/Cocoa/WKBackForwardListItem.mm:

(-[WKBackForwardListItem _copySnapshotForTesting]):

• UIProcess/API/Cocoa/WKBackForwardListItemInternal.h:
• UIProcess/API/Cocoa/WKBackForwardListItemPrivate.h: Added.

Add WKBackForwardListItem SPI to retrieve the snapshot for a given back-forward list item.

• UIProcess/API/Cocoa/WKWebViewPrivate.h:
• UIProcess/API/Cocoa/WKWebView.mm:

(-[WKWebView _disableBackForwardSnapshotVolatilityForTesting]):
Add WKWebView SPI to disable volatility of snapshots, for testing purposes.

• UIProcess/Cocoa/WebViewImpl.h:
• UIProcess/Cocoa/WebViewImpl.mm:

(WebKit::WebViewImpl::takeViewSnapshot):
Adopt ViewSnapshot::setVolatile instead of reaching into the surface.

• UIProcess/mac/ViewGestureControllerMac.mm:

(WebKit::ViewGestureController::removeSwipeSnapshot):
Adopt ViewSnapshot::setVolatile instead of reaching into the surface.

• UIProcess/mac/ViewSnapshotStore.h:

(WebKit::ViewSnapshotStore::setDisableSnapshotVolatilityForTesting):
(WebKit::ViewSnapshotStore::disableSnapshotVolatilityForTesting):

• UIProcess/mac/ViewSnapshotStore.mm:

(WebKit::ViewSnapshotStore::ViewSnapshotStore):
(WebKit::ViewSnapshot::setVolatile):
(WebKit::ViewSnapshot::asLayerContents):
Move all touching of snapshot volatility into the new setVolatile,
and short-circuit if the testing-only m_disableSnapshotVolatility bit is set

(WebKit::ViewSnapshot::asImageForTesting):
Make a CGImageRef for the aforementioned WKBackForwardListItem SPI.
This is currently not implemented for the iOS Simulator, because we
don't have access to the bits of the image in that case.

• TestWebKitAPI/TestWebKitAPI.xcodeproj/project.pbxproj:
• TestWebKitAPI/Tests/WebKit2Cocoa/SnapshotStore.mm: Added.

(-[SnapshotTestWKWebView init]):
(-[SnapshotTestWKWebView loadPageNamed:]):
(imagesAreEqual):
Add tests for existing basic snapshotting behavior:

• a back-forward item shouldn't have a snapshot until it is navigated away from

• causing a back-forward item to fall out of the back-forward list should remove the snapshot

• calling the explicit snapshotting SPI, then navigating away from the page, should result in the snapshot being retaken

2:12 PM Changeset in webkit [206828] by Chris Dumez

• 23 edits in trunk

KeyboardEvent.getModifierState() should support "CapsLock" modifier
​https://bugs.webkit.org/show_bug.cgi?id=162861

Reviewed by Darin Adler.

Source/WebCore:

Add support for "CapsLock" modifier on Mac / iOS for KeyboardEvent's
getModifierState() API, as per:

• ​https://w3c.github.io/uievents/#dom-keyboardevent-getmodifierstate

No new tests, updated existing test.

• dom/KeyboardEvent.cpp:

(WebCore::KeyboardEvent::KeyboardEvent):
(WebCore::KeyboardEvent::getModifierState):

• dom/KeyboardEvent.h:
• platform/PlatformEvent.h:
• platform/ios/PlatformEventFactoryIOS.mm:

(WebCore::modifiersForEvent):

• platform/mac/PlatformEventFactoryMac.mm:

(WebCore::keyIdentifierForKeyEvent):

Source/WebKit2:

Add support for CapsLock modifier on events.

• Shared/API/c/WKEvent.h:
• Shared/API/c/WKSharedAPICast.h:

(WebKit::toAPI):

• Shared/WebEventConversion.cpp:

(WebKit::WebKit2PlatformMouseEvent::WebKit2PlatformMouseEvent):
(WebKit::WebKit2PlatformWheelEvent::WebKit2PlatformWheelEvent):
(WebKit::WebKit2PlatformKeyboardEvent::WebKit2PlatformKeyboardEvent):
(WebKit::WebKit2PlatformTouchEvent::WebKit2PlatformTouchEvent):
(WebKit::WebKit2PlatformGestureEvent::WebKit2PlatformGestureEvent):

• Shared/efl/WebEventFactory.cpp:

(WebKit::toWebEventModifiers):

• Shared/ios/WebIOSEventFactory.mm:

(modifiersForEvent):

• Shared/mac/WebEventFactory.mm:

(WebKit::modifiersForEvent):

• UIProcess/API/Cocoa/WKNavigationAction.mm:

(toNSEventModifierFlags):

• UIProcess/Cocoa/WebAutomationSessionCocoa.mm:

(WebKit::WebAutomationSession::platformSimulateMouseInteraction):

Tools:

Add support for 'capsLockKey' modifier for EventSender.keyDown() so
that we can test this new modifier support for layout tests.

• TestWebKitAPI/mac/PlatformWebViewMac.mm:

(TestWebKitAPI::modifierFlagsForWKModifiers):

• WebKitTestRunner/InjectedBundle/EventSendingController.cpp:

(WTR::parseModifier):

• WebKitTestRunner/mac/EventSenderProxy.mm:

(WTR::buildModifierFlags):

LayoutTests:

Extend layout test coverage to cover 'Caps Lock' modifier.

• fast/events/special-key-events-in-input-text-expected.txt:
• fast/events/special-key-events-in-input-text.html:

1:46 PM Changeset in webkit [206827] by commit-queue@webkit.org

• 3 edits
  2 adds in trunk/Tools

Add MobileMiniBrowser tests for control center and looping fullscreen.
​https://bugs.webkit.org/show_bug.cgi?id=162748

Patch by Jeremy Jones <​jeremyj@apple.com> on 2016-10-05
Reviewed by Jer Noble.

Refactor existing MobileMiniBrowserTests and add new tests for

• fullscreen rotation
• fullscreen looping (failing)
• control center interruption (failing)

• MobileMiniBrowser/MobileMiniBrowser.xcodeproj/project.pbxproj:
• MobileMiniBrowser/MobileMiniBrowserFramework/Resources/looping2s.html: Added.
• MobileMiniBrowser/MobileMiniBrowserFramework/test2s.mp4: Added.
• MobileMiniBrowser/MobileMiniBrowserUITests/MobileMiniBrowserUITests.m:

(-[MobileMiniBrowserUITests setUp]):
(-[MobileMiniBrowserUITests waitToTapButtonNamed:forApp:]):
(-[MobileMiniBrowserUITests loadURL:]):
(-[MobileMiniBrowserUITests requireMinFPS:sampleDurationSeconds:message:]):
(-[MobileMiniBrowserUITests ensureFullscreenControls]):
(-[MobileMiniBrowserUITests timeForTimeString:]):
(-[MobileMiniBrowserUITests testBasicVideoPlayback]):
(-[MobileMiniBrowserUITests testBasicVideoFullscreen]):
(-[MobileMiniBrowserUITests testVideoFullscreenAndRotationAnimation]):
(-[MobileMiniBrowserUITests testVideoFullscreenControlCenter]):
(-[MobileMiniBrowserUITests testLoopingFullscreenLockup]):

1:21 PM Changeset in webkit [206826] by Michael Catanzaro

• 2 edits in trunk/Source/WebCore

[GTK] Improve OS X UA quirk
​https://bugs.webkit.org/show_bug.cgi?id=162613

Reviewed by Carlos Garcia Campos.

Include OS X version number in user agent, like Safari does.

• platform/gtk/UserAgentGtk.cpp:

(WebCore::buildUserAgentString):

1:20 PM Changeset in webkit [206825] by Michael Catanzaro

• 2 edits in trunk/Source/WebCore

[GTK] Improve comment in platformVersionForUAString
​https://bugs.webkit.org/show_bug.cgi?id=162612

Reviewed by Carlos Garcia Campos.

• platform/gtk/UserAgentGtk.cpp:

(WebCore::platformVersionForUAString):

12:02 PM Changeset in webkit [206824] by bshafiei@apple.com

• 5 edits in branches/safari-602.2.14.1-branch/Source

Versioning.

12:02 PM Changeset in webkit [206823] by bshafiei@apple.com

• 5 edits in branches/safari-602.2.14.0-branch/Source

Versioning.

12:00 PM Changeset in webkit [206822] by bshafiei@apple.com

• 5 edits in branches/safari-602-branch/Source

Versioning.

11:50 AM Changeset in webkit [206821] by Alan Bujtas

• 5 edits
  2 adds in trunk

Missing quotation mark when <q> gets reparented.
​https://bugs.webkit.org/show_bug.cgi?id=162940

Reviewed by Darin Adler.

Source/WebCore:

When an opening RenderQuote gets removed due to some change in the render tree,
we notify the closing RenderQuote so that it can update its content (") accordingly (and vice versa).
However when the same RenderQuote is added back to the render tree (aka reparenting), we fail to
let the other RenderQuote know about it and its content becomes stale.
This patch ensures that when either the opening or closing part of the quotation mark gets reparented,
we inform the other, corresponding opening/closing renderer about it.

Test: fast/css-generated-content/missing-quotes-with-dynamic-content.html

• dom/PseudoElement.cpp:

(WebCore::PseudoElement::didAttachRenderers):

• rendering/RenderQuote.cpp:

(WebCore::RenderQuote::insertedIntoTree):

• rendering/RenderQuote.h:

LayoutTests:

The position of the closing quotation mark is driven by the multicolumn code hence the cover div.

• fast/css-generated-content/missing-quotes-with-dynamic-content-expected.html: Added.
• fast/css-generated-content/missing-quotes-with-dynamic-content.html: Added.

11:42 AM Changeset in webkit [206820] by bshafiei@apple.com

• 5 edits in trunk/Source

Versioning.

11:27 AM Changeset in webkit [206819] by achristensen@apple.com

• 4 edits in trunk

URLParser should correctly strip unnecessary 0's in IPv6 addresses
​https://bugs.webkit.org/show_bug.cgi?id=162979

Reviewed by Brady Eidson.

Source/WebCore:

Covered by new API tests.

• platform/URLParser.cpp:

(WebCore::URLParser::parseIPv6Host):
Strip leading 0's and any 0's before the last 0 if the value is 0.

Tools:

• TestWebKitAPI/Tests/WebCore/URLParser.cpp:

(TestWebKitAPI::TEST_F):

11:25 AM Changeset in webkit [206818] by achristensen@apple.com

• 4 edits in trunk

UTF-8 encode queries of nonspecial and websocket schemes
​https://bugs.webkit.org/show_bug.cgi?id=162956

Reviewed by Geoffrey Garen and Brady Eidson.

Source/WebCore:

The URL spec says in the query state:
'If url is not special or url's scheme is either "ws" or "wss", set encoding to UTF-8.'
This should be determined as soon as we are done parsing the scheme.

Covered by new API tests.
This also fixes tests like fast/loader/javascript-url-encoding-2.html when URLParser is enabled.

• platform/URLParser.cpp:

(WebCore::isValidSchemeCharacter):
Renamed Scheme to ValidScheme so I can use Scheme as the name of an enum class in the same namespace.
(WebCore::isSpecial):
(WebCore::scheme):
Separate functionality so we can have different behavior for different sets of
ws and wss schemes, special, and non-special schemes.
(WebCore::URLParser::copyURLPartsUntil):
(WebCore::URLParser::parse):
Set isUTF8Encoding to true when we finish parsing the scheme if the scheme is ws, wss, or non-special,
according to spec. This also matches existing behavior. This way we will already know whether to go
into UTF8Query or NonUTF8Query state when we see a '?'.
(WebCore::isSpecialScheme): Deleted.

Tools:

• TestWebKitAPI/Tests/WebCore/URLParser.cpp:

(TestWebKitAPI::TEST_F):

11:19 AM Changeset in webkit [206817] by achristensen@apple.com

• 4 edits in trunk

Prepare to enable URLParser
​https://bugs.webkit.org/show_bug.cgi?id=162974

Reviewed by Brady Eidson.

Source/WebCore:

• platform/URL.cpp:

(WebCore::URL::serialize):
When we enable the URLParser, this attempt to make serialized URLs more spec compliant
won't be needed any more.

Tools:

• TestWebKitAPI/Tests/WebCore/URLParser.cpp:

(TestWebKitAPI::TEST_F):
Added some tests that are unrelated to the code change, but these are
reflected in changed LayoutTest results when enabling the URLParser,
and they weren't in the URLParser tests yet. They should be.

10:21 AM Changeset in webkit [206816] by Brent Fulgham

• 6 edits in trunk/Source/WebKit/win

[Win][Direct2D] Connect WebCore/WebKit Drawing
​https://bugs.webkit.org/show_bug.cgi?id=162924

Reviewed by Alex Christensen.

Update the Windows WebKit layer to optionally use the Direct2D
backend.

• FullscreenVideoController.cpp: Disable some CoreAnimation-based

accelerated compositing logic.

• FullscreenVideoController.h:
• WebFrame.cpp:

(WebFrame::spoolPages): Disable in Direct2D.

• WebView.cpp:

(WebView::ensureBackingStore): Add Direct2D logic.
(WebView::scrollBackingStore): Ditto.
(WebView::sizeChanged): Ditto.
(WebView::paintWithDirect2D): Added.
(WebView::WebViewWndProc): Call Direct2D paint routine.

• WebView.h:

10:18 AM Changeset in webkit [206815] by Brent Fulgham

• 4 edits
  2 adds in trunk/Source/WebCore

[Win][Direct2D] Add initial D2D Path and Gradient implementation
​https://bugs.webkit.org/show_bug.cgi?id=162923

Reviewed by Alex Christensen.

This patch lands a set of new files that implements initial Path
and Gradient features on Windows using Direct2D.

No new tests until complete backend lands.

• platform/graphics/Gradient.h:
• platform/graphics/Path.cpp:
• platform/graphics/Path.h:
• platform/graphics/win/GradientDirect2D.cpp: Added.
• platform/graphics/win/PathDirect2D.cpp: Added.

9:46 AM Changeset in webkit [206814] by commit-queue@webkit.org

• 7 edits in trunk/Source/WebCore

[Readable Streams API] Align function names with spec
​https://bugs.webkit.org/show_bug.cgi?id=162967

Patch by Romain Bellessort <​romain.bellessort@crf.canon.fr> on 2016-10-05
Reviewed by Darin Adler.

Align function names with spec. Behaviour was already in line with spec, but previous function names had
sometimes been preserved to track the changes more easily. This led to some inconsistencies (for instance,
enqueueInReadableStream was still used even though it works on a controller instead of a stream) and it
also made it more difficult to compare spec and implementation.

No change in behaviour.

• Modules/fetch/FetchInternals.js:

(consumeStream): Updated with new function names.

• Modules/fetch/FetchResponse.js:

(clone): Updated with new function names.

• Modules/streams/ReadableStream.js:

(tee): Updated with new function names.

• Modules/streams/ReadableStreamDefaultController.js:

(enqueue): Updated with new function names.
(error): Updated with new function names.

• Modules/streams/ReadableStreamDefaultReader.js:

(read): Updated with new function names.

• Modules/streams/ReadableStreamInternals.js:

(privateInitializeReadableStreamDefaultController): Updated with new function names.
(readableStreamDefaultControllerError): Updated with new function names.
(readableStreamTee): Updated with new function names.
(readableStreamTeePullFunction): Updated with new function names.
(readableStreamTeeBranch2CancelFunction): Updated with new function names.
(readableStreamError): Updated with new function names.
(readableStreamCancel): Updated with new function names.
(readableStreamDefaultControllerPull): Updated with new function names.
(readableStreamDefaultControllerClose): Updated with new function names.
(readableStreamClose): Updated with new function names.
(readableStreamDefaultControllerEnqueue): Updated with new function names.
(readableStreamDefaultReaderRead): Updated with new function names.

9:29 AM Changeset in webkit [206813] by dbates@webkit.org

• 2 edits
  2 moves in trunk/LayoutTests

[WK2][NetworkSession] Ping requests should follow redirects if applicable
​https://bugs.webkit.org/show_bug.cgi?id=162580
<rdar://problem/28631274>

Rename test anchor-ping-and-do-not-follow-redirect-when-sending-ping.html to anchor-ping-and-follow-redirect-when-sending-ping.html
to reflect what it is actually testing - that we follow a redirect when sending a ping for a <a ping>.

• http/tests/navigation/ping-attribute/anchor-ping-and-follow-redirect-when-sending-ping-expected.txt: Renamed from LayoutTests/http/tests/navigation/ping-attribute/anchor-ping-and-do-not-follow-redirect-when-sending-ping-expected.txt.
• http/tests/navigation/ping-attribute/anchor-ping-and-follow-redirect-when-sending-ping.html: Renamed from LayoutTests/http/tests/navigation/ping-attribute/anchor-ping-and-do-not-follow-redirect-when-sending-ping.html.
• http/tests/navigation/resources/save-ping-and-redirect-to-save-ping.php: Only append the query string

if we have a non-empty query string to avoid appending a superfluous "?" on redirect.

9:16 AM WebKitGTK/2.14.x edited by Michael Catanzaro

(diff)

9:09 AM Changeset in webkit [206812] by Chris Dumez

• 7 edits in trunk/Source/WebCore

[Bindings] Declare dictionary / enumeration template specializations in the header
​https://bugs.webkit.org/show_bug.cgi?id=162929

Reviewed by Darin Adler.

Declare dictionary / enumeration template specializations in the header
so that:

• They can be used from custom bindings code.
• Dictionaries / enumerations can be used to their own IDL files to share them (Bug 162912).

No new tests, rebaselined bindings tests.

• WebCore.xcodeproj/project.pbxproj:
• bindings/generic/IDLTypes.h:
• bindings/js/JSDOMConvert.h:
• bindings/scripts/CodeGeneratorJS.pm:

(GenerateInterface):
(GenerateEnumerationImplementationContent):
(GenerateEnumerationHeaderContent):
(GenerateDictionaryHeaderContent):
(GenerateHeader):
(GenerateCallbackHeader):
(GenerateCallbackImplementation):
(GenerateDefaultValue): Deleted.
(GenerateDictionaryImplementationContent): Deleted.
(GenerateParametersCheck): Deleted.

• bindings/scripts/test/JS/JSTestObj.cpp:
• bindings/scripts/test/JS/JSTestObj.h:

9:02 AM Changeset in webkit [206811] by zandobersek@gmail.com

• 92 edits in trunk

Rename ENABLE_ENCRYPTED_MEDIA_V2 to ENABLE_LEGACY_ENCRYPTED_MEDIA
​https://bugs.webkit.org/show_bug.cgi?id=162903

Reviewed by Alex Christensen.

Rename build guards for the remaining implementation of the legacy EME API
to ENABLE_LEGACY_ENCRYPTED_MEDIA. This will allow for the future implementation
of the near-finished API to be guarded with the simple ENABLE_ENCRYPTED_MEDIA guards.

.:

• Source/cmake/OptionsEfl.cmake:
• Source/cmake/OptionsMac.cmake:
• Source/cmake/OptionsWin.cmake:
• Source/cmake/WebKitFeatures.cmake:
• Source/cmake/tools/vsprops/FeatureDefines.props:
• Source/cmake/tools/vsprops/FeatureDefinesCairo.props:

Source/JavaScriptCore:

• Configurations/FeatureDefines.xcconfig:

Source/WebCore:

No new tests -- no change in behavior.

• CMakeLists.txt:
• Configurations/FeatureDefines.xcconfig:
• DerivedSources.cpp:
• DerivedSources.make:
• Modules/encryptedmedia/CDM.cpp:
• Modules/encryptedmedia/CDM.h:
• Modules/encryptedmedia/CDMPrivate.h:
• Modules/encryptedmedia/CDMPrivateClearKey.cpp:
• Modules/encryptedmedia/CDMPrivateClearKey.h:
• Modules/encryptedmedia/CDMPrivateMediaPlayer.cpp:
• Modules/encryptedmedia/CDMPrivateMediaPlayer.h:
• Modules/encryptedmedia/CDMSessionClearKey.cpp:
• Modules/encryptedmedia/CDMSessionClearKey.h:
• Modules/encryptedmedia/MediaKeyMessageEvent.cpp:
• Modules/encryptedmedia/MediaKeyMessageEvent.h:
• Modules/encryptedmedia/MediaKeyMessageEvent.idl:
• Modules/encryptedmedia/MediaKeyNeededEvent.cpp:
• Modules/encryptedmedia/MediaKeyNeededEvent.h:
• Modules/encryptedmedia/MediaKeyNeededEvent.idl:
• Modules/encryptedmedia/MediaKeySession.cpp:
• Modules/encryptedmedia/MediaKeySession.h:
• Modules/encryptedmedia/MediaKeySession.idl:
• Modules/encryptedmedia/MediaKeys.cpp:
• Modules/encryptedmedia/MediaKeys.h:
• Modules/encryptedmedia/MediaKeys.idl:
• bindings/js/JSDictionary.cpp:
• bindings/js/JSDictionary.h:
• dom/Element.idl:
• dom/EventNames.in:
• dom/EventTargetFactory.in:
• html/HTMLMediaElement.cpp:

(WebCore::HTMLMediaElement::~HTMLMediaElement):
(WebCore::HTMLMediaElement::webkitGenerateKeyRequest):
(WebCore::HTMLMediaElement::webkitAddKey):

• html/HTMLMediaElement.h:
• html/HTMLMediaElement.idl:
• html/MediaError.h:
• html/MediaError.idl:
• html/MediaKeyError.h:
• html/MediaKeyError.idl:
• page/Settings.h:
• platform/graphics/CDMSession.h:
• platform/graphics/MediaPlayer.cpp:
• platform/graphics/MediaPlayer.h:
• platform/graphics/MediaPlayerPrivate.h:
• platform/graphics/avfoundation/CDMPrivateMediaSourceAVFObjC.h:
• platform/graphics/avfoundation/CDMPrivateMediaSourceAVFObjC.mm:
• platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.cpp:
• platform/graphics/avfoundation/MediaPlayerPrivateAVFoundation.h:
• platform/graphics/avfoundation/cf/CDMSessionAVFoundationCF.cpp:
• platform/graphics/avfoundation/cf/CDMSessionAVFoundationCF.h:
• platform/graphics/avfoundation/cf/MediaPlayerPrivateAVFoundationCF.cpp:

(WebCore::MediaPlayerPrivateAVFoundationCF::supportsKeySystem):
(WebCore::AVFWrapper::shouldWaitForLoadingOfResource):

• platform/graphics/avfoundation/cf/MediaPlayerPrivateAVFoundationCF.h:
• platform/graphics/avfoundation/objc/CDMSessionAVContentKeySession.h:
• platform/graphics/avfoundation/objc/CDMSessionAVContentKeySession.mm:
• platform/graphics/avfoundation/objc/CDMSessionAVFoundationObjC.h:
• platform/graphics/avfoundation/objc/CDMSessionAVFoundationObjC.mm:
• platform/graphics/avfoundation/objc/CDMSessionAVStreamSession.h:
• platform/graphics/avfoundation/objc/CDMSessionAVStreamSession.mm:
• platform/graphics/avfoundation/objc/CDMSessionMediaSourceAVFObjC.h:
• platform/graphics/avfoundation/objc/CDMSessionMediaSourceAVFObjC.mm:
• platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.h:
• platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:

(WebCore::MediaPlayerPrivateAVFoundationObjC::supportsKeySystem):
(WebCore::MediaPlayerPrivateAVFoundationObjC::shouldWaitForLoadingOfResource):
(WebCore::playerKVOProperties):
(-[WebCoreAVFMovieObserver observeValueForKeyPath:ofObject:change:context:]):

• platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaSourceAVFObjC.h:
• platform/graphics/avfoundation/objc/MediaPlayerPrivateMediaSourceAVFObjC.mm:
• platform/graphics/avfoundation/objc/MediaSourcePrivateAVFObjC.h:
• platform/graphics/avfoundation/objc/MediaSourcePrivateAVFObjC.mm:
• platform/graphics/avfoundation/objc/SourceBufferPrivateAVFObjC.mm:

(WebCore::SourceBufferPrivateAVFObjC::willProvideContentKeyRequestInitializationDataForTrackID):
(WebCore::SourceBufferPrivateAVFObjC::didProvideContentKeyRequestInitializationDataForTrackID):

• testing/Internals.cpp:
• testing/Internals.h:
• testing/Internals.idl:
• testing/MockCDM.cpp:
• testing/MockCDM.h:

Source/WebKit/mac:

• Configurations/FeatureDefines.xcconfig:
• WebView/WebPreferences.mm:

(+[WebPreferences initialize]):

• WebView/WebView.mm:

(-[WebView _preferencesChanged:]):

Source/WebKit2:

• Configurations/FeatureDefines.xcconfig:
• WebProcess/WebPage/WebPage.cpp:

(WebKit::m_userInterfaceLayoutDirection):

• WebProcess/WebProcess.cpp:

(WebKit::m_resourceLoadStatisticsStorage):

Source/WTF:

• wtf/FeatureDefines.h:

Tools:

• TestWebKitAPI/Configurations/FeatureDefines.xcconfig: Also remove the

ENABLE_IOS_TEXT_AUTOSIZING feature define that was renamed in r206395 and was
throwing up style checker errors.

7:26 AM Changeset in webkit [206810] by dbates@webkit.org

• 20 edits
  7 adds in trunk

[WK2][NetworkSession] Ping requests should follow redirects if applicable
​https://bugs.webkit.org/show_bug.cgi?id=162580
<rdar://problem/28631274>

Reviewed by Alex Christensen.

Source/WebKit2:

Implement support for following redirects, if applicable, of a ping request when using the
Network Session (ENABLE(NETWORK_SESSION)) code path in WebKit2. For ping requests that are
in response to hyperlink auditing (e.g. <a ping>) we should follow redirect responses.
For Content Security Policy and XSS Auditor violation reports we should not follow redirect
responses.

Currently we always ignore redirects for ping requests that are sent using the Network Session
code path in WebKit2. This behavior disagrees with the behavior described in section
"Hyperlink auditing" of the HTML standard, <​https://html.spec.whatwg.org/multipage/semantics.html#hyperlink-auditing> (27 September 2016),
as well as the behavior in Legacy WebKit and WebKit2 with Network Session disabled. We should
follow redirects for ping requests initiated as part of hyperlink auditing to more closely
conform to the HTML standard and match the behavior in Legacy WebKit and WebKit2 with Network
Session disabled.

• NetworkProcess/PingLoad.h:

LayoutTests:

Add tests to ensure that we follow redirects for ping requests that are initiated as part
of hyperlink auditing.

Modified http/tests/navigation/resources/save-Ping.php to include the host and request URI
information in the saved report and updated expected results for existing tests as a result
of this change.

• http/tests/contentextensions/block-cookies-in-csp-report-expected.txt:
• http/tests/contentextensions/block-cookies-in-image-load-in-onunload-expected.txt:
• http/tests/contentextensions/block-cookies-in-ping-expected.txt:
• http/tests/contentextensions/hide-on-csp-report-expected.txt:
• http/tests/contentextensions/hide-on-ping-expected.txt:
• http/tests/contentextensions/hide-on-ping-with-ping-that-redirects-expected.txt: Added.
• http/tests/contentextensions/hide-on-ping-with-ping-that-redirects.html: Added.
• http/tests/contentextensions/hide-on-ping-with-ping-that-redirects.html.json: Added.
• http/tests/contentextensions/resources/save-ping-and-redirect-to-save-ping.php: Added.
• http/tests/contentextensions/resources/save-ping.php:
• http/tests/navigation/image-load-in-pagehide-handler-expected.txt:
• http/tests/navigation/image-load-in-unload-handler-expected.txt:
• http/tests/navigation/ping-attribute/anchor-cookie-expected.txt:
• http/tests/navigation/ping-attribute/anchor-cross-origin-expected.txt:
• http/tests/navigation/ping-attribute/anchor-cross-origin-from-https-expected.txt:
• http/tests/navigation/ping-attribute/anchor-ping-and-do-not-follow-redirect-when-sending-ping-expected.txt: Added.
• http/tests/navigation/ping-attribute/anchor-ping-and-do-not-follow-redirect-when-sending-ping.html: Added.
• http/tests/navigation/ping-attribute/anchor-same-origin-expected.txt:
• http/tests/navigation/ping-attribute/area-cookie-expected.txt:
• http/tests/navigation/ping-attribute/area-cross-origin-expected.txt:
• http/tests/navigation/ping-attribute/area-cross-origin-from-https-expected.txt:
• http/tests/navigation/ping-attribute/area-same-origin-expected.txt:
• http/tests/navigation/resources/save-Ping.php:
• http/tests/navigation/resources/save-ping-and-redirect-to-save-ping.php: Added.

7:22 AM Changeset in webkit [206809] by dbates@webkit.org

• 59 edits
  7 adds in trunk

Do not follow redirects when sending violation report
​https://bugs.webkit.org/show_bug.cgi?id=162520
<rdar://problem/27957639>

Reviewed by Alex Christensen.

Source/WebCore:

Do not follow redirects when sending a Content Security Policy or XSS Auditor violation report
as redirects can be used to forward report details to a third-party.

This changes makes WebKit more closely conform to the reporting requirements in section Reporting
of the Content Security Level 2 standard: <​https://w3c.github.io/webappsec-csp/2/#violation-reports>
(Editor's Draft, 25 April 2016).

Tests: http/tests/security/contentSecurityPolicy/report-blocked-uri-and-do-not-follow-redirect-when-sending-report.php

http/tests/security/xssAuditor/report-script-tag-and-do-not-follow-redirect-when-sending-report.html
http/tests/security/xssAuditor/report-script-tag-full-block-and-do-not-follow-redirect-when-sending-report.html

• loader/LoaderStrategy.h: Modified createPingHandle() to take a boolean, shouldFollowRedirects,

whether to follow redirect responses for a ping request.

• loader/PingLoader.cpp:

(WebCore::PingLoader::loadImage): Pass ShouldFollowRedirects::Yes to PingLoader::startPingLoad to
keep our current behavior.
(WebCore::PingLoader::sendPing): Ditto. Note our current behavior of following redirects matches
the behavior described in the section "Hyperlink auditing" of the HTML standard:
<​https://html.spec.whatwg.org/multipage/semantics.html#hyperlink-auditing> (23 September 2016).
(WebCore::PingLoader::sendViolationReport): Pass ShouldFollowRedirects::No to PingLoader::startPingLoad
so that we do not follow redirects when sending a violation report.
(WebCore::PingLoader::startPingLoad): Modified to take argument shouldFollowRedirects whether to
follow redirect responses for a ping request.

• loader/PingLoader.h:
• platform/network/PingHandle.h: Add boolean m_shouldFollowRedirects. I grouped this boolean with

the existing boolean, m_shouldUseCredentialStorage, as opposed to appending to the end of the class
definition to avoid increasing object size as clang will coalesces the two bools into a single
machine word. Override ResourceHandleClient::willSendRequest() and ResourceHandleClient::willSendRequestAsync()
to follow a redirect, if applicable.

Source/WebKit:

Update implementation of legacy WebKit loader strategy to pass through a boolean,
shouldFollowRedirects, to PingHandle as to whether to follow redirect responses
for a ping request.

• WebCoreSupport/WebResourceLoadScheduler.cpp:

(WebResourceLoadScheduler::createPingHandle):

• WebCoreSupport/WebResourceLoadScheduler.h:

Source/WebKit2:

Update the non-Network Session WebKit2 implementation to follow redirect responses for a ping
request, if applicable. I did not update the Network Session implementation at this time. I
will fix it in <​https://bugs.webkit.org/show_bug.cgi?id=162580>.

• NetworkProcess/NetworkConnectionToWebProcess.cpp:

(WebKit::NetworkConnectionToWebProcess::loadPing): Tell PingHandle whether to follow redirects.

• NetworkProcess/NetworkLoadParameters.h:
• NetworkProcess/NetworkResourceLoadParameters.cpp:

(WebKit::NetworkResourceLoadParameters::encode): Encode NetworkResourceLoadParameters::shouldFollowRedirects.
(WebKit::NetworkResourceLoadParameters::decode): Decode NetworkResourceLoadParameters::shouldFollowRedirects.

• NetworkProcess/PingLoad.h: Added FIXME comment to implement support for following redirects,

if applicable (for hyperlink auditing). See <​https://bugs.webkit.org/show_bug.cgi?id=162580>
for more details.

• WebProcess/Network/WebLoaderStrategy.cpp:

(WebKit::WebLoaderStrategy::createPingHandle): Modified to take a boolean whether to follow
redirects responses and set NetworkResourceLoadParameters::shouldFollowRedirects as appropriate.

• WebProcess/Network/WebLoaderStrategy.h:

LayoutTests:

Add tests for Content Security Policy and XSS Auditor to ensure that we do not follow redirects
when sending a violation report. Modified http/tests/security/contentSecurityPolicy/resources/save-report.php
to save the URL of the original ping request and conditionally clear cookies. Modified
http/tests/security/xssAuditor/resources/echo-intertag.pl to support testing for the XSS Auditor.
These changes together with the existing HTTP Host information that is saved with the report we can detect
if a redirect occurred when saving a report.

Updated expected results of existing tests now that we emit the URL of the ping request in the saved report.

• http/tests/security/contentSecurityPolicy/1.1/frame-ancestors/report-frame-ancestors-cross-origin-expected.txt:
• http/tests/security/contentSecurityPolicy/1.1/frame-ancestors/report-frame-ancestors-cross-origin-https-expected.txt:
• http/tests/security/contentSecurityPolicy/1.1/frame-ancestors/report-frame-ancestors-same-origin-expected.txt:
• http/tests/security/contentSecurityPolicy/1.1/frame-ancestors/report-frame-ancestors-same-origin-https-expected.txt:
• http/tests/security/contentSecurityPolicy/1.1/report-uri-effective-directive-expected.txt:
• http/tests/security/contentSecurityPolicy/1.1/script-blocked-sends-multiple-reports-expected.txt:
• http/tests/security/contentSecurityPolicy/1.1/scripthash-allowed-by-enforced-policy-and-blocked-by-report-policy-expected.txt:
• http/tests/security/contentSecurityPolicy/1.1/scripthash-allowed-by-enforced-policy-and-blocked-by-report-policy2-expected.txt:
• http/tests/security/contentSecurityPolicy/1.1/scripthash-allowed-by-legacy-enforced-policy-and-blocked-by-report-policy-expected.txt:
• http/tests/security/contentSecurityPolicy/1.1/scripthash-allowed-by-legacy-enforced-policy-and-blocked-by-report-policy2-expected.txt:
• http/tests/security/contentSecurityPolicy/1.1/scripthash-blocked-by-enforced-policy-and-allowed-by-report-policy-expected.txt:
• http/tests/security/contentSecurityPolicy/1.1/scripthash-blocked-by-legacy-enforced-policy-and-allowed-by-report-policy-expected.txt:
• http/tests/security/contentSecurityPolicy/1.1/scripthash-blocked-by-legacy-enforced-policy-and-blocked-by-report-policy-expected.txt:
• http/tests/security/contentSecurityPolicy/1.1/scripthash-blocked-by-legacy-enforced-policy-and-blocked-by-report-policy2-expected.txt:
• http/tests/security/contentSecurityPolicy/1.1/scriptnonce-blocked-by-enforced-policy-and-allowed-by-report-policy-expected.txt:
• http/tests/security/contentSecurityPolicy/1.1/scriptnonce-blocked-by-legacy-enforced-policy-and-allowed-by-report-policy-expected.txt:
• http/tests/security/contentSecurityPolicy/1.1/scriptnonce-blocked-by-legacy-enforced-policy-and-blocked-by-report-policy-expected.txt:
• http/tests/security/contentSecurityPolicy/1.1/scriptnonce-blocked-by-legacy-enforced-policy-and-blocked-by-report-policy2-expected.txt:
• http/tests/security/contentSecurityPolicy/report-and-enforce-expected.txt:
• http/tests/security/contentSecurityPolicy/report-blocked-data-uri-expected.txt:
• http/tests/security/contentSecurityPolicy/report-blocked-file-uri-expected.txt:
• http/tests/security/contentSecurityPolicy/report-blocked-uri-and-do-not-follow-redirect-when-sending-report-expected.txt: Added.
• http/tests/security/contentSecurityPolicy/report-blocked-uri-and-do-not-follow-redirect-when-sending-report.php: Added.
• http/tests/security/contentSecurityPolicy/report-blocked-uri-cross-origin-expected.txt:
• http/tests/security/contentSecurityPolicy/report-blocked-uri-expected.txt:
• http/tests/security/contentSecurityPolicy/report-cross-origin-no-cookies-expected.txt:
• http/tests/security/contentSecurityPolicy/report-cross-origin-no-cookies-when-private-browsing-enabled-expected.txt:
• http/tests/security/contentSecurityPolicy/report-cross-origin-no-cookies-when-private-browsing-toggled-expected.txt:
• http/tests/security/contentSecurityPolicy/report-only-expected.txt:
• http/tests/security/contentSecurityPolicy/report-only-from-header-expected.txt:
• http/tests/security/contentSecurityPolicy/report-only-upgrade-insecure-expected.txt:
• http/tests/security/contentSecurityPolicy/report-same-origin-no-cookies-when-private-browsing-toggled-expected.txt:
• http/tests/security/contentSecurityPolicy/report-same-origin-with-cookies-expected.txt:
• http/tests/security/contentSecurityPolicy/report-same-origin-with-cookies-when-private-browsing-enabled-expected.txt:
• http/tests/security/contentSecurityPolicy/report-status-code-zero-when-using-https-expected.txt:
• http/tests/security/contentSecurityPolicy/report-uri-expected.txt:
• http/tests/security/contentSecurityPolicy/report-uri-from-child-frame-expected.txt:
• http/tests/security/contentSecurityPolicy/report-uri-from-inline-javascript-expected.txt:
• http/tests/security/contentSecurityPolicy/report-uri-from-javascript-expected.txt:
• http/tests/security/contentSecurityPolicy/report-uri-scheme-relative-expected.txt:
• http/tests/security/contentSecurityPolicy/resources/save-report-and-redirect-to-save-report.php: Added.
• http/tests/security/contentSecurityPolicy/resources/save-report.php:
• http/tests/security/xssAuditor/report-script-tag-and-do-not-follow-redirect-when-sending-report-expected.txt: Added.
• http/tests/security/xssAuditor/report-script-tag-and-do-not-follow-redirect-when-sending-report.html: Added.
• http/tests/security/xssAuditor/report-script-tag-expected.txt:
• http/tests/security/xssAuditor/report-script-tag-full-block-and-do-not-follow-redirect-when-sending-report-expected.txt: Added.
• http/tests/security/xssAuditor/report-script-tag-full-block-and-do-not-follow-redirect-when-sending-report.html: Added.
• http/tests/security/xssAuditor/report-script-tag-full-block-expected.txt:
• http/tests/security/xssAuditor/report-script-tag-replace-state-expected.txt:
• http/tests/security/xssAuditor/resources/echo-intertag.pl:

3:51 AM Changeset in webkit [206808] by Csaba Osztrogonác

• 2 edits in trunk/Source/JavaScriptCore

ARM EABI buildfix after r206778
​https://bugs.webkit.org/show_bug.cgi?id=162964

Unreviewed trivial fix.

• jit/CCallHelpers.h:

(JSC::CCallHelpers::setupArgumentsWithExecState):

2:43 AM Changeset in webkit [206807] by Carlos Garcia Campos

• 7 edits in trunk/Source

[SOUP] Move global TLS errors handling from ResourceHandle to SoupNetworkSession
​https://bugs.webkit.org/show_bug.cgi?id=162910

Reviewed by Alex Christensen.

Source/WebCore:

So that it will be shared with network session code. Also remove setHostAllowsAnyHTTPSCertificate() that we have
never actually used.

• platform/network/ResourceHandle.h:
• platform/network/soup/ResourceHandleSoup.cpp:

(WebCore::tlsErrorsChangedCallback): Use SoupNetworkSession::checkTLSErrors().

• platform/network/soup/SoupNetworkSession.cpp:

(WebCore::HostTLSCertificateSet::add):
(WebCore::HostTLSCertificateSet::contains):
(WebCore::HostTLSCertificateSet::computeCertificateHash):
(WebCore::clientCertificates):
(WebCore::SoupNetworkSession::setShouldIgnoreTLSErrors):
(WebCore::SoupNetworkSession::checkTLSErrors):
(WebCore::SoupNetworkSession::allowSpecificHTTPSCertificateForHost):

• platform/network/soup/SoupNetworkSession.h:

Source/WebKit2:

Use SoupNetworkSession instead of ResourceHandle.

• NetworkProcess/soup/NetworkProcessSoup.cpp:

(WebKit::NetworkProcess::setIgnoreTLSErrors):
(WebKit::NetworkProcess::allowSpecificHTTPSCertificateForHost):

2:23 AM Changeset in webkit [206806] by Carlos Garcia Campos

• 2 edits in trunk/Source/WebKit2

Unreviewed. Fix GTK+ build with USE_REDIRECTED_XCOMPOSITE_WINDOW disabled.

• UIProcess/AcceleratedDrawingAreaProxy.cpp:

(WebKit::AcceleratedDrawingAreaProxy::setNativeSurfaceHandleForCompositing):

12:08 AM Changeset in webkit [206805] by Carlos Garcia Campos

• 6 edits in trunk/Source/WebCore

[SOUP] Move request HTTP body handling to ResourceRequestSoup and simplify it
​https://bugs.webkit.org/show_bug.cgi?id=162891

Reviewed by Michael Catanzaro.

We are currently adding the HTTP body to the message in ResourceHandleSoup, but we could do it as part of the
ResourceRequest::updateSoupMessage() method, since the HTTP body is indeed part of the request. The code can be
also simplified, especially the handling of files, by using SharedBuffer that already mmaps the file when
possible, instead of using GMappedFile API.

• platform/SharedBuffer.h: Add createSoupBuffer() method.
• platform/network/soup/ResourceHandleSoup.cpp:

(WebCore::createSoupMessageForHandleAndRequest):
(WebCore::milisecondsSinceRequest): Deleted.
(WebCore::ResourceHandle::didStartRequest): Deleted.
(WebCore::startingCallback): Deleted.
(WebCore::networkEventCallback): Deleted.
(WebCore::ResourceHandle::start): Deleted.
(WebCore::ResourceHandle::releaseForDownload): Deleted.
(WebCore::ResourceHandle::timeoutFired): Deleted.

• platform/network/soup/ResourceRequest.h:
• platform/network/soup/ResourceRequestSoup.cpp:

(WebCore::appendEncodedBlobItemToSoupMessageBody): Helper method to append the blob to the request message body.
(WebCore::ResourceRequest::updateSoupMessageBody): Add HTTPBody to the request message body.
(WebCore::ResourceRequest::updateSoupMessage): Call updateSoupMessageBody() to also update the body.

• platform/soup/SharedBufferSoup.cpp:

(WebCore::SharedBuffer::createSoupBuffer): Returns a new SoupBuffer for the SharedBuffer.

Oct 4, 2016:

11:16 PM Changeset in webkit [206804] by sbarati@apple.com

• 27 edits
  2 adds in trunk

String.prototype.toLowerCase should be a DFG/FTL intrinsic
​https://bugs.webkit.org/show_bug.cgi?id=162887

Reviewed by Filip Pizlo and Yusuke Suzuki.

JSTests:

• microbenchmarks/to-lower-case.js: Added.

(assert):
(foo):
(bar):

• stress/to-lower-case.js: Added.

(assert):
(foo):

Source/JavaScriptCore:

This patch makes ToLowerCase an intrinsic in the DFG/FTL. On the fast
path, the intrinsic will loop over an 8-bit string ensuring it's already
lower case, and simply return the string. In the slow path, it'll call
into C code to make a new string.

This is a 7-8% speedup on ES6SampleBench/Basic.

• dfg/DFGAbstractInterpreterInlines.h:

(JSC::DFG::AbstractInterpreter<AbstractStateType>::executeEffects):

• dfg/DFGByteCodeParser.cpp:

(JSC::DFG::ByteCodeParser::handleIntrinsicCall):

• dfg/DFGClobberize.h:

(JSC::DFG::clobberize):

• dfg/DFGDoesGC.cpp:

(JSC::DFG::doesGC):

• dfg/DFGFixupPhase.cpp:

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

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

(JSC::DFG::safeToExecute):

• dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compileToLowerCase):

• dfg/DFGSpeculativeJIT.h:

(JSC::DFG::SpeculativeJIT::callOperation):

• dfg/DFGSpeculativeJIT32_64.cpp:

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

• dfg/DFGSpeculativeJIT64.cpp:

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

• ftl/FTLAbstractHeapRepository.h:
• ftl/FTLCapabilities.cpp:

(JSC::FTL::canCompile):

• ftl/FTLLowerDFGToB3.cpp:

(JSC::FTL::DFG::LowerDFGToB3::compileNode):
(JSC::FTL::DFG::LowerDFGToB3::compileToLowerCase):

• jit/JITOperations.h:
• runtime/Intrinsic.h:
• runtime/StringPrototype.cpp:

(JSC::StringPrototype::finishCreation):

Source/WTF:

This patch exposes a new StringImpl function called convertToLowercaseWithoutLocaleStartingAtFailingIndex8Bit
which extracts slow path for the 8-bit part of convertToLowercaseWithoutLocale
into a helper function. I decided to extract this into its own function because
it may be the case that JSCs JITs will want to continue the operation
after it has already ensured that part of an 8-bit string is lower case.

• wtf/text/StringImpl.cpp:

(WTF::StringImpl::convertToLowercaseWithoutLocale):
(WTF::StringImpl::convertToLowercaseWithoutLocaleStartingAtFailingIndex8Bit):

• wtf/text/StringImpl.h:
• wtf/text/WTFString.cpp:

(WTF::String::convertToLowercaseWithoutLocaleStartingAtFailingIndex8Bit):

• wtf/text/WTFString.h:

8:13 PM Changeset in webkit [206803] by Chris Dumez

• 29 edits
  2 copies
  3 adds in trunk

Implement KeyboardEvent.code from the UI Event spec
​https://bugs.webkit.org/show_bug.cgi?id=149584

Reviewed by Darin Adler.

LayoutTests/imported/w3c:

Rebaseline W3C test now that more checks are passing.

• web-platform-tests/dom/events/Event-subclasses-constructors-expected.txt:

Source/WebCore:

Add support for Keyboard.code attribute:

• ​https://w3c.github.io/uievents/#dom-keyboardevent-code
• ​https://w3c.github.io/uievents-code/

The feature is only enabled on Mac for now as the implementation is
platform-specific and Mac only.

Test: fast/events/keyboardevent-code.html

• dom/KeyboardEvent.cpp:

(WebCore::KeyboardEvent::KeyboardEvent):

• dom/KeyboardEvent.h:
• dom/KeyboardEvent.idl:
• platform/PlatformKeyboardEvent.h:

(WebCore::PlatformKeyboardEvent::PlatformKeyboardEvent):
(WebCore::PlatformKeyboardEvent::code):
(WebCore::PlatformKeyboardEvent::unmodifiedText): Deleted.

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

(WebCore::codeForKeyEvent):
(WebCore::keyIdentifierForKeyEvent):

• replay/SerializationMethods.cpp:

(JSC::EncodingTraits<PlatformKeyboardEvent>::encodeValue):
(JSC::EncodingTraits<PlatformKeyboardEvent>::decodeValue):

Source/WebKit2:

Add support for Keyboard.code attribute:

• ​https://w3c.github.io/uievents/#dom-keyboardevent-code
• ​https://w3c.github.io/uievents-code/

• Shared/WebEvent.h:

(WebKit::WebKeyboardEvent::code):

• Shared/WebEventConversion.cpp:

(WebKit::WebKit2PlatformKeyboardEvent::WebKit2PlatformKeyboardEvent):

• Shared/WebKeyboardEvent.cpp:

(WebKit::WebKeyboardEvent::WebKeyboardEvent):
(WebKit::WebKeyboardEvent::encode):
(WebKit::WebKeyboardEvent::decode):

• Shared/mac/WebEventFactory.mm:

(WebKit::WebEventFactory::createWebKeyboardEvent):

Source/WTF:

Add build time flag to toggle support for the code attribute on
KeyboardEvent and only enable it on Mac for now, given that the
implementation is missing on other platforms.

• wtf/FeatureDefines.h:

LayoutTests:

Extend layout test coverage.

• fast/events/arrow-keys-on-body-expected.txt:
• fast/events/arrow-keys-on-body.html:
• fast/events/constructors/keyboard-event-constructor-expected.txt:
• fast/events/constructors/keyboard-event-constructor.html:
• fast/events/key-events-in-input-button-expected.txt:
• fast/events/key-events-in-input-button.html:
• fast/events/key-events-in-input-text-expected.txt:
• fast/events/key-events-in-input-text.html:
• fast/events/keyboardevent-code-expected.txt: Added.
• fast/events/keyboardevent-code.html: Added.
• fast/events/special-key-events-in-input-text-expected.txt:
• fast/events/special-key-events-in-input-text.html:

6:52 PM Changeset in webkit [206802] by commit-queue@webkit.org

• 17 edits in trunk/Source

The dragged image should be the current frame only of the animated image
​https://bugs.webkit.org/show_bug.cgi?id=162109

Patch by Said Abou-Hallawa <​sabouhallawa@apple.com> on 2016-10-04
Reviewed by Tim Horton.

Source/WebCore:

Instead of creating an NSImage with all the frames for the dragImage,
create an NSImage with the current frame only.

• dom/DataTransferMac.mm:

(WebCore::DataTransfer::createDragImage): Call snapshotNSImage() to create the dragImage.

• editing/cocoa/HTMLConverter.mm:

(fileWrapperForElement): Call the Image function with its new name.

• platform/graphics/BitmapImage.h:
• platform/graphics/Image.h:

(WebCore::Image::nsImage): Rename getNSImage() to nsImage().
(WebCore::Image::snapshotNSImage): Returns the NSImage of the current frame.
(WebCore::Image::tiffRepresentation): Rename getTIFFRepresentation() to tiffRepresentation().
(WebCore::Image::getNSImage): Deleted.
(WebCore::Image::getTIFFRepresentation): Deleted.

• platform/graphics/mac/ImageMac.mm:

(WebCore::BitmapImage::tiffRepresentation): Rename getTIFFRepresentation() to tiffRepresentation().
(WebCore::BitmapImage::nsImage): Rename getNSImage() to nsImage().
(WebCore::BitmapImage::snapshotNSImage): Returns the NSImage of the current frame.
(WebCore::BitmapImage::getTIFFRepresentation): Deleted.
(WebCore::BitmapImage::getNSImage): Deleted.

• platform/mac/CursorMac.mm:

(WebCore::createCustomCursor): Call snapshotNSImage() since the cursor does not animate anyway.

• platform/mac/DragImageMac.mm:

(WebCore::createDragImageFromImage): Use snapshotNSImage() for the dragImage.

• platform/mac/PasteboardMac.mm:

(WebCore::Pasteboard::write): Call the Image function with its new name.

Source/WebKit/mac:

• DOM/DOM.mm:

(-[DOMElement image]): Call the Image function with its new name.
(-[DOMElement _imageTIFFRepresentation]): Ditto.

• Misc/WebElementDictionary.mm:

(-[WebElementDictionary _image]): Call the Image function with its new name.

• Misc/WebIconDatabase.mm:

(-[WebIconDatabase defaultIconWithSize:]): Call snapshotNSImage() to create the icon image.
(webGetNSImage): Call the Image function with its new name.

• WebCoreSupport/WebContextMenuClient.mm:

(WebContextMenuClient::imageForCurrentSharingServicePickerItem): Call snapshotNSImage() instead of nsImage()..
(WebContextMenuClient::contextMenuForEvent): Ditto.

• WebView/WebHTMLView.mm:

(-[WebHTMLView pasteboard:provideDataForType:]): Call the Image function with its new name.

Source/WebKit2:

• UIProcess/Cocoa/WebViewImpl.mm:

(WebKit::WebViewImpl::provideDataForPasteboard): Call the Image function with its new name.

6:02 PM Changeset in webkit [206801] by aestes@apple.com

• 4 edits in trunk

[iOS] Crash in WebResourceLoaderQuickLookDelegate when the client cancels the navigation to a QuickLook resource
​https://bugs.webkit.org/show_bug.cgi?id=162950
<rdar://problem/23759114>

Reviewed by Brady Eidson.

Source/WebCore:

When we receive data from QLPreviewConverter for the first time, we call
ResourceLoader::didReceiveResponse() with the preview NSURLResponse from QuickLook. If the
client decides to cancel this navigation in decidePolicyForResponse(),
WebResourceLoaderQuickLookDelegate will end up with a null _resourceLoader after
didReceiveResponse() returns. This change adds null checks in the methods that use
_resourceLoader after calling -_sendDidReceiveResponseIfNecessary.

New API test: QuickLook.CancelNavigationAfterResponse

• platform/network/ios/QuickLook.mm:

(-[WebResourceLoaderQuickLookDelegate connection:didReceiveDataArray:]): Changed to only
call ResourceLoader::didReceiveDataArray() if _resourceLoader is non-null.
(-[WebResourceLoaderQuickLookDelegate connection:didReceiveData:lengthReceived:]): Ditto for
ResourceLoader::didReceiveData().
(-[WebResourceLoaderQuickLookDelegate connection:didFailWithError:]): Ditto for
ResourceLoader::didFail().

Tools:

Added a new API test.

• TestWebKitAPI/Tests/WebKit2Cocoa/QuickLook.mm: Sorted imports and removed redundant

initialization of static bools.
(runTest): Factored out the common test logic between QuickLook.NavigationDelegate and
QuickLook.CancelNavigationAfterResponse.
(TEST): Added QuickLook.CancelNavigationAfterResponse.
(-[QuickLookDecidePolicyDelegate
webView:decidePolicyForNavigationResponse:decisionHandler:]): Canceled the navigation.
(-[QuickLookDecidePolicyDelegate webView:didFailProvisionalNavigation:withError:]): Set
isDone to true.

5:42 PM Changeset in webkit [206800] by rniwa@webkit.org

• 2 edits
  2 adds in trunk/Tools

Add the support for running ES6SampleBench to run-benchmark
​https://bugs.webkit.org/show_bug.cgi?id=162890

Reviewed by Saam Barati.

Added the support for running ES6SampleBench as "es6bench".
e.g. ./Tools/Scripts/run-benchmark --platform osx --plan es6bench --browser safari

Instead of the default 10 iterations, run the test 4 iterations using 5 instances of browser
for the total of 20 iterations.

• Scripts/webkitpy/benchmark_runner/data/patches/ES6SampleBench.patch: Added.
• Scripts/webkitpy/benchmark_runner/data/plans/es6bench.plan: Added.
• Scripts/webkitpy/benchmark_runner/run_benchmark.py:

(start): Fixed the bug that the linter will complain about the presence of "debugOutput" by
removing it if it's present.

5:40 PM Changeset in webkit [206799] by rniwa@webkit.org

• 2 edits in trunk/Source/WebKit2

Revert a change erroneously committed in r206795.

• WebProcess/WebPage/WebPage.cpp:

(WebKit::WebPage::updatePreferences):

5:23 PM Changeset in webkit [206798] by Simon Fraser

• 10 edits
  2 adds in trunk

[iOS WK2] Make it possible for a test to describe a user gesture as a stream of events in JSON format
​https://bugs.webkit.org/show_bug.cgi?id=162934

Reviewed by Dean Jackson.

Tools:

With this change, a test can describe a user gesture in an "event stream", which is
some JSON describing an array of events with their underlying touches. The added
test describes a single tap.

The implementation fires up an NSThread, and sleeps the thread between events to dispatch
them at close to real time.

In future, HIDEventGenerator could use this internally for all of the "compound" interactions.

• DumpRenderTree/ios/UIScriptControllerIOS.mm:

(WTR::UIScriptController::sendEventStream):

• TestRunnerShared/UIScriptContext/Bindings/UIScriptController.idl:
• TestRunnerShared/UIScriptContext/UIScriptController.cpp:

(WTR::UIScriptController::sendEventStream):

• TestRunnerShared/UIScriptContext/UIScriptController.h:
• WebKitTestRunner/ios/HIDEventGenerator.h:
• WebKitTestRunner/ios/HIDEventGenerator.mm:

(transducerTypeFromString):
(phaseFromString):
(-[HIDEventGenerator eventMaskFromEventInfo:]):
(-[HIDEventGenerator touchFromEventInfo:]):
(-[HIDEventGenerator _createIOHIDEventWithInfo:]):
(-[HIDEventGenerator dispatchEventWithInfo:]):
(-[HIDEventGenerator eventDispatchThreadEntry:]):
(-[HIDEventGenerator sendEventStream:completionBlock:]):

• WebKitTestRunner/ios/UIScriptControllerIOS.mm:

(WTR::UIScriptController::sendEventStream):

LayoutTests:

• fast/events/ios/event-stream-single-tap-expected.txt: Added.
• fast/events/ios/event-stream-single-tap.html: Added.

5:15 PM Changeset in webkit [206797] by BJ Burg

• 2 edits in trunk/Source/JavaScriptCore

Web Inspector: don't synchronously send a listing message if we might need to query _WKAutomationDelegate
​https://bugs.webkit.org/show_bug.cgi?id=162810
<rdar://problem/28571460>

Reviewed by Timothy Hatcher.

We shouldn't ever access the _WKAutomationDelegate through RemoteInspector::Client methods
off of the main thread, because it could cause problems. This happens when we pushListingsNow()
in response to a WIRApplicationGetListingMessage XPC message. In this case, just use
pushListingsSoon() since it dispatches on the correct (main) queue to gather listing information.

This may induce a slight update delay when first connecting to the UIProcess through RemoteInspector,
but this is at most 200ms and will coalesce with other updates that happen when UIProcess gets set up.

There are no other code paths through RemoteInspector (for UIProcess) that could cause a call
to pushListingsNow(), so this only needs to be changed in the XPC message handler.

• inspector/remote/RemoteInspector.mm:

(Inspector::RemoteInspector::receivedGetListingMessage):

5:02 PM Changeset in webkit [206796] by Chris Dumez

• 9 edits in trunk

Add support for KeyboardEvent.isComposing attribute
​https://bugs.webkit.org/show_bug.cgi?id=162921

Reviewed by Darin Adler.

Source/WebCore:

Add support for KeyboardEvent.isComposing attribute:

• ​https://www.w3.org/TR/uievents/#dom-keyboardevent-iscomposing
• ​https://www.w3.org/TR/uievents/#ref-for-dom-keyboardevent-iscomposing-3

Manual test: ​https://jsfiddle.net/ay92sr08/9/

No new tests, extended existing tests.

• dom/KeyboardEvent.cpp:

(WebCore::KeyboardEvent::KeyboardEvent):

• dom/KeyboardEvent.h:
• dom/KeyboardEvent.idl:

LayoutTests:

Extend layout test coverage to test KeyboardEvent.isComposing.

• editing/mac/input/kotoeri-enter-to-confirm-and-newline-expected.txt:
• editing/mac/input/kotoeri-enter-to-confirm-and-newline.html:
• fast/events/constructors/keyboard-event-constructor-expected.txt:
• fast/events/constructors/keyboard-event-constructor.html:

4:42 PM Changeset in webkit [206795] by rniwa@webkit.org

• 13 edits
  3 adds in trunk

ShadowRoot interface should have elementFromPoint
​https://bugs.webkit.org/show_bug.cgi?id=162882

Reviewed by Chris Dumez.

Source/WebCore:

Add elementFromPoint to ShadowRoot's prototype as specified at:
​https://www.w3.org/TR/shadow-dom/#extensions-to-the-documentorshadowroot-mixin
with changes proposed at ​https://github.com/w3c/csswg-drafts/issues/556

Added TreeScope::retargetToScope which implements

This patch also factors DocumentOrShadowRoot.idl out of Document and ShadowRoot interfaces to better match
the latest DOM specification: ​https://dom.spec.whatwg.org/#mixin-documentorshadowroot

Test: fast/shadow-dom/Document-prototype-elementFromPoint.html

• CMakeLists.txt:
• DerivedSources.make:
• WebCore.xcodeproj/project.pbxproj:
• dom/Document.cpp:

(WebCore::Document::nodeFromPoint): Moved to TreeScope.
(WebCore::Document::elementFromPoint): Moved to TreeScope.

• dom/Document.h:
• dom/Document.idl: Moved elementFromPoint and activeElement to DocumentOrShadowRoot.idl.
• dom/DocumentOrShadowRoot.idl: Added.
• dom/EventPath.cpp:

(WebCore::RelatedNodeRetargeter::checkConsistency): Use newly added TreeScope::retargetToScope.

• dom/ShadowRoot.idl: Moved activeElement to DocumentOrShadowRoot.idl.
• dom/TreeScope.cpp:

(WebCore::TreeScope::retargetToScope): Added. Implements ​https://dom.spec.whatwg.org/#retarget efficiently.
Instead of checking whether A (node) is a shadow-including inclusive ancestor of B (this scope) at each
parent, find the lowest ancestor which contains both A and B, and return the self-inclusive ancestor of B
in that tree. To find the lowest common ancestor in O(n), traverse all ancestors of A and B separately and
do a top-down traversal. The last tree scope in which A's ancestor and B's ancestor match is the lowest
common ancestor.
(WebCore::TreeScope::nodeFromPoint): Moved from Document.
(WebCore::TreeScope::elementFromPoint): Moved from Document. Use retargetToScope and parentInComposedTree
instead of parentNode and ancestorInThisScope to match the semantics proposed in
​https://github.com/w3c/csswg-drafts/issues/556

• dom/TreeScope.h:

LayoutTests:

Add a W3C style testharness.js test for elementFromPoint on ShadowRoot.

• fast/shadow-dom/DocumentOrShadowRoot-prototype-elementFromPoint-expected.txt: Added.
• fast/shadow-dom/DocumentOrShadowRoot-prototype-elementFromPoint.html: Added.

4:25 PM Changeset in webkit [206794] by jfbastien@apple.com

• 7 edits in trunk

WebAssembly: handle a few corner cases
​https://bugs.webkit.org/show_bug.cgi?id=162884

Reviewed by Keith Miller.

JSTests:

• stress/wasm/generate-wasmops-header.js:

(const.opcodeIterator): max opcode value

Source/JavaScriptCore:

• wasm/JSWASMModule.cpp: missing include broke cmake build
• wasm/WASMFunctionParser.h:

(JSC::WASM::FunctionParser<Context>::parseBlock): check op is valid
(JSC::WASM::FunctionParser<Context>::parseExpression): switch covers all cases

• wasm/WASMOps.h:

(JSC::WASM::isValidOpType): op is valid

• wasm/WASMParser.h:

(JSC::WASM::Parser::consumeString): avoid str[i] being one-past-the-end
(JSC::WASM::Parser::parseUInt32): shift math around to avoid overflow

4:20 PM Changeset in webkit [206793] by commit-queue@webkit.org

• 2 edits in trunk/Tools

Add Megan to contributor's list
​https://bugs.webkit.org/show_bug.cgi?id=162944

Patch by Megan Gardner <Megan Gardner> on 2016-10-04
Reviewed by Tim Horton.

Add Megan to the contributor's lists so that her e-mail autocompletes for CC

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

4:13 PM Changeset in webkit [206792] by achristensen@apple.com

• 2 edits in trunk/Tools

Add tests verifying non-special URL hosts are parsed according to spec
​https://bugs.webkit.org/show_bug.cgi?id=162885

Reviewed by Sam Weinig.

• TestWebKitAPI/Tests/WebCore/URLParser.cpp:

(TestWebKitAPI::TEST_F):

4:10 PM Changeset in webkit [206791] by mmaxfield@apple.com

• 3 edits
  2 adds in trunk

font-family: cursive should map to KaiTi in Chinese
​https://bugs.webkit.org/show_bug.cgi?id=162889
<rdar://problem/28363547>

Reviewed by Tim Horton.

Source/WebCore:

According to Bobby Tung, this is a better mapping. Previously, we were
relying on the system fallback mechanism. However, an explicit mapping
is more robust.

Test: fast/text/cursive-zh.html

• page/cocoa/SettingsCocoa.mm:

(WebCore::Settings::initializeDefaultFontFamilies):

LayoutTests:

• fast/text/cursive-zh-expected.html: Added.
• fast/text/cursive-zh.html: Added.

3:51 PM WebKitGTK/Gardening/Calendar edited by Michael Catanzaro

Remove gardening calendar through October; we'll be experimenting with … (diff)

3:51 PM Changeset in webkit [206790] by Yusuke Suzuki

• 4 edits in trunk/Source/JavaScriptCore

REGRESSION (r206778): Release JSC test ChakraCore.yaml/ChakraCore/test/Error/validate_line_column.js.default failing
​https://bugs.webkit.org/show_bug.cgi?id=162937

Reviewed by Saam Barati.

We dropped expression info accidentally at r206777.

• bytecompiler/BytecodeGenerator.cpp:

(JSC::BytecodeGenerator::emitCallDefineProperty):

• bytecompiler/BytecodeGenerator.h:
• bytecompiler/NodesCodegen.cpp:

(JSC::PropertyListNode::emitPutConstantProperty):
(JSC::ClassExprNode::emitBytecode):

3:41 PM Changeset in webkit [206789] by andersca@apple.com

• 7 edits in trunk

Properly kill web processes in the launching state
​https://bugs.webkit.org/show_bug.cgi?id=162938

Reviewed by Tim Horton.

Source/WebKit2:

• UIProcess/Launcher/ProcessLauncher.h:

Add m_xpcConnection member.

• UIProcess/Launcher/mac/ProcessLauncherMac.mm:

(WebKit::ProcessLauncher::launchProcess):
Store the XPC connection in the m_xpcConnection member variable.
In the reply handler, handle m_xpcConnection being null.

(WebKit::ProcessLauncher::platformInvalidate):
Cancel and kill the connection.

• UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::terminateProcess):
Get rid of an assertion.

• UIProcess/WebProcessProxy.cpp:

(WebKit::WebProcessProxy::requestTermination):
Handle the launching state.

Tools:

• TestWebKitAPI/Tests/WebKit2/TerminateTwice.cpp:

Enable this test again.

3:26 PM Changeset in webkit [206788] by Ryan Haddad

• 2 edits in trunk/LayoutTests

Marking http/tests/cache/disk-cache/disk-cache-media.html as flaky on mac-wk2.
​https://bugs.webkit.org/show_bug.cgi?id=162946

Unreviewed test gardening.

• platform/mac-wk2/TestExpectations:

3:26 PM Changeset in webkit [206787] by Ryan Haddad

• 2 edits in trunk/LayoutTests

Marking http/tests/cache/disk-cache/disk-cache-request-max-stale.html as flaky on mac-wk2.
​https://bugs.webkit.org/show_bug.cgi?id=162945

Unreviewed test gardening.

• platform/mac-wk2/TestExpectations:

3:11 PM Changeset in webkit [206786] by Ryan Haddad

• 2 edits in trunk/LayoutTests

Marking http/tests/cache/disk-cache/resource-becomes-uncacheable.html as flaky on mac-wk2.
​https://bugs.webkit.org/show_bug.cgi?id=161649

Unreviewed test gardening.

• platform/mac-wk2/TestExpectations:

2:52 PM Changeset in webkit [206785] by Ryan Haddad

• 2 edits in trunk/LayoutTests

Marking transitions/background-position-transitions.html as flaky.
​https://bugs.webkit.org/show_bug.cgi?id=162881

Unreviewed test gardening.

• TestExpectations:

2:22 PM Changeset in webkit [206784] by achristensen@apple.com

• 4 edits in trunk

URLParser: query-only URLs relative to file URLs should just add a query
​https://bugs.webkit.org/show_bug.cgi?id=162888

Reviewed by Tim Horton.

Source/WebCore:

Covered by new API tests.

• platform/URLParser.cpp:

(WebCore::URLParser::parse):
Like bug 206749 but with queries instead of fragments.

Tools:

• TestWebKitAPI/Tests/WebCore/URLParser.cpp:

(TestWebKitAPI::TEST_F):

2:20 PM Changeset in webkit [206783] by achristensen@apple.com

• 4 edits in trunk

URLParser should match URL::parse and other browsers when parsing a URL containing only scheme://
​https://bugs.webkit.org/show_bug.cgi?id=162909

Reviewed by Tim Horton.

Source/WebCore:

If there's no host in this case we shouldn't fail, but rather make a valid URL with the in the path.
This matches Chrome, Firefox, and Safari's behavior.
Covered by API tests.

• platform/URLParser.cpp:

(WebCore::URLParser::parse):

Tools:

• TestWebKitAPI/Tests/WebCore/URLParser.cpp:

(TestWebKitAPI::TEST_F):

2:13 PM Changeset in webkit [206782] by timothy_horton@apple.com

• 2 edits in trunk/Source/WebKit2

Fix the build.

• Shared/WebBackForwardListItem.cpp:

(WebKit::WebBackForwardListItem::WebBackForwardListItem):
(WebKit::WebBackForwardListItem::highestUsedItemID):

2:08 PM Changeset in webkit [206781] by timothy_horton@apple.com

• 4 edits in trunk/Source/WebKit2

Fix a typo in WebBackForwardListItem (highed->highest)
​https://bugs.webkit.org/show_bug.cgi?id=162935

Reviewed by Anders Carlsson.

• Shared/WebBackForwardListItem.cpp:

(WebKit::WebBackForwardListItem::highestUsedItemID):
(WebKit::WebBackForwardListItem::highedUsedItemID): Deleted.

• Shared/WebBackForwardListItem.h:
• UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::creationParameters):

12:37 PM Changeset in webkit [206780] by Brent Fulgham

• 2 edits in trunk/Source/WebCore

Unreviewed build fix after r206773.

Exclude Direct2D definitions when building for normal CG Windows.

• platform/graphics/GraphicsContext.h:

12:34 PM Changeset in webkit [206779] by Yusuke Suzuki

• 23 edits
  5 adds in trunk

[DOMJIT] Introduce DOMJIT::GetterSetter to tell JIT information
​https://bugs.webkit.org/show_bug.cgi?id=162916

Reviewed by Filip Pizlo.

Source/JavaScriptCore:

In this patch, we introduce DOMJIT::GetterSetter.
This class maintains information required to emit JIT code in DFG and FTL.
DOMJIT::GetterSetter has 2 virtual functions: checkDOM and callDOM.
These functions can return a DOMJIT::Patchpoint that allows us to inject
appropriate machine code during DFG and FTL phases. DFG and FTL will invoke
these functions to get a patchpoint. And this patchpoint will be used to
emit code corresponding to CheckDOM and CallDOM DFG nodes, which will be added
in subsqeunt patch.

We propagate DOMJIT::GetterSetter through PropertySlot, AccessCase, GetByIdVariant,
and GetByIdStatus along with CustomGetter to teach DFG that this custom access
code has a chance to be inlined with this DOMJIT::GetterSetter information.
Instead of propagating CustomGetterSetter holding DOMJIT::GetterSetter and CustomGetter,
we propagate CustomGetter and DOMJIT::GetterSetter. This is because of the current
CustomGetterSetter design that we reify CustomGetterSetters only when we need to reify
all the properties. This design allows us to avoid frequent CustomGetterSetter allocations
and structure transitions.

Currently, domJIT field is always nullptr since there is no DOMJITAttribute user.
When we add this, we will add code handling this DOMJIT::GetterSetter in DFG::ByteCodeParser.

• CMakeLists.txt:
• JavaScriptCore.xcodeproj/project.pbxproj:
• bytecode/GetByIdStatus.cpp:

(JSC::GetByIdStatus::computeForStubInfoWithoutExitSiteFeedback):

• bytecode/GetByIdVariant.cpp:

(JSC::GetByIdVariant::GetByIdVariant):
(JSC::GetByIdVariant::operator=):
(JSC::GetByIdVariant::attemptToMerge):
(JSC::GetByIdVariant::dumpInContext):

• bytecode/GetByIdVariant.h:

(JSC::GetByIdVariant::domJIT):
(JSC::GetByIdVariant::intrinsic): Deleted.

• bytecode/PolymorphicAccess.cpp:

(JSC::AccessCase::get):
(JSC::AccessCase::clone):

• bytecode/PolymorphicAccess.h:

(JSC::AccessCase::domJIT):
(JSC::AccessCase::RareData::RareData):

• dfg/DFGNode.h:
• domjit/DOMJITGetterSetter.h: Added.

(JSC::DOMJIT::GetterSetter::GetterSetter):
(JSC::DOMJIT::GetterSetter::~GetterSetter):
(JSC::DOMJIT::GetterSetter::getter):
(JSC::DOMJIT::GetterSetter::setter):
(JSC::DOMJIT::GetterSetter::thisClassInfo):

• domjit/DOMJITPatchpoint.h: Added.

(JSC::DOMJIT::Patchpoint::create):
(JSC::DOMJIT::Patchpoint::setGenerator):
(JSC::DOMJIT::Patchpoint::generator):

• jit/Repatch.cpp:

(JSC::tryCacheGetByID):

• runtime/CustomGetterSetter.h:
• runtime/JSObject.h:

(JSC::JSObject::fillCustomGetterPropertySlot):

• runtime/Lookup.h:

(JSC::HashTableValue::domJIT):
(JSC::getStaticPropertySlotFromTable):
(JSC::putEntry):
(JSC::reifyStaticProperty):

• runtime/PropertySlot.h:

(JSC::PropertySlot::domJIT):
(JSC::PropertySlot::setCacheableCustom):

Source/WebCore:

• ForwardingHeaders/domjit/DOMJITGetterSetter.h: Added.
• PlatformEfl.cmake:
• PlatformGTK.cmake:
• PlatformMac.cmake:
• PlatformWin.cmake:

Source/WebKit2:

• CMakeLists.txt:

Tools:

• DumpRenderTree/CMakeLists.txt:

12:31 PM Changeset in webkit [206778] by Yusuke Suzuki

• 40 edits
  1 add in trunk/Source/JavaScriptCore

[JSC] Add a new byte code op_define_property instead of calling defineProperty
​https://bugs.webkit.org/show_bug.cgi?id=162108

Reviewed by Saam Barati.

To construct ES6 class, we emitted bytecode that performs the following operations.

1. construct a new object
2. put "configurable", "enumerable" etc. fields
3. call "defineProperty" function

However, this approach has problems. Every time we define a class method, we need to create
a new object to represent property descriptor. This can be removed if we can introduce
a special bytecode or special function.

This patch introduces new bytecodes, op_define_data_property and op_define_accessor_property.
Instead of taking an object, they takes several registers to avoid object allocations.
We're planning to use this bytecode to implement Object.defineProperty in builtin JS next.
This allows us to leverage object allocation sinking. And it also gives us a chance to use
faster ::get and ::hasProperty in JS.

Originally, I attempted to create one bytecode, op_define_property. However, it takes too many
children in DFG and uses so many registers in DFG. This leads tricky program in 32bit platforms.
Furthermore, it does not fit to the number of x64 argument registers. So instead, we introduce
two bytecodes.

And for op_define_accessor_property, we perform CellUse edge filter to getter and setter children.
This edge filter makes us possible to use SpeculateCellOperand and reduce the number of used registers
in comparison with JSValueOperand. To make children Cells even if we do not specify some accessors (for
example, { get: func, set: null } case), we fill registers with special throwTypeErrorFunction.
The attributes bitset keep information like "This property descriptor only has getter slot".

In these two bytecodes, we take attributes (configurable, enumerable, writable, hasGetter etc.) as
register instead of embedding constant int value because we will use these bytecodes to implement
Object.defineProperty next. In Object.defineProperty case, an attributes are not statically defined
at bytecode compiling time.

Run ES6SampleBench/Air 20 times. The result shows ~2% performance improvement.

Baseline:

firstIteration: 84.05 ms +- 4.37 ms
averageWorstCase: 40.54 ms +- 2.81 ms
steadyState: 3317.49 ms +- 48.25 ms
summary: 223.51 ms +- 5.07 ms

Patched:

firstIteration: 84.46 ms +- 4.22 ms
averageWorstCase: 41.48 ms +- 2.33 ms
steadyState: 3253.48 ms +- 29.31 ms
summary: 224.40 ms +- 4.72 ms

• JavaScriptCore.xcodeproj/project.pbxproj:
• bytecode/BytecodeList.json:
• bytecode/BytecodeUseDef.h:

(JSC::computeUsesForBytecodeOffset):
(JSC::computeDefsForBytecodeOffset):

• bytecode/CodeBlock.cpp:

(JSC::CodeBlock::dumpBytecode):

• bytecode/SpecialPointer.h:
• bytecompiler/BytecodeGenerator.cpp:

(JSC::BytecodeGenerator::emitMoveLinkTimeConstant):
(JSC::BytecodeGenerator::emitCallDefineProperty):

• bytecompiler/BytecodeGenerator.h:
• bytecompiler/NodesCodegen.cpp:

(JSC::PropertyListNode::emitPutConstantProperty):
(JSC::BitwiseNotNode::emitBytecode):
(JSC::ClassExprNode::emitBytecode):
(JSC::ObjectPatternNode::bindValue):

• dfg/DFGAbstractInterpreterInlines.h:

(JSC::DFG::AbstractInterpreter<AbstractStateType>::executeEffects):

• dfg/DFGByteCodeParser.cpp:

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

• dfg/DFGCapabilities.cpp:

(JSC::DFG::capabilityLevel):

• dfg/DFGClobberize.h:

(JSC::DFG::clobberize):

• dfg/DFGDoesGC.cpp:

(JSC::DFG::doesGC):

• dfg/DFGFixupPhase.cpp:

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

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

(JSC::DFG::safeToExecute):

• dfg/DFGSpeculativeJIT.cpp:

(JSC::DFG::SpeculativeJIT::compileDefineDataProperty):
(JSC::DFG::SpeculativeJIT::compileDefineAccessorProperty):

• dfg/DFGSpeculativeJIT.h:

(JSC::DFG::SpeculativeJIT::callOperation):

• dfg/DFGSpeculativeJIT32_64.cpp:

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

• dfg/DFGSpeculativeJIT64.cpp:

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

• ftl/FTLCapabilities.cpp:

(JSC::FTL::canCompile):

• ftl/FTLLowerDFGToB3.cpp:

(JSC::FTL::DFG::LowerDFGToB3::compileNode):
(JSC::FTL::DFG::LowerDFGToB3::compileDefineDataProperty):
(JSC::FTL::DFG::LowerDFGToB3::compileDefineAccessorProperty):
(JSC::FTL::DFG::LowerDFGToB3::compilePutByValWithThis): Deleted.

• jit/CCallHelpers.cpp:

(JSC::CCallHelpers::setupFourStubArgsGPR): Deleted.

• jit/CCallHelpers.h:

(JSC::CCallHelpers::setupFourStubArgsGPR):
(JSC::CCallHelpers::setupFiveStubArgsGPR):
(JSC::CCallHelpers::setupArgumentsWithExecState):
(JSC::CCallHelpers::setupStubArgsGPR):
(JSC::CCallHelpers::prepareForTailCallSlow): Deleted.

• jit/JIT.cpp:

(JSC::JIT::privateCompileMainPass):

• jit/JIT.h:
• jit/JITOperations.h:
• jit/JITPropertyAccess.cpp:

(JSC::JIT::emit_op_define_data_property):
(JSC::JIT::emit_op_define_accessor_property):

• llint/LowLevelInterpreter.asm:
• runtime/CommonSlowPaths.cpp:

(JSC::SLOW_PATH_DECL):

• runtime/CommonSlowPaths.h:
• runtime/DefinePropertyAttributes.h: Added.

(JSC::DefinePropertyAttributes::DefinePropertyAttributes):
(JSC::DefinePropertyAttributes::rawRepresentation):
(JSC::DefinePropertyAttributes::hasValue):
(JSC::DefinePropertyAttributes::setValue):
(JSC::DefinePropertyAttributes::hasGet):
(JSC::DefinePropertyAttributes::setGet):
(JSC::DefinePropertyAttributes::hasSet):
(JSC::DefinePropertyAttributes::setSet):
(JSC::DefinePropertyAttributes::writable):
(JSC::DefinePropertyAttributes::configurable):
(JSC::DefinePropertyAttributes::enumerable):
(JSC::DefinePropertyAttributes::setWritable):
(JSC::DefinePropertyAttributes::setConfigurable):
(JSC::DefinePropertyAttributes::setEnumerable):
(JSC::DefinePropertyAttributes::fillWithTriState):
(JSC::DefinePropertyAttributes::extractTriState):

• runtime/JSGlobalObject.cpp:

(JSC::JSGlobalObject::init):
(JSC::JSGlobalObject::visitChildren):

• runtime/JSGlobalObject.h:

(JSC::JSGlobalObject::throwTypeErrorFunction):
(JSC::JSGlobalObject::definePropertyFunction): Deleted.

• runtime/ObjectConstructor.cpp:

(JSC::ObjectConstructor::addDefineProperty): Deleted.

• runtime/ObjectConstructor.h:
• runtime/PropertyDescriptor.h:

(JSC::toPropertyDescriptor):

12:00 PM Changeset in webkit [206777] by Chris Dumez

• 17 edits in trunk

Return null when window.open() is popup blocked
​https://bugs.webkit.org/show_bug.cgi?id=162897

Reviewed by Michael Catanzaro.

Source/WebCore:

The return value of window.open() when the popup is blocked should be null:

• ​https://github.com/whatwg/html/pull/1854
• ​https://github.com/whatwg/html/issues/1851

Firefox / Edge return null.

No new tests, rebaselined existing tests.

• bindings/js/JSDOMWindowCustom.cpp:

(WebCore::JSDOMWindow::open):

LayoutTests:

Rebaseline existing tests to reflect behavior change.

• fast/events/popup-blocked-from-fake-button-click-expected.txt:
• fast/events/popup-blocked-from-fake-button-click.html:
• fast/events/popup-blocked-from-fake-focus-expected.txt:
• fast/events/popup-blocked-from-fake-focus.html:
• fast/events/popup-blocked-from-fake-user-gesture-expected.txt:
• fast/events/popup-blocked-from-fake-user-gesture.html:
• fast/events/popup-blocked-from-untrusted-click-event-on-anchor-expected.txt:
• fast/events/popup-blocked-from-untrusted-click-event-on-anchor.html:
• fast/events/popup-blocking-timers3-expected.txt:
• fast/events/popup-blocking-timers3.html:
• fast/events/popup-blocking-timers4-expected.txt:
• fast/events/popup-blocking-timers4.html:
• fast/events/popup-blocking-timers6-expected.txt:
• fast/events/popup-blocking-timers6.html:

11:51 AM Changeset in webkit [206776] by Chris Dumez

• 7 edits in trunk/Source/WebCore

[Web IDL] Add support for dictionary inheritance
​https://bugs.webkit.org/show_bug.cgi?id=162907

Reviewed by Sam Weinig.

Add support for dictionary inheritance to our Web IDL parser and
bindings generator. Leverage this new support for EventTarget's
EventListenerOptions / AddEventListenerOptions dictionaries, in
order to match the DOM specification:

• ​https://dom.spec.whatwg.org/#interface-eventtarget

Also update the bindings generator to convert the dictionary
members in lexicographical order, as per:

• ​https://heycam.github.io/webidl/#es-dictionary (Step 5.1)

No new tests, rebaselined bindings tests.

• bindings/scripts/CodeGenerator.pm:

(GetDictionaryByName):

• bindings/scripts/CodeGeneratorJS.pm:

(GenerateDictionaryImplementationContent):

• bindings/scripts/IDLParser.pm:

(parseDictionary):

• bindings/scripts/test/JS/JSTestObj.cpp:

(WebCore::convertDictionary<TestObj::Dictionary>):
(WebCore::convertDictionary<TestObj::DictionaryThatShouldNotTolerateNull>):
(WebCore::convertDictionary<TestObj::DictionaryThatShouldTolerateNull>):
(WebCore::convertDictionary<AlternateDictionaryName>):
(WebCore::convertDictionary<TestObj::ParentDictionary>):
(WebCore::convertDictionary<TestObj::ChildDictionary>):

• bindings/scripts/test/TestObj.idl:
• dom/EventTarget.idl:

11:45 AM Changeset in webkit [206775] by Ryan Haddad

• 2 edits in trunk/LayoutTests

Marking http/tests/cache/disk-cache/disk-cache-204-status-code.html as flaky on mac-wk2.
​https://bugs.webkit.org/show_bug.cgi?id=162685

Unreviewed test gardening.

• platform/mac-wk2/TestExpectations:

11:41 AM Changeset in webkit [206774] by Ryan Haddad

• 2 edits in trunk/LayoutTests

Land TestExpectations for rdar://problem/28586837.

Unreviewed test gardening.

• platform/mac/TestExpectations:

11:31 AM Changeset in webkit [206773] by Brent Fulgham

• 9 edits
  2 adds in trunk/Source/WebCore

[Win][Direct2D] Add initial D2D GraphicsContext implementation
​https://bugs.webkit.org/show_bug.cgi?id=162917

Reviewed by Dean Jackson.

This patch lands a set of new files that implement the GraphicsContext
features on Windows using Direct2D.

No new tests until complete backend lands.

• platform/graphics/Color.h: Add new casting operator.
• platform/graphics/FloatSize.h: Ditto.
• platform/graphics/GraphicsContext.cpp: Update for Direct2D

implementation.

• platform/graphics/GraphicsContext.h:
• platform/graphics/win/ColorDirect2D.cpp:

(WebCore::Color::operator D2D1_VECTOR_4F): Added.

• platform/graphics/win/GraphicsContextCGWin.cpp: Add compiler

guard to avoid building when using Direct2D.

• platform/graphics/win/GraphicsContextDirect2D.cpp: Added.
• platform/graphics/win/GraphicsContextPlatformPrivateDirect2D.h: Added.
• platform/graphics/win/GraphicsContextWin.cpp: Update for Direct2D

includes.

• platform/graphics/win/TransformationMatrixDirect2D.cpp: Fix comment.

9:53 AM Changeset in webkit [206772] by commit-queue@webkit.org

• 5 edits in trunk/Source

[SOUP] Remove SSLPolicyFlags from SoupNetworkSession
​https://bugs.webkit.org/show_bug.cgi?id=162906

Patch by Carlos Garcia Campos <​cgarcia@igalia.com> on 2016-10-04
Reviewed by Michael Catanzaro.

Source/WebCore:

All soup based ports are setting SSLUseSystemCAFile flag unconditionally, so we can just use that when creating
the session like we do for all other construct parameters.

• platform/network/soup/SoupNetworkSession.cpp:

(WebCore::SoupNetworkSession::createTestingSession):
(WebCore::SoupNetworkSession::setupLogger):
(WebCore::SoupNetworkSession::SoupNetworkSession): Deleted.
(WebCore::SoupNetworkSession::clearOldSoupCache): Deleted.
(WebCore::SoupNetworkSession::setHTTPProxy): Deleted.

• platform/network/soup/SoupNetworkSession.h:

(WebCore::SoupNetworkSession::soupSession): Deleted.

Source/WebKit2:

• NetworkProcess/soup/NetworkProcessMainSoup.cpp:

(WebKit::NetworkProcessMainUnix):

8:35 AM Changeset in webkit [206771] by Wenson Hsieh

• 17 edits in trunk

Media controls are displayed in the incorrect state momentarily after switching between tabs playing media
​https://bugs.webkit.org/show_bug.cgi?id=162766
<rdar://problem/28533523>

Reviewed by Jer Noble.

Source/WebCore:

When showing Now Playing controls for a media session, we should first set up the Now Playing info and
playback state before telling MediaRemote to make the session visible. This is WebKit work in ensuring that
when switching Now Playing sessions by switching tabs, we do not first display an invalid Now Playing state
before updating to the expected state.

Adds 2 new WebKit API tests in NowPlayingControlsTests: NowPlayingControlsHideAfterShowingClearsInfo and
NowPlayingControlsClearInfoAfterSessionIsNoLongerValid.

• platform/audio/PlatformMediaSessionManager.h:

(WebCore::PlatformMediaSessionManager::lastUpdatedNowPlayingTitle):
(WebCore::PlatformMediaSessionManager::lastUpdatedNowPlayingDuration):
(WebCore::PlatformMediaSessionManager::lastUpdatedNowPlayingElapsedTime):
(WebCore::PlatformMediaSessionManager::hasActiveNowPlayingSession): Deleted.

• platform/audio/mac/MediaSessionManagerMac.h:
• platform/audio/mac/MediaSessionManagerMac.mm:

(WebCore::MediaSessionManagerMac::updateNowPlayingInfo):

Source/WebKit2:

Plumbs some more Now Playing information from the web process to the UI process for testing purposes. See
WebCore ChangeLog for more details.

• UIProcess/API/Cocoa/WKWebView.mm:

(-[WKWebView _handleActiveNowPlayingSessionInfoResponse:title:duration:elapsedTime:]):
(-[WKWebView _handleActiveNowPlayingSessionInfoResponse:]): Deleted.

• UIProcess/API/Cocoa/WKWebViewPrivate.h:
• UIProcess/PageClient.h:
• UIProcess/WebPageProxy.cpp:

(WebKit::WebPageProxy::handleActiveNowPlayingSessionInfoResponse):

• UIProcess/WebPageProxy.h:
• UIProcess/WebPageProxy.messages.in:
• UIProcess/mac/PageClientImpl.h:
• UIProcess/mac/PageClientImpl.mm:

(WebKit::PageClientImpl::handleActiveNowPlayingSessionInfoResponse):

• WebProcess/WebPage/mac/WebPageMac.mm:

(WebKit::WebPage::requestActiveNowPlayingSessionInfo):

Tools:

Adds new tests and tweaks existing tests to verify last updated Now Playing information.

• TestWebKitAPI/Tests/WebKit2Cocoa/NowPlayingControlsTests.mm:

(-[NowPlayingTestWebView _handleActiveNowPlayingSessionInfoResponse:title:duration:elapsedTime:]):
(TestWebKitAPI::TEST):
(-[NowPlayingTestWebView _handleActiveNowPlayingSessionInfoResponse:]): Deleted.

• TestWebKitAPI/Tests/WebKit2Cocoa/large-video-test-now-playing.html:

8:17 AM Changeset in webkit [206770] by commit-queue@webkit.org

• 5 edits
  3 adds in trunk

[Fetch API] ReadableStream should be errored with TypeError values
​https://bugs.webkit.org/show_bug.cgi?id=162895

Patch by Youenn Fablet <​youenn@apple.com> on 2016-10-04
Reviewed by Alex Christensen.

LayoutTests/imported/w3c:

• web-platform-tests/fetch/api/basic/error-after-response-expected.txt: Added.
• web-platform-tests/fetch/api/basic/error-after-response.html: Added.
• web-platform-tests/fetch/api/resources/bad-chunk-encoding.py: Added.

(main):

• web-platform-tests/fetch/api/response/response-consume-stream-expected.txt:

Source/WebCore:

Test: imported/w3c/web-platform-tests/fetch/api/basic/error-after-response.html

• Modules/fetch/FetchResponse.cpp:

(WebCore::FetchResponse::BodyLoader::didFail): Removing FIXME.

• bindings/js/ReadableStreamDefaultController.h:

(WebCore::ReadableStreamDefaultController::error<String>): Erroring with TypeError values.

8:15 AM Changeset in webkit [206769] by commit-queue@webkit.org

• 4 edits in trunk/LayoutTests

[mac-wk1] LayoutTest http/tests/security/cached-cross-origin-preloading-css-stylesheet.html is a flaky failure
​https://bugs.webkit.org/show_bug.cgi?id=162791

Patch by Youenn Fablet <​youenn@apple.com> on 2016-10-04
Reviewed by Alex Christensen.

Flackiness was due to one preload sometimes arriving after the test was finished.
This causes the issue of not having the preload actually being checked against CORS and the console error was not displayed.
Making the test wait 1 second after all loads finish to ensure the preload will also finishes.

Cleaning the test and improving it by:

• making each load to take 1 second (so that preload is not finished when actual load happens
• adding some comments explaining each subtest
• printing DONE at the end of all loads.

• http/tests/security/cached-cross-origin-preloading-css-stylesheet-expected.txt:
• http/tests/security/cached-cross-origin-preloading-css-stylesheet.html:
• platform/mac-wk1/TestExpectations: Removed flaky expectation.

6:49 AM Changeset in webkit [206768] by zandobersek@gmail.com

• 2 edits in trunk/Source/WebKit2

Unreviewed GTK+ build fix.

• UIProcess/API/C/WKPage.cpp:

(WKPageSetPageNavigationClient): Guard the call to getDefaultWebCryptoMasterKey()
with ENABLE(SUBTLE_CRYPTO), since the same guard is used for the definition. Fixes
GTK+ production builds where the feature is disabled.

2:19 AM Changeset in webkit [206767] by sbarati@apple.com

• 2 edits in trunk/Source/JavaScriptCore

Follow up fix to GetMapBucket and MapHash speculating on child node types.
To fix this, on 32-bit platforms, we do not speculate on the child
type since we just call into C code for these nodes.

• dfg/DFGFixupPhase.cpp:

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