Add support for loading pak files from arbitrary file regions.

base/files/file.cc

 // Copyright (c) 2011 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 "base/files/file.h"
 #include "base/files/file_path.h"
 
 namespace base {
 
+const File::Region File::Region::kWholeFile;
+
 File::Info::Info()
     : size(0),
       is_directory(false),
       is_symbolic_link(false) {
 }
 
 File::Info::~Info() {
 }
 
+File::Region::Region() : offset(0), size(0) {

[Primiano Tucci (use gerrit), 2014/07/23 23:17:38]

This is a private ctor and can be used only by kWholeFile above.
Essentially,  only kWholeFile will have a size = 0 (which means no RO
initializers for kWholeFile -> save 16 bytes of .rodata binary size -> just
increment the .bss counter without impact on binary), hence any comparison
(other == kWholeFile) would be true iff the other = kWholeFile (modulo hacking
the public fields as I mentioned in my comment in file.h).

[willchan no longer on Chromium, 2014/07/25 21:21:23]

Sounds great.

+}
+
+File::Region::Region(int64 offset, int64 size) : offset(offset), size(size) {
+  DCHECK_GE(offset, 0);
+  DCHECK_GT(size, 0);
+}
+
+bool File::Region::operator==(const File::Region& other) const {
+  return other.offset == offset && other.size == size;
+}
+
 File::File()
     : error_details_(FILE_ERROR_FAILED),
       created_(false),
       async_(false) {
 }
 
 #if !defined(OS_NACL)
 File::File(const FilePath& name, uint32 flags)
     : error_details_(FILE_OK),
       created_(false),
@@ skipping 89 matching lines @@
       return "FILE_ERROR_IO";
     case FILE_ERROR_MAX:
       break;
   }
 
   NOTREACHED();
   return "";
 }
 
 }  // namespace base