Don't check if SerializedScriptValue's Reader::m_version is less than 0

Source/bindings/v8/SerializedScriptValue.cpp

 /*
  * Copyright (C) 2010 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 1619 matching lines @@
 
     v8::Isolate* isolate() const { return m_isolate; }
 
     bool read(v8::Handle<v8::Value>* value, CompositeCreator& creator)
     {
         SerializationTag tag;
         if (!readTag(&tag))
             return false;
         switch (tag) {
         case ReferenceCountTag: {
-            if (m_version <= 0)
+            if (!m_version)
                 return false;
             uint32_t referenceTableSize;
             if (!doReadUint32(&referenceTableSize))
                 return false;
             // If this test fails, then the serializer and deserializer disagree about the assignment
             // of object reference IDs. On the deserialization side, this means there are too many or too few
             // calls to pushObjectReference.
             if (referenceTableSize != creator.objectReferenceCount())
                 return false;
             return true;
@@ skipping 120 matching lines @@
             uint32_t length;
             if (!doReadUint32(&numProperties))
                 return false;
             if (!doReadUint32(&length))
                 return false;
             if (!creator.completeDenseArray(numProperties, length, value))
                 return false;
             break;
         }
         case ArrayBufferViewTag: {
-            if (m_version <= 0)
+            if (!m_version)
                 return false;
             if (!readArrayBufferView(value, creator))
                 return false;
             creator.pushObjectReference(*value);
             break;
         }
         case ArrayBufferTag: {
-            if (m_version <= 0)
+            if (!m_version)
                 return false;
             if (!readArrayBuffer(value))
                 return false;
             creator.pushObjectReference(*value);
             break;
         }
         case GenerateFreshObjectTag: {
-            if (m_version <= 0)
+            if (!m_version)
                 return false;
             if (!creator.newObject())
                 return false;
             return true;
         }
         case GenerateFreshSparseArrayTag: {
-            if (m_version <= 0)
+            if (!m_version)
                 return false;
             uint32_t length;
             if (!doReadUint32(&length))
                 return false;
             if (!creator.newSparseArray(length))
                 return false;
             return true;
         }
         case GenerateFreshDenseArrayTag: {
-            if (m_version <= 0)
+            if (!m_version)
                 return false;
             uint32_t length;
             if (!doReadUint32(&length))
                 return false;
             if (!creator.newDenseArray(length))
                 return false;
             return true;
         }
         case MessagePortTag: {
-            if (m_version <= 0)
+            if (!m_version)
                 return false;
             uint32_t index;
             if (!doReadUint32(&index))
                 return false;
             if (!creator.tryGetTransferredMessagePort(index, value))
                 return false;
             break;
         }
         case ArrayBufferTransferTag: {
-            if (m_version <= 0)
+            if (!m_version)
                 return false;
             uint32_t index;
             if (!doReadUint32(&index))
                 return false;
             if (!creator.tryGetTransferredArrayBuffer(index, value))
                 return false;
             break;
         }
         case ObjectReferenceTag: {
-            if (m_version <= 0)
+            if (!m_version)
                 return false;
             uint32_t reference;
             if (!doReadUint32(&reference))
                 return false;
             if (!creator.tryGetObjectFromObjectReference(reference, value))
                 return false;
             break;
         }
         default:
             return false;
@@ skipping 1200 matching lines @@
     // If the allocated memory was not registered before, then this class is likely
     // used in a context other then Worker's onmessage environment and the presence of
     // current v8 context is not guaranteed. Avoid calling v8 then.
     if (m_externallyAllocatedMemory) {
         ASSERT(v8::Isolate::GetCurrent());
         v8::Isolate::GetCurrent()->AdjustAmountOfExternalAllocatedMemory(-m_externallyAllocatedMemory);
     }
 }
 
 } // namespace WebCore