diff options
| author | samuong <samuong@chromium.org> | 2016-01-15 09:55:07 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-01-15 17:56:11 +0000 |
| commit | 1c392f8d584a4ccae3264679483d26bdaddb146f (patch) | |
| tree | 4d376e454f0484771a58650f6ba455ca6997da54 | |
| parent | defa24fe63af53a3551702b1dfde2416b48453c8 (diff) | |
| download | chromium_src-1c392f8d584a4ccae3264679483d26bdaddb146f.zip chromium_src-1c392f8d584a4ccae3264679483d26bdaddb146f.tar.gz chromium_src-1c392f8d584a4ccae3264679483d26bdaddb146f.tar.bz2 | |
Revert of Remove Pass on OS X. (patchset #1 id:1 of https://codereview.chromium.org/1586743002/ )
Reason for revert:
This has broken compiles on this Mac bot: https://build.chromium.org/p/chromium.chrome/builders/Google%20Chrome%20Mac/builds/6887
Original issue's description:
> Remove Pass on OS X.
>
> Pass() is a legacy of the original C++03 move emulation implementation.
> New code should use std::move from <utility>.
>
> BUG=557422
> R=danakj@chromium.org
>
> Committed: https://crrev.com/04e90d5757b393018d1b608fd67608d460bb97f6
> Cr-Commit-Position: refs/heads/master@{#369765}
TBR=danakj@chromium.org,dcheng@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=557422
Review URL: https://codereview.chromium.org/1597543002
Cr-Commit-Position: refs/heads/master@{#369787}
| -rw-r--r-- | base/memory/scoped_ptr_unittest.cc | 2 | ||||
| -rw-r--r-- | base/move.h | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/base/memory/scoped_ptr_unittest.cc b/base/memory/scoped_ptr_unittest.cc index 8bea43b..4f0e784 100644 --- a/base/memory/scoped_ptr_unittest.cc +++ b/base/memory/scoped_ptr_unittest.cc @@ -402,7 +402,7 @@ TEST(ScopedPtrTest, MoveBehavior) { EXPECT_TRUE(scoper3.get()); } -#if !defined(OS_ANDROID) && !defined(OS_LINUX) && !defined(OS_MACOSX) +#if !defined(OS_ANDROID) && !defined(OS_LINUX) // Test uncaught Pass() does not have side effects, because Pass() // is implemented by std::move(). // TODO(danakj): Remove this test case when we remove Pass(). diff --git a/base/move.h b/base/move.h index 0af416e..24bf9d75 100644 --- a/base/move.h +++ b/base/move.h @@ -31,7 +31,7 @@ // for more details. // TODO(crbug.com/566182): Remove this macro and use DISALLOW_COPY_AND_ASSIGN // everywhere instead. -#if defined(OS_ANDROID) || defined(OS_LINUX) || defined(OS_MACOSX) +#if defined(OS_ANDROID) || defined(OS_LINUX) #define DISALLOW_COPY_AND_ASSIGN_WITH_MOVE_FOR_BIND(type) \ private: \ type(const type&) = delete; \ |