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

Side by Side Diff: gpu/command_buffer/service/context_group.cc

Issue 301793003: During image destroy, delete textures only if we have a GL context. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: WIP - Avoid optional image manager creation/destroy in context group Created 6 years, 6 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "gpu/command_buffer/service/context_group.h" 5 #include "gpu/command_buffer/service/context_group.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <string> 8 #include <string>
9 9
10 #include "base/command_line.h" 10 #include "base/command_line.h"
(...skipping 18 matching lines...) Expand all
29 namespace gles2 { 29 namespace gles2 {
30 30
31 ContextGroup::ContextGroup( 31 ContextGroup::ContextGroup(
32 MailboxManager* mailbox_manager, 32 MailboxManager* mailbox_manager,
33 ImageManager* image_manager, 33 ImageManager* image_manager,
34 MemoryTracker* memory_tracker, 34 MemoryTracker* memory_tracker,
35 ShaderTranslatorCache* shader_translator_cache, 35 ShaderTranslatorCache* shader_translator_cache,
36 FeatureInfo* feature_info, 36 FeatureInfo* feature_info,
37 bool bind_generates_resource) 37 bool bind_generates_resource)
38 : mailbox_manager_(mailbox_manager ? mailbox_manager : new MailboxManager), 38 : mailbox_manager_(mailbox_manager ? mailbox_manager : new MailboxManager),
39 image_manager_(image_manager ? image_manager : new ImageManager), 39 image_manager_(image_manager),
40 memory_tracker_(memory_tracker), 40 memory_tracker_(memory_tracker),
41 shader_translator_cache_(shader_translator_cache), 41 shader_translator_cache_(shader_translator_cache),
42 enforce_gl_minimums_(CommandLine::ForCurrentProcess()->HasSwitch( 42 enforce_gl_minimums_(CommandLine::ForCurrentProcess()->HasSwitch(
43 switches::kEnforceGLMinimums)), 43 switches::kEnforceGLMinimums)),
44 bind_generates_resource_(bind_generates_resource), 44 bind_generates_resource_(bind_generates_resource),
45 max_vertex_attribs_(0u), 45 max_vertex_attribs_(0u),
46 max_texture_units_(0u), 46 max_texture_units_(0u),
47 max_texture_image_units_(0u), 47 max_texture_image_units_(0u),
48 max_vertex_texture_image_units_(0u), 48 max_vertex_texture_image_units_(0u),
49 max_fragment_uniform_vectors_(0u), 49 max_fragment_uniform_vectors_(0u),
(...skipping 337 matching lines...) Expand 10 before | Expand all | Expand 10 after
387 GLenum pname, GLint min_required, uint32* v) { 387 GLenum pname, GLint min_required, uint32* v) {
388 uint32 value = 0; 388 uint32 value = 0;
389 GetIntegerv(pname, &value); 389 GetIntegerv(pname, &value);
390 bool result = CheckGLFeatureU(min_required, &value); 390 bool result = CheckGLFeatureU(min_required, &value);
391 *v = value; 391 *v = value;
392 return result; 392 return result;
393 } 393 }
394 394
395 } // namespace gles2 395 } // namespace gles2
396 } // namespace gpu 396 } // namespace gpu
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698