commit d10db2d699241253ae9a0e84285648244ec1f498
author rvargas@chromium.org <rvargas@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Sat Feb 22 02:25:53 2014 +0900
committer Qijiang Fan <fqj@google.com> Thu Jun 04 16:07:56 2020 +0900
tree 9b32eb87dd27e5b9d8682225e25d3f502988dcca
parent d6f029272a624ba1d410ad4d5926f1b5b7868040

Base: Don't check for thread restrictions when closing a file doesn't do IO.

Also, clarify the File API a little more.

BUG=322664
R=thakis@chromium.org

Review URL: https://codereview.chromium.org/172673002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@252585 0039d316-1c4b-4281-b951-d872f2087c98


CrOS-Libchrome-Original-Commit: 48abb285fb4cd2c8fab6d01f26bb78700404a01f

base/files/file_posix.cc

diff --git a/base/files/file_posix.cc b/base/files/file_posix.cc
index a37de53..a62ad98 100644
--- a/base/files/file_posix.cc
+++ b/base/files/file_posix.cc
@@ -217,10 +217,10 @@
 }
 
 void File::Close() {
-  base::ThreadRestrictions::AssertIOAllowed();
   if (!IsValid())
     return;
 
+  base::ThreadRestrictions::AssertIOAllowed();
   if (!IGNORE_EINTR(close(file_)))
     file_ = kInvalidPlatformFileValue;
 }