Hash :
d7518622
Author :
Date :
2018-03-27T09:44:31
Buffer11: Refactor Subject/Observer pattern. Instead of having a direct/static observer distinction, add two messages for 'Contents Changed' and 'Storage Changed'. This makes Buffer11 itself the subject with two different message handling cases in the onSubjectStateChange methods. Bug: angleproject:2389 Change-Id: I645cd4b7cc7ce51cb7f48a01c7fc72939cbe89fe Reviewed-on: https://chromium-review.googlesource.com/957940 Commit-Queue: Jamie Madill <jmadill@chromium.org> Reviewed-by: Geoff Lang <geofflang@chromium.org> Reviewed-by: Yuly Novikov <ynovikov@chromium.org>
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43
//
// Copyright (c) 2016 The ANGLE Project Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
//
// Observer_unittest:
// Unit tests for Observers and related classes.
#include <gtest/gtest.h>
#include "libANGLE/Observer.h"
using namespace angle;
using namespace testing;
namespace
{
struct ObserverClass : public ObserverInterface
{
void onSubjectStateChange(const gl::Context *context,
SubjectIndex index,
SubjectMessage message) override
{
wasNotified = true;
}
bool wasNotified = false;
};
// Test that Observer/Subject state change notifications work.
TEST(ObserverTest, BasicUsage)
{
Subject subject;
ObserverClass observer;
ObserverBinding binding(&observer, 0u);
binding.bind(&subject);
ASSERT_FALSE(observer.wasNotified);
subject.onStateChange(nullptr, SubjectMessage::STORAGE_CHANGED);
ASSERT_TRUE(observer.wasNotified);
}
} // anonymous namespace