aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAlexei Svitkine <asvitkine@chromium.org>2021-11-16 22:01:32 +0000
committerCopybara-Service <copybara-worker@google.com>2021-11-16 14:09:10 -0800
commit261f4df0801de80106dd28a47914131c9b897058 (patch)
treeb65f66e91da145f35a2d240a8ea1a41eb20c892d
parent8bb965d29e918d0559589a215ff7f4bd0874bc08 (diff)
downloadzucchini-261f4df0801de80106dd28a47914131c9b897058.tar.gz
Update callsites to use WIN_-prefixed base::File flags.
The flags have been renamed to make it clear that they only apply to Windows (see associated bug). If you're reviewing this CL, please verify your code's usage of the flag to ensure that the behavior is as intended. Thanks! This CL was uploaded by git cl split. R=huangs@chromium.org Bug: 1244149 Change-Id: I08d4f6e886a699d852470e08025edb85b0a695ab Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3279224 Auto-Submit: Alexei Svitkine <asvitkine@chromium.org> Reviewed-by: Samuel Huang <huangs@chromium.org> Commit-Queue: Samuel Huang <huangs@chromium.org> Cr-Commit-Position: refs/heads/main@{#942295} NOKEYCHECK=True GitOrigin-RevId: 9f7f7c8f781c5b7946dc9be3f80cc28cda9e971c
-rw-r--r--mapped_file_unittest.cc4
-rw-r--r--zucchini_commands.cc10
-rw-r--r--zucchini_integration.cc13
3 files changed, 14 insertions, 13 deletions
diff --git a/mapped_file_unittest.cc b/mapped_file_unittest.cc
index e3ee6dc..56014b0 100644
--- a/mapped_file_unittest.cc
+++ b/mapped_file_unittest.cc
@@ -33,7 +33,7 @@ TEST_F(MappedFileWriterTest, Keep) {
{
using base::File;
File file(file_path_, File::FLAG_CREATE_ALWAYS | File::FLAG_READ |
- File::FLAG_WRITE | File::FLAG_SHARE_DELETE |
+ File::FLAG_WRITE | File::FLAG_WIN_SHARE_DELETE |
File::FLAG_CAN_DELETE_ON_CLOSE);
MappedFileWriter file_writer(file_path_, std::move(file), 10);
EXPECT_FALSE(file_writer.HasError());
@@ -49,7 +49,7 @@ TEST_F(MappedFileWriterTest, DeleteOnClose) {
{
using base::File;
File file(file_path_, File::FLAG_CREATE_ALWAYS | File::FLAG_READ |
- File::FLAG_WRITE | File::FLAG_SHARE_DELETE |
+ File::FLAG_WRITE | File::FLAG_WIN_SHARE_DELETE |
File::FLAG_CAN_DELETE_ON_CLOSE);
MappedFileWriter file_writer(file_path_, std::move(file), 10);
EXPECT_FALSE(file_writer.HasError());
diff --git a/zucchini_commands.cc b/zucchini_commands.cc
index 0699cbe..225fc25 100644
--- a/zucchini_commands.cc
+++ b/zucchini_commands.cc
@@ -60,7 +60,7 @@ zucchini::status::Code MainRead(MainParams params) {
CHECK_EQ(1U, params.file_paths.size());
base::File input_file(params.file_paths[0],
base::File::FLAG_OPEN | base::File::FLAG_READ |
- base::File::FLAG_SHARE_DELETE);
+ base::File::FLAG_WIN_SHARE_DELETE);
zucchini::MappedFileReader input(std::move(input_file));
if (input.HasError()) {
LOG(ERROR) << "Error with file " << params.file_paths[0].value() << ": "
@@ -80,7 +80,7 @@ zucchini::status::Code MainDetect(MainParams params) {
CHECK_EQ(1U, params.file_paths.size());
base::File input_file(params.file_paths[0],
base::File::FLAG_OPEN | base::File::FLAG_READ |
- base::File::FLAG_SHARE_DELETE);
+ base::File::FLAG_WIN_SHARE_DELETE);
zucchini::MappedFileReader input(std::move(input_file));
if (input.HasError()) {
LOG(ERROR) << "Error with file " << params.file_paths[0].value() << ": "
@@ -100,7 +100,7 @@ zucchini::status::Code MainMatch(MainParams params) {
CHECK_EQ(2U, params.file_paths.size());
using base::File;
File old_file(params.file_paths[0], File::FLAG_OPEN | File::FLAG_READ |
- base::File::FLAG_SHARE_DELETE);
+ base::File::FLAG_WIN_SHARE_DELETE);
zucchini::MappedFileReader old_image(std::move(old_file));
if (old_image.HasError()) {
LOG(ERROR) << "Error with file " << params.file_paths[0].value() << ": "
@@ -108,7 +108,7 @@ zucchini::status::Code MainMatch(MainParams params) {
return zucchini::status::kStatusFileReadError;
}
File new_file(params.file_paths[1], File::FLAG_OPEN | File::FLAG_READ |
- base::File::FLAG_SHARE_DELETE);
+ base::File::FLAG_WIN_SHARE_DELETE);
zucchini::MappedFileReader new_image(std::move(new_file));
if (new_image.HasError()) {
LOG(ERROR) << "Error with file " << params.file_paths[1].value() << ": "
@@ -131,7 +131,7 @@ zucchini::status::Code MainCrc32(MainParams params) {
CHECK_EQ(1U, params.file_paths.size());
base::File image_file(params.file_paths[0],
base::File::FLAG_OPEN | base::File::FLAG_READ |
- base::File::FLAG_SHARE_DELETE);
+ base::File::FLAG_WIN_SHARE_DELETE);
zucchini::MappedFileReader image(std::move(image_file));
if (image.HasError()) {
LOG(ERROR) << "Error with file " << params.file_paths[0].value() << ": "
diff --git a/zucchini_integration.cc b/zucchini_integration.cc
index bf28b3c..c654a08 100644
--- a/zucchini_integration.cc
+++ b/zucchini_integration.cc
@@ -184,11 +184,12 @@ status::Code Generate(const base::FilePath& old_path,
std::string imposed_matches) {
using base::File;
File old_file(old_path, File::FLAG_OPEN | File::FLAG_READ |
- base::File::FLAG_SHARE_DELETE);
+ base::File::FLAG_WIN_SHARE_DELETE);
File new_file(new_path, File::FLAG_OPEN | File::FLAG_READ |
- base::File::FLAG_SHARE_DELETE);
+ base::File::FLAG_WIN_SHARE_DELETE);
File patch_file(patch_path, File::FLAG_CREATE_ALWAYS | File::FLAG_READ |
- File::FLAG_WRITE | File::FLAG_SHARE_DELETE |
+ File::FLAG_WRITE |
+ File::FLAG_WIN_SHARE_DELETE |
File::FLAG_CAN_DELETE_ON_CLOSE);
const FileNames file_names(old_path, new_path, patch_path);
return GenerateCommon(std::move(old_file), std::move(new_file),
@@ -211,11 +212,11 @@ status::Code Apply(const base::FilePath& old_path,
bool force_keep) {
using base::File;
File old_file(old_path, File::FLAG_OPEN | File::FLAG_READ |
- base::File::FLAG_SHARE_DELETE);
+ base::File::FLAG_WIN_SHARE_DELETE);
File patch_file(patch_path, File::FLAG_OPEN | File::FLAG_READ |
- base::File::FLAG_SHARE_DELETE);
+ base::File::FLAG_WIN_SHARE_DELETE);
File new_file(new_path, File::FLAG_CREATE_ALWAYS | File::FLAG_READ |
- File::FLAG_WRITE | File::FLAG_SHARE_DELETE |
+ File::FLAG_WRITE | File::FLAG_WIN_SHARE_DELETE |
File::FLAG_CAN_DELETE_ON_CLOSE);
const FileNames file_names(old_path, new_path, patch_path);
return ApplyCommon(std::move(old_file), std::move(patch_file),