WebKit-https.git
2 years agorun-jsc should pass the option to use dollar vm by default
keith_miller@apple.com [Sun, 3 Jun 2018 00:49:13 +0000 (00:49 +0000)]
run-jsc should pass the option to use dollar vm by default
https://bugs.webkit.org/show_bug.cgi?id=186170

Reviewed by Yusuke Suzuki.

* Scripts/run-jsc:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232448 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoUnreviewed, rolling out r232275.
cdumez@apple.com [Sat, 2 Jun 2018 23:34:43 +0000 (23:34 +0000)]
Unreviewed, rolling out r232275.

May have caused a ~1% PLT regression on iOS

Reverted changeset:

"Store 0-lifetime stylesheets / scripts into the disk cache
for faster history navigations"
https://bugs.webkit.org/show_bug.cgi?id=186060
https://trac.webkit.org/changeset/232275

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232447 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoUnreviewed, rolling out r232439.
commit-queue@webkit.org [Sat, 2 Jun 2018 23:03:36 +0000 (23:03 +0000)]
Unreviewed, rolling out r232439.
https://bugs.webkit.org/show_bug.cgi?id=186238

It breaks gtk-linux-32-release (Requested by caiolima on
#webkit).

Reverted changeset:

"[ESNext][BigInt] Implement support for addition operations"
https://bugs.webkit.org/show_bug.cgi?id=179002
https://trac.webkit.org/changeset/232439

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232446 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoUnreviewed, rolling out r232421.
commit-queue@webkit.org [Sat, 2 Jun 2018 23:02:50 +0000 (23:02 +0000)]
Unreviewed, rolling out r232421.
https://bugs.webkit.org/show_bug.cgi?id=186240

Breaks webkitpy tests (Requested by aakashjain on #webkit).

Reverted changeset:

"Add some tests for lldb_webkit.py"
https://bugs.webkit.org/show_bug.cgi?id=183744
https://trac.webkit.org/changeset/232421

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232445 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoBaseline op_jtrue emits an insane amount of code
utatane.tea@gmail.com [Sat, 2 Jun 2018 21:13:47 +0000 (21:13 +0000)]
Baseline op_jtrue emits an insane amount of code
https://bugs.webkit.org/show_bug.cgi?id=185708

Reviewed by Filip Pizlo.

JSTests:

* stress/logical-not-masquerades-as-undefined.js: Added.
(shouldBe):
(test):
* stress/logical-not.js: Added.
(shouldBe):
(test):

Source/JavaScriptCore:

op_jtrue / op_jfalse bloats massive amount of code. This patch attempts to reduce the size of this code by,

1. op_jtrue / op_jfalse immediately jumps if the condition met. We add AssemblyHelpers::branchIf{Truthy,Falsey}
   to jump directly. This tightens the code.

2. Align our emitConvertValueToBoolean implementation to FTL's boolify function. It emits less code.

This reduces the code size of op_jtrue in x64 from 220 bytes to 164 bytes.

[  12] jtrue             arg1, 6(->18)
      0x7f233170162c: mov 0x30(%rbp), %rax
      0x7f2331701630: mov %rax, %rsi
      0x7f2331701633: xor $0x6, %rsi
      0x7f2331701637: test $0xfffffffffffffffe, %rsi
      0x7f233170163e: jnz 0x7f2331701654
      0x7f2331701644: cmp $0x7, %eax
      0x7f2331701647: setz %sil
      0x7f233170164b: movzx %sil, %esi
      0x7f233170164f: jmp 0x7f2331701705
      0x7f2331701654: test %rax, %r14
      0x7f2331701657: jz 0x7f233170169c
      0x7f233170165d: cmp %r14, %rax
      0x7f2331701660: jb 0x7f2331701675
      0x7f2331701666: test %eax, %eax
      0x7f2331701668: setnz %sil
      0x7f233170166c: movzx %sil, %esi
      0x7f2331701670: jmp 0x7f2331701705
      0x7f2331701675: lea (%r14,%rax), %rsi
      0x7f2331701679: movq %rsi, %xmm0
      0x7f233170167e: xorps %xmm1, %xmm1
      0x7f2331701681: ucomisd %xmm1, %xmm0
      0x7f2331701685: jz 0x7f2331701695
      0x7f233170168b: mov $0x1, %esi
      0x7f2331701690: jmp 0x7f2331701705
      0x7f2331701695: xor %esi, %esi
      0x7f2331701697: jmp 0x7f2331701705
      0x7f233170169c: test %rax, %r15
      0x7f233170169f: jnz 0x7f2331701703
      0x7f23317016a5: cmp $0x1, 0x5(%rax)
      0x7f23317016a9: jnz 0x7f23317016c1
      0x7f23317016af: mov 0x8(%rax), %esi
      0x7f23317016b2: test %esi, %esi
      0x7f23317016b4: setnz %sil
      0x7f23317016b8: movzx %sil, %esi
      0x7f23317016bc: jmp 0x7f2331701705
      0x7f23317016c1: test $0x1, 0x6(%rax)
      0x7f23317016c5: jz 0x7f23317016f9
      0x7f23317016cb: mov (%rax), %esi
      0x7f23317016cd: mov $0x7f23315000c8, %rdx
      0x7f23317016d7: mov (%rdx), %rdx
      0x7f23317016da: mov (%rdx,%rsi,8), %rsi
      0x7f23317016de: mov $0x7f2330de0000, %rdx
      0x7f23317016e8: cmp %rdx, 0x18(%rsi)
      0x7f23317016ec: jnz 0x7f23317016f9
      0x7f23317016f2: xor %esi, %esi
      0x7f23317016f4: jmp 0x7f2331701705
      0x7f23317016f9: mov $0x1, %esi
      0x7f23317016fe: jmp 0x7f2331701705
      0x7f2331701703: xor %esi, %esi
      0x7f2331701705: test %esi, %esi
      0x7f2331701707: jnz 0x7f233170171b

[  12] jtrue             arg1, 6(->18)
      0x7f6c8710156c: mov 0x30(%rbp), %rax
      0x7f6c87101570: test %rax, %r15
      0x7f6c87101573: jnz 0x7f6c871015c8
      0x7f6c87101579: cmp $0x1, 0x5(%rax)
      0x7f6c8710157d: jnz 0x7f6c87101592
      0x7f6c87101583: cmp $0x0, 0x8(%rax)
      0x7f6c87101587: jnz 0x7f6c87101623
      0x7f6c8710158d: jmp 0x7f6c87101615
      0x7f6c87101592: test $0x1, 0x6(%rax)
      0x7f6c87101596: jz 0x7f6c87101623
      0x7f6c8710159c: mov (%rax), %esi
      0x7f6c8710159e: mov $0x7f6c86f000e0, %rdx
      0x7f6c871015a8: mov (%rdx), %rdx
      0x7f6c871015ab: mov (%rdx,%rsi,8), %rsi
      0x7f6c871015af: mov $0x7f6c867e0000, %rdx
      0x7f6c871015b9: cmp %rdx, 0x18(%rsi)
      0x7f6c871015bd: jnz 0x7f6c87101623
      0x7f6c871015c3: jmp 0x7f6c87101615
      0x7f6c871015c8: cmp %r14, %rax
      0x7f6c871015cb: jb 0x7f6c871015de
      0x7f6c871015d1: test %eax, %eax
      0x7f6c871015d3: jnz 0x7f6c87101623
      0x7f6c871015d9: jmp 0x7f6c87101615
      0x7f6c871015de: test %rax, %r14
      0x7f6c871015e1: jz 0x7f6c87101602
      0x7f6c871015e7: lea (%r14,%rax), %rsi
      0x7f6c871015eb: movq %rsi, %xmm0
      0x7f6c871015f0: xorps %xmm1, %xmm1
      0x7f6c871015f3: ucomisd %xmm1, %xmm0
      0x7f6c871015f7: jz 0x7f6c87101615
      0x7f6c871015fd: jmp 0x7f6c87101623
      0x7f6c87101602: mov $0x7, %r11
      0x7f6c8710160c: cmp %r11, %rax
      0x7f6c8710160f: jz 0x7f6c87101623

* dfg/DFGSpeculativeJIT32_64.cpp:
(JSC::DFG::SpeculativeJIT::emitBranch):
* dfg/DFGSpeculativeJIT64.cpp:
(JSC::DFG::SpeculativeJIT::emitBranch):
* jit/AssemblyHelpers.cpp:
(JSC::AssemblyHelpers::emitConvertValueToBoolean):
(JSC::AssemblyHelpers::branchIfValue):
* jit/AssemblyHelpers.h:
(JSC::AssemblyHelpers::branchIfTruthy):
(JSC::AssemblyHelpers::branchIfFalsey):
* jit/JIT.h:
* jit/JITInlines.h:
(JSC::JIT::addJump):
* jit/JITOpcodes.cpp:
(JSC::JIT::emit_op_jfalse):
(JSC::JIT::emit_op_jtrue):
* jit/JITOpcodes32_64.cpp:
(JSC::JIT::emit_op_jfalse):
(JSC::JIT::emit_op_jtrue):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232444 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[JSC] Remove WeakReferenceHarvester
utatane.tea@gmail.com [Sat, 2 Jun 2018 21:11:19 +0000 (21:11 +0000)]
[JSC] Remove WeakReferenceHarvester
https://bugs.webkit.org/show_bug.cgi?id=186102

Reviewed by Filip Pizlo.

JSTests:

* microbenchmarks/create-many-weak-map.js: Added.

Source/JavaScriptCore:

After several cleanups, now JSWeakMap becomes the last user of WeakReferenceHarvester.
Since JSWeakMap is already managed in IsoSubspace, we can iterate marked JSWeakMap
by using output constraints & Subspace iteration.

This patch removes WeakReferenceHarvester. Instead of managing this linked-list, our
output constraint set iterates marked JSWeakMap by using Subspace.

And we also add locking for JSWeakMap's rehash and output constraint visiting.

Attached microbenchmark does not show any regression.

* API/JSAPIWrapperObject.h:
* CMakeLists.txt:
* JavaScriptCore.xcodeproj/project.pbxproj:
* heap/Heap.cpp:
(JSC::Heap::endMarking):
(JSC::Heap::addCoreConstraints):
* heap/Heap.h:
* heap/SlotVisitor.cpp:
(JSC::SlotVisitor::addWeakReferenceHarvester): Deleted.
* heap/SlotVisitor.h:
* heap/WeakReferenceHarvester.h: Removed.
* runtime/WeakMapImpl.cpp:
(JSC::WeakMapImpl<WeakMapBucket>::visitChildren):
(JSC::WeakMapImpl<WeakMapBucket<WeakMapBucketDataKey>>::visitOutputConstraints):
(JSC::WeakMapImpl<WeakMapBucket<WeakMapBucketDataKeyValue>>::visitOutputConstraints):
(JSC::WeakMapImpl<WeakMapBucket<WeakMapBucketDataKey>>::visitWeakReferences): Deleted.
(JSC::WeakMapImpl<WeakMapBucket<WeakMapBucketDataKeyValue>>::visitWeakReferences): Deleted.
* runtime/WeakMapImpl.h:
(JSC::WeakMapImpl::WeakMapImpl):
(JSC::WeakMapImpl::finishCreation):
(JSC::WeakMapImpl::rehash):
(JSC::WeakMapImpl::makeAndSetNewBuffer):
(JSC::WeakMapImpl::DeadKeyCleaner::target): Deleted.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232443 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[JSC] Object.create should have intrinsic
utatane.tea@gmail.com [Sat, 2 Jun 2018 21:08:10 +0000 (21:08 +0000)]
[JSC] Object.create should have intrinsic
https://bugs.webkit.org/show_bug.cgi?id=186200

Reviewed by Filip Pizlo.

JSTests:

* microbenchmarks/object-create-null.js: Added.
(test):
* microbenchmarks/object-create-unknown-object-prototype.js: Added.
(test):
* microbenchmarks/object-create-untyped-prototype.js: Added.
(test):
* stress/get-by-pname-only-prototype-properties.js: Added.
(foo):
* stress/object-create-define.js: Added.
(shouldBe):
(test):
* stress/object-create-null-external.js: Added.
(shouldBe):
(test):
* stress/object-create-null.js: Added.
(shouldBe):
(test):
* stress/object-create-prototype.js: Added.
(shouldBe):
(test):
* stress/object-create-undefined.js: Added.
(shouldThrow):
(i.shouldThrow):

Source/JavaScriptCore:

Object.create is used in various JS code. `Object.create(null)` is particularly used
to create empty plain object with null [[Prototype]]. We can find `Object.create(null)`
call in ARES-6/Babylon code.

This patch adds ObjectCreateIntrinsic to JSC. DFG recognizes it and produces ObjectCreate
DFG node. DFG AI and constant folding attempt to convert it to NewObject when prototype
object is null. It offers significant performance boost for `Object.create(null)`.

                                                 baseline                  patched

object-create-null                           53.7940+-1.5297     ^     19.8846+-0.6584        ^ definitely 2.7053x faster
object-create-unknown-object-prototype       38.9977+-1.1364     ^     37.2207+-0.6143        ^ definitely 1.0477x faster
object-create-untyped-prototype              22.5632+-0.6917           22.2539+-0.6876          might be 1.0139x faster

* dfg/DFGAbstractInterpreterInlines.h:
(JSC::DFG::AbstractInterpreter<AbstractStateType>::executeEffects):
* dfg/DFGByteCodeParser.cpp:
(JSC::DFG::ByteCodeParser::handleIntrinsicCall):
* 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::convertToNewObject):
* dfg/DFGNodeType.h:
* dfg/DFGOperations.cpp:
* dfg/DFGOperations.h:
* dfg/DFGPredictionPropagationPhase.cpp:
* dfg/DFGSafeToExecute.h:
(JSC::DFG::safeToExecute):
* dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::SpeculativeJIT::compileObjectCreate):
* dfg/DFGSpeculativeJIT.h:
* 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::compileObjectCreate):
* runtime/Intrinsic.cpp:
(JSC::intrinsicName):
* runtime/Intrinsic.h:
* runtime/JSGlobalObject.cpp:
(JSC::JSGlobalObject::init):
(JSC::JSGlobalObject::visitChildren):
* runtime/JSGlobalObject.h:
(JSC::JSGlobalObject::nullPrototypeObjectStructure const):
* runtime/ObjectConstructor.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232442 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[ESNext][BigInt] Implement support for addition operations
ticaiolima@gmail.com [Sat, 2 Jun 2018 16:03:51 +0000 (16:03 +0000)]
[ESNext][BigInt] Implement support for addition operations
https://bugs.webkit.org/show_bug.cgi?id=179002

Reviewed by Yusuke Suzuki.

JSTests:

* bigIntTests.yaml:
* stress/addition-order-evaluation.js: Added.
* stress/big-int-add-wrapped-value.js: Added.
* stress/big-int-addition-basic.js: Added.
* stress/big-int-addition-jit.js: Added.
* stress/big-int-addition-memory-stress.js: Added.
* stress/big-int-addition-string-coercion.js: Added.
* stress/big-int-addition-to-primitive-precedence.js: Added.
* stress/big-int-addition-to-primitive.js: Added.
* stress/big-int-addition-type-error.js: Added.
* stress/big-int-no-conversion-to-number.js:
* stress/big-int-sub-wrapped-value.js: Added.
* stress/big-int-subtraction-basic.js: Added.
* stress/big-int-subtraction-jit.js: Added.
* stress/big-int-subtraction-type-error.js: Added.
* stress/sub-order-evaluation.js: Added.

Source/JavaScriptCore:

This patch is implementing support to BigInt Operands into binary "+"
and binary "-" operators. Right now, we have limited support to DFG
and FTL JIT layers, but we plan to fix this support in future
patches.

* jit/JITOperations.cpp:
* runtime/CommonSlowPaths.cpp:
(JSC::SLOW_PATH_DECL):
* runtime/JSBigInt.cpp:
(JSC::JSBigInt::parseInt):
(JSC::JSBigInt::stringToBigInt):
(JSC::JSBigInt::toString):
(JSC::JSBigInt::multiply):
(JSC::JSBigInt::divide):
(JSC::JSBigInt::remainder):
(JSC::JSBigInt::add):
(JSC::JSBigInt::sub):
(JSC::JSBigInt::absoluteAdd):
(JSC::JSBigInt::absoluteSub):
(JSC::JSBigInt::toStringGeneric):
(JSC::JSBigInt::allocateFor):
(JSC::JSBigInt::toNumber const):
(JSC::JSBigInt::getPrimitiveNumber const):
* runtime/JSBigInt.h:
* runtime/JSCJSValueInlines.h:
* runtime/Operations.cpp:
(JSC::jsAddSlowCase):
* runtime/Operations.h:
(JSC::jsSub):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232439 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoMake WKWebView firstResponder after entering or exiting fullscreen.
commit-queue@webkit.org [Sat, 2 Jun 2018 16:03:31 +0000 (16:03 +0000)]
Make WKWebView firstResponder after entering or exiting fullscreen.
https://bugs.webkit.org/show_bug.cgi?id=186088
rdar://problem/40387859

Patch by Jeremy Jones <jeremyj@apple.com> on 2018-06-02
Reviewed by Jer Noble.

Update first responder when WKWebView moves to a new window, otherwise there won't be a first responder.

* UIProcess/ios/fullscreen/WKFullScreenWindowControllerIOS.mm:
(-[WKFullScreenWindowController beganEnterFullScreenWithInitialFrame:finalFrame:]):
(-[WKFullScreenWindowController _completedExitFullScreen]):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232438 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoExit fullscreen when javascript alerts are presented.
commit-queue@webkit.org [Sat, 2 Jun 2018 16:02:14 +0000 (16:02 +0000)]
Exit fullscreen when javascript alerts are presented.
https://bugs.webkit.org/show_bug.cgi?id=185619
rdar://problem/35724264

Patch by Jeremy Jones <jeremyj@apple.com> on 2018-06-02
Reviewed by Jer Noble.

Prevent users from being trapped in fullscreen by alert cycles.
Prevent fullscreen from becoming unresponsive due to alerts hidden behind fullscreen.

Fullscreen exit is initiated here in the UI process so that fullscreen UI is torn down immediately,
before the alert is shown.

* UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::exitFullscreenImmediately):
(WebKit::WebPageProxy::runJavaScriptAlert):
(WebKit::WebPageProxy::runJavaScriptConfirm):
(WebKit::WebPageProxy::runJavaScriptPrompt):
* UIProcess/WebPageProxy.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232437 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoAdd a sandbox profile for com.cisco.webex.plugin.gpc64 plugin
youenn@apple.com [Sat, 2 Jun 2018 15:59:53 +0000 (15:59 +0000)]
Add a sandbox profile for com.cisco.webex.plugin.gpc64 plugin
https://bugs.webkit.org/show_bug.cgi?id=186110

Reviewed by Brent Fulgham.

* PluginProcess/mac/com.apple.WebKit.plugin-common.sb.in: Webex needs to create some symlinks.
* Resources/PlugInSandboxProfiles/com.cisco.webex.plugin.gpc64.sb: Added.
* WebKit.xcodeproj/project.pbxproj:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232436 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[Extra zoom mode] The user should always be able to double tap to zoom to a scale...
wenson_hsieh@apple.com [Sat, 2 Jun 2018 06:54:12 +0000 (06:54 +0000)]
[Extra zoom mode] The user should always be able to double tap to zoom to a scale of at least 1
https://bugs.webkit.org/show_bug.cgi?id=186209
<rdar://problem/40529255>

Reviewed by Tim Horton.

Source/WebKit:

Tweaks the way double-tap-to-zoom scales are determined in extra zoom mode. Rather than zooming to make the 50th
and 90th percentiles of text in the document legible, only consider the 90th percentile of text size when
determining zoom scale, and fix the other potential zoom scale at 1; additionally, if the zoom scales are close
(within 0.3 of each other), snap the lower zoom scale to the higher value.

This results in the following changes in behavior:
-   Enables double tap to zoom in cases where all the text in the page is already legible.
-   On pages with mobile viewports, usually allows the user to toggle between initial scale and a scale of 1.
-   If a significant portion of text is unusually small, the zoomed-in scale may exceed 1.

