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

Side by Side Diff: cc/test/test_context_provider.cc

Issue 1317743002: cc: Implement shared worker contexts. (v1) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: DetachFromThread after SetupLock Created 5 years, 3 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
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "cc/test/test_context_provider.h" 5 #include "cc/test/test_context_provider.h"
6 6
7 #include <set> 7 #include <set>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/bind.h" 10 #include "base/bind.h"
11 #include "base/callback_helpers.h" 11 #include "base/callback_helpers.h"
12 #include "base/logging.h" 12 #include "base/logging.h"
13 #include "cc/test/test_gles2_interface.h" 13 #include "cc/test/test_gles2_interface.h"
14 #include "cc/test/test_web_graphics_context_3d.h" 14 #include "cc/test/test_web_graphics_context_3d.h"
15 #include "third_party/skia/include/gpu/GrContext.h" 15 #include "third_party/skia/include/gpu/GrContext.h"
16 #include "third_party/skia/include/gpu/gl/SkNullGLContext.h" 16 #include "third_party/skia/include/gpu/gl/SkNullGLContext.h"
17 17
18 namespace cc { 18 namespace cc {
19 19
20 // static 20 // static
21 scoped_refptr<TestContextProvider> TestContextProvider::Create() { 21 scoped_refptr<TestContextProvider> TestContextProvider::Create() {
22 return Create(TestWebGraphicsContext3D::Create().Pass()); 22 return Create(TestWebGraphicsContext3D::Create().Pass());
23 } 23 }
24 24
25 // static 25 // static
26 scoped_refptr<TestContextProvider> TestContextProvider::CreateWorker() {
27 scoped_refptr<TestContextProvider> worker_context_provider =
28 Create(TestWebGraphicsContext3D::Create().Pass());
danakj 2015/08/26 18:48:50 you don't need .Pass() for a return value
reveman 2015/08/26 22:02:10 Done.
29 if (!worker_context_provider)
30 return nullptr;
31 worker_context_provider->bound_ = true;
danakj 2015/08/26 18:48:50 Can you explain this with a comment? Prod code in
reveman 2015/08/26 22:02:10 Done.
32 return worker_context_provider;
33 }
34
35 // static
26 scoped_refptr<TestContextProvider> TestContextProvider::Create( 36 scoped_refptr<TestContextProvider> TestContextProvider::Create(
27 scoped_ptr<TestWebGraphicsContext3D> context) { 37 scoped_ptr<TestWebGraphicsContext3D> context) {
28 if (!context) 38 if (!context)
29 return NULL; 39 return NULL;
30 return new TestContextProvider(context.Pass()); 40 return new TestContextProvider(context.Pass());
31 } 41 }
32 42
33 TestContextProvider::TestContextProvider( 43 TestContextProvider::TestContextProvider(
34 scoped_ptr<TestWebGraphicsContext3D> context) 44 scoped_ptr<TestWebGraphicsContext3D> context)
35 : context3d_(context.Pass()), 45 : context3d_(context.Pass()),
(...skipping 156 matching lines...) Expand 10 before | Expand all | Expand 10 after
192 DCHECK(memory_policy_changed_callback_.is_null() || cb.is_null()); 202 DCHECK(memory_policy_changed_callback_.is_null() || cb.is_null());
193 memory_policy_changed_callback_ = cb; 203 memory_policy_changed_callback_ = cb;
194 } 204 }
195 205
196 void TestContextProvider::SetMaxTransferBufferUsageBytes( 206 void TestContextProvider::SetMaxTransferBufferUsageBytes(
197 size_t max_transfer_buffer_usage_bytes) { 207 size_t max_transfer_buffer_usage_bytes) {
198 context3d_->SetMaxTransferBufferUsageBytes(max_transfer_buffer_usage_bytes); 208 context3d_->SetMaxTransferBufferUsageBytes(max_transfer_buffer_usage_bytes);
199 } 209 }
200 210
201 } // namespace cc 211 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698