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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: base/files/file_win.cc

Issue 323683002: net: FileStream cleanup (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 6 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
« no previous file with comments | « base/files/file_unittest.cc ('k') | chrome/browser/chromeos/drive/local_file_reader.cc » ('j') | chrome/browser/extensions/api/messaging/native_message_process_host.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/files/file_win.cc
diff --git a/base/files/file_win.cc b/base/files/file_win.cc
index 1e13381e255f00be678f0c224b933b729bd47ab8..d037989c2d7777ef7fe6d448b7547e63cdba3a49 100644
--- a/base/files/file_win.cc
+++ b/base/files/file_win.cc
@@ -13,6 +13,11 @@
namespace base {
+// Make sure our Whence mappings match the system headers.
+COMPILE_ASSERT(File::FROM_BEGIN == FILE_BEGIN &&
+ File::FROM_CURRENT == FILE_CURRENT &&
+ File::FROM_END == FILE_END, whence_matches_system);
+
void File::InitializeUnsafe(const FilePath& name, uint32 flags) {
base::ThreadRestrictions::AssertIOAllowed();
DCHECK(!IsValid());
@@ -118,8 +123,6 @@ void File::Close() {
int64 File::Seek(Whence whence, int64 offset) {
base::ThreadRestrictions::AssertIOAllowed();
DCHECK(IsValid());
- if (offset < 0)
- return -1;
LARGE_INTEGER distance, res;
distance.QuadPart = offset;
« no previous file with comments | « base/files/file_unittest.cc ('k') | chrome/browser/chromeos/drive/local_file_reader.cc » ('j') | chrome/browser/extensions/api/messaging/native_message_process_host.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698