Test: fast/events/extrazoom/double-tap-to-zoom-with-large-text.html

* WebProcess/WebPage/ViewGestureGeometryCollector.cpp:
(WebKit::ViewGestureGeometryCollector::collectGeometryForSmartMagnificationGesture):
(WebKit::ViewGestureGeometryCollector::computeTextLegibilityScales):

LayoutTests:

Add a test to verify that double tapping zooms in on a page where all the text is large enough to be legible at
initial scale.

* fast/events/extrazoom/double-tap-to-zoom-with-large-text-expected.txt: Added.
* fast/events/extrazoom/double-tap-to-zoom-with-large-text.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232435 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoEditor can hold references to Documents after you navigate away
rniwa@webkit.org [Sat, 2 Jun 2018 04:09:30 +0000 (04:09 +0000)]
Editor can hold references to Documents after you navigate away
https://bugs.webkit.org/show_bug.cgi?id=186215

Reviewed by Simon Fraser.

Source/WebCore:

Clear the various member variables that can hold onto a document in Editor::clear and FrameSelection::prepareForDestruction.

Test: editing/selection/navigation-clears-editor-state.html

* editing/Editor.cpp:
(WebCore::Editor::clear):
* editing/Editor.h:
* editing/FrameSelection.cpp:
(WebCore::FrameSelection::FrameSelection):
(WebCore::FrameSelection::prepareForDestruction):
* editing/FrameSelection.h:

LayoutTests:

Added a regression test.

* editing/selection/navigation-clears-editor-state-expected.txt: Added.
* editing/selection/navigation-clears-editor-state.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232434 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoResourceLoader::cancel() shouldn't synchronously fire load event on document
rniwa@webkit.org [Sat, 2 Jun 2018 00:21:58 +0000 (00:21 +0000)]
ResourceLoader::cancel() shouldn't synchronously fire load event on document
https://bugs.webkit.org/show_bug.cgi?id=185284

Revert the erroneous change to SSFontSelector::beginLoadTimerFired(), which broke CancelLoading.CancelFontSubresource.
FrameLoader::loadDone calls checkCompleted, not checkLoadComplete so we still need to call checkLoadComplete here.

* css/CSSFontSelector.cpp:
(WebCore::CSSFontSelector::beginLoadTimerFired):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232433 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoMove tests in LayoutTests/editing to appropriate subdirectories
rniwa@webkit.org [Sat, 2 Jun 2018 00:04:43 +0000 (00:04 +0000)]
Move tests in LayoutTests/editing to appropriate subdirectories
https://bugs.webkit.org/show_bug.cgi?id=186212

Rubber-stamped by Wenson Hsieh.

Moved the following four tests in LayoutTests/editing to its subdirectories.

* editing/input/composition-underline-color-expected-mismatch.html: Moved from editing/composition-underline-color-expected-mismatch.html.
* editing/input/composition-underline-color.html: Move from editing/composition-underline-color.html.
* editing/input/marked-text-appearance-expected-mismatch.html: Copied from LayoutTests/editing/marked-text-appearance-expected-mismatch.html.
* editing/input/marked-text-appearance.html: Moved from editing/marked-text-appearance.html.
* editing/selection/rtl-to-ltr-editing-word-move-spin-expected.txt: Moved from editing/rtl-to-ltr-editing-word-move-spin-expected.txt.
* editing/selection/rtl-to-ltr-editing-word-move-spin.html: Moved from editing/rtl-to-ltr-editing-word-move-spin.html.
* editing/selection/simple-line-layout-caret-is-gone-expected.txt: Moved from editing/simple-line-layout-caret-is-gone-expected.txt.
* editing/selection/simple-line-layout-caret-is-gone.html: Moved from editing/simple-line-layout-caret-is-gone.html.
* editing/style/apply-style-iframe-crash-expected.txt: Moved from editing/apply-style-iframe-crash-expected.txt.
* editing/style/apply-style-iframe-crash.html: Moved from editing/apply-style-iframe-crash.html.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232430 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoSkip fetch/api/redirect/redirect-method.html and fetch/api/redirect/redirect-method...
ryanhaddad@apple.com [Fri, 1 Jun 2018 23:25:41 +0000 (23:25 +0000)]
Skip fetch/api/redirect/redirect-method.html and fetch/api/redirect/redirect-method-worker.html on macOS debug.
https://bugs.webkit.org/show_bug.cgi?id=184365

Unreviewed test gardening.

Patch by David Fenton <david_fenton@apple.com> on 2018-06-01

* platform/mac/TestExpectations:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232428 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoKeyboard focus should exit fullscreen.
commit-queue@webkit.org [Fri, 1 Jun 2018 23:22:57 +0000 (23:22 +0000)]
Keyboard focus should exit fullscreen.
https://bugs.webkit.org/show_bug.cgi?id=185617
rdar://problem/34697938

Patch by Jeremy Jones <jeremyj@apple.com> on 2018-06-01
Reviewed by Ryosuke Niwa.

Source/WebCore:

No opensource test because feature is not enabled.

While in element fullscreen, initiating keyboard focus should exit fullscreen in iOS.

Improve fullscreen testability by exposing isAnimatingFullScreen.

* dom/Document.h:
* testing/Internals.cpp:
(WebCore::Internals::isAnimatingFullScreen const):
* testing/Internals.h:
* testing/Internals.idl:

Source/WebKit:

While in element fullscreen, initiating keyboard focus should exit fullscreen in iOS.

* WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::elementDidFocus):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232427 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoDon't continue playing in background when auto-picture-in-picture is disabled.
commit-queue@webkit.org [Fri, 1 Jun 2018 23:19:58 +0000 (23:19 +0000)]
Don't continue playing in background when auto-picture-in-picture is disabled.
https://bugs.webkit.org/show_bug.cgi?id=186089
rdar://problem/40314314

Patch by Jeremy Jones <jeremyj@apple.com> on 2018-06-01
Reviewed by Eric Carlson.

No new tests because we don't have a way to test auto-pip.

With the refactoring work done in VideoFullscreenInterfaceAVKit, HTMLMediaElement no longer has to anticipate
auto-pip when if it should pause on suspend. Instead VideoFullscreenInterfaceAVKit proactively updates
videoFullscreenMode, so checking for PIP is sufficient.

* html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::shouldOverrideBackgroundPlaybackRestriction const):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232426 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoREGRESSION (r231456): Colloquy is broken
cdumez@apple.com [Fri, 1 Jun 2018 23:09:14 +0000 (23:09 +0000)]
REGRESSION (r231456): Colloquy is broken
https://bugs.webkit.org/show_bug.cgi?id=186197
<rdar://problem/40722960>

Reviewed by Timothy Hatcher.

Restore pre-r231456 behavior for Colloquy app so that it works again.

Source/WebCore:

* html/HTMLFrameElementBase.cpp:
(WebCore::HTMLFrameElementBase::openURL):
* page/Settings.yaml:

Source/WebKitLegacy/mac:

* WebView/WebView.mm:
(-[WebView _needsFrameNameFallbackToIdQuirk]):
(-[WebView _preferencesChanged:]):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232424 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoRegression(r230567): Unable to log into twitter.com in private sessions
cdumez@apple.com [Fri, 1 Jun 2018 22:56:29 +0000 (22:56 +0000)]
Regression(r230567): Unable to log into twitter.com in private sessions
https://bugs.webkit.org/show_bug.cgi?id=186205
<rdar://problem/40670799>

Reviewed by Youenn Fablet.

We were using the same SWServer for all private sessions and the SWServer's sessionID would
be legacyPrivateSessionID(). As a result, the service worker's sessionID would be legacyPrivateSessionID()
Source/WebCore:

as well and would not match the sessionID of its client pages. This sessionID mismatch was
causing the breakage.

Instead of using the same SWServer of all private sessions, we now go back to using a SWServer
per private session. However, we now make sure that the SWServer gets destroyed whenever its
corresponding session gets destroyed.

* workers/service/server/SWServer.cpp:
(WebCore::SWServer::~SWServer):

Source/WebKit:

as well and would not match the sessionID of its client pages. This sessionID mismatch was
causing the breakage.

Instead of using the same SWServer of all private sessions, we now go back to using a SWServer
per private session. However, we now make sure that the SWServer gets destroyed whenever its
corresponding session gets destroyed.

* NetworkProcess/NetworkProcess.cpp:
(WebKit::NetworkProcess::destroySession):
* NetworkProcess/cache/CacheStorageEngine.cpp:
(WebKit::CacheStorage::Engine::from):
* StorageProcess/StorageProcess.cpp:
(WebKit::StorageProcess::destroySession):
(WebKit::StorageProcess::swServerForSession):
* StorageProcess/StorageProcess.h:
* StorageProcess/StorageProcess.messages.in:
* UIProcess/WebProcessPool.cpp:
(WebKit::WebProcessPool::setAnyPageGroupMightHavePrivateBrowsingEnabled):
* UIProcess/WebsiteData/WebsiteDataStore.cpp:
(WebKit::WebsiteDataStore::~WebsiteDataStore):

(WebKit::WebsiteDataStore::enableResourceLoadStatisticsAndSetTestingCallback):
* UIProcess/WebsiteData/WebsiteDataStore.h:
(WebKit::WebsiteDataStore::weakPtrFactory const):
Fix memory leak caused by a reference cycle between the WebsiteDataStore and its
WebResourceLoadStatisticsStore, by using WeakPtr to break the cycle. This was causing
us to leak WebsiteDataStore objects, which would prevent the destruction of sessions.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232423 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoAdd some tests for lldb_webkit.py
dbates@webkit.org [Fri, 1 Jun 2018 22:54:26 +0000 (22:54 +0000)]
Add some tests for lldb_webkit.py
https://bugs.webkit.org/show_bug.cgi?id=183744

Reviewed by Alexey Proskuryakov.

Adds some tests to ensure we do not regress LLDB pretty-printing of WTF::StringImpl
and WTF::String objects.

The tests make use of the LLDB Python API (lldb.py) and a simple debug-built test
program, lldbWebKitTester, to run. For now, we only support building lldbWebKitTester
on Mac.

* Makefile: Build the simple test tool lldbWebKitTester on Mac.
* Scripts/build-lldbwebkittester: Added.
(buildProjectOrDie):
* Scripts/dump-class-layout: Extract logic to compute the path to the LLDB Python module
from here to Scripts/webkitpy/common/system/systemhost.py so that it can used by both
this script and lldb/lldb_webkit_unittest.py. Also import the lldb module at the top of
the file and take advantage of Python's default error semantics to throw an exception
if the import fails instead of handling it ourself. This has the side effect that we
now always import the LLDB Python module even if this script is invoked with --help.
If this turns out to be a significant annoyance then we can look to dynamically import
the module as we did before this change.
(webkit_build_dir):
(main):
(developer_dir): Deleted.
(import_lldb): Deleted.
* Scripts/webkitpy/common/checkout/scm/scm_unittest.py: Update FIXME comment to reflect
that fact that test-webkitpy does not support class and module fixtures. This is because
test-webkitpy currently implements parallelism by breaking down existing test classes
into individual test methods itself and having each worker run exactly one test method (via
unittest.TestLoader.loadTestsFromName()) at a time. As a result of this reorganization,
setUpModule()/setUpClass() are called for each test method as opposed to once per test
class/test module.
(remove_dir): Ditto.
* Scripts/webkitpy/common/system/systemhost.py:
(SystemHost):
(SystemHost.path_to_lldb_python_directory): Added.
* Scripts/webkitpy/test/main.py:
(_find_lldb_webkit_tester): Returns whether there exists a Debug or Release-built lldbWebKitTester.
(_build_lldb_webkit_tester): Builds lldbWebKitTester. For now, we only support building
lldbWebKitTester on Mac.
(main): Add Tools/lldb to the test search path if the platform has lldb.py.
(Tester.run): Pass a boolean as to whether we will run the lldb_webkit unit tests.
(Tester._run_tests): Modified to take a boolean as to whether to run the lldb_webkit unit tests.
If we will run these tests then build lldbWebKitTester if it has not already been built as the
unit tests depend on this program.
* lldb/lldbWebKitTester/Configurations/Base.xcconfig: Added.
* lldb/lldbWebKitTester/Configurations/DebugRelease.xcconfig: Added.
* lldb/lldbWebKitTester/Configurations/lldbWebKitTester.xcconfig: Added.
* lldb/lldbWebKitTester/Makefile: Added.
* lldb/lldbWebKitTester/lldbWebKitTester.xcodeproj/project.pbxproj: Added.
* lldb/lldbWebKitTester/main.cpp: Added.
(breakForTestingSummaryProviders):
(utf16String):
(testSummaryProviders):
(main):
* lldb/lldb_webkit_unittest.py: Added.
(destroy_cached_debug_session):
(LLDBDebugSession):
(LLDBDebugSession.setup):
(LLDBDebugSession.tearDown):
(TestSummaryProviders):
(TestSummaryProviders.setUpClass):
(TestSummaryProviders._sbFrame):
(TestSummaryProviders.serial_test_WTFStringImpl_SummaryProvider_null_string):
(TestSummaryProviders.serial_test_WTFStringImpl_SummaryProvider_empty_string):
(TestSummaryProviders.serial_test_WTFStringImpl_SummaryProvider_8bit_string):
(TestSummaryProviders.serial_test_WTFStringImpl_SummaryProvider_16bit_string):
(TestSummaryProviders.serial_test_WTFString_SummaryProvider_null_string):
(TestSummaryProviders.serial_test_WTFString_SummaryProvider_empty_string):
(TestSummaryProviders.serial_test_WTFString_SummaryProvider_8bit_string):
(TestSummaryProviders.serial_test_WTFString_SummaryProvider_16bit_string):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232421 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoAdd an option to restrict communication to localhost sockets
youenn@apple.com [Fri, 1 Jun 2018 22:34:31 +0000 (22:34 +0000)]
Add an option to restrict communication to localhost sockets
https://bugs.webkit.org/show_bug.cgi?id=186208

Reviewed by Eric Carlson.

Source/WebCore:

Covered by existing tests.
Add an option in LibWebRTCProvider to restrict to localhost sockets.
Use that option when Internals is used.

* platform/mediastream/libwebrtc/LibWebRTCProvider.h:
* testing/Internals.cpp:
(WebCore::Internals::resetToConsistentState):

Source/WebKit:

Implement restriction to localhost sockets by setting any IP address to 127.0.0.1.
This is done on WebProcess side just before requesting to open the socket by NetworkProcess.

* WebProcess/Network/webrtc/LibWebRTCNetwork.h:
(WebKit::LibWebRTCNetwork::disableNonLocalhostConnections):
* WebProcess/Network/webrtc/LibWebRTCProvider.cpp:
(WebKit::LibWebRTCProvider::disableNonLocalhostConnections):
(WebKit::LibWebRTCProvider::registerMDNSName):
* WebProcess/Network/webrtc/LibWebRTCProvider.h:
* WebProcess/Network/webrtc/LibWebRTCSocketFactory.cpp:
(WebKit::prepareSocketAddress):
(WebKit::LibWebRTCSocketFactory::CreateServerTcpSocket):
(WebKit::LibWebRTCSocketFactory::CreateUdpSocket):
(WebKit::LibWebRTCSocketFactory::CreateClientTcpSocket):
* WebProcess/Network/webrtc/LibWebRTCSocketFactory.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232420 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoResourceLoader::cancel() shouldn't synchronously fire load event on document
rniwa@webkit.org [Fri, 1 Jun 2018 22:30:02 +0000 (22:30 +0000)]
ResourceLoader::cancel() shouldn't synchronously fire load event on document
https://bugs.webkit.org/show_bug.cgi?id=185284
Source/WebCore:

Reviewed by Antti Koivisto.

Because a resource loading can be canceled as a node is removed a document or CachedResource is destructed,
it's not safe to synchronously fire load event on document upon cancelation. This patch makes the cancellation
of a resource load schedule m_checkTimer in FrameLoader to fire a load event asynchronously instead.

Specifically, this patch makes FrameLoader::loadDone call FrameLoader::scheduleCheckCompleted when the load
had failed or cancled instead of calling FrameLoader::checkCompleted which can synchronously fire load event.
To differentiate the two cases, new enum LoadCompletionType has been added to FrameLoader::loadDone and related
functions. To avoid calling the navigation delegate too early, the same abstraction for checkLoadComplete()
has been added in the form of FrameLoader::subresourceLoadDone.

Unfortunately, delaying calls to checkCompleted() and checkLoadComplete() by a timer can result in client
callbacks such as didFinishLoadForFrame and didFailLoadWithError to never get called when the frame gets
detached from the parent after the last resource had stopped loading but before the timer fires. To preserve
these deleagte callbacks, this patch expedites the timer in FrameLoader::frameDetached and Page::goToItem by
by invoking newly added stopAllLoadersAndCheckCompleteness, which stops all loading and then immediately invokes
checkCompleted() and checkLoadComplete() synchronously if m_checkTimer had been started.

Tests: http/tests/preload/dynamic_removing_preload.html

* css/CSSFontSelector.cpp:
(WebCore::CSSFontSelector::beginLoadTimerFired): Removed superfluous call to checkLoadComplete since
cachedResourceLoader's loadDone would call checkLoadComplete anyway.
* html/HTMLFrameOwnerElement.cpp:
(WebCore::HTMLFrameOwnerElement::disconnectContentFrame): Removed the misleading comment added in r140090.
Firefox DOES indeed fire unload event in the content document of a removed frame. While this comment made
it sound like this function isn't called when a frame is removed from the tree when in reality we simply
remove a frame prior to removing the node via disconnectSubframesIfNeeded.
* loader/DocumentLoader.cpp:
(WebCore::DocumentLoader::removeSubresourceLoader):
* loader/DocumentLoader.h:
* loader/FrameLoader.cpp:
(WebCore::FrameLoader::FrameLoader):
(WebCore::FrameLoader::loadDone):
(WebCore::FrameLoader::subresourceLoadDone):
(WebCore::FrameLoader::checkCompleted): Added a release assert that this function is only called when it's safe
to execute scripts.
(WebCore::FrameLoader::checkTimerFired):
(WebCore::FrameLoader::checkCompletenessNow):Extracted from checkTimerFired.
(WebCore::FrameLoader::stopAllLoaders): Removed the code to stop m_checkTimer introduced in r53655.
Stopping the timer here would prevent FrameLoader::frameDetached to detect the case when stopping the loader
scheduled a load completion check. Also stopping this timer without clearing the corresponding booleans:
m_checkingLoadCompleteForDetachment and m_checkingLoadCompleteForDetachment is problematic. The assertion
r53655 addressed is now addressed by explicitly checking & clearing the timer in frameDetached.
(WebCore::FrameLoader::stopAllLoadersAndCheckCompleteness): Added.
(WebCore::FrameLoader::checkLoadCompleteForThisFrame): Avoid an early exit when the newly added boolean
m_checkingLoadCompleteForDetachment is set since m_isStopping is no longer set in frameDetached in order
to invoke didFailLoadWithError when detaching a frame.
(WebCore::FrameLoader::frameDetached): Call checkCompletenessNow in the case the frame had already been
completed loading. Also call stopAllLoadersAndCheckCompleteness in the case stopping loading would complete
the loading before stopping active DOM objects.
* loader/FrameLoader.h:
(WebCore::FrameLoader::m_checkingLoadCompleteForDetachment): Added.
* loader/FrameLoaderTypes.h:
(WebCore::LoadCompletionType): Added.
* loader/SubresourceLoader.cpp:
(WebCore::SubresourceLoader::didFinishLoading):
(WebCore::SubresourceLoader::didFail):
(WebCore::SubresourceLoader::didCancel):
(WebCore::SubresourceLoader::notifyDone):
* loader/SubresourceLoader.h:
* loader/cache/CachedResourceLoader.cpp:
(WebCore::CachedResourceLoader::loadDone):
* loader/cache/CachedResourceLoader.h:
* page/Page.cpp:
(WebCore::Page::goToItem): Call stopAllLoadersAndCheckCompleteness instead of stopAllLoaders since stopping
loading here may complete loading.

