Chromium Code Reviews| Index: Source/bindings/dart/DartDOMWrapper.h |
| diff --git a/Source/bindings/dart/DartDOMWrapper.h b/Source/bindings/dart/DartDOMWrapper.h |
| index b2da8f71b1e3a70098d8f205eef3b96f0ed67299..4e880c4e3987c684600de398eb0a4443ca1a337f 100644 |
| --- a/Source/bindings/dart/DartDOMWrapper.h |
| +++ b/Source/bindings/dart/DartDOMWrapper.h |
| @@ -65,9 +65,11 @@ public: |
| ASSERT(domData); |
| if (ScriptWrappable::wrapperCanBeStoredInObject(domObject)) { |
| Dart_WeakPersistentHandle wrapper = (Dart_WeakPersistentHandle)(ScriptWrappable::fromObject(domObject)->getDartWrapper(domData)); |
| - if (wrapper) |
| - return wrapper; |
| - // FIXMEDART: Assert absence from map when multi-isolate wrapper info is implemeted. |
| + |
| +#ifdef DEBUG |
|
vsm
2014/06/06 13:39:45
Isn't this redundant with the ASSERT?
rmacnak
2014/06/06 21:26:31
Leftover from copy-paste from asserts that needed
|
| + ASSERT(wrapper == Traits::MapTraits::domMap(domData)->get(domObject)); |
| +#endif |
| + return wrapper; |
| } |
| return Traits::MapTraits::domMap(domData)->get(domObject); |
| } |
| @@ -301,10 +303,13 @@ private: |
| Dart_WeakPersistentHandle currentWrapper = 0; |
| if (ScriptWrappable::wrapperCanBeStoredInObject(domObject)) { |
| currentWrapper = (Dart_WeakPersistentHandle)(ScriptWrappable::fromObject(domObject)->getDartWrapper(domData)); |
| - } |
| - if (!currentWrapper) { |
| +#ifdef DEBUG |
|
vsm
2014/06/06 13:39:46
ditto
|
| + ASSERT(currentWrapper == Traits::MapTraits::domMap(domData)->get(domObject)); |
| +#endif |
| + } else { |
| currentWrapper = Traits::MapTraits::domMap(domData)->get(domObject); |
| } |
| + |
| // This could be an old wrapper which has been replaced with a custom element. |
| if (currentWrapper != wrapper) { |
| #ifdef DEBUG |