diff options
| author | hbono@chromium.org <hbono@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-26 04:14:21 +0000 |
|---|---|---|
| committer | hbono@chromium.org <hbono@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-26 04:14:21 +0000 |
| commit | 6999fd893401c26e6dca64e471231aa456cef55c (patch) | |
| tree | ae39bda5bd674d32112bfff3ecf331aa034e3b2a /ui/base/x | |
| parent | 93e7901867cfa9264d743b891027ab1a48ed4547 (diff) | |
| download | chromium_src-6999fd893401c26e6dca64e471231aa456cef55c.zip chromium_src-6999fd893401c26e6dca64e471231aa456cef55c.tar.gz chromium_src-6999fd893401c26e6dca64e471231aa456cef55c.tar.bz2 | |
A blind fix for Bug 70870
It seems r72539 <http://crrev.com/72539> changed ui::GetXWindowStack() so it returns false if XGetWindowProperty() returns 0 (Success). This change just compares the return value of XGetWindowProperty() with Success as the function did before r72539.
BUG=70870
TEST=make the "Linux Tests (valgrind)(1)" bot green.
Review URL: http://codereview.chromium.org/6350017
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72608 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ui/base/x')
| -rw-r--r-- | ui/base/x/x11_util.cc | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/ui/base/x/x11_util.cc b/ui/base/x/x11_util.cc index fda637c..56e53f9 100644 --- a/ui/base/x/x11_util.cc +++ b/ui/base/x/x11_util.cc @@ -441,13 +441,13 @@ bool GetXWindowStack(Window window, std::vector<XID>* windows) { int format; unsigned long count; unsigned char *data = NULL; - if (!GetProperty(window, - "_NET_CLIENT_LIST_STACKING", - ~0L, - &type, - &format, - &count, - &data)) { + if (GetProperty(window, + "_NET_CLIENT_LIST_STACKING", + ~0L, + &type, + &format, + &count, + &data) != Success) { return false; } |