Commit cdeb8039 authored by Noel Gordon's avatar Noel Gordon Committed by Commit Bot

DriveTestVolume: Remove extra CheckForUpdates calls

DriveTestVolume::CreateEntry calls CreateFile|Directory and then calls
CheckForUpdates. CreateFile|Directory also call CheckForUpdates: seems
redundant given that the caller does it for them.

Bug: 836254
Change-Id: Ia778a64d0195925a827c40dc6dfdf3877a6611d3
Reviewed-on: https://chromium-review.googlesource.com/1110077
Commit-Queue: Noel Gordon <noel@chromium.org>
Reviewed-by: default avatarStuart Langley <slangley@chromium.org>
Reviewed-by: default avatarSam McNally <sammc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#569958}
parent cc4fa88c
...@@ -580,8 +580,6 @@ class DriveTestVolume : public TestVolume { ...@@ -580,8 +580,6 @@ class DriveTestVolume : public TestVolume {
base::RunLoop().RunUntilIdle(); base::RunLoop().RunUntilIdle();
ASSERT_TRUE(error == google_apis::HTTP_SUCCESS); ASSERT_TRUE(error == google_apis::HTTP_SUCCESS);
ASSERT_TRUE(entry); ASSERT_TRUE(entry);
CheckForUpdates();
} }
// Creates a test file with the given spec. // Creates a test file with the given spec.
...@@ -623,8 +621,6 @@ class DriveTestVolume : public TestVolume { ...@@ -623,8 +621,6 @@ class DriveTestVolume : public TestVolume {
base::RunLoop().RunUntilIdle(); base::RunLoop().RunUntilIdle();
ASSERT_TRUE(error == google_apis::HTTP_SUCCESS); ASSERT_TRUE(error == google_apis::HTTP_SUCCESS);
ASSERT_TRUE(entry); ASSERT_TRUE(entry);
CheckForUpdates();
} }
// Notifies FileSystem that the contents in FakeDriveService have changed, // Notifies FileSystem that the contents in FakeDriveService have changed,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment