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

Side by Side Diff: cc/output/managed_memory_policy_unittest.cc

Issue 1490513005: Add support for ManagedMemoryPolicy (de)serialization (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@renderer-settings
Patch Set: Add missing return-statements for conversions Created 5 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
(Empty)
1 // Copyright 2015 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 "cc/output/managed_memory_policy.h"
6
7 #include "cc/proto/managed_memory_policy.pb.h"
8 #include "cc/proto/memory_allocation.pb.h"
9 #include "gpu/command_buffer/common/gpu_memory_allocation.h"
10 #include "testing/gtest/include/gtest/gtest.h"
11
12 namespace cc {
13 namespace {
14
15 void VerifySerializeAndDeserialize(
16 size_t bytes_limit_when_visible,
17 gpu::MemoryAllocation::PriorityCutoff priority_cutoff_when_visible,
18 size_t num_resources_limit) {
19 ManagedMemoryPolicy policy1(bytes_limit_when_visible,
20 gpu::MemoryAllocation::CUTOFF_ALLOW_REQUIRED_ONLY,
21 num_resources_limit);
22 proto::ManagedMemoryPolicy proto;
23 policy1.ToProtobuf(&proto);
24 ManagedMemoryPolicy policy2(1);
25 policy2.FromProtobuf(proto);
26 EXPECT_EQ(policy1, policy2);
27 }
28
29 TEST(ManagedMemoryPolicyTest, SerializeDeserialize) {
30 VerifySerializeAndDeserialize(
31 42, gpu::MemoryAllocation::CUTOFF_ALLOW_REQUIRED_ONLY, 24);
32 VerifySerializeAndDeserialize(0, gpu::MemoryAllocation::CUTOFF_ALLOW_NOTHING,
33 1);
34 VerifySerializeAndDeserialize(
35 1, gpu::MemoryAllocation::CUTOFF_ALLOW_NICE_TO_HAVE, 0);
36 VerifySerializeAndDeserialize(
37 1024, gpu::MemoryAllocation::CUTOFF_ALLOW_EVERYTHING, 4096);
38 }
39
40 } // namespace
41 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698