<html>
    <head>
      <base href="https://bugs.webkit.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Correctly support fragment-only URLs in CSS images"
   href="https://bugs.webkit.org/show_bug.cgi?id=229917#c17">Comment # 17</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Correctly support fragment-only URLs in CSS images"
   href="https://bugs.webkit.org/show_bug.cgi?id=229917">bug 229917</a>
              from <span class="vcard"><a class="email" href="mailto:darin@apple.com" title="Darin Adler <darin@apple.com>"> <span class="fn">Darin Adler</span></a>
</span></b>
        <pre>Comment on <span class=""><a href="attachment.cgi?id=444322&action=diff" name="attach_444322" title="Patch">attachment 444322</a> <a href="attachment.cgi?id=444322&action=edit" title="Patch">[details]</a></span>
Patch

View in context: <a href="https://bugs.webkit.org/attachment.cgi?id=444322&action=review">https://bugs.webkit.org/attachment.cgi?id=444322&action=review</a>

<span class="quote">> Source/WebCore/dom/Document.cpp:5606
> +    // <a href="https://www.w3.org/TR/css-values-4/#local-urls">https://www.w3.org/TR/css-values-4/#local-urls</a>
> +    // Empty URLs and fragment-only URLs should not be resolved relative to the base
> +    // URL.
> +    if (url.isEmpty() || url.startsWith('#'))
> +        return URL(URL(), url);</span >

I think you are correct to say that it may be incorrect to have this here. We’ll need to see which regression tests fail.

Instead we may want to change CSSImageValue to use another function specific to CSS that adds this behavior and then calls through to Document::completeURL. I might put it in CSSParserContext as a static member function that takes a document, or somewhere else that CSSImageValue can find it. I think there are possibly other call sites like CSSCursorImageValue that may need the same code.</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>