Bug 172888

Summary: [JSC] Create a fast path to Object.defineProperty
Product: WebKit Reporter: Caio Lima <ticaiolima>
Component: JavaScriptCoreAssignee: Caio Lima <ticaiolima>
Status: NEW ---    
Severity: Normal CC: buildbot, fpizlo, keith_miller, mark.lam, mjs, msaboff, rniwa, saam, ticaiolima, ysuzuki
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Simple benchmark
none
WIP - It starts
buildbot: commit-queue-
Archive of layout-test-results from ews114 for mac-elcapitan
none
Archive of layout-test-results from ews104 for mac-elcapitan-wk2
none
Archive of layout-test-results from ews125 for ios-simulator-wk2
none
Archive of layout-test-results from ews100 for mac-elcapitan
none
RFC - Patch proposal
none
Patch
none
Benchmarks
none
Archive of layout-test-results from ews112 for mac-elcapitan
none
Archive of layout-test-results from ews101 for mac-elcapitan
none
Patch
none
Archive of layout-test-results from ews102 for mac-elcapitan
none
Archive of layout-test-results from ews112 for mac-elcapitan
none
Archive of layout-test-results from ews123 for ios-simulator-wk2
none
Archive of layout-test-results from ews104 for mac-elcapitan-wk2
none
Patch
none
Archive of layout-test-results from ews102 for mac-elcapitan
none
Archive of layout-test-results from ews106 for mac-elcapitan-wk2
none
Archive of layout-test-results from ews115 for mac-elcapitan
none
Archive of layout-test-results from ews125 for ios-simulator-wk2
none
Patch
mjs: review-, buildbot: commit-queue-
Archive of layout-test-results from ews124 for ios-simulator-wk2 none

Description Caio Lima 2017-06-02 18:56:45 PDT
We can create a fast path to Object.defineProperty to get improvements due Object Allocation Sink
Comment 1 Yusuke Suzuki 2017-06-03 02:52:02 PDT
The following is the design in my mind

We can find that Object.defineProperty is frequently called in some realistic frameworks such as Ember.js.
Object.defineProperty is typically used in the following form.

Object.defineProperty(object, name, {
    // property descriptor object
});

If we can create Object.defineProperty's prologue in JS, we can encourage Object Allocation Sinking to sink the above property descriptor object.
My design is simple. We already have some bytecodes like op_define_data_property / op_define_accessor_property.
So, let's add bytecode intrinsic for them and wrap them with JS function.

