Support optional arguments in SendRequestNatives::StartRequest

content/public/renderer/v8_value_converter.h

Index: content/public/renderer/v8_value_converter.h
diff --git a/content/public/renderer/v8_value_converter.h b/content/public/renderer/v8_value_converter.h
index 6e785c143a070ea4c0d5e5a639380a128b178efc..9506c6703849285954d593c307111844f4c6b52b 100644
--- a/content/public/renderer/v8_value_converter.h
+++ b/content/public/renderer/v8_value_converter.h
@@ -28,6 +28,17 @@ class CONTENT_EXPORT V8ValueConverter {
 
   virtual ~V8ValueConverter() {}
 
+  // Use the following setters to support additional types other than the
+  // default ones.
+  virtual bool IsAllowIndefined() const = 0;

[asargent_no_longer_on_chrome, 2012/05/10 20:17:08]

Indefined -> Undefined

Also, "IsAllow" sounds very strange (in conversation, you'd say "is undefined
allowed?" and not "is allow undefined?"). My preference would be to name the
getter just "AllowUndefined". 

similar issue for for IsAllowDate and IsAllowRegexp.

[battre, 2012/05/11 12:16:28]

Done.

+  virtual void SetAllowUndefined(bool val) = 0;
+
+  virtual bool IsAllowDate() const = 0;
+  virtual void SetAllowDate(bool val) = 0;
+
+  virtual bool IsAllowRegexp() const = 0;
+  virtual void SetAllowRegexp(bool val) = 0;
+
   // Converts Value to v8::Value. Unsupported types are replaced with null.
   // If an array or object throws while setting a value, that property or item
   // is skipped, leaving a hole in the case of arrays.