Relax the EncryptMetadata check.

core/fpdfapi/parser/cpdf_security_handler_embeddertest.cpp

Index: core/fpdfapi/parser/cpdf_security_handler_embeddertest.cpp
diff --git a/core/fpdfapi/parser/cpdf_security_handler_embeddertest.cpp b/core/fpdfapi/parser/cpdf_security_handler_embeddertest.cpp
index 37b6d8fc33a1d34c31dc6394222e8bde75b154d1..120fb1be2a074cfa8dc50425e57d38f0fab8074e 100644
--- a/core/fpdfapi/parser/cpdf_security_handler_embeddertest.cpp
+++ b/core/fpdfapi/parser/cpdf_security_handler_embeddertest.cpp
@@ -21,6 +21,10 @@ TEST_F(CPDFSecurityHandlerEmbeddertest, NoPassword) {
   EXPECT_FALSE(OpenDocument("encrypted.pdf"));
 }
 
+TEST_F(CPDFSecurityHandlerEmbeddertest, BadPassword) {
+  EXPECT_FALSE(OpenDocument("encrypted.pdf", "tiger"));
+}
+
 TEST_F(CPDFSecurityHandlerEmbeddertest, UserPassword) {
   ASSERT_TRUE(OpenDocument("encrypted.pdf", "1234"));
   EXPECT_EQ(0xFFFFF2C0, FPDF_GetDocPermissions(document()));
@@ -30,3 +34,21 @@ TEST_F(CPDFSecurityHandlerEmbeddertest, OwnerPassword) {
   ASSERT_TRUE(OpenDocument("encrypted.pdf", "5678"));
   EXPECT_EQ(0xFFFFFFFC, FPDF_GetDocPermissions(document()));
 }
+
+TEST_F(CPDFSecurityHandlerEmbeddertest, NoPasswordVersion5) {
+  ASSERT_FALSE(OpenDocument("bug_644.pdf", "1234"));

[Wei Li, 2016/12/16 23:45:57]

Does no-password case test empty password? Otherwise this is exactly same as
BadPasswordVersion5()

[Tom Sepez, 2016/12/17 00:23:48]

Nice catch.

+}
+
+TEST_F(CPDFSecurityHandlerEmbeddertest, BadPasswordVersion5) {
+  ASSERT_FALSE(OpenDocument("bug_644.pdf", "1234"));
+}
+
+TEST_F(CPDFSecurityHandlerEmbeddertest, OnwerPasswordVersion5) {

[Wei Li, 2016/12/16 23:45:57]

s/Onwer/Owner

[Tom Sepez, 2016/12/17 00:24:35]

Done.

+  ASSERT_TRUE(OpenDocument("bug_644.pdf", "a"));
+  EXPECT_EQ(0xFFFFFFFC, FPDF_GetDocPermissions(document()));
+}
+
+TEST_F(CPDFSecurityHandlerEmbeddertest, UserPasswordVersion5) {
+  ASSERT_TRUE(OpenDocument("bug_644.pdf", "b"));
+  EXPECT_EQ(0xFFFFFFFC, FPDF_GetDocPermissions(document()));
+}