[Webkit-unassigned] [Bug 55644] [fileapi] Add URI resolution support to WorkerContext

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=55644





--- Comment #6 from Kinuko Yasuda <kinuko at chromium.org>  2011-03-03 12:24:13 PST ---
(In reply to comment #5)
> Created an attachment (id=84604)
 --> (https://bugs.webkit.org/attachment.cgi?id=84604&action=review) [details]
> Patch

LGTM.

(In reply to comment #4)
> > The implementation of this method is mostly same as that of requestFileSystem-- maybe we could factor out a common private method that takes all of parameters plus boolean create flag and make both readFileSystem and requestFileSystem a wrapper of the method.
> 
> Good idea.  I've done that.  The only thing that I don't like about all this is the fact that there's two bools and a long long all together in one signature.  Quite easy to screw up the order and break stuff.  I'd like to come back to this code and redo the API with enums instead.

Agreed, the signature looks a bit messed up after repeated ad-hoc changes.  Anyway the two public methods look nicer now, thanks for doing this.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.