LayoutTests:

<rdar://problem/39994507>

Reviewed by Antti Koivisto.

Fixed the tests as needed, and skipped more drag & drop tests in WebKitTestRunner as drag & drop isn't supported.

* editing/pasteboard/drag-image-to-contenteditable-in-iframe.html: Fixed the test. Explicitly invoke
testRunner.waitUntilDone() to wait until the iframe is loaded. Because "load" event in DOM only fires after
all subframes are loaded but the load delegate callback fires as long as subresources in the main frame
had finished loading, DumpRenderTree would finish the test prematurely otherwise. The old code happens to
work before this patch because we happen to not invoke FrameLoader::checkComplete at the "right" moment.
The WebCore change now triggers such a check and prematurely end the test without this fix to the test.
* http/tests/xmlhttprequest/reentrant-cancel-expected.txt:
* http/tests/xmlhttprequest/reentrant-cancel.html: Canceling XHR inside addElement is no longer
firing load event synchronously as expected. Added a code to end the test after the load event.
* http/wpt/service-workers/clone-opaque-being-loaded-response.html:
* platform/mac-wk2/TestExpectations:
* platform/wk2/TestExpectations:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232419 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoSome tests for webkitdirectory API fail when tests are in an APFS file system
rniwa@webkit.org [Fri, 1 Jun 2018 21:28:40 +0000 (21:28 +0000)]
Some tests for webkitdirectory API fail when tests are in an APFS file system
https://bugs.webkit.org/show_bug.cgi?id=186184

Reviewed by Chris Dumez.

On an APFS volume, files may not be listed in the lexicologically order and the size of a directory
may defer from that of HFS+. To make the test results consistent on either file systems,
sort file entries lexicologically by the name and list two possible directory sizes as the expectation.

Added a new helper function, shouldBeOneOfValues, which takes an array of values as the expectation.

* editing/pasteboard/entries-api/datatransfer-items-drop-directoryReader.html:
* editing/pasteboard/entries-api/datatransfer-items-drop-getAsEntry-expected.txt:
* editing/pasteboard/entries-api/datatransfer-items-drop-getAsEntry.html:
* fast/forms/file/entries-api/drag-folder-webkitEntries-expected.txt:
* fast/forms/file/entries-api/drag-folder-webkitEntries.html:
* fast/forms/file/entries-api/webkitdirectory-drag-folder.html:
* resources/js-test.js:
(shouldBeOneOfValues): Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232417 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoRegression(r230876): Swipe navigation snapshot may get removed too early
cdumez@apple.com [Fri, 1 Jun 2018 20:58:27 +0000 (20:58 +0000)]
Regression(r230876): Swipe navigation snapshot may get removed too early
https://bugs.webkit.org/show_bug.cgi?id=186168
<rdar://problem/39743617>

Reviewed by Tim Horton.

The swipe navigation snapshot would get removed too early when receiving a paint
event after requesting a history navigation but before the provisional load has
actually started. This is because of the asynchronous navigation policy decision
which occurs after requesting to navigate. To address the issue, we now start
listening for events only after the provisional load has started.

* UIProcess/API/Cocoa/WKWebView.mm:
(-[WKWebView _didStartProvisionalLoadForMainFrame]):
* UIProcess/API/Cocoa/WKWebViewInternal.h:
* UIProcess/Cocoa/ViewGestureController.cpp:
(WebKit::ViewGestureController::didStartProvisionalLoadForMainFrame):
(WebKit::ViewGestureController::didReachMainFrameLoadTerminalState):
(WebKit::ViewGestureController::didSameDocumentNavigationForMainFrame):
* UIProcess/Cocoa/ViewGestureController.h:
* UIProcess/ios/PageClientImplIOS.mm:
(WebKit::PageClientImpl::didStartProvisionalLoadForMainFrame):
* UIProcess/ios/ViewGestureControllerIOS.mm:
(WebKit::ViewGestureController::endSwipeGesture):
* UIProcess/mac/PageClientImplMac.h:
* UIProcess/mac/PageClientImplMac.mm:
(WebKit::PageClientImpl::didStartProvisionalLoadForMainFrame):
* UIProcess/mac/ViewGestureControllerMac.mm:
(WebKit::ViewGestureController::endSwipeGesture):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232416 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoFix the !ENABLE(DFG_JIT) build.
aestes@apple.com [Fri, 1 Jun 2018 20:57:45 +0000 (20:57 +0000)]
Fix the !ENABLE(DFG_JIT) build.

* bytecode/InByIdStatus.cpp:
(JSC::InByIdStatus::computeFor):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232415 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoUnreviewed, build fix for r232276.
jiewen_tan@apple.com [Fri, 1 Jun 2018 20:52:48 +0000 (20:52 +0000)]
Unreviewed, build fix for r232276.

iOS sandbox profiles don't preprocess macros. Therefore, remove the whole
macro condition block.

* Resources/SandboxProfiles/ios/com.apple.WebKit.Networking.sb:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232414 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agohttp/tests/inspector/network/har/har-page.html should actually sort dumped cookies...
dbates@webkit.org [Fri, 1 Jun 2018 20:41:04 +0000 (20:41 +0000)]
http/tests/inspector/network/har/har-page.html should actually sort dumped cookies by name
https://bugs.webkit.org/show_bug.cgi?id=186137

Reviewed by Joseph Pecoraro.

JavaScript does not support comparison of String data types using the minus operator. Instead
implement the string comparator using relational operators and the identity operator.

* http/tests/inspector/network/har/har-page-expected.txt:
* http/tests/inspector/network/har/har-page.html:
* platform/gtk/TestExpectations: Unskip test.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232413 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoFix the watchOS build after r232385
wenson_hsieh@apple.com [Fri, 1 Jun 2018 20:13:15 +0000 (20:13 +0000)]
Fix the watchOS build after r232385
https://bugs.webkit.org/show_bug.cgi?id=186203

Reviewed by Keith Miller.

Add a missing header include for JSImmutableButterfly.

* runtime/ArrayPrototype.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232411 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoStop using StorageTracker.db in LocalStorageDatabaseTracker
sihui_liu@apple.com [Fri, 1 Jun 2018 20:13:08 +0000 (20:13 +0000)]
Stop using StorageTracker.db in LocalStorageDatabaseTracker
https://bugs.webkit.org/show_bug.cgi?id=186104

Reviewed by Geoffrey Garen.

Source/WebCore:

No behavior change.

* platform/sql/SQLiteFileSystem.h:

Source/WebKit:

Stop using StorageTracker.db and stop caching origins in LocalStorageDatabaseTracker for efficiency
and simplicity. Since functions in LocalStorageDatabaseTracker are not frequently called, we get
little benefits from caching origins.

* Platform/Logging.h:
* UIProcess/API/C/WKKeyValueStorageManager.cpp:
(WKKeyValueStorageManagerGetStorageDetailsByOrigin):
* UIProcess/WebStorage/LocalStorageDatabaseTracker.cpp:
(WebKit::LocalStorageDatabaseTracker::LocalStorageDatabaseTracker):
(WebKit::LocalStorageDatabaseTracker::didOpenDatabaseWithOrigin):
(WebKit::LocalStorageDatabaseTracker::deleteDatabaseWithOrigin):
(WebKit::LocalStorageDatabaseTracker::deleteAllDatabases):
(WebKit::LocalStorageDatabaseTracker::databasesModifiedSince):
(WebKit::LocalStorageDatabaseTracker::origins const):
(WebKit::LocalStorageDatabaseTracker::originDetails):
(WebKit::LocalStorageDatabaseTracker::databasePath const):
(WebKit::fileCreationTime): Deleted.
(WebKit::fileModificationTime): Deleted.
(WebKit::LocalStorageDatabaseTracker::trackerDatabasePath const): Deleted.
(WebKit::LocalStorageDatabaseTracker::openTrackerDatabase): Deleted.
(WebKit::LocalStorageDatabaseTracker::importOriginIdentifiers): Deleted.
(WebKit::LocalStorageDatabaseTracker::updateTrackerDatabaseFromLocalStorageDatabaseFiles): Deleted.
(WebKit::LocalStorageDatabaseTracker::addDatabaseWithOriginIdentifier): Deleted.
(WebKit::LocalStorageDatabaseTracker::removeDatabaseWithOriginIdentifier): Deleted.
(WebKit::LocalStorageDatabaseTracker::pathForDatabaseWithOriginIdentifier): Deleted.
* UIProcess/WebStorage/LocalStorageDatabaseTracker.h:
* UIProcess/WebStorage/StorageManager.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232410 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agohttp/tests/appcache/deferred-events.html is flaky due to console log messages
youenn@apple.com [Fri, 1 Jun 2018 19:53:56 +0000 (19:53 +0000)]
http/tests/appcache/deferred-events.html is flaky due to console log messages
https://bugs.webkit.org/show_bug.cgi?id=185543

Reviewed by Chris Dumez.

Update the test to wait for the onerror callback.
At that time, all console logging should be done.

* http/tests/appcache/deferred-events-expected.txt:
* http/tests/appcache/deferred-events.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232409 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[JSC] Add Symbol.prototype.description getter
utatane.tea@gmail.com [Fri, 1 Jun 2018 19:06:36 +0000 (19:06 +0000)]
[JSC] Add Symbol.prototype.description getter
https://bugs.webkit.org/show_bug.cgi?id=186053

Reviewed by Keith Miller.

JSTests:

* stress/symbol-description.js: Added.
(shouldBe):
(shouldThrow):

Source/JavaScriptCore:

Symbol.prototype.description accessor  is now stage 3[1].
This adds a getter to retrieve [[Description]] value from Symbol.
Previously, Symbol#toString() returns `Symbol(${description})` value.
So users need to extract `description` part if they want it.

[1]: https://tc39.github.io/proposal-Symbol-description/

* runtime/Symbol.cpp:
(JSC::Symbol::description const):
* runtime/Symbol.h:
* runtime/SymbolPrototype.cpp:
(JSC::tryExtractSymbol):
(JSC::symbolProtoGetterDescription):
(JSC::symbolProtoFuncToString):
(JSC::symbolProtoFuncValueOf):

LayoutTests:

* js/Object-getOwnPropertyNames-expected.txt:
* js/script-tests/Object-getOwnPropertyNames.js:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232404 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoAdd a demo for passive touch listeners.
dino@apple.com [Fri, 1 Jun 2018 18:55:45 +0000 (18:55 +0000)]
Add a demo for passive touch listeners.

* demos/passive-touches/index.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232402 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[JSC] Correct values and members of JSBigInt appropriately
utatane.tea@gmail.com [Fri, 1 Jun 2018 18:47:26 +0000 (18:47 +0000)]
[JSC] Correct values and members of JSBigInt appropriately
https://bugs.webkit.org/show_bug.cgi?id=186196

Reviewed by Darin Adler.

This patch cleans up a bit to select more appropriate values and members of JSBigInt.

1. JSBigInt's structure should be StructureIsImmortal.
2. JSBigInt::allocationSize should be annotated with `inline`.
3. Remove JSBigInt::visitChildren since it is completely the same to JSCell::visitChildren.
4. Remove JSBigInt::finishCreation since it is completely the same to JSCell::finishCreation.

* runtime/JSBigInt.cpp:
(JSC::JSBigInt::allocationSize):
(JSC::JSBigInt::allocateFor):
(JSC::JSBigInt::compareToDouble):
(JSC::JSBigInt::visitChildren): Deleted.
(JSC::JSBigInt::finishCreation): Deleted.
* runtime/JSBigInt.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232401 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[DFG] InById should be converted to MatchStructure
utatane.tea@gmail.com [Fri, 1 Jun 2018 18:39:57 +0000 (18:39 +0000)]
[DFG] InById should be converted to MatchStructure
https://bugs.webkit.org/show_bug.cgi?id=185803

Reviewed by Keith Miller.

JSTests:

* microbenchmarks/in-by-id-match.js: Added.
(test):
* microbenchmarks/in-by-id-removed.js: Added.
(test):
* stress/in-by-id-constant.js: Added.
(shouldBe):
(test1):
* stress/in-by-id-match-invalid.js: Added.
(shouldBe):
(test1):
* stress/in-by-id-match.js: Added.
(shouldBe):
(test1):

Source/JavaScriptCore:

MatchStructure is introduced for instanceof optimization. But this node
is also useful for InById node. This patch converts InById to MatchStructure
node with CheckStructures if possible by using InByIdStatus.

Added microbenchmarks show improvements.

                           baseline                  patched

in-by-id-removed       18.1196+-0.8108     ^     16.1702+-0.9773        ^ definitely 1.1206x faster
in-by-id-match         16.3912+-0.2608     ^     15.2736+-0.8173        ^ definitely 1.0732x faster

* JavaScriptCore.xcodeproj/project.pbxproj:
* Sources.txt:
* bytecode/InByIdStatus.cpp: Added.
(JSC::InByIdStatus::appendVariant):
(JSC::InByIdStatus::computeFor):
(JSC::InByIdStatus::hasExitSite):
(JSC::InByIdStatus::computeForStubInfo):
(JSC::InByIdStatus::computeForStubInfoWithoutExitSiteFeedback):
(JSC::InByIdStatus::filter):
(JSC::InByIdStatus::dump const):
* bytecode/InByIdStatus.h: Added.
(JSC::InByIdStatus::InByIdStatus):
(JSC::InByIdStatus::state const):
(JSC::InByIdStatus::isSet const):
(JSC::InByIdStatus::operator bool const):
(JSC::InByIdStatus::isSimple const):
(JSC::InByIdStatus::numVariants const):
(JSC::InByIdStatus::variants const):
(JSC::InByIdStatus::at const):
(JSC::InByIdStatus::operator[] const):
(JSC::InByIdStatus::takesSlowPath const):
* bytecode/InByIdVariant.cpp: Added.
(JSC::InByIdVariant::InByIdVariant):
(JSC::InByIdVariant::attemptToMerge):
(JSC::InByIdVariant::dump const):
(JSC::InByIdVariant::dumpInContext const):
* bytecode/InByIdVariant.h: Added.
(JSC::InByIdVariant::isSet const):
(JSC::InByIdVariant::operator bool const):
(JSC::InByIdVariant::structureSet const):
(JSC::InByIdVariant::structureSet):
(JSC::InByIdVariant::conditionSet const):
(JSC::InByIdVariant::offset const):
(JSC::InByIdVariant::isHit const):
* bytecode/PolyProtoAccessChain.h:
* dfg/DFGByteCodeParser.cpp:
(JSC::DFG::ByteCodeParser::parseBlock):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232400 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agomove should only emit the move if it's actually needed
keith_miller@apple.com [Fri, 1 Jun 2018 18:11:36 +0000 (18:11 +0000)]
move should only emit the move if it's actually needed
https://bugs.webkit.org/show_bug.cgi?id=186123

Reviewed by Saam Barati.

This patch relpaces move with moveToDestinationIfNeeded. This
will prevent us from emiting moves to the same location. The old
move, has been renamed to emitMove and made private.

* bytecompiler/BytecodeGenerator.cpp:
(JSC::BytecodeGenerator::BytecodeGenerator):
(JSC::BytecodeGenerator::emitMove):
(JSC::BytecodeGenerator::emitGetGlobalPrivate):
(JSC::BytecodeGenerator::emitGetAsyncIterator):
(JSC::BytecodeGenerator::move): Deleted.
* bytecompiler/BytecodeGenerator.h:
(JSC::BytecodeGenerator::move):
(JSC::BytecodeGenerator::moveToDestinationIfNeeded): Deleted.
* bytecompiler/NodesCodegen.cpp:
(JSC::ThisNode::emitBytecode):
(JSC::SuperNode::emitBytecode):
(JSC::NewTargetNode::emitBytecode):
(JSC::ResolveNode::emitBytecode):
(JSC::TaggedTemplateNode::emitBytecode):
(JSC::ArrayNode::emitBytecode):
(JSC::ObjectLiteralNode::emitBytecode):
(JSC::EvalFunctionCallNode::emitBytecode):
(JSC::FunctionCallResolveNode::emitBytecode):
(JSC::BytecodeIntrinsicNode::emit_intrinsic_putByIdDirect):
(JSC::BytecodeIntrinsicNode::emit_intrinsic_putByIdDirectPrivate):
(JSC::BytecodeIntrinsicNode::emit_intrinsic_putByValDirect):
(JSC::BytecodeIntrinsicNode::emit_intrinsic_toNumber):
(JSC::BytecodeIntrinsicNode::emit_intrinsic_toString):
(JSC::BytecodeIntrinsicNode::emit_intrinsic_toObject):
(JSC::BytecodeIntrinsicNode::emit_intrinsic_idWithProfile):
(JSC::BytecodeIntrinsicNode::emit_intrinsic_isJSArray):
(JSC::BytecodeIntrinsicNode::emit_intrinsic_isProxyObject):
(JSC::BytecodeIntrinsicNode::emit_intrinsic_isRegExpObject):
(JSC::BytecodeIntrinsicNode::emit_intrinsic_isObject):
(JSC::BytecodeIntrinsicNode::emit_intrinsic_isDerivedArray):
(JSC::BytecodeIntrinsicNode::emit_intrinsic_isMap):
(JSC::BytecodeIntrinsicNode::emit_intrinsic_isSet):
(JSC::CallFunctionCallDotNode::emitBytecode):
(JSC::ApplyFunctionCallDotNode::emitBytecode):
(JSC::emitPostIncOrDec):
(JSC::PostfixNode::emitBracket):
(JSC::PostfixNode::emitDot):
(JSC::PrefixNode::emitResolve):
(JSC::PrefixNode::emitBracket):
(JSC::PrefixNode::emitDot):
(JSC::LogicalOpNode::emitBytecode):
(JSC::ReadModifyResolveNode::emitBytecode):
(JSC::AssignResolveNode::emitBytecode):
(JSC::AssignDotNode::emitBytecode):
(JSC::AssignBracketNode::emitBytecode):
(JSC::FunctionNode::emitBytecode):
(JSC::ClassExprNode::emitBytecode):
(JSC::DestructuringAssignmentNode::emitBytecode):
(JSC::ArrayPatternNode::emitDirectBinding):
(JSC::ObjectPatternNode::bindValue const):
(JSC::AssignmentElementNode::bindValue const):
(JSC::ObjectSpreadExpressionNode::emitBytecode):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232399 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoexport-w3c-test-changes cannot apply patches when some w3c-import.log files are changed
commit-queue@webkit.org [Fri, 1 Jun 2018 16:36:01 +0000 (16:36 +0000)]
export-w3c-test-changes cannot apply patches when some w3c-import.log files are changed
https://bugs.webkit.org/show_bug.cgi?id=186133

Patch by Frederic Wang <fwang@igalia.com> on 2018-06-01
Reviewed by Youenn Fablet.

* Scripts/webkitpy/w3c/test_exporter.py: Exclude w3c-import.log from the diff.
* Scripts/webkitpy/w3c/test_exporter_unittest.py: Add unit test.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232398 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[GTK] Crash in WebKitFaviconDatabase when pageURL is unset
mcatanzaro@igalia.com [Fri, 1 Jun 2018 16:12:14 +0000 (16:12 +0000)]
[GTK] Crash in WebKitFaviconDatabase when pageURL is unset
https://bugs.webkit.org/show_bug.cgi?id=186164

Reviewed by Carlos Garcia Campos.

PageURL can legitimately be null here if JavaScript does something silly with window.open.

* UIProcess/API/glib/WebKitFaviconDatabase.cpp:
(webkitFaviconDatabaseSetIconURLForPageURL):
(webkitFaviconDatabaseSetIconForPageURL):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232397 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[Cocoa] Web Automation: use the session delegate to perform window maximize
bburg@apple.com [Fri, 1 Jun 2018 16:08:18 +0000 (16:08 +0000)]
[Cocoa] Web Automation: use the session delegate to perform window maximize
https://bugs.webkit.org/show_bug.cgi?id=186167
<rdar://problem/40544391>

Reviewed by Timothy Hatcher.

Add plumbing to hand off window maximizing to the session delegate.

