Index: third_party/WebKit/Source/bindings/core/v8/custom/V8WindowCustom.cpp |
diff --git a/third_party/WebKit/Source/bindings/core/v8/custom/V8WindowCustom.cpp b/third_party/WebKit/Source/bindings/core/v8/custom/V8WindowCustom.cpp |
index 2f4c84c920020006ea7928839e18aaf1a9fdb334..327e8b0f5a33c5b393cedc95ade51ff03b41d81a 100644 |
--- a/third_party/WebKit/Source/bindings/core/v8/custom/V8WindowCustom.cpp |
+++ b/third_party/WebKit/Source/bindings/core/v8/custom/V8WindowCustom.cpp |
@@ -75,8 +75,7 @@ void V8Window::locationAttributeGetterCustom( |
Location* location = window->location(); |
DCHECK(location); |
- // Keep the wrapper object for the return value alive as long as |this| |
- // object is alive in order to save creation time of the wrapper object. |
+ // If we have already created a wrapper object in this world, returns it. |
if (DOMDataStore::SetReturnValue(info.GetReturnValue(), location)) |
return; |
@@ -101,17 +100,6 @@ void V8Window::locationAttributeGetterCustom( |
wrapper = ToV8(location, holder, isolate); |
} |
- // Keep the wrapper object for the return value alive as long as |this| |
- // object is alive in order to save creation time of the wrapper object. |
- // |
- // TODO(dcheng): The hidden reference behavior is broken in many ways. We |
- // should be caching for all DOM attributes. Even if it's not critical for |
- // remote Location objects, we should clean this up to improve |
- // maintainability. In the long-term, this will be superseded by wrapper |
- // tracing. |
- V8PrivateProperty::GetSymbol(isolate, "KeepAlive#Window#location") |
- .Set(holder, wrapper); |
- |
V8SetReturnValue(info, wrapper); |
} |