Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4)

Issue 242113007: Revert 264780 "[fsp] Add FileSystemURLParser to the file system ..." (Closed)

Created:
6 years, 8 months ago by Ken Rockot(use gerrit already)
Modified:
6 years, 8 months ago
Reviewers:
mtomasz
CC:
chromium-reviews
Visibility:
Public.

Description

Revert 264780 "[fsp] Add FileSystemURLParser to the file system ..." Dependent on 264778 which broke Linux ChromeOS ASan. > [fsp] Add FileSystemURLParser to the file system provider. > > The FileSystemURLParser is a class which parses a cracked FileSystemURL to > associate it with the proper ProvidedFileSystem class handling this URL. > > This will be necessary to route operations from AsyncFileUtil to a proper > providing extension. > > Along the way, the Service class has been fixed. Now, the provided file systems > are gracefully unmounted in destructor. As a result, the external mount points > are cleared neatly. > > TEST=unit_tests: FileSystemProviderMountPathUtilTest* > BUG=248427 > > Review URL: https://codereview.chromium.org/237583015 TBR=mtomasz@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=264800

Patch Set 1 #

Messages

Total messages: 2 (0 generated)
Ken Rockot(use gerrit already)
6 years, 8 months ago (2014-04-18 15:59:47 UTC) #1
Ken Rockot(use gerrit already)
6 years, 8 months ago (2014-04-18 15:59:55 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r264800.

Powered by Google App Engine
This is Rietveld 408576698