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

Side by Side Diff: webkit/plugins/npapi/plugin_instance.cc

Issue 16950028: Move file_util::Delete to the base namespace (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 5 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/plugins/npapi/plugin_instance.h" 5 #include "webkit/plugins/npapi/plugin_instance.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/message_loop.h" 9 #include "base/message_loop.h"
10 #include "base/strings/string_number_conversions.h" 10 #include "base/strings/string_number_conversions.h"
(...skipping 225 matching lines...) Expand 10 before | Expand all | Expand 10 after
236 // saved on a per-URL basis, and then only passed 236 // saved on a per-URL basis, and then only passed
237 // to new instances of the plugin at the same URL. 237 // to new instances of the plugin at the same URL.
238 // Sounds like a huge security risk. When we do support 238 // Sounds like a huge security risk. When we do support
239 // these, we should pass them back to the PluginLib 239 // these, we should pass them back to the PluginLib
240 // to be stored there. 240 // to be stored there.
241 DCHECK(savedData == 0); 241 DCHECK(savedData == 0);
242 } 242 }
243 243
244 for (unsigned int file_index = 0; file_index < files_created_.size(); 244 for (unsigned int file_index = 0; file_index < files_created_.size();
245 file_index++) { 245 file_index++) {
246 file_util::Delete(files_created_[file_index], false); 246 base::Delete(files_created_[file_index], false);
247 } 247 }
248 248
249 // Ensure that no timer callbacks are invoked after NPP_Destroy. 249 // Ensure that no timer callbacks are invoked after NPP_Destroy.
250 timers_.clear(); 250 timers_.clear();
251 } 251 }
252 252
253 NPError PluginInstance::NPP_SetWindow(NPWindow *window) { 253 NPError PluginInstance::NPP_SetWindow(NPWindow *window) {
254 DCHECK(npp_functions_ != 0); 254 DCHECK(npp_functions_ != 0);
255 DCHECK(npp_functions_->setwindow != 0); 255 DCHECK(npp_functions_->setwindow != 0);
256 256
(...skipping 423 matching lines...) Expand 10 before | Expand all | Expand 10 after
680 plugin_stream->UpdateUrl( 680 plugin_stream->UpdateUrl(
681 plugin_stream->pending_redirect_url().c_str()); 681 plugin_stream->pending_redirect_url().c_str());
682 } 682 }
683 break; 683 break;
684 } 684 }
685 } 685 }
686 } 686 }
687 687
688 } // namespace npapi 688 } // namespace npapi
689 } // namespace webkit 689 } // namespace webkit
OLDNEW
« no previous file with comments | « webkit/browser/fileapi/sandbox_origin_database.cc ('k') | webkit/plugins/npapi/plugin_stream_posix.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698