Revert of IDL: Null values should be converted for non-nullable dictionary members

Source/bindings/tests/results/core/V8TestDictionaryDerived.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // This file has been auto-generated by code_generator_v8.py. DO NOT MODIFY!
 
 #include "config.h"
 #include "V8TestDictionaryDerived.h"
 
 #include "bindings/core/v8/ExceptionState.h"
@@ skipping 10 matching lines @@
         return;
     }
 
     V8TestDictionary::toImpl(isolate, v8Value, impl, exceptionState);
     if (exceptionState.hadException())
         return;
 
     v8::Local<v8::Object> v8Object = v8Value->ToObject(isolate);
     v8::TryCatch block;
     v8::Local<v8::Value> derivedStringMemberValue = v8Object->Get(v8String(isolate, "derivedStringMember"));
-    if (block.HasCaught()) {
+    if (!derivedStringMemberValue.IsEmpty() && !isUndefinedOrNull(derivedStringMemberValue)) {
+        TOSTRING_VOID(V8StringResource<>, derivedStringMember, derivedStringMemberValue);
+        impl.setDerivedStringMember(derivedStringMember);
+    } else if (block.HasCaught()) {
         exceptionState.rethrowV8Exception(block.Exception());
         return;
     }
-    if (derivedStringMemberValue.IsEmpty() || derivedStringMemberValue->IsUndefined()) {
-        // Do nothing.
-    } else {
-        TOSTRING_VOID(V8StringResource<>, derivedStringMember, derivedStringMemberValue);
-        impl.setDerivedStringMember(derivedStringMember);
-    }
 
     v8::Local<v8::Value> derivedStringMemberWithDefaultValue = v8Object->Get(v8String(isolate, "derivedStringMemberWithDefault"));
-    if (block.HasCaught()) {
+    if (!derivedStringMemberWithDefaultValue.IsEmpty() && !isUndefinedOrNull(derivedStringMemberWithDefaultValue)) {
+        TOSTRING_VOID(V8StringResource<>, derivedStringMemberWithDefault, derivedStringMemberWithDefaultValue);
+        impl.setDerivedStringMemberWithDefault(derivedStringMemberWithDefault);
+    } else if (block.HasCaught()) {
         exceptionState.rethrowV8Exception(block.Exception());
         return;
     }
-    if (derivedStringMemberWithDefaultValue.IsEmpty() || derivedStringMemberWithDefaultValue->IsUndefined()) {
-        // Do nothing.
-    } else {
-        TOSTRING_VOID(V8StringResource<>, derivedStringMemberWithDefault, derivedStringMemberWithDefaultValue);
-        impl.setDerivedStringMemberWithDefault(derivedStringMemberWithDefault);
-    }
 
 }
 
 v8::Handle<v8::Value> toV8(const TestDictionaryDerivedImplementedAs& impl, v8::Handle<v8::Object> creationContext, v8::Isolate* isolate)
 {
     v8::Handle<v8::Object> v8Object = v8::Object::New(isolate);
     toV8TestDictionary(impl, v8Object, creationContext, isolate);
     toV8TestDictionaryDerivedImplementedAs(impl, v8Object, creationContext, isolate);
     return v8Object;
 }
@@ skipping 19 matching lines @@
 }
 
 TestDictionaryDerivedImplementedAs NativeValueTraits<TestDictionaryDerivedImplementedAs>::nativeValue(const v8::Handle<v8::Value>& value, v8::Isolate* isolate, ExceptionState& exceptionState)
 {
     TestDictionaryDerivedImplementedAs impl;
     V8TestDictionaryDerivedImplementedAs::toImpl(isolate, value, impl, exceptionState);
     return impl;
 }
 
 } // namespace blink