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

Side by Side Diff: mojo/services/view_manager/context_factory_impl.h

Issue 553213003: Avoid destroying surface before the parent surface stops referencing it. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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 #ifndef MOJO_SERVICES_VIEW_MANAGER_CONTEXT_FACTORY_IMPL_H_ 5 #ifndef MOJO_SERVICES_VIEW_MANAGER_CONTEXT_FACTORY_IMPL_H_
6 #define MOJO_SERVICES_VIEW_MANAGER_CONTEXT_FACTORY_IMPL_H_ 6 #define MOJO_SERVICES_VIEW_MANAGER_CONTEXT_FACTORY_IMPL_H_
7 7
8 #include "mojo/public/cpp/system/core.h" 8 #include "mojo/public/cpp/system/core.h"
9 #include "ui/compositor/compositor.h" 9 #include "ui/compositor/compositor.h"
10 10
(...skipping 20 matching lines...) Expand all
31 ui::Compositor* compositor, 31 ui::Compositor* compositor,
32 ui::Layer* layer) OVERRIDE; 32 ui::Layer* layer) OVERRIDE;
33 virtual void RemoveReflector(scoped_refptr<ui::Reflector> reflector) OVERRIDE; 33 virtual void RemoveReflector(scoped_refptr<ui::Reflector> reflector) OVERRIDE;
34 34
35 virtual scoped_refptr<cc::ContextProvider> 35 virtual scoped_refptr<cc::ContextProvider>
36 SharedMainThreadContextProvider() OVERRIDE; 36 SharedMainThreadContextProvider() OVERRIDE;
37 virtual void RemoveCompositor(ui::Compositor* compositor) OVERRIDE; 37 virtual void RemoveCompositor(ui::Compositor* compositor) OVERRIDE;
38 virtual bool DoesCreateTestContexts() OVERRIDE; 38 virtual bool DoesCreateTestContexts() OVERRIDE;
39 virtual cc::SharedBitmapManager* GetSharedBitmapManager() OVERRIDE; 39 virtual cc::SharedBitmapManager* GetSharedBitmapManager() OVERRIDE;
40 virtual base::MessageLoopProxy* GetCompositorMessageLoop() OVERRIDE; 40 virtual base::MessageLoopProxy* GetCompositorMessageLoop() OVERRIDE;
41 virtual scoped_ptr<cc::SurfaceIdAllocator> CreateSurfaceIdAllocator()
42 OVERRIDE;
41 43
42 private: 44 private:
43 scoped_refptr<webkit::gpu::ContextProviderInProcess> 45 scoped_refptr<webkit::gpu::ContextProviderInProcess>
44 shared_main_thread_contexts_; 46 shared_main_thread_contexts_;
45 47
46 ScopedMessagePipeHandle command_buffer_handle_; 48 ScopedMessagePipeHandle command_buffer_handle_;
49 uint32_t next_surface_id_namespace_;
47 50
48 // TODO(sky): for debugging, remove. 51 // TODO(sky): for debugging, remove.
49 bool did_create_; 52 bool did_create_;
50 53
51 DISALLOW_COPY_AND_ASSIGN(ContextFactoryImpl); 54 DISALLOW_COPY_AND_ASSIGN(ContextFactoryImpl);
52 }; 55 };
53 56
54 } // namespace service 57 } // namespace service
55 } // namespace mojo 58 } // namespace mojo
56 59
57 #endif // MOJO_SERVICES_VIEW_MANAGER_CONTEXT_FACTORY_IMPL_H_ 60 #endif // MOJO_SERVICES_VIEW_MANAGER_CONTEXT_FACTORY_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698