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

Side by Side Diff: sky/compositor/surface_allocator.cc

Issue 807733002: Split surface id and simplify connecting to surfaces service (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 6 years 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 2014 The Chromium Authors. All rights reserved. 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 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 "sky/compositor/surface_allocator.h" 5 #include "sky/compositor/surface_allocator.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 8
9 namespace sky { 9 namespace sky {
10 10
11 SurfaceAllocator::SurfaceAllocator(uint32_t id_namespace) 11 SurfaceAllocator::SurfaceAllocator(uint32_t id_namespace)
12 : id_namespace_(id_namespace), next_id_(1) { 12 : id_namespace_(id_namespace), next_id_(1) {
13 DCHECK(id_namespace); 13 DCHECK(id_namespace);
14 } 14 }
15 15
16 SurfaceAllocator::~SurfaceAllocator() { 16 SurfaceAllocator::~SurfaceAllocator() {
17 } 17 }
18 18
19 uint64_t SurfaceAllocator::CreateSurfaceId() { 19 mojo::SurfaceIdPtr SurfaceAllocator::CreateSurfaceId() {
20 // Surface IDs are 64 integers. The high 32 bits are the namespace of the ID, 20 auto id = mojo::SurfaceId::New();
21 // which is assigned to us by the surfaces service. The lower 32 bits are ours 21 id->local = next_id_++;
22 // to allocate as we see fit. For simplicity, we just allocate them 22 id->id_namespace = id_namespace_;
23 // sequentially. In principle, we could run out, but at 60 Hz, it takes 23 return id.Pass();
24 // several years to run out.
25 return static_cast<uint64_t>(id_namespace_) << 32 | next_id_++;
26 } 24 }
27 25
28 } // namespace sky 26 } // namespace sky
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698