[Webkit-unassigned] [Bug 27821] ApplicationCacheHost refactoring
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Jul 30 15:24:11 PDT 2009
https://bugs.webkit.org/show_bug.cgi?id=27821
--- Comment #8 from Michael Nordman <michaeln at google.com> 2009-07-30 15:24:11 PDT ---
> nit: It is encouraged to delete these lines that list functions in new
> files being added. The point, I gather, is that the function list is
> most interesting when existing files are being modified.
done
> nit: isEmpty() is all you need since it returns true if the string is
> either null or zero-length.
done, thnx
> nit: Can this just be a non-pointer member variable? It isn't lazily
> allocated.
done, involved 'forwarding' ApplicationCacheHost.hand DOMApplicationCache.h) to
the webkit layer, so the xcodeproject file changed too
> nit: you can probably get away with just forward declaring KURL, right?
done, thnx
> nit: Instead of creating a new enum type, perhaps it would have been better
> to just pass an AtomicString for the event name? What does the enum buy us
> over just using the event name directly?
leaving as is, its used as a 'c' array index too
> I don't think this change is correct or necessary. See page/ChromeClient.h.
done
--
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