function defineProperty(object, name, descriptor) {
    // Retrieve necessary values.
    var enumerable = descriptor.enumerable;
    ...
    // And call intrinsics.
    if (accessor property)
        @defineAccessorProperty(...);
    else
        @defineDataProperty(...);
    ....
}
Comment 2 Yusuke Suzuki 2017-06-03 04:23:05 PDT
(In reply to Yusuke Suzuki from comment #1)
> The following is the design in my mind
> 
> We can find that Object.defineProperty is frequently called in some
> realistic frameworks such as Ember.js.
> Object.defineProperty is typically used in the following form.
> 
> Object.defineProperty(object, name, {
>     // property descriptor object
> });
> 
> If we can create Object.defineProperty's prologue in JS, we can encourage
> Object Allocation Sinking to sink the above property descriptor object.
> My design is simple. We already have some bytecodes like
> op_define_data_property / op_define_accessor_property.
> So, let's add bytecode intrinsic for them and wrap them with JS function.
> 
> function defineProperty(object, name, descriptor) {
>     // Retrieve necessary values.
>     var enumerable = descriptor.enumerable;
>     ...
>     // And call intrinsics.
>     if (accessor property)
>         @defineAccessorProperty(...);
>     else
>         @defineDataProperty(...);
>     ....
> }

At that time, we need extra carefully construct this defineProperty function to be inlined.
Comment 3 Caio Lima 2017-07-10 11:19:58 PDT
WIP. Patch coming soon.
Comment 4 Caio Lima 2017-07-16 12:05:25 PDT
Created attachment 315614 [details]
Simple benchmark

Just a microbenchmark to see the speedup in this operation.
Comment 5 Caio Lima 2017-07-16 12:26:02 PDT
Created attachment 315616 [details]
WIP - It starts

RFC here. It is just starting the implementation.

This implementation isn't spec compliant and I'm thinking create intrinsic functions to verify if the descriptor has some property without user observable behavior, otherwise it will be classified as escaped object if we use "in" operation.

I'm also in doubt if there is another case where the user-observable behavior can be captured other than proxies case, because according the spec[1], ToPropertyDescriptor operation is called.

The fast case I'm planing here is when we have "Object.defineProperty(a, "foo", {/* descriptor */})" and descriptor is a JS Object allocated in DFG with "NewObject".

[1]-https://tc39.github.io/ecma262/#sec-object.defineproperty
Comment 6 Build Bot 2017-07-16 12:29:16 PDT
Attachment 315616 [details] did not pass style-queue:


ERROR: Source/JavaScriptCore/runtime/JSGlobalObjectFunctions.cpp:827:  One line control clauses should not use braces.  [whitespace/braces] [4]
Total errors found: 1 in 13 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 7 Build Bot 2017-07-16 13:10:10 PDT
Comment on attachment 315616 [details]
WIP - It starts

Attachment 315616 [details] did not pass jsc-ews (mac):
Output: http://webkit-queues.webkit.org/results/4131901

New failing tests:
ChakraCore.yaml/ChakraCore/test/es5/InsufficientArguments.js.default
jsc-layout-tests.yaml/js/script-tests/function-toString-vs-name.js.layout-ftl-no-cjit
jsc-layout-tests.yaml/js/script-tests/function-toString-vs-name.js.layout-dfg-eager-no-cjit
jsc-layout-tests.yaml/js/script-tests/array-defineOwnProperty.js.layout
jsc-layout-tests.yaml/js/script-tests/array-defineOwnProperty.js.layout-dfg-eager-no-cjit
ChakraCore.yaml/ChakraCore/test/es5/enumerable.js.default
stress/array-species-functions.js.ftl-eager
jsc-layout-tests.yaml/js/script-tests/function-toString-vs-name.js.layout-no-cjit
jsc-layout-tests.yaml/js/script-tests/function-toString-vs-name.js.layout-no-ftl
stress/array-species-functions.js.no-llint
stress/array-species-functions.js.dfg-maximal-flush-validate-no-cjit
stress/array-species-functions.js.no-cjit-validate-phases
stress/array-species-functions.js.ftl-eager-no-cjit-b3o1
stress/array-species-functions.js.no-ftl
stress/array-species-functions.js.ftl-no-cjit-no-put-stack-validate
ChakraCore.yaml/ChakraCore/test/es6/proxyprotobug.js.default
stress/array-species-functions.js.no-cjit-collect-continuously
jsc-layout-tests.yaml/js/script-tests/function-toString-vs-name.js.layout-ftl-eager-no-cjit
stress/array-species-functions.js.ftl-no-cjit-b3o1
stress/array-species-functions.js.dfg-eager-no-cjit-validate
jsc-layout-tests.yaml/js/script-tests/array-defineOwnProperty.js.layout-no-cjit
ChakraCore.yaml/ChakraCore/test/es5/defineProperty.js.default
stress/array-species-functions.js.default
jsc-layout-tests.yaml/js/script-tests/array-defineOwnProperty.js.layout-ftl-no-cjit
stress/array-species-functions.js.ftl-no-cjit-validate-sampling-profiler
ChakraCore.yaml/ChakraCore/test/es5/defineIndexProperty.js.default
stress/array-species-functions.js.dfg-eager
jsc-layout-tests.yaml/js/script-tests/function-toString-vs-name.js.layout
stress/array-species-functions.js.ftl-eager-no-cjit
stress/array-species-functions.js.ftl-no-cjit-small-pool
stress/array-species-functions.js.ftl-no-cjit-no-inline-validate
jsc-layout-tests.yaml/js/script-tests/array-defineOwnProperty.js.layout-ftl-eager-no-cjit
jsc-layout-tests.yaml/js/script-tests/array-defineOwnProperty.js.layout-no-llint
jsc-layout-tests.yaml/js/script-tests/function-toString-vs-name.js.layout-no-llint
jsc-layout-tests.yaml/js/script-tests/array-defineOwnProperty.js.layout-no-ftl
Comment 8 Build Bot 2017-07-16 13:22:41 PDT
Comment on attachment 315616 [details]
WIP - It starts

Attachment 315616 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/4131930

Number of test failures exceeded the failure limit.
Comment 9 Build Bot 2017-07-16 13:22:42 PDT
Created attachment 315620 [details]
Archive of layout-test-results from ews114 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews114  Port: mac-elcapitan  Platform: Mac OS X 10.11.6
Comment 10 Build Bot 2017-07-16 13:32:06 PDT
Comment on attachment 315616 [details]
WIP - It starts

Attachment 315616 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/4131956

New failing tests:
imported/w3c/web-platform-tests/streams/readable-streams/bad-underlying-sources.dedicatedworker.html
streams/shadowing-defineProperty.html
imported/w3c/web-platform-tests/streams/piping/close-propagation-forward.html
imported/w3c/web-platform-tests/streams/piping/error-propagation-forward.html
imported/w3c/web-platform-tests/streams/piping/general.html
js/dom/Object-defineProperty.html
imported/w3c/web-platform-tests/streams/count-queuing-strategy.dedicatedworker.html
imported/w3c/web-platform-tests/streams/piping/close-propagation-forward.dedicatedworker.html
imported/w3c/web-platform-tests/streams/readable-streams/count-queuing-strategy-integration.dedicatedworker.html
imported/w3c/web-platform-tests/streams/readable-streams/bad-underlying-sources.html
imported/w3c/web-platform-tests/streams/byte-length-queuing-strategy.html
imported/w3c/web-platform-tests/streams/piping/error-propagation-forward.dedicatedworker.html
imported/w3c/web-platform-tests/streams/byte-length-queuing-strategy.dedicatedworker.html
imported/w3c/web-platform-tests/streams/piping/flow-control.dedicatedworker.html
streams/reference-implementation/count-queuing-strategy.html
js/function-toString-vs-name.html
imported/w3c/web-platform-tests/streams/readable-streams/count-queuing-strategy-integration.html
imported/w3c/web-platform-tests/streams/piping/error-propagation-backward.dedicatedworker.html
streams/reference-implementation/byte-length-queuing-strategy.html
js/array-defineOwnProperty.html
imported/w3c/web-platform-tests/streams/piping/flow-control.html
imported/w3c/web-platform-tests/streams/count-queuing-strategy.html
Comment 11 Build Bot 2017-07-16 13:32:07 PDT
Created attachment 315621 [details]
Archive of layout-test-results from ews104 for mac-elcapitan-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews104  Port: mac-elcapitan-wk2  Platform: Mac OS X 10.11.6
Comment 12 Build Bot 2017-07-16 13:56:11 PDT
Comment on attachment 315616 [details]
WIP - It starts

Attachment 315616 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/4131974

New failing tests:
imported/w3c/web-platform-tests/streams/readable-streams/bad-underlying-sources.dedicatedworker.html
streams/shadowing-defineProperty.html
imported/w3c/web-platform-tests/streams/piping/close-propagation-forward.html
imported/w3c/web-platform-tests/streams/piping/error-propagation-forward.html
imported/w3c/web-platform-tests/streams/piping/close-propagation-forward.dedicatedworker.html
js/dom/Object-defineProperty.html
imported/w3c/web-platform-tests/streams/count-queuing-strategy.dedicatedworker.html
imported/w3c/web-platform-tests/streams/piping/general.html
imported/w3c/web-platform-tests/streams/readable-streams/count-queuing-strategy-integration.dedicatedworker.html
imported/w3c/web-platform-tests/streams/readable-streams/bad-underlying-sources.html
imported/w3c/web-platform-tests/streams/byte-length-queuing-strategy.html
imported/w3c/web-platform-tests/streams/piping/error-propagation-forward.dedicatedworker.html
imported/w3c/web-platform-tests/streams/byte-length-queuing-strategy.dedicatedworker.html
imported/w3c/web-platform-tests/streams/piping/flow-control.dedicatedworker.html
streams/reference-implementation/count-queuing-strategy.html
js/function-toString-vs-name.html
imported/w3c/web-platform-tests/streams/readable-streams/count-queuing-strategy-integration.html
imported/w3c/web-platform-tests/streams/piping/error-propagation-backward.dedicatedworker.html
streams/reference-implementation/byte-length-queuing-strategy.html
js/array-defineOwnProperty.html
imported/w3c/web-platform-tests/streams/piping/flow-control.html
imported/w3c/web-platform-tests/streams/count-queuing-strategy.html
Comment 13 Build Bot 2017-07-16 13:56:13 PDT
Created attachment 315625 [details]
Archive of layout-test-results from ews125 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews125  Port: ios-simulator-wk2  Platform: Mac OS X 10.12.5
Comment 14 Build Bot 2017-07-16 14:01:50 PDT
Comment on attachment 315616 [details]
WIP - It starts

Attachment 315616 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/4132053

New failing tests:
imported/w3c/web-platform-tests/streams/readable-streams/bad-underlying-sources.dedicatedworker.html
streams/shadowing-defineProperty.html
imported/w3c/web-platform-tests/streams/piping/close-propagation-forward.html
imported/w3c/web-platform-tests/streams/piping/error-propagation-forward.html
imported/w3c/web-platform-tests/streams/piping/general.html
js/dom/Object-defineProperty.html
imported/w3c/web-platform-tests/streams/count-queuing-strategy.dedicatedworker.html
imported/w3c/web-platform-tests/streams/piping/close-propagation-forward.dedicatedworker.html
imported/w3c/web-platform-tests/streams/readable-streams/count-queuing-strategy-integration.dedicatedworker.html
imported/w3c/web-platform-tests/streams/readable-streams/bad-underlying-sources.html
imported/w3c/web-platform-tests/streams/byte-length-queuing-strategy.html
imported/w3c/web-platform-tests/streams/piping/error-propagation-forward.dedicatedworker.html
imported/w3c/web-platform-tests/streams/byte-length-queuing-strategy.dedicatedworker.html
imported/w3c/web-platform-tests/streams/piping/flow-control.dedicatedworker.html
streams/reference-implementation/count-queuing-strategy.html
js/function-toString-vs-name.html
imported/w3c/web-platform-tests/streams/readable-streams/count-queuing-strategy-integration.html
imported/w3c/web-platform-tests/streams/piping/error-propagation-backward.dedicatedworker.html
streams/reference-implementation/byte-length-queuing-strategy.html
js/array-defineOwnProperty.html
imported/w3c/web-platform-tests/streams/piping/flow-control.html
imported/w3c/web-platform-tests/streams/count-queuing-strategy.html
Comment 15 Build Bot 2017-07-16 14:01:52 PDT
Created attachment 315626 [details]
Archive of layout-test-results from ews100 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews100  Port: mac-elcapitan  Platform: Mac OS X 10.11.6
Comment 16 Caio Lima 2017-08-06 13:13:36 PDT
Created attachment 317382 [details]
RFC - Patch proposal

It's a WIP. Not all test passing, but I would like to get feedback in the Path I'm following here. I'm not 100% sure if the Machinery into FTL analysis are right.

The main idea into FTL is that since the descriptor allocation won't exist, I need to check the presence of properties from structure.

Also, I think that fast path rule should allow just descriptors with prototype === Object. Otherwise it should fall to slow path, since we can have proxies into prototype chain.
Comment 17 Caio Lima 2017-10-04 02:14:49 PDT
Created attachment 322641 [details]
Patch
Comment 18 Caio Lima 2017-10-04 02:20:25 PDT
Created attachment 322642 [details]
Benchmarks

These are the results of current benchmarks. I can see some regressions on "regexp-prototype-*-observable-side-effects" and "string-prototype-*-observable-side-effects", but as the function is called 4 times in each test, there is no JIT and we can't benefit from it.
Comment 19 Build Bot 2017-10-04 11:44:36 PDT
Comment on attachment 322641 [details]
Patch

Attachment 322641 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/4755956

New failing tests:
imported/w3c/web-platform-tests/streams/readable-streams/bad-underlying-sources.dedicatedworker.html
ietestcenter/Javascript/15.2.3.6-3-5.html
ietestcenter/Javascript/15.2.3.6-3-11.html
imported/w3c/web-platform-tests/streams/piping/close-propagation-forward.html
imported/w3c/web-platform-tests/streams/piping/error-propagation-forward.html
ietestcenter/Javascript/15.2.3.6-3-14.html
imported/w3c/web-platform-tests/streams/piping/close-propagation-forward.dedicatedworker.html
js/dom/Object-defineProperty.html
imported/w3c/web-platform-tests/streams/count-queuing-strategy.dedicatedworker.html
ietestcenter/Javascript/15.2.3.6-3-10.html
streams/shadowing-defineProperty.html
imported/w3c/web-platform-tests/streams/piping/general.html
ietestcenter/Javascript/15.2.3.6-3-12.html
imported/w3c/web-platform-tests/streams/readable-streams/count-queuing-strategy-integration.dedicatedworker.html
ietestcenter/Javascript/15.2.3.6-3-7.html
imported/w3c/web-platform-tests/streams/readable-streams/bad-underlying-sources.html
imported/w3c/web-platform-tests/streams/byte-length-queuing-strategy.html
ietestcenter/Javascript/15.2.3.6-3-8.html
imported/w3c/web-platform-tests/streams/byte-length-queuing-strategy.dedicatedworker.html
ietestcenter/Javascript/15.2.3.6-3-9.html
imported/w3c/web-platform-tests/streams/piping/flow-control.dedicatedworker.html
streams/reference-implementation/count-queuing-strategy.html
ietestcenter/Javascript/15.2.3.6-3-13.html
imported/w3c/web-platform-tests/streams/readable-streams/count-queuing-strategy-integration.html
imported/w3c/web-platform-tests/streams/piping/error-propagation-backward.dedicatedworker.html
streams/reference-implementation/byte-length-queuing-strategy.html
ietestcenter/Javascript/15.2.3.6-3-6.html
imported/w3c/web-platform-tests/streams/piping/flow-control.html
imported/w3c/web-platform-tests/streams/count-queuing-strategy.html
Comment 20 Build Bot 2017-10-04 11:44:38 PDT
Created attachment 322697 [details]
Archive of layout-test-results from ews112 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews112  Port: mac-elcapitan  Platform: Mac OS X 10.11.6
Comment 21 Build Bot 2017-10-04 14:24:08 PDT
Comment on attachment 322641 [details]
Patch

Attachment 322641 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/4757981

New failing tests:
imported/w3c/web-platform-tests/streams/readable-streams/bad-underlying-sources.dedicatedworker.html
ietestcenter/Javascript/15.2.3.6-3-5.html
ietestcenter/Javascript/15.2.3.6-3-11.html
imported/w3c/web-platform-tests/streams/piping/close-propagation-forward.html
imported/w3c/web-platform-tests/streams/piping/error-propagation-forward.html
ietestcenter/Javascript/15.2.3.6-3-14.html
imported/w3c/web-platform-tests/streams/piping/close-propagation-forward.dedicatedworker.html
js/dom/Object-defineProperty.html
imported/w3c/web-platform-tests/streams/count-queuing-strategy.dedicatedworker.html
ietestcenter/Javascript/15.2.3.6-3-10.html
streams/shadowing-defineProperty.html
imported/w3c/web-platform-tests/streams/piping/general.html
imported/w3c/web-platform-tests/streams/readable-streams/count-queuing-strategy-integration.dedicatedworker.html
ietestcenter/Javascript/15.2.3.6-3-12.html
ietestcenter/Javascript/15.2.3.6-3-7.html
imported/w3c/web-platform-tests/streams/readable-streams/bad-underlying-sources.html
imported/w3c/web-platform-tests/streams/byte-length-queuing-strategy.html
ietestcenter/Javascript/15.2.3.6-3-8.html
imported/w3c/web-platform-tests/streams/byte-length-queuing-strategy.dedicatedworker.html
ietestcenter/Javascript/15.2.3.6-3-9.html
imported/w3c/web-platform-tests/streams/piping/flow-control.dedicatedworker.html
streams/reference-implementation/count-queuing-strategy.html
ietestcenter/Javascript/15.2.3.6-3-13.html
imported/w3c/web-platform-tests/streams/readable-streams/count-queuing-strategy-integration.html
imported/w3c/web-platform-tests/streams/piping/error-propagation-backward.dedicatedworker.html
streams/reference-implementation/byte-length-queuing-strategy.html
ietestcenter/Javascript/15.2.3.6-3-6.html
imported/w3c/web-platform-tests/streams/piping/flow-control.html
imported/w3c/web-platform-tests/streams/count-queuing-strategy.html
Comment 22 Build Bot 2017-10-04 14:24:09 PDT
Created attachment 322727 [details]
Archive of layout-test-results from ews101 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews101  Port: mac-elcapitan  Platform: Mac OS X 10.11.6
Comment 23 Caio Lima 2017-10-04 16:12:51 PDT
Created attachment 322736 [details]
Patch
Comment 24 Build Bot 2017-10-04 16:57:14 PDT
Comment on attachment 322736 [details]
Patch

Attachment 322736 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/4759348

New failing tests:
ietestcenter/Javascript/15.2.3.6-3-10.html
ietestcenter/Javascript/15.2.3.6-3-8.html
ietestcenter/Javascript/15.2.3.6-3-9.html
ietestcenter/Javascript/15.2.3.6-3-5.html
ietestcenter/Javascript/15.2.3.6-3-12.html
ietestcenter/Javascript/15.2.3.6-3-7.html
ietestcenter/Javascript/15.2.3.6-3-14.html
ietestcenter/Javascript/15.2.3.6-3-13.html
js/dom/Object-defineProperty.html
ietestcenter/Javascript/15.2.3.6-3-6.html
ietestcenter/Javascript/15.2.3.6-3-11.html
Comment 25 Build Bot 2017-10-04 16:57:16 PDT
Created attachment 322743 [details]
Archive of layout-test-results from ews102 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-elcapitan  Platform: Mac OS X 10.11.6
Comment 26 Build Bot 2017-10-04 17:40:10 PDT
Comment on attachment 322736 [details]
Patch

Attachment 322736 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/4759749

New failing tests:
ietestcenter/Javascript/15.2.3.6-3-10.html
ietestcenter/Javascript/15.2.3.6-3-8.html
ietestcenter/Javascript/15.2.3.6-3-9.html
ietestcenter/Javascript/15.2.3.6-3-5.html
ietestcenter/Javascript/15.2.3.6-3-12.html
ietestcenter/Javascript/15.2.3.6-3-7.html
ietestcenter/Javascript/15.2.3.6-3-14.html
ietestcenter/Javascript/15.2.3.6-3-13.html
js/dom/Object-defineProperty.html
ietestcenter/Javascript/15.2.3.6-3-6.html
ietestcenter/Javascript/15.2.3.6-3-11.html
Comment 27 Build Bot 2017-10-04 17:40:12 PDT
Created attachment 322749 [details]
Archive of layout-test-results from ews112 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews112  Port: mac-elcapitan  Platform: Mac OS X 10.11.6
Comment 28 Build Bot 2017-10-04 17:47:05 PDT
Comment on attachment 322736 [details]
Patch

Attachment 322736 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/4759773

New failing tests:
ietestcenter/Javascript/15.2.3.6-3-10.html
ietestcenter/Javascript/15.2.3.6-3-8.html
ietestcenter/Javascript/15.2.3.6-3-9.html
ietestcenter/Javascript/15.2.3.6-3-5.html
ietestcenter/Javascript/15.2.3.6-3-12.html
ietestcenter/Javascript/15.2.3.6-3-7.html
ietestcenter/Javascript/15.2.3.6-3-14.html
ietestcenter/Javascript/15.2.3.6-3-13.html
js/dom/Object-defineProperty.html
ietestcenter/Javascript/15.2.3.6-3-6.html
ietestcenter/Javascript/15.2.3.6-3-11.html
Comment 29 Build Bot 2017-10-04 17:47:07 PDT
Created attachment 322751 [details]
Archive of layout-test-results from ews123 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews123  Port: ios-simulator-wk2  Platform: Mac OS X 10.12.6
Comment 30 Build Bot 2017-10-04 18:11:39 PDT
Comment on attachment 322736 [details]
Patch

Attachment 322736 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/4760307

New failing tests:
ietestcenter/Javascript/15.2.3.6-3-10.html
ietestcenter/Javascript/15.2.3.6-3-8.html
ietestcenter/Javascript/15.2.3.6-3-9.html
ietestcenter/Javascript/15.2.3.6-3-5.html
ietestcenter/Javascript/15.2.3.6-3-12.html
ietestcenter/Javascript/15.2.3.6-3-7.html
ietestcenter/Javascript/15.2.3.6-3-14.html
ietestcenter/Javascript/15.2.3.6-3-13.html
js/dom/Object-defineProperty.html
ietestcenter/Javascript/15.2.3.6-3-6.html
ietestcenter/Javascript/15.2.3.6-3-11.html
Comment 31 Build Bot 2017-10-04 18:11:41 PDT
Created attachment 322754 [details]
Archive of layout-test-results from ews104 for mac-elcapitan-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews104  Port: mac-elcapitan-wk2  Platform: Mac OS X 10.11.6
Comment 32 Caio Lima 2017-10-05 04:34:58 PDT
Created attachment 322822 [details]
Patch
Comment 33 Build Bot 2017-10-05 05:19:28 PDT
Comment on attachment 322822 [details]
Patch

Attachment 322822 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/4766568

New failing tests:
js/dom/Object-defineProperty.html
ietestcenter/Javascript/15.2.3.6-3-8.html
ietestcenter/Javascript/15.2.3.6-3-13.html
Comment 34 Build Bot 2017-10-05 05:19:30 PDT
Created attachment 322826 [details]
Archive of layout-test-results from ews102 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-elcapitan  Platform: Mac OS X 10.11.6
Comment 35 Build Bot 2017-10-05 05:25:59 PDT
Comment on attachment 322822 [details]
Patch

Attachment 322822 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/4766583

New failing tests:
js/dom/Object-defineProperty.html
ietestcenter/Javascript/15.2.3.6-3-8.html
ietestcenter/Javascript/15.2.3.6-3-13.html
Comment 36 Build Bot 2017-10-05 05:26:00 PDT
Created attachment 322827 [details]
Archive of layout-test-results from ews106 for mac-elcapitan-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-elcapitan-wk2  Platform: Mac OS X 10.11.6
Comment 37 Build Bot 2017-10-05 05:51:51 PDT
Comment on attachment 322822 [details]
Patch

Attachment 322822 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/4766600

New failing tests:
js/dom/Object-defineProperty.html
ietestcenter/Javascript/15.2.3.6-3-8.html
ietestcenter/Javascript/15.2.3.6-3-13.html
Comment 38 Build Bot 2017-10-05 05:51:53 PDT
Created attachment 322831 [details]
Archive of layout-test-results from ews115 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews115  Port: mac-elcapitan  Platform: Mac OS X 10.11.6
Comment 39 Build Bot 2017-10-05 06:10:22 PDT
Comment on attachment 322822 [details]
Patch

Attachment 322822 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/4766715

New failing tests:
js/dom/Object-defineProperty.html
ietestcenter/Javascript/15.2.3.6-3-8.html
ietestcenter/Javascript/15.2.3.6-3-13.html
Comment 40 Build Bot 2017-10-05 06:10:23 PDT
Created attachment 322832 [details]
Archive of layout-test-results from ews125 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews125  Port: ios-simulator-wk2  Platform: Mac OS X 10.12.6
Comment 41 Caio Lima 2017-10-05 08:31:17 PDT
Created attachment 322839 [details]
Patch
Comment 42 Build Bot 2017-10-05 10:04:15 PDT
Comment on attachment 322839 [details]
Patch

Attachment 322839 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/4768418

New failing tests:
accessibility/ios-simulator/video-elements-ios.html
Comment 43 Build Bot 2017-10-05 10:04:17 PDT
Created attachment 322846 [details]
Archive of layout-test-results from ews124 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews124  Port: ios-simulator-wk2  Platform: Mac OS X 10.12.6
Comment 44 Maciej Stachowiak 2020-05-30 19:46:53 PDT
Comment on attachment 322839 [details]
Patch

Unfortunately, this old patch no longer applies.
Comment 45 Maciej Stachowiak 2020-05-30 19:47:10 PDT
(Perf wins are good of course, if this is still relevant.)