• Jian Li's avatar
    Revert "SequenceBound<T> for classes with affinity to a sequence." · 45cb1008
    Jian Li authored
    This reverts commit d91dacc0.
    
    Reason for revert: Failed base_unittests Run on OS: 'Ubuntu-14.04'
    SequenceBoundTest.MultiplyDerivedDestructionWorksLeftSuper
    SequenceBoundTest.MoveAssignmentFromDerivedClass
    SequenceBoundTest.MoveConstructionFromSameClass
    SequenceBoundTest.MoveAssignmentFromSameClass
    SequenceBoundTest.PostBeforeConstruction
    SequenceBoundTest.MoveConstructionFromDerivedClass
    SequenceBoundTest.MultiplyDerivedPostToLeftBaseClass
    SequenceBoundTest.ConstructThenPostThenReset
    SequenceBoundTest.MoveAssignmentFromDerivedClassDestroysOldObject
    SequenceBoundTest.MultiplyDerivedDestructionWorksRightSuper
    SequenceBoundTest.MultiplyDerivedPostToRightBaseClass
    
    https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Linux%20CFI/10148
    
    Original change's description:
    > SequenceBound<T> for classes with affinity to a sequence.
    > 
    > SequenceBound<T> allows for easy creation, posting, and destruction
    > objects of type T on a designated Sequence.  It does not expose the
    > underlying pointer to T, so it's hard to accidentally pass them around
    > in callbacks that are then run on the wrong thread.
    > 
    > A SequenceBound<> may be created anywhere, passed (moved) between
    > threads, etc., safely.  Destruction of the SequenceBound schedules
    > destruction of the underlying instance of T on the appropriate thread.
    > 
    > Not implemented here (but prototyped elsewhere) are weak pointers
    > to these objects that may be passed and used between threads safely;
    > utility functions to generate thread-hopping callbacks; support for
    > strong shared ownership semantics.
    > 
    > Tests: sequence_bound_unittest.cc
    > 
    > Cq-Include-Trybots: luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
    > Change-Id: I3bbf7e975884fce8e18f3bb15a0d211ead58a619
    > Reviewed-on: https://chromium-review.googlesource.com/875241
    > Commit-Queue: Frank Liberato <liberato@chromium.org>
    > Reviewed-by: Daniel Cheng <dcheng@chromium.org>
    > Reviewed-by: Taiju Tsuiki <tzik@chromium.org>
    > Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#591429}
    
    TBR=dcheng@chromium.org,dalecurtis@chromium.org,liberato@chromium.org,tzik@chromium.org
    
    Change-Id: I122ffbe8eb1670e7a252392fb7c28652672d1bc7
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Cq-Include-Trybots: luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
    Reviewed-on: https://chromium-review.googlesource.com/1227356Reviewed-by: default avatarJian Li <jianli@chromium.org>
    Commit-Queue: Jian Li <jianli@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#591477}
    45cb1008
BUILD.gn 114 KB