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

Side by Side Diff: Source/core/dom/DOMDataView.cpp

Issue 606653006: bindings: Adds DOMArrayBuffer, etc. as thin wrappers for ArrayBuffer, etc. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fixed Win GPU tests (DOMDataView). Created 6 years, 2 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 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "config.h"
6 #include "core/dom/DOMDataView.h"
7
8 #include "bindings/core/v8/DOMDataStore.h"
9 #include "bindings/core/v8/V8ArrayBuffer.h"
10 #include "bindings/core/v8/V8DOMWrapper.h"
11
12 namespace blink {
13
14 PassRefPtr<DOMDataView> DOMDataView::create(PassRefPtr<DOMArrayBuffer> buffer, u nsigned byteOffset, unsigned byteLength)
haraken 2014/10/14 15:11:19 buffer => prpBuffer (We normally use prpXXX)
Yuki 2014/10/15 09:35:24 Done.
15 {
16 // Don't use |buffer| directly here to avoid passing PassRefPtr to another
17 // PassRefPtr, which invalidates the original PassRefPtr.
18 // See PassRefPtr::PassRefPtr(const PassRefPtr&).
19 RefPtr<DOMArrayBuffer> buf = buffer;
haraken 2014/10/14 15:11:19 buf => buffer
Yuki 2014/10/15 09:35:24 Done.
20 return adoptRef(new DOMDataView(DataView::create(buf->buffer(), byteOffset, byteLength), buf));
21 }
22
23 v8::Handle<v8::Object> DOMDataView::wrap(v8::Handle<v8::Object> creationContext, v8::Isolate* isolate)
24 {
25 // It's possible that no one except for the new wrapper owns this object at
26 // this moment, so we have to prevent GC to collect this object until the
27 // object gets associated with the wrapper.
28 RefPtrWillBeRawPtr<DOMDataView> protect(this);
haraken 2014/10/14 15:11:19 DOMDataView is not yet on Oilpan's heap, so this s
Yuki 2014/10/15 09:35:24 Done.
29
30 ASSERT(!DOMDataStore::containsWrapperNonTemplate(this, isolate));
31
32 const WrapperTypeInfo* wrapperTypeInfo = this->wrapperTypeInfo();
33 RefPtr<DOMArrayBuffer> buffer = this->buffer();
34 v8::Local<v8::Value> v8Buffer = toV8(buffer.get(), creationContext, isolate) ;
haraken 2014/10/14 15:11:19 buffer.get() => buffer.release()
Yuki 2014/10/15 09:35:24 Done.
35 ASSERT(v8Buffer->IsArrayBuffer());
36
37 v8::Handle<v8::Object> wrapper = v8::DataView::New(v8Buffer.As<v8::ArrayBuff er>(), byteOffset(), byteLength());
38
39 wrapperTypeInfo->installConditionallyEnabledProperties(wrapper, isolate);
40 return associateWithWrapper(wrapperTypeInfo, wrapper, isolate);
41 }
42
43 v8::Handle<v8::Object> DOMDataView::associateWithWrapper(const WrapperTypeInfo* wrapperTypeInfo, v8::Handle<v8::Object> wrapper, v8::Isolate* isolate)
44 {
45 return V8DOMWrapper::associateObjectWithWrapperNonTemplate(this, wrapperType Info, wrapper, isolate);
46 }
47
48 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698