Fix lint bugs.

fpdfsdk/javascript/JS_Define.h

 // Copyright 2014 PDFium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
 
 #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"
 #include "fpdfsdk/javascript/JS_Value.h"
 #include "fpdfsdk/javascript/resource.h"
 #include "fpdfsdk/jsapi/include/fxjs_v8.h"
 
 struct JSConstSpec {
   const wchar_t* pName;
   double number;
@@ skipping 23 matching lines @@
   { const_name, 0, pValue, 1 }                           \
   ,
 
 #define END_JS_STATIC_CONST() \
   { 0, 0, 0, 0 }              \
   }                           \
   ;  // NOLINT
 
 #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 }                \
   }                          \
   ;  // NOLINT
 
 #define BEGIN_JS_STATIC_METHOD(js_class_name) \
   JSMethodSpec js_class_name::JS_Class_Methods[] = {
 #define JS_STATIC_METHOD_ENTRY(method_name)            \
   { JS_WIDESTRING(method_name), method_name##_static } \
@@ skipping 415 matching lines @@
     for (size_t i = 0; i < FX_ArraySize(global_methods) - 1; ++i) {          \
       FXJS_DefineGlobalMethod(pIsolate,                                      \
                               js_class_name::global_methods[i].pName,        \
                               js_class_name::global_methods[i].pMethodCall); \
     }                                                                        \
   }
 
 CJS_Value::Type GET_VALUE_TYPE(v8::Local<v8::Value> p);
 
 #endif  // FPDFSDK_JAVASCRIPT_JS_DEFINE_H_