[NaCl SDK] nacl_io: Fix bug in Path::Range introduced in recent change.

native_client_sdk/src/tests/nacl_io_test/path_test.cc

 // Copyright (c) 2012 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 <fcntl.h>
 
 #include "gtest/gtest.h"
 
 #include "nacl_io/kernel_proxy.h"
 #include "nacl_io/path.h"
@@ skipping 52 matching lines @@
   EXPECT_TRUE(p.IsAbsolute());
   EXPECT_FALSE(p.IsRoot());
   EXPECT_EQ(2, p.Size());
   EXPECT_EQ("/", p.Part(0));
   EXPECT_EQ("foo", p.Part(1));
   EXPECT_EQ("foo", p.Basename());
   EXPECT_EQ("/foo", p.Join());
   EXPECT_EQ("/", p.Range(0, 1));
   EXPECT_EQ("foo", p.Range(1, 2));
   EXPECT_EQ("/foo", p.Range(0, 2));
+  EXPECT_EQ("", p.Range(2, 2));
   EXPECT_EQ("/", p.Parent().Join());
 }
 
 TEST(PathTest, TwoPart_Relative) {
   Path p("foo/bar");
   EXPECT_FALSE(p.IsAbsolute());
   EXPECT_FALSE(p.IsRoot());
   EXPECT_EQ(2, p.Size());
   EXPECT_EQ("foo", p.Part(0));
   EXPECT_EQ("bar", p.Part(1));
   EXPECT_EQ("bar", p.Basename());
   EXPECT_EQ("foo/bar", p.Join());
   EXPECT_EQ("foo", p.Range(0, 1));
   EXPECT_EQ("bar", p.Range(1, 2));
   EXPECT_EQ("foo/bar", p.Range(0, 2));
+  EXPECT_EQ("", p.Range(2, 2));
   EXPECT_EQ("foo", p.Parent().Join());
 }
 
 TEST(PathTest, MakeRelative) {
   EXPECT_EQ("", Path("/").MakeRelative().Join());
   EXPECT_EQ("foo/bar/baz", Path("/foo/bar/baz").MakeRelative().Join());
   EXPECT_EQ("foo/bar/baz", Path("foo/bar/baz").MakeRelative().Join());
 }
 
 TEST(PathTest, Normalize_EmptyComponent) {
@@ skipping 80 matching lines @@
   Path p("a/relative/path");
 
   EXPECT_EQ("a", p.Range(0, 1));
   EXPECT_EQ("a/relative", p.Range(0, 2));
   EXPECT_EQ("a/relative/path", p.Range(0, 3));
 
   EXPECT_EQ("relative", p.Range(1, 2));
   EXPECT_EQ("relative/path", p.Range(1, 3));
 
   EXPECT_EQ("path", p.Range(2, 3));
+
+  EXPECT_EQ("", p.Range(2, 100));
+  EXPECT_EQ("", p.Range(42, 67));
 }
 
 TEST(PathTest, Range_Absolute) {
   Path p("/an/absolute/path");
 
   EXPECT_EQ("/", p.Range(0, 1));
   EXPECT_EQ("/an", p.Range(0, 2));
   EXPECT_EQ("/an/absolute", p.Range(0, 3));
   EXPECT_EQ("/an/absolute/path", p.Range(0, 4));
 
   EXPECT_EQ("an", p.Range(1, 2));
   EXPECT_EQ("an/absolute", p.Range(1, 3));
   EXPECT_EQ("an/absolute/path", p.Range(1, 4));
 
   EXPECT_EQ("absolute", p.Range(2, 3));
   EXPECT_EQ("absolute/path", p.Range(2, 4));
 
   EXPECT_EQ("path", p.Range(3, 4));
+
+  EXPECT_EQ("", p.Range(2, 100));
+  EXPECT_EQ("", p.Range(42, 67));
 }
 
 TEST(PathTest, Assign) {
   Path p;
 
   p = "foo/bar";
   EXPECT_EQ("foo/bar", p.Join());
 
   // Should normalize.
   p = "/foo/../bar";
   EXPECT_EQ("/bar", p.Join());
 
   p = Path("hi/planet");
   EXPECT_EQ("hi/planet", p.Join());
 }
 
 TEST(PathTest, Equals) {
   EXPECT_TRUE(Path("/foo") == Path("/foo"));
   EXPECT_TRUE(Path("foo/../bar") == Path("bar"));
   EXPECT_TRUE(Path("one/path") != Path("another/path"));
 }