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

Unified Diff: chrome/browser/media_galleries/linux/snapshot_file_details.cc

Issue 2358493002: Remove MTP support on Linux. (Closed)
Patch Set: move files Created 4 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/media_galleries/linux/snapshot_file_details.cc
diff --git a/chrome/browser/media_galleries/linux/snapshot_file_details.cc b/chrome/browser/media_galleries/linux/snapshot_file_details.cc
deleted file mode 100644
index a596389a99e874e41615d6f3cbdab968a7260c27..0000000000000000000000000000000000000000
--- a/chrome/browser/media_galleries/linux/snapshot_file_details.cc
+++ /dev/null
@@ -1,72 +0,0 @@
-// Copyright 2013 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "chrome/browser/media_galleries/linux/snapshot_file_details.h"
-
-#include <limits>
-
-#include "base/numerics/safe_conversions.h"
-
-////////////////////////////////////////////////////////////////////////////////
-// SnapshotRequestInfo //
-////////////////////////////////////////////////////////////////////////////////
-
-SnapshotRequestInfo::SnapshotRequestInfo(
- uint32_t file_id,
- const base::FilePath& snapshot_file_path,
- const MTPDeviceAsyncDelegate::CreateSnapshotFileSuccessCallback&
- success_callback,
- const MTPDeviceAsyncDelegate::ErrorCallback& error_callback)
- : file_id(file_id),
- snapshot_file_path(snapshot_file_path),
- success_callback(success_callback),
- error_callback(error_callback) {}
-
-SnapshotRequestInfo::SnapshotRequestInfo(const SnapshotRequestInfo& other) =
- default;
-
-SnapshotRequestInfo::~SnapshotRequestInfo() {
-}
-
-////////////////////////////////////////////////////////////////////////////////
-// SnapshotFileDetails //
-////////////////////////////////////////////////////////////////////////////////
-
-SnapshotFileDetails::SnapshotFileDetails(
- const SnapshotRequestInfo& request_info,
- const base::File::Info& file_info)
- : request_info_(request_info),
- file_info_(file_info),
- bytes_written_(0),
- error_occurred_(false) {
-}
-
-SnapshotFileDetails::~SnapshotFileDetails() {
-}
-
-void SnapshotFileDetails::set_error_occurred(bool error) {
- error_occurred_ = error;
-}
-
-bool SnapshotFileDetails::AddBytesWritten(uint32_t bytes_written) {
- if ((bytes_written == 0) ||
- (bytes_written_ > std::numeric_limits<uint32_t>::max() - bytes_written) ||
- (bytes_written_ + bytes_written > file_info_.size))
- return false;
-
- bytes_written_ += bytes_written;
- return true;
-}
-
-bool SnapshotFileDetails::IsSnapshotFileWriteComplete() const {
- return !error_occurred_ && (bytes_written_ == file_info_.size);
-}
-
-uint32_t SnapshotFileDetails::BytesToRead() const {
- // Read data in 1MB chunks.
- static const uint32_t kReadChunkSize = 1024 * 1024;
- return std::min(
- kReadChunkSize,
- base::checked_cast<uint32_t>(file_info_.size) - bytes_written_);
-}

Powered by Google App Engine
This is Rietveld 408576698