* UIProcess/API/Cocoa/_WKAutomationSessionDelegate.h:
* UIProcess/Cocoa/AutomationSessionClient.h:
* UIProcess/Cocoa/AutomationSessionClient.mm:
(WebKit::AutomationSessionClient::AutomationSessionClient):
(WebKit::AutomationSessionClient::requestMaximizeWindowOfPage):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232396 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoUnskip fast/html/marquee-reparent-check.html on macOS debug.
bfulgham@apple.com [Fri, 1 Jun 2018 16:00:45 +0000 (16:00 +0000)]
Unskip fast/html/marquee-reparent-check.html on macOS debug.
https://bugs.webkit.org/show_bug.cgi?id=178465
<rdar://problem/35052953>

Reviewed by Geoffrey Garen.

Unskipping test since the debug assertion is no longer reproducible in
current sources on shipping software.

* platform/mac/TestExpectations:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232395 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[LFC] Simplify the formatting class implementation by pushing down some of the logic...
zalan@apple.com [Fri, 1 Jun 2018 14:39:38 +0000 (14:39 +0000)]
[LFC] Simplify the formatting class implementation by pushing down some of the logic to the Geometry class
https://bugs.webkit.org/show_bug.cgi?id=186179

Reviewed by Antti Koivisto.

* layout/FormattingContext.cpp:
(WebCore::Layout::FormattingContext::computeOutOfFlowPosition const):
(WebCore::Layout::FormattingContext::computeOutOfFlowWidth const):
(WebCore::Layout::FormattingContext::computeFloatingWidth const):
(WebCore::Layout::FormattingContext::computeOutOfFlowHeight const):
(WebCore::Layout::FormattingContext::computeFloatingHeight const):
* layout/FormattingContext.h:
* layout/FormattingContextGeometry.cpp:
(WebCore::Layout::outOfFlowNonReplacedHeight):
(WebCore::Layout::outOfFlowNonReplacedWidth):
(WebCore::Layout::outOfFlowReplacedHeight):
(WebCore::Layout::outOfFlowReplacedWidth):
(WebCore::Layout::floatingNonReplacedHeight):
(WebCore::Layout::floatingNonReplacedWidth):
(WebCore::Layout::floatingReplacedHeight):
(WebCore::Layout::floatingReplacedWidth):
(WebCore::Layout::outOfFlowNonReplacedPosition):
(WebCore::Layout::outOfFlowReplacedPosition):
(WebCore::Layout::FormattingContext::Geometry::outOfFlowHeight):
(WebCore::Layout::FormattingContext::Geometry::outOfFlowWidth):
(WebCore::Layout::FormattingContext::Geometry::floatingHeight):
(WebCore::Layout::FormattingContext::Geometry::floatingWidth):
(WebCore::Layout::FormattingContext::Geometry::outOfFlowPosition):
(WebCore::Layout::FormattingContext::Geometry::inlineReplacedHeight):
(WebCore::Layout::FormattingContext::Geometry::inlineReplacedWidth):
(WebCore::Layout::FormattingContext::Geometry::outOfFlowNonReplacedHeight): Deleted.
(WebCore::Layout::FormattingContext::Geometry::outOfFlowNonReplacedWidth): Deleted.
(WebCore::Layout::FormattingContext::Geometry::outOfFlowReplacedHeight): Deleted.
(WebCore::Layout::FormattingContext::Geometry::outOfFlowReplacedWidth): Deleted.
(WebCore::Layout::FormattingContext::Geometry::floatingNonReplacedHeight): Deleted.
(WebCore::Layout::FormattingContext::Geometry::floatingNonReplacedWidth): Deleted.
(WebCore::Layout::FormattingContext::Geometry::floatingReplacedHeight): Deleted.
(WebCore::Layout::FormattingContext::Geometry::floatingReplacedWidth): Deleted.
(WebCore::Layout::FormattingContext::Geometry::outOfFlowNonReplacedPosition): Deleted.
(WebCore::Layout::FormattingContext::Geometry::outOfFlowReplacedPosition): Deleted.
(WebCore::Layout::FormattingContext::Geometry::replacedHeight): Deleted.
(WebCore::Layout::FormattingContext::Geometry::replacedWidth): Deleted.
* layout/blockformatting/BlockFormattingContext.cpp:
(WebCore::Layout::BlockFormattingContext::computeStaticPosition const):
(WebCore::Layout::BlockFormattingContext::computeInFlowPositionedPosition const):
(WebCore::Layout::BlockFormattingContext::computeInFlowHeight const):
(WebCore::Layout::BlockFormattingContext::computeInFlowWidth const):
* layout/blockformatting/BlockFormattingContext.h:
* layout/blockformatting/BlockFormattingContextGeometry.cpp:
(WebCore::Layout::inFlowNonReplacedHeight):
(WebCore::Layout::inFlowNonReplacedWidth):
(WebCore::Layout::BlockFormattingContext::Geometry::inFlowHeight):
(WebCore::Layout::BlockFormattingContext::Geometry::inFlowWidth):
(WebCore::Layout::BlockFormattingContext::Geometry::inFlowNonReplacedHeight): Deleted.
(WebCore::Layout::BlockFormattingContext::Geometry::inFlowNonReplacedWidth): Deleted.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232394 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[LFC] Push FormattingContext::computeWidth/computeHeight logic from the base to the...
zalan@apple.com [Fri, 1 Jun 2018 14:07:24 +0000 (14:07 +0000)]
[LFC] Push FormattingContext::computeWidth/computeHeight logic from the base to the subclasses.
https://bugs.webkit.org/show_bug.cgi?id=186172

Reviewed by Antti Koivisto.

FormattingContext::computeWidth/computeHeight virtual functions have very little benefit and in the long run
it's better if we keep the number of virtual functions to the minimum necessary.

* layout/FormattingContext.cpp:
(WebCore::Layout::FormattingContext::computeFloatingWidth const):
(WebCore::Layout::FormattingContext::computeStaticPosition const): Deleted.
(WebCore::Layout::FormattingContext::computeInFlowPositionedPosition const): Deleted.
(WebCore::Layout::FormattingContext::computeWidth const): Deleted.
(WebCore::Layout::FormattingContext::computeHeight const): Deleted.
(WebCore::Layout::FormattingContext::computeMargin const): Deleted.
* layout/FormattingContext.h:
* layout/FormattingContextGeometry.cpp:
(WebCore::Layout::FormattingContext::Geometry::outOfFlowReplacedHeight):
(WebCore::Layout::FormattingContext::Geometry::outOfFlowReplacedWidth):
(WebCore::Layout::FormattingContext::Geometry::floatingReplacedHeight):
(WebCore::Layout::FormattingContext::Geometry::floatingReplacedWidth):
(WebCore::Layout::FormattingContext::Geometry::inlineReplacedHeight):
(WebCore::Layout::FormattingContext::Geometry::inlineReplacedWidth):
(WebCore::Layout::FormattingContext::Geometry::replacedHeight): Deleted.
(WebCore::Layout::FormattingContext::Geometry::replacedWidth): Deleted.
* layout/blockformatting/BlockFormattingContext.cpp:
(WebCore::Layout::BlockFormattingContext::computeWidth const):
(WebCore::Layout::BlockFormattingContext::computeHeight const):
(WebCore::Layout::BlockFormattingContext::computeInFlowHeight const):
(WebCore::Layout::BlockFormattingContext::computeInFlowWidth const):
* layout/blockformatting/BlockFormattingContext.h:
* layout/inlineformatting/InlineFormattingContext.cpp:
(WebCore::Layout::InlineFormattingContext::computeStaticPosition const):
(WebCore::Layout::InlineFormattingContext::computeInFlowPositionedPosition const):
(WebCore::Layout::InlineFormattingContext::computeInFlowWidth const): Deleted.
(WebCore::Layout::InlineFormattingContext::computeInFlowHeight const): Deleted.
* layout/inlineformatting/InlineFormattingContext.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232393 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[GTK] Difficult to read combo box text in dark theme
commit-queue@webkit.org [Fri, 1 Jun 2018 13:02:39 +0000 (13:02 +0000)]
[GTK] Difficult to read combo box text in dark theme
https://bugs.webkit.org/show_bug.cgi?id=165072

Patch by Carlos Eduardo Ramalho <cadubentzen@gmail.com> on 2018-06-01
Reviewed by Carlos Garcia Campos.

Set listbox foreground color to theme color to fix hard-to-read
text color in combo box while using dark theme.

No new tests required. ManualTests/gtk/theme.html already covers it.
See the "Option Lists" section.

* rendering/RenderTheme.cpp:
(WebCore::RenderTheme::adjustStyle):
Adjust style for ListboxPart as well.
* rendering/RenderTheme.h: Add adjustListboxStyle()
* rendering/RenderThemeGtk.cpp:
(WebCore::RenderThemeGtk::adjustListboxStyle const):
Set color to the theme foreground color.
* rendering/RenderThemeGtk.h: adjustListboxStyle() overriden.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232392 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoUnreviewed. Try to fix GTK+ build with old versions of GTK+ after r232390.
carlosgc@webkit.org [Fri, 1 Jun 2018 08:09:44 +0000 (08:09 +0000)]
Unreviewed. Try to fix GTK+ build with old versions of GTK+ after r232390.

* UIProcess/gtk/WebPopupMenuProxyGtk.cpp:
(WebKit::WebPopupMenuProxyGtk::showPopupMenu):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232391 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[GTK] Switch to use a popup window with a tree view instead of a menu for option...
carlosgc@webkit.org [Fri, 1 Jun 2018 07:44:31 +0000 (07:44 +0000)]
[GTK] Switch to use a popup window with a tree view instead of a menu for option menu default implementation
https://bugs.webkit.org/show_bug.cgi?id=186146

Reviewed by Michael Catanzaro.

Source/WebCore:

Make it possible to use GUniquePtr with GtkTreePath.

* platform/gtk/GUniquePtrGtk.h:

Source/WebKit:

It's more convenient to use than the menu.

* UIProcess/API/gtk/WebKitPopupMenu.cpp:
(WebKit::menuCloseCallback):
(WebKit::WebKitPopupMenu::activateItem):
* UIProcess/API/gtk/WebKitPopupMenu.h:
* UIProcess/gtk/WebPopupMenuProxyGtk.cpp:
(WebKit::WebPopupMenuProxyGtk::WebPopupMenuProxyGtk):
(WebKit::WebPopupMenuProxyGtk::selectItem):
(WebKit::WebPopupMenuProxyGtk::activateItem):
(WebKit::WebPopupMenuProxyGtk::activateItemAtPath):
(WebKit::WebPopupMenuProxyGtk::treeViewRowActivatedCallback):
(WebKit::WebPopupMenuProxyGtk::treeViewButtonReleaseEventCallback):
(WebKit::WebPopupMenuProxyGtk::buttonPressEventCallback):
(WebKit::WebPopupMenuProxyGtk::keyPressEventCallback):
(WebKit::WebPopupMenuProxyGtk::createPopupMenu):
(WebKit::WebPopupMenuProxyGtk::show):
(WebKit::WebPopupMenuProxyGtk::showPopupMenu):
(WebKit::WebPopupMenuProxyGtk::hidePopupMenu):
(WebKit::WebPopupMenuProxyGtk::cancelTracking):
(WebKit::WebPopupMenuProxyGtk::typeAheadFindIndex):
(WebKit::WebPopupMenuProxyGtk::typeAheadFind):
* UIProcess/gtk/WebPopupMenuProxyGtk.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232390 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoexport-w3c-test-changes should use the new location & name of the WPT repo
fred.wang@free.fr [Fri, 1 Jun 2018 06:06:43 +0000 (06:06 +0000)]
export-w3c-test-changes should use the new location & name of the WPT repo
https://bugs.webkit.org/show_bug.cgi?id=186135

Patch by Frederic Wang <fwang@igalia.com> on 2018-05-31
Reviewed by Youenn Fablet.

Web platform tests have been moved to GitHub's web-platform-tests/wpt. This patch updates the
corresponding variables to use the new organization and repository name. It also refactors a
bit the code to reduce the amount of hardcoded strings.

* Scripts/webkitpy/w3c/common.py: Use org web-platform-tests, repo name wpt. Also removes the
final slash to WPT_GH_URL, so it can be used for repo URL too.
* Scripts/webkitpy/w3c/test_exporter.py: Import more symbols and define WPT_PR_URL from
WPT_GH_URL.
(WebPlatformTestExporter._wpt_fork_branch_github_url): Use WPT_GH_REPO_NAME.
(WebPlatformTestExporter._wpt_fork_push_url): Use WPT_GH_REPO_NAME.
(WebPlatformTestExporter._git): Use  WPT_GH_URL.
(WebPlatformTestExporter._ensure_username_and_token): Use WPT_GH_ORG, WPT_GH_REPO_NAME.
(WebPlatformTestExporter.make_pull_request): Use WPT_GH_ORG, WPT_GH_REPO_NAME.
(parse_args): Use WPT_GH_REPO_NAME, WPT_GH_URL.
* Scripts/webkitpy/w3c/test_exporter_unittest.py:
(TestExporterTest.test_export): Update organization and repository names for the remote
branch and pull request.
(TestExporterTest.test_export_with_specific_branch): Update repository name.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232389 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoSkip http/tests/security/referrer-policy-header.html on Windows.
pvollan@apple.com [Fri, 1 Jun 2018 04:46:05 +0000 (04:46 +0000)]
Skip http/tests/security/referrer-policy-header.html on Windows.
https://bugs.webkit.org/show_bug.cgi?id=186183

Unreviewed test gardening.

* platform/win/TestExpectations:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232388 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[Baseline] Store constant directly in emit_op_mov
utatane.tea@gmail.com [Fri, 1 Jun 2018 04:45:24 +0000 (04:45 +0000)]
[Baseline] Store constant directly in emit_op_mov
https://bugs.webkit.org/show_bug.cgi?id=186182

Reviewed by Saam Barati.

In the old code, we first move a constant to a register and store it to the specified address.
But in 64bit JSC, we can directly store a constant to the specified address. This reduces the
generated code size. Since the old code was emitting a constant in a code anyway, this change
never increases the size of the generated code.

* jit/JITInlines.h:
(JSC::JIT::emitGetVirtualRegister):
We remove this obsolete comment. Our OSR relies on the fact that values are stored and loaded
from the stack. If we transfer values in registers without loading values from the stack, it
breaks this assumption.

* jit/JITOpcodes.cpp:
(JSC::JIT::emit_op_mov):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232387 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[ESNext][BigInt] Implement support for "=<" and ">=" relational operation
ticaiolima@gmail.com [Fri, 1 Jun 2018 03:30:22 +0000 (03:30 +0000)]
[ESNext][BigInt] Implement support for "=<" and ">=" relational operation
https://bugs.webkit.org/show_bug.cgi?id=185929

Reviewed by Yusuke Suzuki.

JSTests:

* bigIntTests.yaml:
* stress/big-int-greater-than-or-equal-jit.js: Added.
* stress/big-int-greater-than-or-equal-order-of-evaluation.js: Added.
* stress/big-int-greater-than-or-equal-wrapped-values.js: Added.
* stress/big-int-greater-than-or-equal.js: Added.
* stress/big-int-less-than-or-equal-general.js: Added.
* stress/big-int-less-than-or-equal-jit.js: Added.
* stress/big-int-less-than-or-equal-order-of-evaluation.js: Added.
* stress/big-int-less-than-or-equal-wrapped-values.js: Added.

Source/JavaScriptCore:

This patch is introducing support to BigInt operands into ">=" and
"<=" operators.
Here we introduce ```bigIntCompareResult``` that is a helper function
to reuse code between "less than" and "less than or equal" operators.

* runtime/JSBigInt.h:
* runtime/Operations.h:
(JSC::bigIntCompareResult):
(JSC::bigIntCompare):
(JSC::jsLess):
(JSC::jsLessEq):
(JSC::bigIntCompareLess): Deleted.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232386 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoCache toString results for CoW arrays
sbarati@apple.com [Fri, 1 Jun 2018 03:07:28 +0000 (03:07 +0000)]
Cache toString results for CoW arrays
https://bugs.webkit.org/show_bug.cgi?id=186160

Reviewed by Keith Miller.

JSTests:

* microbenchmarks/to-string-on-cow-array.js: Added.
(foo):

Source/JavaScriptCore:

This patch makes it so that we cache the result of toString on
arrays with a CoW butterfly. This cache lives on Heap and is
cleared after every GC. We only cache the toString result when
the CoW butterfly doesn't have a hole (currently, all CoW arrays
have a hole, but this isn't an invariant we want to rely on). The
reason for this is that if there is a hole, the value may be loaded
from the prototype, and the cache may produce a stale result.

This is a ~4% speedup on the ML subtest in ARES. And is a ~1% overall
progression on ARES.

* heap/Heap.cpp:
(JSC::Heap::finalize):
(JSC::Heap::addCoreConstraints):
* heap/Heap.h:
* runtime/ArrayPrototype.cpp:
(JSC::canUseFastJoin):
(JSC::holesMustForwardToPrototype):
(JSC::isHole):
(JSC::containsHole):
(JSC::fastJoin):
(JSC::arrayProtoFuncToString):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232385 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoPutStructure AI rule needs to call didFoldClobberStructures when the incoming value...
sbarati@apple.com [Fri, 1 Jun 2018 03:06:25 +0000 (03:06 +0000)]
PutStructure AI rule needs to call didFoldClobberStructures when the incoming value's structure set is clear
https://bugs.webkit.org/show_bug.cgi?id=186169

Reviewed by Mark Lam.

If we don't do this, the CFA validation rule about StructureID being
clobbered but AI not clobbering or folding a clobber will cause us
to crash. Simon was running into this yesterday on arstechnica.com.
I couldn't come up with a test case for this, but it's obvious
what the issue is by looking at the IR dump at the time of the crash.

* dfg/DFGAbstractInterpreterInlines.h:
(JSC::DFG::AbstractInterpreter<AbstractStateType>::executeEffects):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232384 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoJSImmutableButterfly should align its variable storage
sbarati@apple.com [Fri, 1 Jun 2018 00:43:19 +0000 (00:43 +0000)]
JSImmutableButterfly should align its variable storage
https://bugs.webkit.org/show_bug.cgi?id=186159

Reviewed by Mark Lam.

I'm also making the use of reinterpret_cast and bitwise_cast consistent
inside of JSImmutableButterfly. I switched everything to use bitwise_cast.

* runtime/JSImmutableButterfly.h:
(JSC::JSImmutableButterfly::toButterfly const):
(JSC::JSImmutableButterfly::fromButterfly):
(JSC::JSImmutableButterfly::offsetOfData):
(JSC::JSImmutableButterfly::allocationSize):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232383 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoDFGArrayModes needs to know more about CoW arrays
keith_miller@apple.com [Thu, 31 May 2018 23:50:15 +0000 (23:50 +0000)]
DFGArrayModes needs to know more about CoW arrays
https://bugs.webkit.org/show_bug.cgi?id=186162

Reviewed by Filip Pizlo.

This patch fixes two issues in DFGArrayMode.

1) fromObserved was missing switch cases for when the only observed ArrayModes are CopyOnWrite.
2) DFGArrayModes needs to track if the ArrayClass is an OriginalCopyOnWriteArray in order
to vend an accurate original structure.

Additionally, this patch fixes some places in Bytecode parsing where we told the array mode
we were doing a read but actually doing a write. Also, DFGArrayMode will now print the
action it is expecting when being dumped.

