The example at https://gist.github.com/robertknight/38784b71ddba63b1604b2c6244f216a1 walks the DOM and builds a map of name -> Element where the name is taken from the `data-ref` attribute. On Firefox, Chrome and Edge it successfully reads the `Element.dataset` values and displays `searchBucket, header, content`. On iOS Safari 9.3.5 (tested on an iPhone 5), Safari 9.1.2 (OS X 10.11) it very occasionally displays the correct values but usually reads all the `Element.dataset.ref` values as undefined. The problem disappears with small modifications to the DOM structure, such as deleting a few of the `div` elements, other than the ones with `data-ref` attributes.
The issues is also very sensitive to how `Element.dataset` is accessed. The `findRefs()` function does not modify the DOM but commenting out the two unused calls to it also result in the problem disappearing.
This seems to be working fine for me in Safari 10 so this may have been fixed already. Could you please try with Safari Technology Preview and confirm this is fixed? -> https://developer.apple.com/safari/technology-preview/
Created attachment 290752 [details] dataset['features'] is returning undefined within function This appears to still be happening within Safari 10.0, on both Mac OS X Sierra, as well as iOS 10. With the previous version of Safari, we did not actually notice this happening, but on Safari 10.0, it is very common. See screenshot.
With the original test case at https://gist.github.com/robertknight/38784b71ddba63b1604b2c6244f216a1 I can no longer reproduce the issue on my iPhone 5 with iOS 10. However, by making a trivial modification to site.js so that line 18/19 (`findRefs(searchEl);`) is repeated several more times, I can reproduce the issue again with iOS 10.
This looks like it might be a JIT-related issue. Using the https://itunes.apple.com/us/app/webview-wkwebview-uiwebview/id928647773?mt=8 app, I can reproduce the problem in WKWebView and SafariViewController-powered web views but not UIWebView.
This bug is hitting me pretty hard in a web app I'm developing. Interestingly, I've found that I can work around it by serializing the dataset into JSON and then parsing the JSON, which contains all the correct properties despite .property and ['property'] returning undefined.
This happened to me today. Still an issue. Workaround from Justin Michael did work.
My workaround: https://gist.github.com/anonymous/db9f7359c81e1887fc72dd918dfe5975
This related to caching of properties. I was able to write a test case for this. I'll attach.
Created attachment 308129 [details] Repro case
<rdar://problem/31818563>
JSDOMStringMap has the JSC::GetOwnPropertySlotIsImpure structure flag which is supposed to prevent caching AFAIK. Clearly, it is still caching though...
I wasn't able to reproduce the issue on WebKit trunk nor on STP.
With my report case, I am able to reproduce with shipping Safari but not in the latest Safari Technology Preview. Therefore, I believe this was fixed recently. I'll bisect to try and figure out which change fixed it.
This was fixed by Sam in http://trac.webkit.org/changeset/210667 as part of a large refactoring of our named property getters :) I'll use this bug to land a layout test.
Created attachment 308136 [details] Patch
Comment on attachment 308136 [details] Patch rs=me
Comment on attachment 308136 [details] Patch Clearing flags on attachment: 308136 Committed r215764: <http://trac.webkit.org/changeset/215764>
All reviewed patches have been landed. Closing bug.