Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(472)

Side by Side Diff: Source/bindings/v8/custom/V8TreeWalkerCustom.cpp

Issue 21274004: Fix Document leak from NodeFilter. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: add assertions to wrap Created 7 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
(Empty)
1 #include "config.h"
haraken 2013/08/01 13:32:20 Please add a copyright.
kouhei (in TOK) 2013/08/02 03:26:57 Done.
2 #include "V8TreeWalker.h"
3
4 #include "V8NodeFilter.h"
5 #include "bindings/v8/V8Binding.h"
6 #include "bindings/v8/V8HiddenPropertyName.h"
7 #include "bindings/v8/V8Utilities.h"
8 #include "core/dom/Node.h"
9 #include "core/dom/TreeWalker.h"
10
11 #include "wtf/RefPtr.h"
12
13 namespace WebCore {
14
15 v8::Handle<v8::Object> wrap(TreeWalker* treeWalker, v8::Handle<v8::Object> creat ionContext, v8::Isolate* isolate)
16 {
17 ASSERT(treeWalker);
18 ASSERT(DOMDataStore::getWrapper<V8TreeWalker>(treeWalker, isolate).IsEmpty() );
19
20 v8::Handle<v8::Object> wrapper = V8TreeWalker::createWrapper(treeWalker, cre ationContext, isolate);
21
22 if (treeWalker->filter())
23 V8HiddenPropertyName::setNamedHiddenReference(wrapper, "filter", toV8(tr eeWalker->filter(), creationContext, isolate));
24
25 return wrapper;
26 }
27
28 } // namespace WebCore
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698