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

Side by Side Diff: webkit/fileapi/file_system_context.cc

Issue 10817006: Add more FS types and introduce type field into IsolatedContext (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: removed Media filesystem type for now Created 8 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "webkit/fileapi/file_system_context.h" 5 #include "webkit/fileapi/file_system_context.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/file_util.h" 8 #include "base/file_util.h"
9 #include "base/stl_util.h" 9 #include "base/stl_util.h"
10 #include "base/single_thread_task_runner.h" 10 #include "base/single_thread_task_runner.h"
(...skipping 105 matching lines...) Expand 10 before | Expand all | Expand 10 after
116 116
117 FileSystemMountPointProvider* FileSystemContext::GetMountPointProvider( 117 FileSystemMountPointProvider* FileSystemContext::GetMountPointProvider(
118 FileSystemType type) const { 118 FileSystemType type) const {
119 switch (type) { 119 switch (type) {
120 case kFileSystemTypeTemporary: 120 case kFileSystemTypeTemporary:
121 case kFileSystemTypePersistent: 121 case kFileSystemTypePersistent:
122 return sandbox_provider_.get(); 122 return sandbox_provider_.get();
123 case kFileSystemTypeExternal: 123 case kFileSystemTypeExternal:
124 return external_provider_.get(); 124 return external_provider_.get();
125 case kFileSystemTypeIsolated: 125 case kFileSystemTypeIsolated:
126 case kFileSystemTypeDragged:
tzik 2012/07/24 23:15:11 Could you move this just above kFileSystemTypeUnkn
kinuko 2012/07/25 00:05:45 In this patch true, though it may not hold forever
126 return isolated_provider_.get(); 127 return isolated_provider_.get();
127 default: 128 default:
128 if (provider_map_.find(type) != provider_map_.end()) 129 if (provider_map_.find(type) != provider_map_.end())
129 return provider_map_.find(type)->second; 130 return provider_map_.find(type)->second;
130 // Fall through. 131 // Fall through.
131 case kFileSystemTypeUnknown: 132 case kFileSystemTypeUnknown:
132 NOTREACHED(); 133 NOTREACHED();
133 return NULL; 134 return NULL;
134 } 135 }
135 } 136 }
(...skipping 67 matching lines...) Expand 10 before | Expand all | Expand 10 after
203 if (!io_task_runner_->RunsTasksOnCurrentThread() && 204 if (!io_task_runner_->RunsTasksOnCurrentThread() &&
204 io_task_runner_->DeleteSoon(FROM_HERE, this)) { 205 io_task_runner_->DeleteSoon(FROM_HERE, this)) {
205 return; 206 return;
206 } 207 }
207 STLDeleteContainerPairSecondPointers(provider_map_.begin(), 208 STLDeleteContainerPairSecondPointers(provider_map_.begin(),
208 provider_map_.end()); 209 provider_map_.end());
209 delete this; 210 delete this;
210 } 211 }
211 212
212 } // namespace fileapi 213 } // namespace fileapi
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698