File constructor understands lastModified.

Source/bindings/v8/custom/V8BlobCustomHelpers.h

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 22 matching lines @@
 
 #include "bindings/v8/V8Binding.h"
 
 namespace WebCore {
 
 class BlobBuilder;
 
 // Shared code between the custom constructor bindings for Blob and File.
 namespace V8BlobCustomHelpers {
 
-// Extracts the "type" and "endings" properties out of the BlobPropertyBag passed to a Blob constructor.
-// http://www.w3.org/TR/FileAPI/#constructorParams
-// Returns true if everything went well, false if a JS exception was thrown.
-bool processBlobPropertyBag(v8::Local<v8::Value> propertyBag, const char* blobClassName, String& contentType, String& endings, v8::Isolate*);
+// Parsed properties from a BlobPropertyBag or a FilePropertyBag.
+//
+// Instances are stack-allocated by the File and Blob constructor.
+//
+// processBlobPropertyBag is only called when constructors receive a value for

[kinuko, 2013/11/20 08:01:22]

nit: comment looks stale

[pwnall-personal, 2013/11/20 09:51:07]

It's still true.

lastModified is a bit of a snowflake, but all the other values come straight
from the constructor if no property bag is passed.

Should I add comments around the methods that deal with lastModified?

[kinuko, 2013/11/20 10:13:39]

I mean, we no longer seem to have the method processBlobPropertyBag.

[pwnall-personal, 2013/11/20 10:40:31]

Done.
Ah, that is a very good point! I updated the name.

+// the optional BlobPropertyBag / FilePropertyBag argument. The defaults set in
+// the constructor are used otherwise.
+class ParsedProperties {
+public:
+    String contentType;
+    String endings;

[kinuko, 2013/11/20 08:01:22]

I think we prefer having accessors for C++ class (if it's not for structs)

[pwnall-personal, 2013/11/20 09:51:07]

Done.
Thank you!

 
-// Appends the blobParts passed to a Blob constructor into a BlobBuilder.
+    // False if this contains the properties of a BlobPropertyBag.
+    bool hasFileProperties;
+
+    void setLastModified(double);
+    void setDefaultLastModified();
+    double lastModified();
+
+    explicit ParsedProperties(bool hasFileProperties);

[kinuko, 2013/11/20 08:01:22]

nit: usually we put ctor at the top

[pwnall-personal, 2013/11/20 09:51:07]

Done.
Thank you for your patience in teaching me!

+
+    // Extracts the "type" and "endings" properties out of the BlobPropertyBag passed to a Blob constructor.
+    // http://www.w3.org/TR/FileAPI/#constructorParams
+    // Returns true if everything went well, false if a JS exception was thrown.
+    bool parseBlobPropertyBag(v8::Local<v8::Value> propertyBag, const char* blobClassName, v8::Isolate*);
+
+private:
+    double m_lastModified;
+#ifndef NDEBUG
+    bool m_hasLastModified;
+#endif // NDEBUG
+};
+
+// Appends the blobParts passed to a Blob or File constructor into a BlobBuilder.
 // http://www.w3.org/TR/FileAPI/#constructorParams
 // Returns true if everything went well, false if a JS exception was thrown.
 bool processBlobParts(v8::Local<v8::Object> blobParts, uint32_t blobPartsLength, const String& endings, BlobBuilder&, v8::Isolate*);
 
 } // namespace V8BlobCustomHelpers
 
 } // namespace WebCore
 
 #endif // V8BlobCustomHelpers_h