Chromium Code Reviews| Index: tools/gn/label_unittest.cc |
| diff --git a/tools/gn/label_unittest.cc b/tools/gn/label_unittest.cc |
| index eb1e16c21cc78709ce58dee3533d1c05e1d0732b..ea3d2fa02befc140a42ad6c5da59760d89a15185 100644 |
| --- a/tools/gn/label_unittest.cc |
| +++ b/tools/gn/label_unittest.cc |
| @@ -30,10 +30,9 @@ TEST(Label, Resolve) { |
| { "//chrome/", "/:", false, "", "", "", "" }, |
| { "//chrome/", "blah", true, "//chrome/blah/", "blah", "//t/", "d" }, |
| { "//chrome/", "blah:bar", true, "//chrome/blah/", "bar", "//t/", "d" }, |
| - // No single-leading slash. |
| - { "//chrome/", "/chrome:bar", false, "", "", "", "" }, |
| - // No trailing slash. |
| - { "//chrome/", "/chrome/:bar", false, "", "", "", "" }, |
| + // Absolute paths are are allowed now. |
|
brettw
2015/01/10 00:35:11
I'd either delete this comment or just say "Absolu
Mostyn Bramley-Moore
2015/01/10 01:00:50
Done.
|
| + { "//chrome/", "/chrome:bar", true , "/chrome/", "bar", "//t/", "d" }, |
| + { "//chrome/", "/chrome/:bar", true, "/chrome/", "bar", "//t/", "d" }, |
| // Refers to root dir. |
| { "//chrome/", "//:bar", true, "//", "bar", "//t/", "d" }, |
| // Implicit directory |
| @@ -57,7 +56,7 @@ TEST(Label, Resolve) { |
| { "//chrome/", "//chrome:bar(()", false, "", "", "", "" }, |
| { "//chrome/", "(t:b)", false, "", "", "", "" }, |
| { "//chrome/", ":bar(//t/b)", true, "//chrome/", "bar", "//t/b/", "b" }, |
| - { "//chrome/", ":bar(/t/b)", false, "", "", "", "" }, |
| + { "//chrome/", ":bar(/t/b)", true, "//chrome/", "bar", "/t/b/", "b" }, |
| { "//chrome/", ":bar(t/b)", true, "//chrome/", "bar", "//chrome/t/b/", "b" }, |
| }; |