* bytecode/ArrayProfile.h:
(JSC::hasSeenWritableArray):
* dfg/DFGArrayMode.cpp:
(JSC::DFG::ArrayMode::fromObserved):
(JSC::DFG::ArrayMode::refine const):
(JSC::DFG::ArrayMode::originalArrayStructure const):
(JSC::DFG::arrayActionToString):
(JSC::DFG::arrayClassToString):
(JSC::DFG::ArrayMode::dump const):
(WTF::printInternal):
* dfg/DFGArrayMode.h:
(JSC::DFG::ArrayMode::withProfile const):
(JSC::DFG::ArrayMode::isJSArray const):
(JSC::DFG::ArrayMode::isJSArrayWithOriginalStructure const):
(JSC::DFG::ArrayMode::arrayModesWithIndexingShape const):
* dfg/DFGByteCodeParser.cpp:
(JSC::DFG::ByteCodeParser::handleIntrinsicCall):
(JSC::DFG::ByteCodeParser::parseBlock):
* dfg/DFGFixupPhase.cpp:
(JSC::DFG::FixupPhase::fixupNode):
* dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::SpeculativeJIT::jumpSlowForUnwantedArrayMode):
* ftl/FTLLowerDFGToB3.cpp:
(JSC::FTL::DFG::LowerDFGToB3::isArrayTypeForArrayify):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232376 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoAdd OpenGL display mask to WebPage creation parameters.
pvollan@apple.com [Thu, 31 May 2018 23:43:13 +0000 (23:43 +0000)]
Add OpenGL display mask to WebPage creation parameters.
https://bugs.webkit.org/show_bug.cgi?id=186163
<rdar://problem/40634504>

Reviewed by Brent Fulgham.

To make sure the OpenGL display mask is always available, include it in the WebPage creation parameters.
The OpenGL display mask is sent to the WebProcess when the platform display ID changes, but that is not
early enough in all cases. If the OpenGL display mask is not set, only OpenGL software rendering is offered
on some hardware configurations.
Source/WebCore:

No new tests, since it is not trivial to test whether OpenGL rendering is hardware accelerated.

* platform/PlatformScreen.h:
* platform/mac/PlatformScreenMac.mm:
(WebCore::displayID):

Source/WebKit:

* Shared/WebPageCreationParameters.cpp:
(WebKit::WebPageCreationParameters::encode const):
(WebKit::WebPageCreationParameters::decode):
* Shared/WebPageCreationParameters.h:
* UIProcess/WebPageProxy.cpp:
(WebKit::WebPageProxy::creationParameters):
* WebProcess/WebPage/WebPage.cpp:
(WebKit::m_credentialsMessenger):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232375 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoVersioning.
kocsen_chung@apple.com [Thu, 31 May 2018 22:02:39 +0000 (22:02 +0000)]
Versioning.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232374 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoAdd a rule to allow reading files with prefix /private/var/db/CVMS/cvmsCodeSignObj
bfulgham@apple.com [Thu, 31 May 2018 21:23:32 +0000 (21:23 +0000)]
Add a rule to allow reading files with prefix /private/var/db/CVMS/cvmsCodeSignObj
https://bugs.webkit.org/show_bug.cgi?id=186157
<rdar://problem/40666437>

Reviewed by Eric Carlson.

* WebProcess/com.apple.WebProcess.sb.in:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232373 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoAdd setting to allow override screen size to be disabled.
megan_gardner@apple.com [Thu, 31 May 2018 21:03:36 +0000 (21:03 +0000)]
Add setting to allow override screen size to be disabled.
https://bugs.webkit.org/show_bug.cgi?id=186109

Reviewed by Andy Estes.

Source/WebCore:

Not Testable, and for debug only.

Pipe settings and use them to ignore override screen size if setting is set.

* page/DeprecatedGlobalSettings.cpp:
* page/DeprecatedGlobalSettings.h:
(WebCore::DeprecatedGlobalSettings::setDisableScreenSizeOverride):
(WebCore::DeprecatedGlobalSettings::disableScreenSizeOverride):
* page/Settings.yaml:
* platform/ios/PlatformScreenIOS.mm:

Source/WebKit:

Pipe the needed settings through to WebCore.

* Shared/WebPreferences.yaml:
* UIProcess/WebPageProxy.cpp:
(WebKit::m_configurationPreferenceValues):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232372 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[css-text] Update test suite
rego@igalia.com [Thu, 31 May 2018 20:58:13 +0000 (20:58 +0000)]
[css-text] Update test suite
https://bugs.webkit.org/show_bug.cgi?id=186145

Reviewed by Youenn Fablet.

These tests used to have problems in Mac and iOS platforms,
but they were reworked in WPT upstream:
https://github.com/web-platform-tests/wpt/pull/10860

Importing the new version of the tests in this patch.

LayoutTests/imported/w3c:

* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-001-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-001.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-002-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-002.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-003-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-003.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-004-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-004.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-005-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-005.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-006-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-006.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-007-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-007.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-008-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-008.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-009-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-009.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-010-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-010.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-011-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-011.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-012-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-012.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-013-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-013.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-014-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-014.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-015-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-015.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-016-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-016.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-017-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-017.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-018-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-018.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-019-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-019.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-020-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-020.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-021-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-021.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-022-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-022.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-023-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-023.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-024-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-024.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-025-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-025.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-026-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-026.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-027-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-027.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-028-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-028.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-029-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-029.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-030-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-030.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-031-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-031.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-032-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-032.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-033-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-033.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-034-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-034.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-035-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-035.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-036-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-036.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-037-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-037.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-038-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-038.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-039-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-039.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-040-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-040.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-041-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-041.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-042-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-042.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-043-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-043.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-044-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-044.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-045-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-045.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-046-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-046.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-047-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-047.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-048-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-048.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-049-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-049.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-050-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-050.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-051-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-051.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-052-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-052.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-054-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-054.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-055-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-055.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-056-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-056.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-057-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-057.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-058-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-058.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-059-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-059.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-060-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-060.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-101-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-101.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-102-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-102.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-103-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-103.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-104-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-104.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-105-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-105.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-106-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-106.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-107-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-107.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-108-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-108.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-109-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-109.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-110-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-110.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-111-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-111.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-112-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-112.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-113-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-113.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-114-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-114.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-115-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-115.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-116-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-116.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-117-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-117.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-118-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-118.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-119-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-119.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-120-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-120.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-121-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-121.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-122-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-122.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-123-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-123.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-124-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-124.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-125-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-125.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-126-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-126.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-127-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-127.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-128-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-128.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-129-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-129.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-130-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-130.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-131-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-131.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-132-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-132.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-133-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-133.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-134-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-134.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-135-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-135.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-138-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-138.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-139-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-139.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-140-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-140.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-141-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-141.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-144-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-144.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-145-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-145.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-146-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-146.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-147-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-147.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-148-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-148.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-149-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-149.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-152-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-152.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-154-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-154.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-155-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-155.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-156-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-156.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-157-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-157.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-158-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-158.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-159-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-159.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-160-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-160.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-201-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-201.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-202-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-202.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-203-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-203.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-204-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-204.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-205-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-205.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-206-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-206.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-207-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-207.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-208-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-208.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-209-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-209.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-210-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-210.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-211-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-211.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-212-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-212.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-213-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-213.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-214-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-214.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-215-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-215.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-216-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-216.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-217-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-217.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-218-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-218.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-219-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-219.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-220-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-220.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-221-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-221.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-222-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-222.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-223-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-223.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-224-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-224.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-225-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-225.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-226-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-226.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-227-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-227.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-228-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-228.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-229-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-229.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-230-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-230.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-231-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-231.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-232-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-232.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-233-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-233.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-234-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-234.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-235-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-235.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-236-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-236.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-237-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-237.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-238-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-238.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-239-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-239.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-240-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-240.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-241-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-241.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-242-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-242.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-243-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-243.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-244-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-244.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-245-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-245.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-246-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-246.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-247-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-247.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-248-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-248.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-249-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-249.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-250-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-250.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-251-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-251.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-252-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-252.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-254-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-254.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-255-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-255.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-256-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-256.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-257-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-257.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-258-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-258.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-259-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-259.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-260-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-260.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-301-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-301.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-302-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-302.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-303-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-303.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-304-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-304.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-305-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-305.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-306-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-306.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-307-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-307.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-308-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-308.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-309-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-309.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-310-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-310.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-311-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-311.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-312-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-312.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-313-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-313.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-314-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-314.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-315-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-315.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-316-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-316.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-317-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-317.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-318-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-318.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-319-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-319.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-320-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-320.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-321-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-321.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-322-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-322.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-323-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-323.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-324-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-324.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-325-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-325.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-326-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-326.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-327-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-327.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-351-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-351.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-352-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-352.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-353-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-353.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-354-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-354.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-355-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-355.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-358-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-358.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-359-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-359.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-362-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-362.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-363-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-363.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-364-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-364.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-365-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-365.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-366-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-366.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-367-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-367.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-370-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-370.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-372-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-372.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-373-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-373.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-374-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-374.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-375-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-375.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-376-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-376.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-377-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-377.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-401-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-401.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-402-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-402.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-403-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-403.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-404-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-404.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-405-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-405.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-406-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-406.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-407-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-407.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-408-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-408.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-409-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-409.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-410-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-410.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-411-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-411.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-412-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-412.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-413-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-413.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-414-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-414.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-415-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-415.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-416-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-416.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-417-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-417.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-418-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-418.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-419-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-419.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-420-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-420.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-421-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-421.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-422-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-422.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-423-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-423.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-424-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-424.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-425-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-425.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-426-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-426.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-427-expected.html: Removed.
* web-platform-tests/css/css-text/i18n/css3-text-line-break-jazh-427.html: Removed.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-cj-loose-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-cj-loose.html: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-cj-normal-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-cj-normal.html: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-cj-strict-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-cj-strict.html: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-cpm-loose-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-cpm-loose.html: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-cpm-normal-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-cpm-normal.html: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-cpm-strict-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-cpm-strict.html: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-hyphens-loose-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-hyphens-loose.html: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-hyphens-normal-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-hyphens-normal.html: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-hyphens-strict-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-hyphens-strict.html: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-in-loose-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-in-loose.html: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-in-normal-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-in-normal.html: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-in-strict-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-in-strict.html: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-iteration-loose-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-iteration-loose.html: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-iteration-normal-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-iteration-normal.html: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-iteration-strict-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-iteration-strict.html: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-po-loose-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-po-loose.html: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-po-normal-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-po-normal.html: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-po-strict-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/ja/css-text-line-break-ja-po-strict.html: Added.
* web-platform-tests/css/css-text/i18n/ja/w3c-import.log: Added.
* web-platform-tests/css/css-text/i18n/w3c-import.log:
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-cpm-loose-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-cpm-loose.html: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-cpm-normal-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-cpm-normal.html: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-cpm-strict-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-cpm-strict.html: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-hyphens-loose-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-hyphens-loose.html: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-hyphens-normal-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-hyphens-normal.html: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-hyphens-strict-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-hyphens-strict.html: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-in-loose-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-in-loose.html: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-in-normal-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-in-normal.html: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-in-strict-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-in-strict.html: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-iteration-loose-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-iteration-loose.html: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-iteration-normal-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-iteration-normal.html: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-iteration-strict-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-iteration-strict.html: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-po-loose-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-po-loose.html: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-po-normal-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-po-normal.html: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-po-strict-expected.txt: Added.
* web-platform-tests/css/css-text/i18n/zh/css-text-line-break-zh-po-strict.html: Added.
* web-platform-tests/css/css-text/i18n/zh/w3c-import.log: Added.

LayoutTests:

* platform/ios/TestExpectations:
* platform/mac/TestExpectations:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232371 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoREGRESSION(r216119): DocumentLoader::detachFromFrame still encounters nullptr frame
bfulgham@apple.com [Thu, 31 May 2018 20:54:24 +0000 (20:54 +0000)]
REGRESSION(r216119): DocumentLoader::detachFromFrame still encounters nullptr frame
https://bugs.webkit.org/show_bug.cgi?id=186081
<rdar://problem/34918109>

Reviewed by David Kilzer.

The nullptr check that was removed in r216119 was still needed because when the DocumentLoader
is in 'm_waitingForNavigationPolicy' or in 'm_waitingForContentPolicy', a call to the policy checker
to stopCheck we will clear the frame as part of the cleanup phase.

If m_frame is nullptr after cancelPolicyCheckIfNeeded, our work is already done so just return.

* loader/DocumentLoader.cpp:
(WebCore::DocumentLoader::detachFromFrame):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232370 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoDon't paint the System Preview badge if the image isn't fully decoded
dino@apple.com [Thu, 31 May 2018 20:37:04 +0000 (20:37 +0000)]
Don't paint the System Preview badge if the image isn't fully decoded
https://bugs.webkit.org/show_bug.cgi?id=186158
<rdar://problem/40454866>

Reviewed by Antoine Quint.

Only paint the system preview badge if the image itself said
that it succeeded in drawing.

* rendering/RenderImage.cpp:
(WebCore::RenderImage::paintIntoRect): Check the ImageDrawResult.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232367 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoFix leak of AudioDeviceID array due to an early return in AudioDeviceMac::GetNumberDe...
ddkilzer@apple.com [Thu, 31 May 2018 19:51:30 +0000 (19:51 +0000)]
Fix leak of AudioDeviceID array due to an early return in AudioDeviceMac::GetNumberDevices()
<https://webkit.org/b/186152>
<rdar://problem/40692824>

Reviewed by Alex Christensen.

* Source/webrtc/modules/audio_device/mac/audio_device_mac.cc:
Use std::make_unique<> so that memory is allocated and
deallocated automatically.  Remove manual calls to free().
* WebKit/0011-Fix-AudioDeviceID-array-leak.patch: Add.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232365 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[macOS] Add JIT entitlements to WebContent process and plugin process on macOS
bfulgham@apple.com [Thu, 31 May 2018 19:46:56 +0000 (19:46 +0000)]
[macOS] Add JIT entitlements to WebContent process and plugin process on macOS
https://bugs.webkit.org/show_bug.cgi?id=184485
<rdar://problem/37556535>

Reviewed by Dan Bernstein.

This patch builds on the changes from Bug 185526 to add support for a JIT entitlement. It makes the
following changes:

1. Adds a new 'WebContent-OSX.entitlements file that unconditionally adds the JIT entitlement for all builds.
2. Modifies the PluginService.entitlements to unconditionally add the JIT entitlement.

* Configurations/PluginService.entitlements: Updated to add the JIT entitlement.
* Configurations/WebContent-OSX.entitlements: Updated to add the JIT entitlement.
* Configurations/WebContentService.xcconfig: Updated to use the new 'WebContent-OSX.entitlements' file.
* WebKit.xcodeproj/project.pbxproj:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232364 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoUpdate expected results for BuildStepsConstructorTest
ryanhaddad@apple.com [Thu, 31 May 2018 19:36:20 +0000 (19:36 +0000)]
Update expected results for BuildStepsConstructorTest
https://bugs.webkit.org/show_bug.cgi?id=186119

Reviewed by Alex Christensen.

* BuildSlaveSupport/build.webkit.org-config/steps_unittest.py: Add missing entries for 'WinCairo 64-bit' bots.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232363 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoImplement checked cast for DDResultRef once DDResultGetCFTypeID() is available
ddkilzer@apple.com [Thu, 31 May 2018 19:33:38 +0000 (19:33 +0000)]
Implement checked cast for DDResultRef once DDResultGetCFTypeID() is available
<https://webkit.org/b/184554>
<rdar://problem/36241894>

Reviewed by Brent Fulgham.

* editing/cocoa/DataDetection.mm:
(WebCore::detectItemAtPositionWithRange): Implement checked cast
for DDResultRef.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232362 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoFix Windows build after r232246.
achristensen@apple.com [Thu, 31 May 2018 19:15:30 +0000 (19:15 +0000)]
Fix Windows build after r232246.
https://bugs.webkit.org/show_bug.cgi?id=186024

* rendering/svg/SVGTextLayoutEngineBaseline.cpp:
(WebCore::SVGTextLayoutEngineBaseline::calculateAlignmentBaselineShift const):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232361 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoDon't attempt to extend sandbox when running tests with mock media devices
bfulgham@apple.com [Thu, 31 May 2018 19:13:35 +0000 (19:13 +0000)]
Don't attempt to extend sandbox when running tests with mock media devices
https://bugs.webkit.org/show_bug.cgi?id=186150
<rdar://problem/40690875>

Reviewed by Alexey Proskuryakov.

Avoid attempting to issue a sandbox extension to actual media devices when
running with Mock capture devices.

Also handle the 'denyNextRequest' test state outside of the USE(APPLE_INTERNAL_SDK)
guards, since it is needed for Open Source builds as well.

* UIProcess/UserMediaProcessManager.cpp:
(WebKit::UserMediaProcessManager::willCreateMediaStream):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232360 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoUnreviewed, rolling out r232212.
commit-queue@webkit.org [Thu, 31 May 2018 18:43:52 +0000 (18:43 +0000)]
Unreviewed, rolling out r232212.
https://bugs.webkit.org/show_bug.cgi?id=186153

