Bugs in CJS_PublicMethods::ParseNumber()

fpdfsdk/src/javascript/JS_Value.h

Index: fpdfsdk/src/javascript/JS_Value.h
diff --git a/fpdfsdk/src/javascript/JS_Value.h b/fpdfsdk/src/javascript/JS_Value.h
index 8517b76dac29422b3833ef1adc3717ae5c7e9408..20a6e38b46965e9cac89264d676b634f285b5492 100644
--- a/fpdfsdk/src/javascript/JS_Value.h
+++ b/fpdfsdk/src/javascript/JS_Value.h
@@ -62,6 +62,11 @@ class CJS_Value {
   v8::Local<v8::Array> ToV8Array() const;
   v8::Local<v8::Value> ToV8Value() const;
 
+  // Replace the current |m_pValue| with a v8::Number if possible
+  // to make one from the current |m_pValue|, updating |m_eType|
+  // as appropriate to indicate the result.
+  void MaybeCoerceToNumber();

[jochen (gone - plz use gerrit), 2016/01/19 17:16:25]

is it ok if the number has values like +infinity or -0?

[Tom Sepez, 2016/01/19 22:19:16]

Probably.  Remember, there isn't really a spec for this.

+
   void operator=(int iValue);
   void operator=(bool bValue);
   void operator=(double);