Fix lint bugs.

fpdfsdk/javascript/JS_Define.h

Index: fpdfsdk/javascript/JS_Define.h
diff --git a/fpdfsdk/javascript/JS_Define.h b/fpdfsdk/javascript/JS_Define.h
index c7872599bf8f54baa815dae3792e0548043769b7..aaae98744acd43e1171ae583bcf1dcc14d9e0a40 100644
--- a/fpdfsdk/javascript/JS_Define.h
+++ b/fpdfsdk/javascript/JS_Define.h
@@ -7,6 +7,7 @@
 #ifndef FPDFSDK_JAVASCRIPT_JS_DEFINE_H_
 #define FPDFSDK_JAVASCRIPT_JS_DEFINE_H_
 
+#include <string>

[Tom Sepez, 2016/04/29 22:29:25]

Shouldn't be necessary, as there isn't any std::string in here.  Lint gets
confused when there is an argument/member named "string", a better fix might be
to rename these to something more descriptive to avoid the lint bug. It would be
a shame to drag std::string into these compilation units since its usage is rare
in pdfium.

[Lei Zhang, 2016/04/30 01:22:49]

Done.

 #include <vector>
 
 #include "fpdfsdk/javascript/JS_Object.h"
@@ -50,12 +51,9 @@ struct JSMethodSpec {
 
 #define BEGIN_JS_STATIC_PROP(js_class_name) \
   JSPropertySpec js_class_name::JS_Class_Properties[] = {
-#define JS_STATIC_PROP_ENTRY(prop_name)                                                    \
-  {                                                                                        \
-    JS_WIDESTRING(prop_name), get_##prop_name##_static, \
-        set_##prop_name##_static \
-  }                                                                                        \
-  ,
+#define JS_STATIC_PROP_ENTRY(prop_name)                \
+  {JS_WIDESTRING(prop_name), get_##prop_name##_static, \
+   set_##prop_name##_static},  // NOLINT
 
 #define END_JS_STATIC_PROP() \
   { 0, 0, 0 }                \