Bug 163429 - [Web IDL] Update [CachedAttribute]'s behavior now that we support [SameObject]
Summary: [Web IDL] Update [CachedAttribute]'s behavior now that we support [SameObject]
Status: ASSIGNED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Bindings (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL:
Keywords:
Depends on: 163414
Blocks:
  Show dependency treegraph
 
Reported: 2016-10-13 21:08 PDT by Chris Dumez
Modified: 2016-10-14 12:16 PDT (History)
6 users (show)

See Also:


Attachments
Patch (24.92 KB, patch)
2016-10-13 21:16 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (26.64 KB, patch)
2016-10-13 21:42 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (25.91 KB, patch)
2016-10-13 22:31 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews125 for ios-simulator-wk2 (9.62 MB, application/zip)
2016-10-13 23:41 PDT, Build Bot
no flags Details
Archive of layout-test-results from ews116 for mac-yosemite (1.71 MB, application/zip)
2016-10-13 23:43 PDT, Build Bot
no flags Details
Archive of layout-test-results from ews102 for mac-yosemite (1.01 MB, application/zip)
2016-10-13 23:54 PDT, Build Bot
no flags Details
Archive of layout-test-results from ews106 for mac-yosemite-wk2 (1.27 MB, application/zip)
2016-10-14 00:02 PDT, Build Bot
no flags Details
Patch (26.57 KB, patch)
2016-10-14 10:49 PDT, Chris Dumez
buildbot: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from ews103 for mac-yosemite (1.13 MB, application/zip)
2016-10-14 11:42 PDT, Build Bot
no flags Details
Archive of layout-test-results from ews113 for mac-yosemite (1.71 MB, application/zip)
2016-10-14 12:02 PDT, Build Bot
no flags Details
Archive of layout-test-results from ews107 for mac-yosemite-wk2 (1.16 MB, application/zip)
2016-10-14 12:08 PDT, Build Bot
no flags Details
Archive of layout-test-results from ews126 for ios-simulator-wk2 (deleted)
2016-10-14 12:16 PDT, Build Bot
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Dumez 2016-10-13 21:08:37 PDT
[SameObject] and [CachedAttribute] currently mean the same thing, so to clean things up:
- Rename [CachedAttribute] to [SameObject] in all IDLs where we always return the same object.
- Update [CachedAttribute] to ask the implementation since if the value has changed since the last access before actually using the cached value. This allows for caching attributes but adds flexibility by supporting invalidation. This is used now used by History.state instead of using custom bindings code to achieve this. In a follow-up patch, this will also be used for MessageEvent.data to fix a bug. I am also considering making [CachedAttribute] being the implicit default for FrozenArray attributes to match the specification more closely.
Comment 1 Chris Dumez 2016-10-13 21:16:08 PDT
Created attachment 291563 [details]
Patch
Comment 2 Chris Dumez 2016-10-13 21:42:41 PDT
Created attachment 291565 [details]
Patch
Comment 3 Chris Dumez 2016-10-13 22:31:03 PDT
Created attachment 291569 [details]
Patch
Comment 4 Build Bot 2016-10-13 23:41:12 PDT
Comment on attachment 291569 [details]
Patch

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

New failing tests:
fast/loader/stateobjects/state-attribute-history-getter.html
Comment 5 Build Bot 2016-10-13 23:41:16 PDT
Created attachment 291575 [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.11.6
Comment 6 Build Bot 2016-10-13 23:43:43 PDT
Comment on attachment 291569 [details]
Patch

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

New failing tests:
fast/loader/stateobjects/state-attribute-history-getter.html
Comment 7 Build Bot 2016-10-13 23:43:47 PDT
Created attachment 291576 [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 8 Build Bot 2016-10-13 23:54:32 PDT
Comment on attachment 291569 [details]
Patch

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

New failing tests:
fast/loader/stateobjects/state-attribute-history-getter.html
Comment 9 Build Bot 2016-10-13 23:54:36 PDT
Created attachment 291577 [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 10 Build Bot 2016-10-14 00:02:22 PDT
Comment on attachment 291569 [details]
Patch

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

New failing tests:
fast/loader/stateobjects/state-attribute-history-getter.html
Comment 11 Build Bot 2016-10-14 00:02:26 PDT
Created attachment 291579 [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 Chris Dumez 2016-10-14 10:49:09 PDT
Created attachment 291647 [details]
Patch
Comment 13 Build Bot 2016-10-14 11:42:45 PDT
Comment on attachment 291647 [details]
Patch

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

New failing tests:
fast/loader/stateobjects/replacestate-then-pushstate.html
fast/loader/stateobjects/pushstate-object-types.html
fast/loader/stateobjects/document-destroyed-navigate-back-with-fragment-scroll.html
fast/loader/stateobjects/state-attribute-popstate-event.html
fast/loader/stateobjects/document-destroyed-navigate-back.html
fast/loader/stateobjects/pushstate-with-fragment-urls-and-hashchange.html
Comment 14 Build Bot 2016-10-14 11:42:49 PDT
Created attachment 291655 [details]
Archive of layout-test-results from ews103 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews103  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 15 Build Bot 2016-10-14 12:02:34 PDT
Comment on attachment 291647 [details]
Patch

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

New failing tests:
fast/loader/stateobjects/replacestate-then-pushstate.html
fast/loader/stateobjects/pushstate-object-types.html
fast/loader/stateobjects/document-destroyed-navigate-back-with-fragment-scroll.html
fast/loader/stateobjects/state-attribute-popstate-event.html
fast/loader/stateobjects/document-destroyed-navigate-back.html
fast/loader/stateobjects/pushstate-with-fragment-urls-and-hashchange.html
Comment 16 Build Bot 2016-10-14 12:02:38 PDT
Created attachment 291656 [details]
Archive of layout-test-results from ews113 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews113  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 17 Build Bot 2016-10-14 12:08:24 PDT
Comment on attachment 291647 [details]
Patch

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

New failing tests:
fast/loader/stateobjects/replacestate-then-pushstate.html
fast/loader/stateobjects/pushstate-object-types.html
fast/loader/stateobjects/document-destroyed-navigate-back-with-fragment-scroll.html
fast/loader/stateobjects/state-attribute-popstate-event.html
fast/loader/stateobjects/document-destroyed-navigate-back.html
fast/loader/stateobjects/pushstate-with-fragment-urls-and-hashchange.html
Comment 18 Build Bot 2016-10-14 12:08:28 PDT
Created attachment 291658 [details]
Archive of layout-test-results from ews107 for mac-yosemite-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews107  Port: mac-yosemite-wk2  Platform: Mac OS X 10.10.5
Comment 19 Build Bot 2016-10-14 12:16:39 PDT
Comment on attachment 291647 [details]
Patch

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

New failing tests:
fast/loader/stateobjects/state-attribute-popstate-event.html
fast/loader/stateobjects/replacestate-then-pushstate.html
fast/loader/stateobjects/document-destroyed-navigate-back.html
fast/loader/stateobjects/pushstate-object-types.html
fast/loader/stateobjects/document-destroyed-navigate-back-with-fragment-scroll.html
Comment 20 Build Bot 2016-10-14 12:16:44 PDT
Created attachment 291659 [details]
Archive of layout-test-results from ews126 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews126  Port: ios-simulator-wk2  Platform: Mac OS X 10.11.6