[Webkit-unassigned] [Bug 74386] Migrate createObjectURL & revokeObjectURL to static (Class) methods
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Jan 19 14:28:49 PST 2012
https://bugs.webkit.org/show_bug.cgi?id=74386
Adam Barth <abarth at webkit.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #123078|review?, commit-queue? |review+, commit-queue-
Flag| |
--- Comment #19 from Adam Barth <abarth at webkit.org> 2012-01-19 14:28:49 PST ---
(From update of attachment 123078)
View in context: https://bugs.webkit.org/attachment.cgi?id=123078&action=review
This looks good. A few minor questions/nits below. Thanks!
> Source/WebCore/html/DOMURL.cpp:-28
> -#if ENABLE(BLOB)
I'm curious why you moved this ifdef around. It looks like almost everything is ifdefed anyway.
> Source/WebCore/html/DOMURL.cpp:59
> + virtual void contextDestroyed()
Please add the OVERRIDE keyword. We're just starting to use that in WebKit.
> Source/WebCore/html/DOMURL.h:41
> +class DOMURL : public RefCounted<DOMURL> {
It looks like DOMURL is now declared outside of ENABLE(BLOB). Is that what you intend?
--
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