broke build (Requested by alexchristensen on #webkit).

Reverted changeset:

"JavaScriptCore: Disable 32-bit JIT on Windows"
https://bugs.webkit.org/show_bug.cgi?id=185989
https://trac.webkit.org/changeset/232212

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232359 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[iOS] TestWebKitAPI.WebKit.WKHTTPCookieStoreWithoutProcessPool fails because cookies...
sihui_liu@apple.com [Thu, 31 May 2018 18:04:54 +0000 (18:04 +0000)]
[iOS] TestWebKitAPI.WebKit.WKHTTPCookieStoreWithoutProcessPool fails because cookies use different files with/without processpool
https://bugs.webkit.org/show_bug.cgi?id=185831
<rdar://problem/40468716>

Reviewed by David Kilzer.

* TestWebKitAPI/Tests/WebKitCocoa/WKHTTPCookieStore.mm:
(-[CookieUIDelegate webView:runJavaScriptAlertPanelWithMessage:initiatedByFrame:completionHandler:]):
(TEST):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232357 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoWebKit memory management: Safari jetsams on some websites when zooming and scrolling
antti@apple.com [Thu, 31 May 2018 17:55:43 +0000 (17:55 +0000)]
WebKit memory management: Safari jetsams on some websites when zooming and scrolling
https://bugs.webkit.org/show_bug.cgi?id=186091
<rdar://problem/36315010>

Reviewed by Geoff Garen.

When zooming a page rapidly the visible rect and the page zoom level may get momentarily out of sync.
When this happens we may generate tiles for a much larger area than needed and run out of memory
building the next layer tree transaction. Running out of memory is more likely if the page has lots of
tiled layers in addition to the main content layer.

We already have code for dealing with the scale mismatch for zoom-out case (where this would cause
visibly missing tiles). This patch enables the same exact adjustment for zoom-in case (where the
symptom is creating too many tiles).

* WebProcess/WebPage/ios/WebPageIOS.mm:
(WebKit::adjustExposedRectForNewScale):

Do some renames to make it clear that this can both expand and contract the visible rect.
Bail out if there is nothing to do.

(WebKit::WebPage::updateVisibleContentRects):

Call adjustExposedRectForNewScale directly and unconditionally.

(WebKit::adjustExposedRectForBoundedScale): Deleted.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232356 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoRebaseline test for change in Error.stack behavior.
keith_miller@apple.com [Thu, 31 May 2018 17:42:04 +0000 (17:42 +0000)]
Rebaseline test for change in Error.stack behavior.

* stress/error-stack-trace-limit.js:
(numberOfFrames):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232355 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoFix memory leak under NetworkStateNotifier::updateStateWithoutNotifying()
cdumez@apple.com [Thu, 31 May 2018 17:38:59 +0000 (17:38 +0000)]
Fix memory leak under NetworkStateNotifier::updateStateWithoutNotifying()
https://bugs.webkit.org/show_bug.cgi?id=186149
<rdar://problem/40674537>

Reviewed by Geoffrey Garen.

* platform/network/mac/NetworkStateNotifierMac.cpp:
(WebCore::NetworkStateNotifier::updateStateWithoutNotifying):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232354 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoUpdate http/wpt/service-workers/header-filtering.https.html to use a valid Referrer...
cdumez@apple.com [Thu, 31 May 2018 16:58:45 +0000 (16:58 +0000)]
Update http/wpt/service-workers/header-filtering.https.html to use a valid Referrer-Policy
https://bugs.webkit.org/show_bug.cgi?id=186148

Reviewed by Youenn Fablet.

Update http/wpt/service-workers/header-filtering.https.html to use a valid Referrer-Policy to
avoid logging a console message.

* http/wpt/service-workers/header-filtering.https-expected.txt:
* http/wpt/service-workers/resources/response-full-of-headers.py:
(main):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232353 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoFix the macOS High Sierra build.
aestes@apple.com [Thu, 31 May 2018 16:52:31 +0000 (16:52 +0000)]
Fix the macOS High Sierra build.

* NetworkProcess/cocoa/NetworkSessionCocoa.mm:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232352 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[iOS] API test ScrollViewInsetTests.InnerHeightWithLargeTopContentInset is a flaky...
cdumez@apple.com [Thu, 31 May 2018 16:47:31 +0000 (16:47 +0000)]
[iOS] API test ScrollViewInsetTests.InnerHeightWithLargeTopContentInset is a flaky failure
https://bugs.webkit.org/show_bug.cgi?id=184893
<rdar://problem/39747271>

Reviewed by Wenson Hsieh.

Update test to wait until the view is the expected height instead of
calling waitForNextPresentationUpdate once and expect the view to
have the right height. This fixes the flakiness.

* TestWebKitAPI/Tests/ios/ScrollViewInsetTests.mm:
(TestWebKitAPI::waitUntilInnerHeightIs):
(TestWebKitAPI::TEST):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232351 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[LFC] Layout code needs to know the type of the Element associated with a Layout...
zalan@apple.com [Thu, 31 May 2018 16:43:52 +0000 (16:43 +0000)]
[LFC] Layout code needs to know the type of the Element associated with a Layout::Box
https://bugs.webkit.org/show_bug.cgi?id=186117

Reviewed by Antti Koivisto.

Since these attributes don't change during layout, we could just pass them in to Layout::Box instead
of keep querying the Element.

* layout/layouttree/LayoutBlockContainer.cpp:
(WebCore::Layout::BlockContainer::BlockContainer):
* layout/layouttree/LayoutBlockContainer.h:
* layout/layouttree/LayoutBox.cpp:
(WebCore::Layout::Box::Box):
(WebCore::Layout::Box::isPaddingApplicable const):
(WebCore::Layout::Box::isDocumentBox const): Deleted.
(WebCore::Layout::Box::isBodyBox const): Deleted.
* layout/layouttree/LayoutBox.h:
(WebCore::Layout::Box::isAnonymous const):
(WebCore::Layout::Box::isDocumentBox const):
(WebCore::Layout::Box::isBodyBox const):
(WebCore::Layout::Box::ElementAttributes::ElementAttributes):
(WebCore::Layout::Box::setPreviousSibling):
(WebCore::Layout::Box::setIsAnonymous): Deleted.
* layout/layouttree/LayoutContainer.cpp:
(WebCore::Layout::Container::Container):
* layout/layouttree/LayoutContainer.h:
* layout/layouttree/LayoutInlineBox.cpp:
(WebCore::Layout::InlineBox::InlineBox):
* layout/layouttree/LayoutInlineBox.h:
* layout/layouttree/LayoutInlineContainer.cpp:
(WebCore::Layout::InlineContainer::InlineContainer):
* layout/layouttree/LayoutInlineContainer.h:
* layout/layouttree/LayoutTreeBuilder.cpp:
(WebCore::Layout::TreeBuilder::createLayoutTree):
(WebCore::Layout::TreeBuilder::createSubTree):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232350 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoUnreviewed iOS build fix after r232335.
cdumez@apple.com [Thu, 31 May 2018 16:24:59 +0000 (16:24 +0000)]
Unreviewed iOS build fix after r232335.

* html/BaseChooserOnlyDateAndTimeInputType.h:
* html/BaseDateAndTimeInputType.h:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232349 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoUnreviewed, apply review comments from Darin after r232310.
cdumez@apple.com [Thu, 31 May 2018 15:47:35 +0000 (15:47 +0000)]
Unreviewed, apply review comments from Darin after r232310.

* platform/ReferrerPolicy.cpp:
(WebCore::parseReferrerPolicy):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232348 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[LFC] Margin box is border box + margins.
zalan@apple.com [Thu, 31 May 2018 14:59:37 +0000 (14:59 +0000)]
[LFC] Margin box is border box + margins.
https://bugs.webkit.org/show_bug.cgi?id=186124

Reviewed by Antti Koivisto.

* layout/displaytree/DisplayBox.cpp:
(WebCore::Display::Box::marginBox const):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232347 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[LFC] Add quirks mode setter/getting to LayoutContext
zalan@apple.com [Thu, 31 May 2018 14:58:24 +0000 (14:58 +0000)]
[LFC] Add quirks mode setter/getting to LayoutContext
https://bugs.webkit.org/show_bug.cgi?id=186122

Reviewed by Antti Koivisto.

* layout/LayoutContext.h:
(WebCore::Layout::LayoutContext::setInQuirksMode):
(WebCore::Layout::LayoutContext::inQuirksMode const):
* page/FrameViewLayoutContext.cpp:
(WebCore::layoutUsingFormattingContext):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232346 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[wpe] update the TestExpectations for encrypted-media after some investigations
commit-queue@webkit.org [Thu, 31 May 2018 11:00:16 +0000 (11:00 +0000)]
[wpe] update the TestExpectations for encrypted-media after some investigations
https://bugs.webkit.org/show_bug.cgi?id=185593

Patch by Yacine Bandou <yacine.bandou_ext@softathome.com> on 2018-05-31
Reviewed by Xabier Rodriguez-Calvar.

After the fix 185725 and some investigations, the WPE platform TestExpectations should be updated

* platform/wpe/TestExpectations:
* platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-retrieve-destroy-persistent-license.https-expected.txt:
* platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-retrieve-persistent-license.https-expected.txt:
* platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-clear-encrypted.https-expected.txt:
* platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-multikey-sequential-readyState.https-expected.txt:
* platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-multikey-sequential.https-expected.txt:
* platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-playback-temporary-waitingforkey.https-expected.txt:
* platform/wpe/imported/w3c/web-platform-tests/encrypted-media/clearkey-mp4-waiting-for-a-key.https-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232345 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoUnreviewed GTK+ gardening after r232336.
magomez@igalia.com [Thu, 31 May 2018 10:27:24 +0000 (10:27 +0000)]
Unreviewed GTK+ gardening after r232336.

* platform/gtk/TestExpectations:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232344 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agotest262-runner should run in verbose mode in automation
commit-queue@webkit.org [Thu, 31 May 2018 10:20:54 +0000 (10:20 +0000)]
test262-runner should run in verbose mode in automation
https://bugs.webkit.org/show_bug.cgi?id=186115

Patch by Leo Balter <leonardo.balter@gmail.com> on 2018-05-31
Reviewed by Yusuke Suzuki.

This patch toggles the verbose mode in the build bot and reports every test file
in order to avoid the '1200 seconds without output' issue.
* BuildSlaveSupport/build.webkit.org-config/steps.py:
(RunTest262Tests):
* Scripts/test262/Runner.pm:
(processResult):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232343 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[GTK] Hardcoded text color in input fields
commit-queue@webkit.org [Thu, 31 May 2018 06:41:41 +0000 (06:41 +0000)]
[GTK] Hardcoded text color in input fields
https://bugs.webkit.org/show_bug.cgi?id=126907

Patch by Carlos Eduardo Ramalho <cadubentzen@gmail.com> on 2018-05-30
Reviewed by Carlos Garcia Campos.

Set text color in input fields to foreground theme color.
Also, set "window.background" as base GtkStyleContext to mimic
GTK applications and fix some theme bugs.

No new tests required. ManualTests/gtk/theme.html already covers it.

* platform/gtk/RenderThemeGadget.cpp:
(WebCore::baseStyleContext): Added.
(WebCore::RenderThemeGadget::RenderThemeGadget):
Use "window.background" GtkStyleContext instead of
null parent for RenderThemeGadgets.
* rendering/RenderThemeGtk.cpp:
(WebCore::RenderThemeGtk::adjustButtonStyle const):
Set color as foreground theme color.
(WebCore::RenderThemeGtk::adjustTextFieldStyle const): Ditto.
(WebCore::RenderThemeGtk::adjustTextAreaStyle const): Ditto.
(WebCore::RenderThemeGtk::adjustSearchFieldStyle const): Ditto.
* rendering/RenderThemeGtk.h: adjustTextAreaStyle() overriden.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232338 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[JSC] Pass VM& parameter as much as possible
utatane.tea@gmail.com [Thu, 31 May 2018 06:19:33 +0000 (06:19 +0000)]
[JSC] Pass VM& parameter as much as possible
https://bugs.webkit.org/show_bug.cgi?id=186085

Reviewed by Saam Barati.

Source/JavaScriptCore:

JSCell::vm() is slow compared to ExecState::vm(). That's why we have bunch of functions in JSCell/JSObject that take VM& as a parameter.
For example, we have JSCell::structure() and JSCell::structure(VM&), the former retrieves VM& from the cell and invokes structure(VM&).
If we can get VM& from ExecState* or the other place, it reduces the inlined code size.
This patch attempts to pass VM& parameter to such functions as much as possible.

* API/APICast.h:
(toJS):
(toJSForGC):
* API/JSCallbackObjectFunctions.h:
(JSC::JSCallbackObject<Parent>::getOwnPropertySlotByIndex):
(JSC::JSCallbackObject<Parent>::deletePropertyByIndex):
(JSC::JSCallbackObject<Parent>::staticFunctionGetter):
* API/JSObjectRef.cpp:
(JSObjectIsConstructor):
* API/JSTypedArray.cpp:
(JSObjectGetTypedArrayBuffer):
* API/JSValueRef.cpp:
(JSValueIsInstanceOfConstructor):
* bindings/ScriptFunctionCall.cpp:
(Deprecated::ScriptFunctionCall::call):
* bindings/ScriptValue.cpp:
(Inspector::jsToInspectorValue):
* bytecode/AccessCase.cpp:
(JSC::AccessCase::generateImpl):
* bytecode/CodeBlock.cpp:
(JSC::CodeBlock::CodeBlock):
* bytecode/ObjectAllocationProfileInlines.h:
(JSC::ObjectAllocationProfile::possibleDefaultPropertyCount):
* bytecode/ObjectPropertyConditionSet.cpp:
(JSC::generateConditionsForInstanceOf):
* bytecode/PropertyCondition.cpp:
(JSC::PropertyCondition::isWatchableWhenValid const):
(JSC::PropertyCondition::attemptToMakeEquivalenceWithoutBarrier const):
* bytecode/StructureStubClearingWatchpoint.cpp:
(JSC::StructureStubClearingWatchpoint::fireInternal):
* debugger/Debugger.cpp:
(JSC::Debugger::detach):
* debugger/DebuggerScope.cpp:
(JSC::DebuggerScope::create):
(JSC::DebuggerScope::put):
(JSC::DebuggerScope::deleteProperty):
(JSC::DebuggerScope::getOwnPropertyNames):
(JSC::DebuggerScope::defineOwnProperty):
* dfg/DFGAbstractInterpreterInlines.h:
(JSC::DFG::AbstractInterpreter<AbstractStateType>::executeEffects):
* dfg/DFGAbstractValue.cpp:
(JSC::DFG::AbstractValue::mergeOSREntryValue):
* dfg/DFGArgumentsEliminationPhase.cpp:
* dfg/DFGArrayMode.cpp:
(JSC::DFG::ArrayMode::refine const):
* dfg/DFGByteCodeParser.cpp:
(JSC::DFG::ByteCodeParser::handleIntrinsicCall):
(JSC::DFG::ByteCodeParser::handleTypedArrayConstructor):
(JSC::DFG::ByteCodeParser::handleConstantInternalFunction):
(JSC::DFG::ByteCodeParser::check):
* dfg/DFGConstantFoldingPhase.cpp:
(JSC::DFG::ConstantFoldingPhase::foldConstants):
(JSC::DFG::ConstantFoldingPhase::addStructureTransitionCheck):
* dfg/DFGFixupPhase.cpp:
(JSC::DFG::FixupPhase::fixupNode):
* dfg/DFGGraph.cpp:
(JSC::DFG::Graph::tryGetConstantProperty):
* dfg/DFGOperations.cpp:
* dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::SpeculativeJIT::compileGetByValOnString):
* dfg/DFGStrengthReductionPhase.cpp:
(JSC::DFG::StrengthReductionPhase::handleNode):
* ftl/FTLLowerDFGToB3.cpp:
(JSC::FTL::DFG::LowerDFGToB3::compileStringCharAt):
* ftl/FTLOperations.cpp:
(JSC::FTL::operationPopulateObjectInOSR):
* inspector/InjectedScriptManager.cpp:
(Inspector::InjectedScriptManager::createInjectedScript):
* inspector/JSJavaScriptCallFrame.cpp:
(Inspector::JSJavaScriptCallFrame::caller const):
(Inspector::JSJavaScriptCallFrame::scopeChain const):
* interpreter/CallFrame.cpp:
(JSC::CallFrame::wasmAwareLexicalGlobalObject):
* interpreter/Interpreter.cpp:
(JSC::Interpreter::executeProgram):
(JSC::Interpreter::executeCall):
(JSC::Interpreter::executeConstruct):
(JSC::Interpreter::execute):
(JSC::Interpreter::executeModuleProgram):
* jit/JITOperations.cpp:
(JSC::getByVal):
* jit/Repatch.cpp:
(JSC::tryCacheInByID):
* jsc.cpp:
(functionDollarAgentReceiveBroadcast):
(functionHasCustomProperties):
* llint/LLIntSlowPaths.cpp:
(JSC::LLInt::LLINT_SLOW_PATH_DECL):
(JSC::LLInt::setupGetByIdPrototypeCache):
(JSC::LLInt::getByVal):
(JSC::LLInt::handleHostCall):
(JSC::LLInt::llint_throw_stack_overflow_error):
* runtime/AbstractModuleRecord.cpp:
(JSC::AbstractModuleRecord::finishCreation):
* runtime/ArrayConstructor.cpp:
(JSC::constructArrayWithSizeQuirk):
* runtime/ArrayPrototype.cpp:
(JSC::speciesWatchpointIsValid):
(JSC::arrayProtoFuncToString):
(JSC::arrayProtoFuncToLocaleString):
(JSC::ArrayPrototype::tryInitializeSpeciesWatchpoint):
* runtime/AsyncFunctionConstructor.cpp:
(JSC::callAsyncFunctionConstructor):
(JSC::constructAsyncFunctionConstructor):
* runtime/AsyncGeneratorFunctionConstructor.cpp:
(JSC::callAsyncGeneratorFunctionConstructor):
(JSC::constructAsyncGeneratorFunctionConstructor):
* runtime/BooleanConstructor.cpp:
(JSC::constructWithBooleanConstructor):
* runtime/ClonedArguments.cpp:
(JSC::ClonedArguments::createEmpty):
(JSC::ClonedArguments::createWithInlineFrame):
(JSC::ClonedArguments::createWithMachineFrame):
(JSC::ClonedArguments::createByCopyingFrom):
(JSC::ClonedArguments::getOwnPropertySlot):
(JSC::ClonedArguments::materializeSpecials):
* runtime/CommonSlowPaths.cpp:
(JSC::SLOW_PATH_DECL):
* runtime/CommonSlowPaths.h:
(JSC::CommonSlowPaths::tryCachePutToScopeGlobal):
(JSC::CommonSlowPaths::tryCacheGetFromScopeGlobal):
(JSC::CommonSlowPaths::canAccessArgumentIndexQuickly):
* runtime/ConstructData.cpp:
(JSC::construct):
* runtime/DateConstructor.cpp:
(JSC::constructWithDateConstructor):
* runtime/DatePrototype.cpp:
(JSC::dateProtoFuncToJSON):
* runtime/DirectArguments.cpp:
(JSC::DirectArguments::overrideThings):
* runtime/Error.cpp:
(JSC::getStackTrace):
* runtime/ErrorConstructor.cpp:
(JSC::Interpreter::constructWithErrorConstructor):
(JSC::Interpreter::callErrorConstructor):
* runtime/FunctionConstructor.cpp:
(JSC::constructWithFunctionConstructor):
(JSC::callFunctionConstructor):
* runtime/GeneratorFunctionConstructor.cpp:
(JSC::callGeneratorFunctionConstructor):
(JSC::constructGeneratorFunctionConstructor):
* runtime/GenericArgumentsInlines.h:
(JSC::GenericArguments<Type>::getOwnPropertySlot):
* runtime/InferredStructureWatchpoint.cpp:
(JSC::InferredStructureWatchpoint::fireInternal):
* runtime/InferredType.cpp:
(JSC::InferredType::removeStructure):
* runtime/InferredType.h:
* runtime/InferredTypeInlines.h:
(JSC::InferredType::finalizeUnconditionally):
* runtime/IntlCollator.cpp:
(JSC::IntlCollator::initializeCollator):
* runtime/IntlCollatorConstructor.cpp:
(JSC::IntlCollatorConstructorFuncSupportedLocalesOf):
* runtime/IntlCollatorPrototype.cpp:
(JSC::IntlCollatorPrototypeGetterCompare):
* runtime/IntlDateTimeFormat.cpp:
(JSC::IntlDateTimeFormat::initializeDateTimeFormat):
(JSC::IntlDateTimeFormat::formatToParts):
* runtime/IntlDateTimeFormatConstructor.cpp:
(JSC::IntlDateTimeFormatConstructorFuncSupportedLocalesOf):
* runtime/IntlDateTimeFormatPrototype.cpp:
(JSC::IntlDateTimeFormatPrototypeGetterFormat):
* runtime/IntlNumberFormat.cpp:
(JSC::IntlNumberFormat::initializeNumberFormat):
(JSC::IntlNumberFormat::formatToParts):
* runtime/IntlNumberFormatConstructor.cpp:
(JSC::IntlNumberFormatConstructorFuncSupportedLocalesOf):
* runtime/IntlNumberFormatPrototype.cpp:
(JSC::IntlNumberFormatPrototypeGetterFormat):
* runtime/IntlObject.cpp:
(JSC::canonicalizeLocaleList):
(JSC::defaultLocale):
(JSC::lookupSupportedLocales):
(JSC::intlObjectFuncGetCanonicalLocales):
* runtime/IntlPluralRules.cpp:
(JSC::IntlPluralRules::initializePluralRules):
(JSC::IntlPluralRules::resolvedOptions):
* runtime/IntlPluralRulesConstructor.cpp:
(JSC::IntlPluralRulesConstructorFuncSupportedLocalesOf):
* runtime/IteratorOperations.cpp:
(JSC::iteratorNext):
(JSC::iteratorClose):
(JSC::iteratorForIterable):
* runtime/JSArray.cpp:
(JSC::JSArray::shiftCountWithArrayStorage):
(JSC::JSArray::unshiftCountWithArrayStorage):
(JSC::JSArray::isIteratorProtocolFastAndNonObservable):
* runtime/JSArrayBufferConstructor.cpp:
(JSC::JSArrayBufferConstructor::finishCreation):
(JSC::constructArrayBuffer):
* runtime/JSArrayBufferPrototype.cpp:
(JSC::arrayBufferProtoFuncSlice):
* runtime/JSArrayBufferView.cpp:
(JSC::JSArrayBufferView::unsharedJSBuffer):
(JSC::JSArrayBufferView::possiblySharedJSBuffer):
* runtime/JSAsyncFunction.cpp:
(JSC::JSAsyncFunction::createImpl):
(JSC::JSAsyncFunction::create):
(JSC::JSAsyncFunction::createWithInvalidatedReallocationWatchpoint):
* runtime/JSAsyncGeneratorFunction.cpp:
(JSC::JSAsyncGeneratorFunction::createImpl):
(JSC::JSAsyncGeneratorFunction::create):
(JSC::JSAsyncGeneratorFunction::createWithInvalidatedReallocationWatchpoint):
* runtime/JSBoundFunction.cpp:
(JSC::boundThisNoArgsFunctionCall):
(JSC::boundFunctionCall):
(JSC::boundThisNoArgsFunctionConstruct):
(JSC::boundFunctionConstruct):
(JSC::getBoundFunctionStructure):
(JSC::JSBoundFunction::create):
(JSC::JSBoundFunction::boundArgsCopy):
* runtime/JSCJSValue.cpp:
(JSC::JSValue::putToPrimitive):
* runtime/JSCellInlines.h:
(JSC::JSCell::setStructure):
(JSC::JSCell::methodTable const):
(JSC::JSCell::toBoolean const):
* runtime/JSFunction.h:
(JSC::JSFunction::createImpl):
* runtime/JSGeneratorFunction.cpp:
(JSC::JSGeneratorFunction::createImpl):
(JSC::JSGeneratorFunction::create):
(JSC::JSGeneratorFunction::createWithInvalidatedReallocationWatchpoint):
* runtime/JSGenericTypedArrayViewConstructorInlines.h:
(JSC::constructGenericTypedArrayViewWithArguments):
(JSC::constructGenericTypedArrayView):
* runtime/JSGenericTypedArrayViewInlines.h:
(JSC::JSGenericTypedArrayView<Adaptor>::getOwnPropertySlotByIndex):
(JSC::JSGenericTypedArrayView<Adaptor>::putByIndex):
(JSC::JSGenericTypedArrayView<Adaptor>::deletePropertyByIndex):
(JSC::JSGenericTypedArrayView<Adaptor>::slowDownAndWasteMemory):
* runtime/JSGenericTypedArrayViewPrototypeFunctions.h:
(JSC::genericTypedArrayViewProtoFuncSlice):
(JSC::genericTypedArrayViewPrivateFuncSubarrayCreate):
* runtime/JSGlobalObject.cpp:
(JSC::JSGlobalObject::init):
(JSC::JSGlobalObject::exposeDollarVM):
(JSC::JSGlobalObject::finishCreation):
* runtime/JSGlobalObject.h:
* runtime/JSGlobalObjectFunctions.cpp:
(JSC::globalFuncEval):
* runtime/JSInternalPromise.cpp:
(JSC::JSInternalPromise::then):
* runtime/JSInternalPromiseConstructor.cpp:
(JSC::constructPromise):
* runtime/JSJob.cpp:
(JSC::JSJobMicrotask::run):
* runtime/JSLexicalEnvironment.cpp:
(JSC::JSLexicalEnvironment::getOwnPropertySlot):
(JSC::JSLexicalEnvironment::put):
* runtime/JSMap.cpp:
(JSC::JSMap::isIteratorProtocolFastAndNonObservable):
* runtime/JSMapIterator.cpp:
(JSC::JSMapIterator::createPair):
* runtime/JSModuleLoader.cpp:
(JSC::JSModuleLoader::provideFetch):
(JSC::JSModuleLoader::loadAndEvaluateModule):
(JSC::JSModuleLoader::loadModule):
(JSC::JSModuleLoader::linkAndEvaluateModule):
(JSC::JSModuleLoader::requestImportModule):
* runtime/JSONObject.cpp:
(JSC::JSONProtoFuncParse):
* runtime/JSObject.cpp:
(JSC::JSObject::putInlineSlow):
(JSC::JSObject::putByIndex):
(JSC::JSObject::notifyPresenceOfIndexedAccessors):
(JSC::JSObject::createInitialIndexedStorage):
(JSC::JSObject::createArrayStorage):
(JSC::JSObject::convertUndecidedToArrayStorage):
(JSC::JSObject::convertInt32ToArrayStorage):
(JSC::JSObject::convertDoubleToArrayStorage):
(JSC::JSObject::convertContiguousToArrayStorage):
(JSC::JSObject::convertFromCopyOnWrite):
(JSC::JSObject::ensureWritableInt32Slow):
(JSC::JSObject::ensureWritableDoubleSlow):
(JSC::JSObject::ensureWritableContiguousSlow):
(JSC::JSObject::ensureArrayStorageSlow):
(JSC::JSObject::setPrototypeDirect):
(JSC::JSObject::deleteProperty):
(JSC::callToPrimitiveFunction):
(JSC::JSObject::hasInstance):
(JSC::JSObject::getOwnNonIndexPropertyNames):
(JSC::JSObject::preventExtensions):
(JSC::JSObject::isExtensible):
(JSC::JSObject::reifyAllStaticProperties):
(JSC::JSObject::fillGetterPropertySlot):
(JSC::JSObject::defineOwnIndexedProperty):
(JSC::JSObject::putByIndexBeyondVectorLengthWithoutAttributes):
(JSC::JSObject::putByIndexBeyondVectorLengthWithArrayStorage):
(JSC::JSObject::putByIndexBeyondVectorLength):
(JSC::JSObject::putDirectIndexBeyondVectorLengthWithArrayStorage):
(JSC::JSObject::putDirectIndexSlowOrBeyondVectorLength):
(JSC::JSObject::getNewVectorLength):
(JSC::JSObject::increaseVectorLength):
(JSC::JSObject::reallocateAndShrinkButterfly):
(JSC::JSObject::shiftButterflyAfterFlattening):
(JSC::JSObject::anyObjectInChainMayInterceptIndexedAccesses const):
(JSC::JSObject::prototypeChainMayInterceptStoreTo):
(JSC::JSObject::needsSlowPutIndexing const):
(JSC::JSObject::suggestedArrayStorageTransition const):
* runtime/JSObject.h:
(JSC::JSObject::mayInterceptIndexedAccesses):
(JSC::JSObject::hasIndexingHeader const):
(JSC::JSObject::hasCustomProperties):
(JSC::JSObject::hasGetterSetterProperties):
(JSC::JSObject::hasCustomGetterSetterProperties):
(JSC::JSObject::isExtensibleImpl):
(JSC::JSObject::isStructureExtensible):
(JSC::JSObject::indexingShouldBeSparse):
(JSC::JSObject::staticPropertiesReified):
(JSC::JSObject::globalObject const):
(JSC::JSObject::finishCreation):
(JSC::JSNonFinalObject::finishCreation):
(JSC::getCallData):
(JSC::getConstructData):
(JSC::JSObject::getOwnNonIndexPropertySlot):
(JSC::JSObject::putOwnDataProperty):
(JSC::JSObject::putOwnDataPropertyMayBeIndex):
(JSC::JSObject::butterflyPreCapacity):
(JSC::JSObject::butterflyTotalSize):
* runtime/JSObjectInlines.h:
(JSC::JSObject::putDirectInternal):
* runtime/JSPromise.cpp:
(JSC::JSPromise::initialize):
(JSC::JSPromise::resolve):
* runtime/JSPromiseConstructor.cpp:
(JSC::constructPromise):
* runtime/JSPromiseDeferred.cpp:
(JSC::newPromiseCapability):
(JSC::callFunction):
* runtime/JSScope.cpp:
(JSC::abstractAccess):
* runtime/JSScope.h:
(JSC::JSScope::globalObject): Deleted.
Remove this JSScope::globalObject function since it is completely the same to JSObject::globalObject().

