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

Side by Side Diff: components/filesystem/public/cpp/prefs/filesystem_json_pref_store.cc

Issue 2492283002: Mojo C++ bindings: switch components/filesystem mojom target to use STL types. (Closed)
Patch Set: 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 // 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 "components/filesystem/public/cpp/prefs/filesystem_json_pref_store.h" 5 #include "components/filesystem/public/cpp/prefs/filesystem_json_pref_store.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <algorithm> 9 #include <algorithm>
10 #include <utility> 10 #include <utility>
(...skipping 373 matching lines...) Expand 10 before | Expand all | Expand 10 after
384 void FilesystemJsonPrefStore::OnTempFileRenamed(mojom::FileError err) {} 384 void FilesystemJsonPrefStore::OnTempFileRenamed(mojom::FileError err) {}
385 385
386 void FilesystemJsonPrefStore::OnPreferencesReadStart() { 386 void FilesystemJsonPrefStore::OnPreferencesReadStart() {
387 directory_->ReadEntireFile( 387 directory_->ReadEntireFile(
388 path_, 388 path_,
389 Bind(&FilesystemJsonPrefStore::OnPreferencesFileRead, AsWeakPtr())); 389 Bind(&FilesystemJsonPrefStore::OnPreferencesFileRead, AsWeakPtr()));
390 } 390 }
391 391
392 void FilesystemJsonPrefStore::OnPreferencesFileRead( 392 void FilesystemJsonPrefStore::OnPreferencesFileRead(
393 mojom::FileError err, 393 mojom::FileError err,
394 mojo::Array<uint8_t> contents) { 394 const std::vector<uint8_t>& contents) {
395 std::unique_ptr<FilesystemJsonPrefStore::ReadResult> read_result( 395 std::unique_ptr<FilesystemJsonPrefStore::ReadResult> read_result(
396 new FilesystemJsonPrefStore::ReadResult); 396 new FilesystemJsonPrefStore::ReadResult);
397 // TODO(erg): Needs even better error handling. 397 // TODO(erg): Needs even better error handling.
398 switch (err) { 398 switch (err) {
399 case mojom::FileError::IN_USE: 399 case mojom::FileError::IN_USE:
400 case mojom::FileError::ACCESS_DENIED: 400 case mojom::FileError::ACCESS_DENIED:
401 case mojom::FileError::NOT_A_FILE: { 401 case mojom::FileError::NOT_A_FILE: {
402 read_only_ = true; 402 read_only_ = true;
403 break; 403 break;
404 } 404 }
(...skipping 11 matching lines...) Expand all
416 reinterpret_cast<char*>(&contents.front()), contents.size())); 416 reinterpret_cast<char*>(&contents.front()), contents.size()));
417 read_result->value = deserializer.Deserialize(&error_code, &error_msg); 417 read_result->value = deserializer.Deserialize(&error_code, &error_msg);
418 read_result->error = HandleReadErrors(read_result->value.get()); 418 read_result->error = HandleReadErrors(read_result->value.get());
419 } 419 }
420 } 420 }
421 421
422 OnFileRead(std::move(read_result)); 422 OnFileRead(std::move(read_result));
423 } 423 }
424 424
425 } // namespace filesystem 425 } // namespace filesystem
OLDNEW
« no previous file with comments | « components/filesystem/public/cpp/prefs/filesystem_json_pref_store.h ('k') | components/filesystem/public/interfaces/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698