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

Side by Side Diff: chrome/browser/chromeos/file_system_provider/operations/copy_entry.cc

Issue 547313002: [fsp] Use auto generated methods for generating dictionaries. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased. Created 6 years, 3 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
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "chrome/browser/chromeos/file_system_provider/operations/copy_entry.h" 5 #include "chrome/browser/chromeos/file_system_provider/operations/copy_entry.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "chrome/common/extensions/api/file_system_provider.h" 9 #include "chrome/common/extensions/api/file_system_provider.h"
10 #include "chrome/common/extensions/api/file_system_provider_internal.h" 10 #include "chrome/common/extensions/api/file_system_provider_internal.h"
(...skipping 10 matching lines...) Expand all
21 : Operation(event_router, file_system_info), 21 : Operation(event_router, file_system_info),
22 source_path_(source_path), 22 source_path_(source_path),
23 target_path_(target_path), 23 target_path_(target_path),
24 callback_(callback) { 24 callback_(callback) {
25 } 25 }
26 26
27 CopyEntry::~CopyEntry() { 27 CopyEntry::~CopyEntry() {
28 } 28 }
29 29
30 bool CopyEntry::Execute(int request_id) { 30 bool CopyEntry::Execute(int request_id) {
31 using extensions::api::file_system_provider::CopyEntryRequestedOptions;
32
31 if (!file_system_info_.writable()) 33 if (!file_system_info_.writable())
32 return false; 34 return false;
33 35
34 scoped_ptr<base::DictionaryValue> values(new base::DictionaryValue); 36 CopyEntryRequestedOptions options;
35 values->SetString("sourcePath", source_path_.AsUTF8Unsafe()); 37 options.file_system_id = file_system_info_.file_system_id();
36 values->SetString("targetPath", target_path_.AsUTF8Unsafe()); 38 options.request_id = request_id;
39 options.source_path = source_path_.AsUTF8Unsafe();
40 options.target_path = target_path_.AsUTF8Unsafe();
37 41
38 return SendEvent( 42 return SendEvent(
39 request_id, 43 request_id,
40 extensions::api::file_system_provider::OnCopyEntryRequested::kEventName, 44 extensions::api::file_system_provider::OnCopyEntryRequested::kEventName,
41 values.Pass()); 45 extensions::api::file_system_provider::OnCopyEntryRequested::Create(
46 options));
42 } 47 }
43 48
44 void CopyEntry::OnSuccess(int /* request_id */, 49 void CopyEntry::OnSuccess(int /* request_id */,
45 scoped_ptr<RequestValue> /* result */, 50 scoped_ptr<RequestValue> /* result */,
46 bool has_more) { 51 bool has_more) {
47 callback_.Run(base::File::FILE_OK); 52 callback_.Run(base::File::FILE_OK);
48 } 53 }
49 54
50 void CopyEntry::OnError(int /* request_id */, 55 void CopyEntry::OnError(int /* request_id */,
51 scoped_ptr<RequestValue> /* result */, 56 scoped_ptr<RequestValue> /* result */,
52 base::File::Error error) { 57 base::File::Error error) {
53 callback_.Run(error); 58 callback_.Run(error);
54 } 59 }
55 60
56 } // namespace operations 61 } // namespace operations
57 } // namespace file_system_provider 62 } // namespace file_system_provider
58 } // namespace chromeos 63 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698