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

Side by Side Diff: third_party/WebKit/Source/modules/filesystem/LocalFileSystem.cpp

Issue 2458003002: Remove ASSERT_WITH_SECURITY_IMPLICATION. (Closed)
Patch Set: Minor formatting fix Created 4 years, 1 month 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
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2013 Google Inc. All rights reserved. 2 * Copyright (C) 2013 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 96 matching lines...) Expand 10 before | Expand all | Expand 10 after
107 WTF::bind(&LocalFileSystem::fileSystemNotAllowedInternal, 107 WTF::bind(&LocalFileSystem::fileSystemNotAllowedInternal,
108 wrapCrossThreadPersistent(this), wrapPersistent(context), 108 wrapCrossThreadPersistent(this), wrapPersistent(context),
109 wrapPersistent(wrapper))); 109 wrapPersistent(wrapper)));
110 } 110 }
111 111
112 void LocalFileSystem::deleteFileSystem( 112 void LocalFileSystem::deleteFileSystem(
113 ExecutionContext* context, 113 ExecutionContext* context,
114 FileSystemType type, 114 FileSystemType type,
115 std::unique_ptr<AsyncFileSystemCallbacks> callbacks) { 115 std::unique_ptr<AsyncFileSystemCallbacks> callbacks) {
116 ASSERT(context); 116 ASSERT(context);
117 ASSERT_WITH_SECURITY_IMPLICATION(context->isDocument()); 117 SECURITY_DCHECK(context->isDocument());
118 118
119 CallbackWrapper* wrapper = new CallbackWrapper(std::move(callbacks)); 119 CallbackWrapper* wrapper = new CallbackWrapper(std::move(callbacks));
120 requestFileSystemAccessInternal( 120 requestFileSystemAccessInternal(
121 context, 121 context,
122 WTF::bind(&LocalFileSystem::deleteFileSystemInternal, 122 WTF::bind(&LocalFileSystem::deleteFileSystemInternal,
123 wrapCrossThreadPersistent(this), wrapPersistent(context), type, 123 wrapCrossThreadPersistent(this), wrapPersistent(context), type,
124 wrapPersistent(wrapper)), 124 wrapPersistent(wrapper)),
125 WTF::bind(&LocalFileSystem::fileSystemNotAllowedInternal, 125 WTF::bind(&LocalFileSystem::fileSystemNotAllowedInternal,
126 wrapCrossThreadPersistent(this), wrapPersistent(context), 126 wrapCrossThreadPersistent(this), wrapPersistent(context),
127 wrapPersistent(wrapper))); 127 wrapPersistent(wrapper)));
(...skipping 114 matching lines...) Expand 10 before | Expand all | Expand 10 after
242 LocalFileSystem::create(std::move(client))); 242 LocalFileSystem::create(std::move(client)));
243 } 243 }
244 244
245 void provideLocalFileSystemToWorker(WorkerClients* clients, 245 void provideLocalFileSystemToWorker(WorkerClients* clients,
246 std::unique_ptr<FileSystemClient> client) { 246 std::unique_ptr<FileSystemClient> client) {
247 clients->provideSupplement(LocalFileSystem::supplementName(), 247 clients->provideSupplement(LocalFileSystem::supplementName(),
248 LocalFileSystem::create(std::move(client))); 248 LocalFileSystem::create(std::move(client)));
249 } 249 }
250 250
251 } // namespace blink 251 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698