Chromium Code Reviews| Index: fpdfsdk/fpdftext_embeddertest.cpp |
| diff --git a/fpdfsdk/fpdftext_embeddertest.cpp b/fpdfsdk/fpdftext_embeddertest.cpp |
| index a5215c20cc9276c7427e004cedba6b32a48c7138..f3e7526c20db3bb734e7175c63162d48247d4060 100644 |
| --- a/fpdfsdk/fpdftext_embeddertest.cpp |
| +++ b/fpdfsdk/fpdftext_embeddertest.cpp |
| @@ -276,10 +276,10 @@ TEST_F(FPDFTextEmbeddertest, WebLinks) { |
| // Page contains two HTTP-style URLs. |
| EXPECT_EQ(2, FPDFLink_CountWebLinks(pagelink)); |
| - // Only a terminating NUL required for bogus links. |
| - EXPECT_EQ(1, FPDFLink_GetURL(pagelink, 2, nullptr, 0)); |
| - EXPECT_EQ(1, FPDFLink_GetURL(pagelink, 1400, nullptr, 0)); |
| - EXPECT_EQ(1, FPDFLink_GetURL(pagelink, -1, nullptr, 0)); |
| + // Zero returned for bogus links. |
|
Lei Zhang
2016/04/20 21:36:49
Isn't this changing the public API behavior? Won't
|
| + EXPECT_EQ(0, FPDFLink_GetURL(pagelink, 2, nullptr, 0)); |
| + EXPECT_EQ(0, FPDFLink_GetURL(pagelink, 1400, nullptr, 0)); |
| + EXPECT_EQ(0, FPDFLink_GetURL(pagelink, -1, nullptr, 0)); |
| // Query the number of characters required for each link (incl NUL). |
| EXPECT_EQ(25, FPDFLink_GetURL(pagelink, 0, nullptr, 0)); |