* runtime/JSSet.cpp:
(JSC::JSSet::isIteratorProtocolFastAndNonObservable):
* runtime/JSSetIterator.cpp:
(JSC::JSSetIterator::createPair):
* runtime/JSStringIterator.cpp:
(JSC::JSStringIterator::clone):
* runtime/Lookup.cpp:
(JSC::reifyStaticAccessor):
(JSC::setUpStaticFunctionSlot):
* runtime/Lookup.h:
(JSC::getStaticPropertySlotFromTable):
(JSC::replaceStaticPropertySlot):
(JSC::reifyStaticProperty):
* runtime/MapConstructor.cpp:
(JSC::constructMap):
* runtime/NumberConstructor.cpp:
(JSC::NumberConstructor::finishCreation):
* runtime/ObjectConstructor.cpp:
(JSC::constructObject):
(JSC::objectConstructorAssign):
(JSC::toPropertyDescriptor):
* runtime/ObjectPrototype.cpp:
(JSC::objectProtoFuncDefineGetter):
(JSC::objectProtoFuncDefineSetter):
(JSC::objectProtoFuncToLocaleString):
* runtime/Operations.cpp:
(JSC::jsIsFunctionType): Deleted.
Replace it with JSValue::isFunction(VM&).

* runtime/Operations.h:
* runtime/ProgramExecutable.cpp:
(JSC::ProgramExecutable::initializeGlobalProperties):
* runtime/RegExpConstructor.cpp:
(JSC::constructWithRegExpConstructor):
(JSC::callRegExpConstructor):
* runtime/SamplingProfiler.cpp:
(JSC::SamplingProfiler::processUnverifiedStackTraces):
(JSC::SamplingProfiler::StackFrame::nameFromCallee):
* runtime/ScopedArguments.cpp:
(JSC::ScopedArguments::overrideThings):
* runtime/ScriptExecutable.cpp:
(JSC::ScriptExecutable::newCodeBlockFor):
(JSC::ScriptExecutable::prepareForExecutionImpl):
* runtime/SetConstructor.cpp:
(JSC::constructSet):
* runtime/SparseArrayValueMap.cpp:
(JSC::SparseArrayValueMap::putEntry):
(JSC::SparseArrayValueMap::putDirect):
* runtime/StringConstructor.cpp:
(JSC::constructWithStringConstructor):
* runtime/StringPrototype.cpp:
(JSC::replaceUsingRegExpSearch):
(JSC::replaceUsingStringSearch):
(JSC::stringProtoFuncIterator):
* runtime/Structure.cpp:
(JSC::Structure::materializePropertyTable):
(JSC::Structure::willStoreValueSlow):
* runtime/StructureCache.cpp:
(JSC::StructureCache::emptyStructureForPrototypeFromBaseStructure):
* runtime/StructureInlines.h:
(JSC::Structure::get):
* runtime/WeakMapConstructor.cpp:
(JSC::constructWeakMap):
* runtime/WeakSetConstructor.cpp:
(JSC::constructWeakSet):
* tools/HeapVerifier.cpp:
(JSC::HeapVerifier::reportCell):
* tools/JSDollarVM.cpp:
(JSC::functionGlobalObjectForObject):
(JSC::JSDollarVM::finishCreation):
* wasm/js/JSWebAssemblyInstance.cpp:
(JSC::JSWebAssemblyInstance::finalizeCreation):
* wasm/js/WasmToJS.cpp:
(JSC::Wasm::handleBadI64Use):
(JSC::Wasm::wasmToJSException):
* wasm/js/WebAssemblyCompileErrorConstructor.cpp:
(JSC::constructJSWebAssemblyCompileError):
(JSC::callJSWebAssemblyCompileError):
* wasm/js/WebAssemblyLinkErrorConstructor.cpp:
(JSC::constructJSWebAssemblyLinkError):
(JSC::callJSWebAssemblyLinkError):
* wasm/js/WebAssemblyModuleRecord.cpp:
(JSC::WebAssemblyModuleRecord::evaluate):
* wasm/js/WebAssemblyPrototype.cpp:
(JSC::instantiate):
* wasm/js/WebAssemblyRuntimeErrorConstructor.cpp:
(JSC::constructJSWebAssemblyRuntimeError):
(JSC::callJSWebAssemblyRuntimeError):
* wasm/js/WebAssemblyToJSCallee.cpp:
(JSC::WebAssemblyToJSCallee::create):

Source/WebCore:

No behavior change.

* bindings/js/JSCSSRuleListCustom.cpp:
(WebCore::JSCSSRuleListOwner::isReachableFromOpaqueRoots):
* bindings/js/JSCallbackData.cpp:
(WebCore::JSCallbackData::invokeCallback):
* bindings/js/JSCustomXPathNSResolver.cpp:
(WebCore::JSCustomXPathNSResolver::lookupNamespaceURI):
* bindings/js/JSDOMBuiltinConstructorBase.cpp:
(WebCore::JSDOMBuiltinConstructorBase::callFunctionWithCurrentArguments):
* bindings/js/JSDOMConvertScheduledAction.h:
(WebCore::Converter<IDLScheduledAction>::convert):
* bindings/js/JSDOMIterator.h:
(WebCore::iteratorForEach):
* bindings/js/JSDOMMapLike.cpp:
(WebCore::forwardFunctionCallToBackingMap):
(WebCore::forwardForEachCallToBackingMap):
* bindings/js/JSDOMPromise.cpp:
(WebCore::callFunction):
* bindings/js/JSDOMPromiseDeferred.cpp:
(WebCore::DeferredPromise::callFunction):
(WebCore::createRejectedPromiseWithTypeError):
* bindings/js/JSDeprecatedCSSOMValueCustom.cpp:
(WebCore::JSDeprecatedCSSOMValueOwner::isReachableFromOpaqueRoots):
* bindings/js/JSEventListener.cpp:
(WebCore::JSEventListener::handleEvent):
* bindings/js/JSNodeListCustom.cpp:
(WebCore::JSNodeListOwner::isReachableFromOpaqueRoots):
* bindings/js/JSPluginElementFunctions.cpp:
(WebCore::pluginElementCustomPut):
(WebCore::callPlugin):
* bindings/js/JSWindowProxy.cpp:
(WebCore::JSWindowProxy::setWindow):
* bindings/js/ReadableStream.cpp:
(WebCore::ReadableStreamInternal::callFunction):
* bindings/js/ReadableStreamDefaultController.cpp:
(WebCore::callFunction):
* bindings/js/ScheduledAction.cpp:
(WebCore::ScheduledAction::executeFunctionInContext):
* bindings/js/SerializedScriptValue.cpp:
(WebCore::CloneSerializer::getProperty):
(WebCore::CloneSerializer::serialize):
* bindings/js/StructuredClone.cpp:
(WebCore::structuredCloneArrayBufferView):
* bindings/js/WorkerScriptController.cpp:
(WebCore::WorkerScriptController::initScript):
* bridge/NP_jsobject.cpp:
* bridge/objc/WebScriptObject.mm:
(-[WebScriptObject callWebScriptMethod:withArguments:]):
* testing/Internals.cpp:
(WebCore::Internals::cloneArrayBuffer):
* testing/js/WebCoreTestSupport.cpp:
(WebCoreTestSupport::injectInternalsObject):

Source/WebKit:

* WebProcess/Plugins/Netscape/NPJSObject.cpp:
(WebKit::NPJSObject::hasMethod):
(WebKit::NPJSObject::construct):
(WebKit::NPJSObject::invoke):

Source/WebKitLegacy/mac:

* Plugins/Hosted/NetscapePluginInstanceProxy.mm:
(WebKit::NetscapePluginInstanceProxy::invoke):

Source/WebKitLegacy/win:

* Plugins/PluginPackage.cpp:
(WebCore::NPN_Invoke):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232337 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoREGRESSION(r232316) [Win] error C2039: 'event': is not a member of 'WebCore::Navigati...
Hironori.Fujii@sony.com [Thu, 31 May 2018 05:34:24 +0000 (05:34 +0000)]
REGRESSION(r232316) [Win] error C2039: 'event': is not a member of 'WebCore::NavigationAction'
https://bugs.webkit.org/show_bug.cgi?id=186125

Unreviewed build fix

* WebActionPropertyBag.cpp:
(WebActionPropertyBag::Read):
(findMouseEvent): Deleted.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232336 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoStraighten out HTMLInputElement attribute handling
darin@apple.com [Thu, 31 May 2018 04:22:59 +0000 (04:22 +0000)]
Straighten out HTMLInputElement attribute handling
https://bugs.webkit.org/show_bug.cgi?id=186043

Reviewed by Daniel Bates.

* dom/RadioButtonGroups.cpp:
(WebCore::RadioButtonGroup::requiredStateChanged): Renamed from requiredAttributeChanged,
and changed to take a reference.
(WebCore::RadioButtonGroups::requiredStateChanged): Ditto.
* dom/RadioButtonGroups.h: Updated for changes above.

* html/BaseChooserOnlyDateAndTimeInputType.cpp:
(WebCore::BaseChooserOnlyDateAndTimeInputType::createShadowSubtree): Call
updateInnerTextValue by its new name.
(WebCore::BaseChooserOnlyDateAndTimeInputType::updateInnerTextValue): Renamed
from updateAppearance.
(WebCore::BaseChooserOnlyDateAndTimeInputType::setValue): Call
updateInnerTextValue by its new name.
(WebCore::BaseChooserOnlyDateAndTimeInputType::attributeChanged): Override this
instead of valueAttributeChanged.
* html/BaseChooserOnlyDateAndTimeInputType.h: Tighten up inheritance a bit.
Also use final a bit more and update for changes above.

* html/BaseDateAndTimeInputType.cpp:
(WebCore::BaseDateAndTimeInputType::attributeChanged): Replaced
minOrMaxAttributeChanged with this override, checking for the attribute names.
* html/BaseDateAndTimeInputType.h: Updated for changes above.

* html/FileInputType.cpp:
(WebCore::FileInputType::disabledStateChanged): Renamed from disabledAttributeChanged.
(WebCore::FileInputType::attributeChanged): Replaced
multipleAttributeChanged with this override, checking for the attribute names.
* html/FileInputType.h: Updated for changes above.

* html/HTMLFormControlElement.cpp:
(WebCore::HTMLFormControlElement::parseAttribute): Updated function names to say
"state changed" rather than "attribute changed" if they are called only when the
computed value is changed, not the raw attribute value.
(WebCore::HTMLFormControlElement::readOnlyStateChanged): Updated name.
(WebCore::HTMLFormControlElement::requiredStateChanged): Ditto.
* html/HTMLFormControlElement.h: Updated for changes above.

* html/HTMLInputElement.cpp:
(WebCore::HTMLInputElement::parseAttribute): Removed the code to call lots of specific
attribute changed functions that have now been deleted. Also removed empty code for a
couple of attributes.
(WebCore::HTMLInputElement::disabledStateChanged): Added. Forward call to input type.
(WebCore::HTMLInputElement::readOnlyStateChanged): Ditto.
(WebCore::HTMLInputElement::requiredStateChanged): Renamed from requiredAttributeChanged.
* html/HTMLInputElement.h: Updated for changes above.

* html/HTMLTextFormControlElement.cpp:
(WebCore::HTMLTextFormControlElement::readOnlyStateChanged): Renamed from
readOnlyAttributeChanged. Also fixed incorrect call to base class. This may have
fixed a bug: With some work we could probably create a test case to demonstrate that.
* html/HTMLTextFormControlElement.h: Updated for changes above.

* html/ImageInputType.cpp:
(WebCore::ImageInputType::attributeChanged): Replaced
altAttributeChanged and srcAttributeChanged with this override, checking for the
attribute names.
* html/ImageInputType.h: Updated for changes above.

* html/InputType.cpp:
(WebCore::InputType::altAttributeChanged): Deleted.
(WebCore::InputType::srcAttributeChanged): Deleted.
(WebCore::InputType::maxResultsAttributeChanged): Deleted.
(WebCore::InputType::minOrMaxAttributeChanged): Deleted.
(WebCore::InputType::stepAttributeChanged): Deleted.
(WebCore::InputType::valueAttributeChanged): Deleted.
(WebCore::InputType::attributeChanged): Moved to header as an inline so derived
classes can call through to the base class with the empty function inlined.
(WebCore::InputType::multipleAttributeChanged): Deleted.
(WebCore::InputType::disabledAttributeChanged): Moved to header as an inline.
(WebCore::InputType::readonlyAttributeChanged): Ditto.
(WebCore::InputType::requiredAttributeChanged): Ditto.
* html/InputType.h: Updated for changes above.

* html/NumberInputType.cpp:
(WebCore::NumberInputType::attributeChanged): Replaced
minOrMaxAttributeChanged and stepAttributeChanged with this override, checking
for the attribute names.
* html/NumberInputType.h: Updated for changes above.

* html/RangeInputType.cpp:
(WebCore::RangeInputType::disabledStateChanged): Renamed from
disabledAttributeChanged.
(WebCore::RangeInputType::attributeChanged): Replaced
minOrMaxAttributeChanged with this override, checking for the attribute names.
* html/RangeInputType.h: Updated for changes above.

* html/SearchInputType.cpp:
(WebCore::SearchInputType::SearchInputType): Removed unneeded initialization of
RefPtr data members to nullptr.
(WebCore::updateResultButtonPseudoType): Removed unneeded if statement; the two
cases above cover 0, and less than 0, so there is no need to check for greater
than 0 for the third case.
(WebCore::SearchInputType::attributeChanged): Replaced
maxResultsAttributeChanged with this override, checking for the attribute name.
* html/SearchInputType.h: Updated for changes above and marked more
member functions final.

