Title: [132489] trunk/Source/WebCore
- Revision
- 132489
- Author
- [email protected]
- Date
- 2012-10-25 08:55:22 -0700 (Thu, 25 Oct 2012)
Log Message
[chromium] External FileSystem should use the root path returned by requestFileSystem
https://bugs.webkit.org/show_bug.cgi?id=100372
Reviewed by Kentaro Hara.
External FileSystem should use the root path returned by
requestFileSystem as other type is doing. Current implementation
completely ignore the returned path.
New tests are to be added in chromium (as the type is only available in chromeos and chrome extensions)
* Modules/filesystem/chromium/DOMFileSystemChromium.cpp:
(WebCore::DOMFileSystemBase::createFileSystemURL):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (132488 => 132489)
--- trunk/Source/WebCore/ChangeLog 2012-10-25 15:40:49 UTC (rev 132488)
+++ trunk/Source/WebCore/ChangeLog 2012-10-25 15:55:22 UTC (rev 132489)
@@ -1,3 +1,19 @@
+2012-10-25 Kinuko Yasuda <[email protected]>
+
+ [chromium] External FileSystem should use the root path returned by requestFileSystem
+ https://bugs.webkit.org/show_bug.cgi?id=100372
+
+ Reviewed by Kentaro Hara.
+
+ External FileSystem should use the root path returned by
+ requestFileSystem as other type is doing. Current implementation
+ completely ignore the returned path.
+
+ New tests are to be added in chromium (as the type is only available in chromeos and chrome extensions)
+
+ * Modules/filesystem/chromium/DOMFileSystemChromium.cpp:
+ (WebCore::DOMFileSystemBase::createFileSystemURL):
+
2012-10-25 Kevin Ellis <[email protected]>
Can't easily position the cursor on an empty line in a textarea with touch if touch adjustment is enabled
Modified: trunk/Source/WebCore/Modules/filesystem/chromium/DOMFileSystemChromium.cpp (132488 => 132489)
--- trunk/Source/WebCore/Modules/filesystem/chromium/DOMFileSystemChromium.cpp 2012-10-25 15:40:49 UTC (rev 132488)
+++ trunk/Source/WebCore/Modules/filesystem/chromium/DOMFileSystemChromium.cpp 2012-10-25 15:55:22 UTC (rev 132489)
@@ -90,7 +90,9 @@
result.append(securityOrigin()->toString());
result.append("/");
result.append(externalPathPrefix);
- result.append(encodeWithURLEscapeSequences(fullPath));
+ result.append(m_filesystemRootURL.path());
+ // Remove the extra leading slash.
+ result.append(encodeWithURLEscapeSequences(fullPath.substring(1)));
return KURL(ParsedURLString, result.toString());
}
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes
