[bindings] Pass NormalConversion to to[U]Int{8/16/32/64}(...) in v8_types.py and reduce conversion …

Source/bindings/core/v8/custom/V8BlobCustomHelpers.cpp

 /*
  * 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 80 matching lines @@
         }
         m_contentType = m_contentType.lower();
     }
 
     if (!m_hasFileProperties)
         return true;
 
     v8::Local<v8::Value> lastModified;
     TONATIVE_DEFAULT(bool, containsLastModified, DictionaryHelper::get(dictionary, "lastModified", lastModified), false);
     if (containsLastModified) {
-        TONATIVE_DEFAULT(long long, lastModifiedInt, toInt64(isolate, lastModified), false);
+        TONATIVE_DEFAULT(long long, lastModifiedInt, toInt64(isolate, lastModified, NormalConversion, exceptionState), false);

[haraken, 2015/03/17 17:19:03]

If toInt64 fails and exceptionState is set, should we probably return early
before executing TONATIVE_DEFAULT?

[Jens Widell, 2015/03/17 17:25:00]

Yeah, we should just have

  long long lastModifiedInt = toInt64(...);
  if (exceptionState.hadException())
    return false;

here.

         setLastModified(static_cast<double>(lastModifiedInt));
     } else {
         setDefaultLastModified();
     }
 
     return true;
 }
 
 bool processBlobParts(v8::Isolate* isolate, v8::Local<v8::Object> blobParts, bool normalizeLineEndingsToNative, BlobData& blobData)
 {
@@ skipping 21 matching lines @@
             TOSTRING_DEFAULT(V8StringResource<>, stringValue, item, false);
             blobData.appendText(stringValue, normalizeLineEndingsToNative);
         }
     }
     return true;
 }
 
 } // namespace V8BlobCustomHelpers
 
 } // namespace blink