[Webkit-unassigned] [Bug 74386] Migrate createObjectURL & revokeObjectURL to static (Class) methods
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Dec 13 10:11:45 PST 2011
https://bugs.webkit.org/show_bug.cgi?id=74386
--- Comment #3 from Erik Arvidsson <arv at chromium.org> 2011-12-13 10:11:45 PST ---
(From update of attachment 118981)
View in context: https://bugs.webkit.org/attachment.cgi?id=118981&action=review
> Source/WebCore/ChangeLog:8
> + Test: fast/dom/DOMURL/check-instanceof-domurl-functions.html
It is probably worth pointing out that there are existing tests that cover webkitURL.
> Source/WebCore/html/DOMURL.cpp:179
> +DOMURL::DOMURL(const String& url, const String& baseURL)
> +{
> + m_href = KURL(KURL(), url);
> + // Resolve the URL relative to baseURL if is provide.
> + if (!baseURL.isEmpty())
> + m_href = KURL(m_href, baseURL);
> +}
Can this be taken out of this patch? In this patch we don't want to be able to instantiate new URL objects.
> Source/WebCore/html/DOMURL.h:59
> + KURL m_href;
Not needed in this patch.
> Source/WebCore/html/DOMURL.idl:-28
> - Conditional=BLOB,
Keep conditional in this patch?
> Source/WebCore/html/DOMURL.idl:32
> + Constructor(in DOMString url, in [Optional=CallWithNullValue] DOMString baseURL)
For this patch I would remove the Constructor since at this point there is no need to be able to instantiate this.
> Source/WebCore/page/DOMWindow.idl:817
> + attribute DOMURLConstructor webkitURL;
Given this patch is only about moving these methods from instance methods to class methods it still makes sense to keep the conditional here and in WorkerContext.idl
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list