Bug 164935 - [WebIDL] Add support for record types
Summary: [WebIDL] Add support for record types
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Sam Weinig
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-11-18 10:14 PST by Sam Weinig
Modified: 2016-11-18 13:47 PST (History)
6 users (show)

See Also:


Attachments
Patch (92.40 KB, patch)
2016-11-18 10:45 PST, Sam Weinig
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews116 for mac-yosemite (1.64 MB, application/zip)
2016-11-18 12:06 PST, Build Bot
no flags Details
Patch (92.48 KB, patch)
2016-11-18 12:09 PST, Sam Weinig
buildbot: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from ews117 for mac-yosemite (1.75 MB, application/zip)
2016-11-18 13:22 PST, Build Bot
no flags Details
Archive of layout-test-results from ews106 for mac-yosemite-wk2 (1001.13 KB, application/zip)
2016-11-18 13:23 PST, Build Bot
no flags Details
Archive of layout-test-results from ews102 for mac-yosemite (1.16 MB, application/zip)
2016-11-18 13:24 PST, Build Bot
no flags Details
Archive of layout-test-results from ews123 for ios-simulator-wk2 (9.07 MB, application/zip)
2016-11-18 13:33 PST, Build Bot
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Sam Weinig 2016-11-18 10:14:48 PST
[WebIDL] Add support for record types
Comment 1 Sam Weinig 2016-11-18 10:45:37 PST
Created attachment 295165 [details]
Patch
Comment 2 WebKit Commit Bot 2016-11-18 10:47:24 PST
Attachment 295165 [details] did not pass style-queue:


ERROR: Source/WebCore/bindings/js/JSDOMConvert.h:1092:  Tests for true/false, null/non-null, and zero/non-zero should all be done without equality comparisons.  [readability/comparison_to_zero] [5]
ERROR: Source/WebCore/bindings/js/JSDOMConvert.h:1097:  Tests for true/false, null/non-null, and zero/non-zero should all be done without equality comparisons.  [readability/comparison_to_zero] [5]
ERROR: Source/WebCore/bindings/js/JSDOMConvert.h:1098:  Tests for true/false, null/non-null, and zero/non-zero should all be done without equality comparisons.  [readability/comparison_to_zero] [5]
Total errors found: 3 in 16 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Tim Horton 2016-11-18 11:25:41 PST
Comment on attachment 295165 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=295165&action=review

Very red.

> Source/WebCore/bindings/js/JSDOMConvert.h:1122
> +                //     1. If types includes a dictionary type, then return the result of converting V to that dictionary type.

Too many leading spaces?
Comment 4 Build Bot 2016-11-18 12:06:06 PST
Comment on attachment 295165 [details]
Patch

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

New failing tests:
js/dom/webidl-type-mapping.html
Comment 5 Build Bot 2016-11-18 12:06:09 PST
Created attachment 295173 [details]
Archive of layout-test-results from ews116 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews116  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 6 Sam Weinig 2016-11-18 12:09:40 PST
Created attachment 295174 [details]
Patch
Comment 7 WebKit Commit Bot 2016-11-18 12:11:56 PST
Attachment 295174 [details] did not pass style-queue:


ERROR: Source/WebCore/bindings/js/JSDOMConvert.h:1092:  Tests for true/false, null/non-null, and zero/non-zero should all be done without equality comparisons.  [readability/comparison_to_zero] [5]
ERROR: Source/WebCore/bindings/js/JSDOMConvert.h:1097:  Tests for true/false, null/non-null, and zero/non-zero should all be done without equality comparisons.  [readability/comparison_to_zero] [5]
ERROR: Source/WebCore/bindings/js/JSDOMConvert.h:1098:  Tests for true/false, null/non-null, and zero/non-zero should all be done without equality comparisons.  [readability/comparison_to_zero] [5]
Total errors found: 3 in 16 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 8 Build Bot 2016-11-18 13:22:39 PST
Comment on attachment 295174 [details]
Patch

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

New failing tests:
js/dom/webidl-type-mapping.html
Comment 9 Build Bot 2016-11-18 13:22:42 PST
Created attachment 295181 [details]
Archive of layout-test-results from ews117 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews117  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 10 Build Bot 2016-11-18 13:23:47 PST
Comment on attachment 295174 [details]
Patch

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

New failing tests:
js/dom/webidl-type-mapping.html
Comment 11 Build Bot 2016-11-18 13:23:51 PST
Created attachment 295182 [details]
Archive of layout-test-results from ews106 for mac-yosemite-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-yosemite-wk2  Platform: Mac OS X 10.10.5
Comment 12 Build Bot 2016-11-18 13:24:13 PST
Comment on attachment 295174 [details]
Patch

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

New failing tests:
js/dom/webidl-type-mapping.html
Comment 13 Build Bot 2016-11-18 13:24:16 PST
Created attachment 295183 [details]
Archive of layout-test-results from ews102 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 14 Build Bot 2016-11-18 13:33:31 PST
Comment on attachment 295174 [details]
Patch

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

New failing tests:
js/dom/webidl-type-mapping.html
Comment 15 Build Bot 2016-11-18 13:33:35 PST
Created attachment 295186 [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.11.6
Comment 16 Sam Weinig 2016-11-18 13:47:49 PST
Committed r208893: <http://trac.webkit.org/changeset/208893>