[webkit-reviews] review granted: [Bug 197229] Visited link hash should be computed only once : [Attachment 368122] patch

[bugzilla-daemon at webkit.org]

Alex Christensen <achristensen at apple.com> has granted Antti Koivisto
<koivisto at iki.fi>'s request for review:
Bug 197229: Visited link hash should be computed only once
https://bugs.webkit.org/show_bug.cgi?id=197229

Attachment 368122: patch

https://bugs.webkit.org/attachment.cgi?id=368122&action=review




--- Comment #6 from Alex Christensen <achristensen at apple.com> ---
Comment on attachment 368122
  --> https://bugs.webkit.org/attachment.cgi?id=368122
patch

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

> Source/WebCore/html/HTMLAnchorElement.h:119
> +    // This is computed only once and must not be affected by subsequent URL
changes.
> +    mutable Optional<SharedStringHash> m_storedVisitedLinkHash;

Would mutable Optional<const SharedStringHash> work?