* html/TextFieldInputType.cpp:
(WebCore::TextFieldInputType::attributeChanged): Added a call through to the
base class to match the style of other overrides of this function.
(WebCore::TextFieldInputType::disabledStateChanged): Renamed from disabledAttributeChanged.
(WebCore::TextFieldInputType::readOnlyStateChanged): Renamed from readonlyAttributeChanged.
* html/TextFieldInputType.h: Updated for changes above. Also made attributeChanged no
longer be marked final since derived classes now override it.

* html/shadow/SliderThumbElement.cpp:
(WebCore::SliderThumbElement::SliderThumbElement): Moved some data member initialization
from this constructor to the class definition.
(WebCore::SliderThumbElement::hostDisabledStateChanged): Renamed from disabledAttributeChanged.
* html/shadow/SliderThumbElement.h: Updated for changes above and marked more
member functions final.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232335 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoDFG combined liveness needs to say that the machine CodeBlock's arguments are live
sbarati@apple.com [Thu, 31 May 2018 03:57:59 +0000 (03:57 +0000)]
DFG combined liveness needs to say that the machine CodeBlock's arguments are live
https://bugs.webkit.org/show_bug.cgi?id=186121
<rdar://problem/39377796>

Reviewed by Keith Miller.

JSTests:

* stress/combined-liveness-needs-to-say-arguments-are-live.js: Added.
(baz):
(foo):

Source/JavaScriptCore:

DFG's combined liveness was reporting that the machine CodeBlock's |this|
argument was dead at certain points in the program. However, a CodeBlock's
arguments are considered live for the entire function. This fixes a bug
where object allocation sinking phase skipped materializing an allocation
because it thought that the argument it was associated with, |this|, was dead.

* dfg/DFGCombinedLiveness.cpp:
(JSC::DFG::liveNodesAtHead):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232334 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[Win][MiniBrowser] Move MainWindow related code from Common.cpp to MainWindow.cpp
Hironori.Fujii@sony.com [Thu, 31 May 2018 03:31:15 +0000 (03:31 +0000)]
[Win][MiniBrowser] Move MainWindow related code from Common.cpp to MainWindow.cpp
https://bugs.webkit.org/show_bug.cgi?id=186079

Reviewed by Alex Christensen.

This is a sub task of Bug 184770.

This change is partial. I'm going to remove global variables
gMainWindow and gMiniBrowser and convert more functions to
MainWindow's methods in follow-up patches.

* MiniBrowser/win/CMakeLists.txt: Added Common.cpp
* MiniBrowser/win/Common.cpp:
(menuItemIsChecked): Moved to MainWindow.cpp.
(turnOffOtherUserAgents): Ditto.
(EditProc): Ditto.
(BackButtonProc): Ditto.
(ForwardButtonProc): Ditto.
(About): Ditto.
(Caches): Ditto.
(CustomUserAgent): Ditto.
(AuthDialogProc): Ditto.
(DisplayAuthDialog): Converted to MainWindow's method.
(ToggleMenuItem): Ditto.
(loadURL): Ditto.
* MiniBrowser/win/Common.h:
* MiniBrowser/win/MainWindow.cpp:
(MainWindow::init): Call SetFocus at the end of the construction.
(MainWindow::WndProc):
(menuItemIsChecked): Moved from Common.cpp.
(turnOffOtherUserAgents): Ditto.
(EditProc): Ditto.
(BackButtonProc): Ditto.
(ForwardButtonProc): Ditto.
(About): Ditto.
(Caches): Ditto.
(CustomUserAgent): Ditto.
(authDialogProc): Ditto.
(MainWindow::displayAuthDialog): Converted from a function of Common.cpp.
(MainWindow::toggleMenuItem): Ditto.
(MainWindow::loadURL): Ditto.
* MiniBrowser/win/MainWindow.h: Added method declarations.
* MiniBrowser/win/MiniBrowser.h: Added pragma once.
* MiniBrowser/win/WinMain.cpp:
(wWinMain): Include Common.h instead of Common.cpp.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232333 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoAX: VoiceOver on macOS does not announce fieldset description from aria-describedby...
n_wang@apple.com [Thu, 31 May 2018 02:27:08 +0000 (02:27 +0000)]
AX: VoiceOver on macOS does not announce fieldset description from aria-describedby when focussing inputs
https://bugs.webkit.org/show_bug.cgi?id=185246
<rdar://problem/39939028>

Reviewed by Chris Fleizach.

Source/WebCore:

Exposed the fieldset's description to its form control children.

Test: accessibility/mac/aria-describedby-fieldset.html

* accessibility/AccessibilityNodeObject.cpp:
(WebCore::AccessibilityNodeObject::helpText const):

LayoutTests:

* accessibility/mac/aria-describedby-fieldset-expected.txt: Added.
* accessibility/mac/aria-describedby-fieldset.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232331 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoAX: Expose link rel type to voiceover
cfleizach@apple.com [Thu, 31 May 2018 01:21:15 +0000 (01:21 +0000)]
AX: Expose link rel type to voiceover
https://bugs.webkit.org/show_bug.cgi?id=186084
<rdar://problem/40640871>

Reviewed by Sam Weinig.

Source/WebCore:

Make the link rel attribute available to accessibility clients.

Test: accessibility/mac/linkrel.html

* accessibility/AccessibilityObject.cpp:
(WebCore::AccessibilityObject::linkRelValue const):
* accessibility/AccessibilityObject.h:
* accessibility/ios/WebAccessibilityObjectWrapperIOS.mm:
(-[WebAccessibilityObjectWrapper accessibilityLinkRelationshipType]):
* accessibility/mac/WebAccessibilityObjectWrapperMac.mm:
(-[WebAccessibilityObjectWrapper accessibilityAttributeNames]):
(-[WebAccessibilityObjectWrapper accessibilityAttributeValue:]):

LayoutTests:

* accessibility/image-link-expected.txt:
* accessibility/image-map2-expected.txt:
* accessibility/mac/document-links-expected.txt:
* accessibility/mac/linkrel-expected.txt: Added.
* accessibility/mac/linkrel.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232326 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoRebaseline http/wpt/service-workers/header-filtering.https.html after r232310.
ryanhaddad@apple.com [Thu, 31 May 2018 00:39:57 +0000 (00:39 +0000)]
Rebaseline http/wpt/service-workers/header-filtering.https.html after r232310.

Unreviewed test gardening.

* http/wpt/service-workers/header-filtering.https-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232324 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoREGRESSION (r224684): User-agent seen by page does not change when modified by the...
beidson@apple.com [Thu, 31 May 2018 00:20:16 +0000 (00:20 +0000)]
REGRESSION (r224684): User-agent seen by page does not change when modified by the develop menu options after reloading.
<rdar://problem/34918109> and https://bugs.webkit.org/show_bug.cgi?id=186111

Reviewed by Andy Estes.

Source/WebCore:

Test: http/tests/navigation/useragent-reload.php

* Modules/websockets/WebSocketHandshake.cpp:
(WebCore::WebSocketHandshake::clientHandshakeRequest const): Use setUserAgent() for consistency.

* loader/DocumentLoader.cpp:
(WebCore::DocumentLoader::startLoadingMainResource): Clear the user agent from the original DocumentLoader request
  before adding additional fields, as reloads should pick up user agent changes.

Tools:

* WebKitTestRunner/InjectedBundle/Bindings/TestRunner.idl:
* WebKitTestRunner/InjectedBundle/TestRunner.cpp:
(WTR::TestRunner::setCustomUserAgent):
* WebKitTestRunner/InjectedBundle/TestRunner.h:
* WebKitTestRunner/TestController.cpp:
(WTR::TestController::resetStateToConsistentValues):
* WebKitTestRunner/TestInvocation.cpp:
(WTR::TestInvocation::didReceiveSynchronousMessageFromInjectedBundle):

LayoutTests:

* TestExpectations: Skip in WK1
* platform/wk2/TestExpectations: Enable in WK2

* http/tests/navigation/resources/user-agent-script.php: Added.
* http/tests/navigation/useragent-reload-expected.txt: Added.
* http/tests/navigation/useragent-reload.php: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232323 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoFix leak of a CVPixelBufferRef due to early rerturn in -[RTCVideoEncoderH264 encode...
ddkilzer@apple.com [Thu, 31 May 2018 00:18:46 +0000 (00:18 +0000)]
Fix leak of a CVPixelBufferRef due to early rerturn in -[RTCVideoEncoderH264 encode:codecSpecificInfo:frameTypes:]
<https://webkit.org/b/186114>
<rdar://problem/40668097>

Reviewed by Eric Carlson.

* Source/webrtc/sdk/objc/Framework/Classes/VideoToolbox/RTCVideoEncoderH264.mm:
(-[RTCVideoEncoderH264 encode:codecSpecificInfo:frameTypes:]):
Call CVBufferRelease(pixelBuffer) before early return to free
it.
* WebKit/0010-Fix-RTCVideoEncoderH264-CVPixelBuffer-leak.patch: Add.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232322 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years ago[macOS] WebProcess needs TCC entitlements for media capture (Take 2)
bfulgham@apple.com [Thu, 31 May 2018 00:08:35 +0000 (00:08 +0000)]
[macOS] WebProcess needs TCC entitlements for media capture (Take 2)
https://bugs.webkit.org/show_bug.cgi?id=185526
<rdar://problem/36674649>

Reviewed by Alexey Proskuryakov.

In Bug 181995 I added TCC entitlements for media capture to the macOS entitlements used for
relocatable builds. These changes also need to apply to system builds of WebKit.

Previously we had not needed an entitlements file for system builds of WebKit, so only provided
an entitlements file for our relocatable build targets. Now we need entitlements when building
on recent macOS targets using internal SDKs.

Since these various scenarios are beginning to multiple, this patch dynamically generates the
entitlements file.

This patch does the following:

1. Changes the "WebContent-OSX.entitlements" to hold the TCC entitlements needed for media capture, and rename
   to "WebContent-OSX-restricted.entitlements".
2. Removes the unneeded "com.apple.private.xpc.domain-extension" from WebContent-OSX.entitlement, and dynamically
   adds it using a new build step using the script 'process-webcontent-entitlements.sh'
3. Updates DebugRelease.xcconfig to use the 'Safari Engineering' signing key for internal builds.
4. Updates WebContentService.xcconfig to remove the CODE_SIGN_ENTITLEMENTS_OSX_WITH_XPC_DOMAIN_EXTENSION_YES target
   since we no longer need a custom entitlement file for this case.
5. Updates WebContentService.Development.xcconfig to remove CODE_SIGN_ENTITLEMENTS_COCOA_TOUCH_NO since it is no
   longer needed.
6. Deletes the file Configurations/WebContent.Development.entitlements since it is no longer needed.
7. Revises the WebContent process sandbox to allow camera and microphone access without needing TCC
   entitlements, since those cannot be applied without an internal SDK.
8. Revises the UserMediaProcessManager to not pass dynamic sandbox extensions to the WebContent process
   when building with the public SDK since those entitlements cannot be added or consumed in Open
   Source builds.

We want to use the TCC entitlements when building with the internal SDK on recent macOS builds.

* Configurations/Base.xcconfig:
* Configurations/DebugRelease.xcconfig:
* Configurations/WebContent-OSX-restricted.entitlements: Renamed from Source/WebKit/Configurations/WebContent-OSX.entitlements.
* Configurations/WebContent.Development.entitlements: Removed.
* Configurations/WebContentService.Development.xcconfig:
* Configurations/WebContentService.xcconfig:
* Scripts/process-webcontent-entitlements.sh: Added.
* UIProcess/UserMediaProcessManager.cpp:
(WebKit::UserMediaProcessManager::willCreateMediaStream): Don't bother creating and sending sandbox extensions for
camera and microphone control for builds that cannot support them.
* WebKit.xcodeproj/project.pbxproj:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232321 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoWebContent crashes with system preview content
dino@apple.com [Thu, 31 May 2018 00:02:07 +0000 (00:02 +0000)]
WebContent crashes with system preview content
https://bugs.webkit.org/show_bug.cgi?id=186118

Reviewed by Myles Maxfield.

We were receiving crash reports on iOS devices when getting
platformContext() from a GraphicsContext. This usually
occurred when the page was invisible. Debugging showed that
it was trying to draw with a disabled context.

* rendering/RenderThemeIOS.mm:
(WebCore::RenderThemeIOS::paintSystemPreviewBadge): Early
return if the context has painting disabled.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232320 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoUnreviewed, uncomment erroneously commented test code.
keith_miller@apple.com [Wed, 30 May 2018 23:48:36 +0000 (23:48 +0000)]
Unreviewed, uncomment erroneously commented test code.

* stress/error-stack-trace-limit.js:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232319 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoWeb Inspector: Annotate Same-Site cookies
dbates@webkit.org [Wed, 30 May 2018 23:42:36 +0000 (23:42 +0000)]
Web Inspector: Annotate Same-Site cookies
https://bugs.webkit.org/show_bug.cgi?id=184897
<rdar://problem/35178209>

Reviewed by Brian Burg.

Source/JavaScriptCore:

Update protocol to include cookie Same-Site policy.

* inspector/protocol/Page.json:

Source/WebCore:

Store and retrieve the Same-Site cookie policy from CFNetwork.

* inspector/agents/InspectorPageAgent.cpp:
(WebCore::cookieSameSitePolicyJSON): Added.
(WebCore::buildObjectForCookie): Modified to include cookie Same-Site policy.
* platform/Cookie.h:
(WebCore::Cookie::encode const): Encode Same-Site policy.
(WebCore::Cookie::decode): Decode Same-Site policy.
* platform/network/cocoa/CookieCocoa.mm:
(WebCore::portStringFromVector):
(WebCore::coreSameSitePolicy):
(WebCore::nsSameSitePolicy):
(WebCore::Cookie::operator NSHTTPCookie * _Nullable  const):
(WebCore::Cookie::operator NSHTTPCookie * const): Deleted.

Source/WebInspectorUI:

Add a new column for the value of the Same-Site cookie attribute to the resource cookie content
view (shown for a resource under the Network tab) and cookie storage content view (shown under
the Storage tab).

The SameSite column in the resource cookie content view reflects the parsing of the Same-Site
attribute from the HTTP response by Web Inspector. This parsing is materially consistent with
the parsing of the SameSite atttribute in CFNetwork. The Same-Site column in the cookie storage
content view reflects the Same-Site cookie policy associated with the cookies provided by the
network stack, if supported. This column will be blank on systems whose network stack does not
support Same-Site cookies (e.g libsoup).

* Localizations/en.lproj/localizedStrings.js:
* UserInterface/Controllers/HARBuilder.js:
(WI.HARBuilder.cookies):
* UserInterface/Models/Cookie.js:
(WI.Cookie):
(WI.Cookie.displayNameForSameSiteType):
(WI.Cookie.parseSameSiteAttributeValue):
(WI.Cookie.parseSetCookieResponseHeader):
* UserInterface/Views/CookieStorageContentView.js:
(WI.CookieStorageContentView.prototype._rebuildTable):
(WI.CookieStorageContentView.prototype._sortDataGrid):
* UserInterface/Views/ResourceCookiesContentView.js:
(WI.ResourceCookiesContentView.prototype.tablePopulateCell):
(WI.ResourceCookiesContentView.prototype._generateSortComparator):
(WI.ResourceCookiesContentView.prototype._refreshResponseCookiesSection):

LayoutTests:

Update an existing test to ensure we include cookie details in the HTTP Archive (HAR) report.

* http/tests/inspector/network/har/har-page-expected.txt:
* http/tests/inspector/network/har/har-page.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232318 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoAdd console logging to grant-storage-access-under-opener.html to assess its flakiness
wilander@apple.com [Wed, 30 May 2018 23:34:12 +0000 (23:34 +0000)]
Add console logging to grant-storage-access-under-opener.html to assess its flakiness
https://bugs.webkit.org/show_bug.cgi?id=183714
<rdar://problem/38666060>

Unreviewed test gardening.

* http/tests/storageAccess/grant-storage-access-under-opener-expected.txt:
* http/tests/storageAccess/grant-storage-access-under-opener.html:
    Now outputs console log entries, step by step, to see where the timeouts happen.
* platform/ios-wk2/TestExpectations:
    Switched to [ Pass ] on iOS Debug to maximize chance of seeing the flakiness.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232317 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoNavigationAction does not need to hold initiating DOM Event
dbates@webkit.org [Wed, 30 May 2018 23:30:09 +0000 (23:30 +0000)]
NavigationAction does not need to hold initiating DOM Event
https://bugs.webkit.org/show_bug.cgi?id=185958
<rdar://problem/40531539>

Reviewed by Simon Fraser.

Source/WebCore:

Take a similar approach as was done in <https://bugs.webkit.org/show_bug.cgi?id=185712>
and have NavigationAction hold only as many details of the initiating DOM Event as
necessary to support WebKit functionality. Retaining a DOM Event effectively retains
the document that associated with the event. And DocumentLoader always keeps around
the NavigationAction of the last navigation for the "benefit of the various policy handlers".
Therefore, having NavigationAction hold only the relevant details of the DOM Event
instead of the DOM Event itself we avoid keeping the document we navigated from alive
longer than necessary.

* dom/MouseRelatedEvent.h:
* dom/UIEventWithKeyState.h:
* loader/NavigationAction.cpp:
(WebCore::NavigationAction::UIEventWithKeyStateData::UIEventWithKeyStateData): Added.
(WebCore::NavigationAction::MouseEventData::MouseEventData): Added.
(WebCore::keyStateDataForFirstEventWithKeyState): Added.
(WebCore::mouseEventDataForFirstMouseEvent): Added.
* loader/NavigationAction.h:
(WebCore::NavigationAction::keyStateEventData const): Added.
(WebCore::NavigationAction::mouseEventData const): Added
(WebCore::NavigationAction::event const): Deleted.

Source/WebKit:

Write in terms of NavigationAction::{keyStateEventData, mouseEventDataForFirstMouseEvent}().

* WebProcess/InjectedBundle/InjectedBundleNavigationAction.cpp:
(WebKit::mouseButtonForMouseEventData):
(WebKit::syntheticClickTypeForMouseEventData):
(WebKit::clickLocationInRootViewCoordinatesForMouseEventData):
(WebKit::InjectedBundleNavigationAction::modifiersForNavigationAction):
(WebKit::InjectedBundleNavigationAction::mouseButtonForNavigationAction):
(WebKit::InjectedBundleNavigationAction::syntheticClickTypeForNavigationAction):
(WebKit::InjectedBundleNavigationAction::clickLocationInRootViewCoordinatesForNavigationAction):
(WebKit::InjectedBundleNavigationAction::InjectedBundleNavigationAction):
(WebKit::mouseEventForNavigationAction): Deleted.
(WebKit::mouseButtonForMouseEvent): Deleted.
(WebKit::syntheticClickTypeForMouseEvent): Deleted.
(WebKit::clickLocationInRootViewCoordinatesForMouseEvent): Deleted.

Source/WebKitLegacy/mac:

Write in terms of NavigationAction::{keyStateEventData, mouseEventDataForFirstMouseEvent}().

* WebCoreSupport/WebFrameLoaderClient.mm:
(WebFrameLoaderClient::actionDictionary const):
(findMouseEvent): Deleted.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232316 268f45cc-cd09-0410-ab3c-d52691b4dbfc

2 years agoRevert changes made to URLWithUserTypedString made in https://trac.webkit.org/changes...
youenn@apple.com [Wed, 30 May 2018 23:23:00 +0000 (23:23 +0000)]
Revert changes made to URLWithUserTypedString made in https://trac.webkit.org/changeset/232281
https://bugs.webkit.org/show_bug.cgi?id=186108
<rdar://problem/40663065>

Reviewed by Alex Christensen.

Covered by existing API tests.

* platform/mac/WebCoreNSURLExtras.mm:
(WebCore::URLWithUserTypedString):
Return early if WebCore::URL is not able to create a CFURL.
Use CFURL otherwise to keep the existing behavior as much as possible.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@232315 268f45cc-cd09-0410-ab3c-d52691b4dbfc