diff options
| author | spang@chromium.org <spang@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-11-21 22:32:25 +0000 |
|---|---|---|
| committer | spang@chromium.org <spang@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-11-21 22:32:25 +0000 |
| commit | f1e3fbe9e0648e47c8d73d1f17beca8a7bc30e44 (patch) | |
| tree | e1b55126add1453b892a962bc6821de612da5c56 | |
| parent | ae4cc391e492828e08c8bd18cf99d775bd628e20 (diff) | |
| download | chromium_src-f1e3fbe9e0648e47c8d73d1f17beca8a7bc30e44.zip chromium_src-f1e3fbe9e0648e47c8d73d1f17beca8a7bc30e44.tar.gz chromium_src-f1e3fbe9e0648e47c8d73d1f17beca8a7bc30e44.tar.bz2 | |
ozone: Fix pipe creation in release builds
Side effects in DCHECK don't happen in release builds.
Review URL: https://codereview.chromium.org/80583004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@236611 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | ui/events/ozone/evdev/touch_event_converter_unittest.cc | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/ui/events/ozone/evdev/touch_event_converter_unittest.cc b/ui/events/ozone/evdev/touch_event_converter_unittest.cc index 1867e82..d99ab72 100644 --- a/ui/events/ozone/evdev/touch_event_converter_unittest.cc +++ b/ui/events/ozone/evdev/touch_event_converter_unittest.cc @@ -69,7 +69,9 @@ MockTouchEventConverterEvdev::MockTouchEventConverterEvdev(int a, int b) int fds[2]; - DCHECK(pipe(fds) >= 0) << "pipe() failed, errno: " << errno; + if (pipe(fds)) + NOTREACHED() << "failed pipe(): " << strerror(errno); + DCHECK(SetNonBlocking(fds[0]) == 0) << "SetNonBlocking for pipe fd[0] failed, errno: " << errno; DCHECK(SetNonBlocking(fds[1]) == 0) |