[Webkit-unassigned] [Bug 116096] DOMFileSystemBase: merge BlackBerry implementation

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue May 14 11:18:32 PDT 2013


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





--- Comment #3 from Alberto Garcia <agarcia at igalia.com>  2013-05-14 11:16:59 PST ---
(In reply to comment #2)
> > +        KURL originURL(ParsedURLString, url.path());

> Is it really OK to parse just the path as a URL and claim it is a
> “parsed URL string”? What are you trying to accomplish here? How
> will originURL.path() be different from url.path()?

The URL is something like this:

filesystem:file:///temporary/foo

In this case originURL would be file:///temporary/foo
and originURL.path() would be /temporary/foo

> > Source/WebCore/Modules/filesystem/DOMFileSystemBase.cpp:90
> > +        if (path.isEmpty() || path[0] != '/')
> > +            return false;

> No need for the path.isEmpty() check here. path[0] will return 0 if
> the path is empty.

You're right, I'll correct this.

-- 
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