[webkit-reviews] review requested: [Bug 137382] AX: Going back is broken for VoiceOver : [Attachment 239762] New patch

[bugzilla-daemon at webkit.org]

chris fleizach <cfleizach at apple.com> has asked	for review:
Bug 137382: AX: Going back is broken for VoiceOver
https://bugs.webkit.org/show_bug.cgi?id=137382

Attachment 239762: New patch
https://bugs.webkit.org/attachment.cgi?id=239762&action=review

------- Additional Comments from chris fleizach <cfleizach at apple.com>
Further investigation uncovered more serious problems

1) I was still missing load notifications for some subset or pages in the
history cache. Luckily there's already an existing mechanism that tracks frame
loads

2) The scroll view sometimes cached its children too long. So we should get rid
of that caching. I think there's little performance penalty to putting the same
3 elements back in, and we won't have to worry about caching the wrong elements