OLD | NEW |
---|---|
1 // Copyright 2016 The Chromium Authors. All rights reserved. | 1 // Copyright 2016 The Chromium Authors. All rights reserved. |
2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
4 | 4 |
5 #include "chrome/browser/media/window_icon_util.h" | 5 #include "chrome/browser/media/window_icon_util.h" |
6 | 6 |
7 #include <X11/Xatom.h> | 7 #include <X11/Xatom.h> |
8 #include <X11/Xutil.h> | 8 #include <X11/Xutil.h> |
9 | 9 |
10 #include "ui/gfx/x/x11_error_tracker.h" | |
10 #include "ui/gfx/x/x11_types.h" | 11 #include "ui/gfx/x/x11_types.h" |
11 | 12 |
12 gfx::ImageSkia GetWindowIcon(content::DesktopMediaID id) { | 13 gfx::ImageSkia GetWindowIcon(content::DesktopMediaID id) { |
13 DCHECK(id.type == content::DesktopMediaID::TYPE_WINDOW); | 14 DCHECK(id.type == content::DesktopMediaID::TYPE_WINDOW); |
14 | 15 |
15 Display* display = gfx::GetXDisplay(); | 16 Display* display = gfx::GetXDisplay(); |
16 Atom property = XInternAtom(display, "_NET_WM_ICON", True); | 17 Atom property = XInternAtom(display, "_NET_WM_ICON", True); |
17 Atom actual_type; | 18 Atom actual_type; |
18 int actual_format; | 19 int actual_format; |
19 unsigned long bytes_after; // NOLINT: type required by XGetWindowProperty | 20 unsigned long bytes_after; // NOLINT: type required by XGetWindowProperty |
20 unsigned long size; | 21 unsigned long size; |
21 long* data; | 22 long* data; |
22 | 23 |
24 // The |error_tracker| essentially provides an empty X error handler for | |
sky
2016/08/26 19:38:36
Excellent description. Thanks!
qiangchen
2016/08/26 19:55:24
Acknowledged.
| |
25 // the call of XGetWindowProperty. The motivation is to guard against crash | |
26 // for any reason that XGetWindowProperty fails. For example, at the time that | |
27 // XGetWindowProperty is called, the window handler (a.k.a |id.id|) may | |
28 // already be invalid due to the fact that the end user has closed the | |
29 // corresponding window, etc. | |
30 std::unique_ptr<gfx::X11ErrorTracker> error_tracker( | |
Lei Zhang
2016/08/30 09:39:34
Can this just be the following?
{
gfx::X11Error
qiangchen
2016/08/30 17:10:45
By other reviewer's comment, as we want to release
| |
31 new gfx::X11ErrorTracker()); | |
23 int status = XGetWindowProperty(display, id.id, property, 0L, ~0L, False, | 32 int status = XGetWindowProperty(display, id.id, property, 0L, ~0L, False, |
24 AnyPropertyType, &actual_type, &actual_format, | 33 AnyPropertyType, &actual_type, &actual_format, |
25 &size, &bytes_after, | 34 &size, &bytes_after, |
26 reinterpret_cast<unsigned char**>(&data)); | 35 reinterpret_cast<unsigned char**>(&data)); |
36 error_tracker.reset(); | |
37 | |
27 if (status != Success) { | 38 if (status != Success) { |
28 return gfx::ImageSkia(); | 39 return gfx::ImageSkia(); |
29 } | 40 } |
30 | 41 |
31 // The format of |data| is concatenation of sections like | 42 // The format of |data| is concatenation of sections like |
32 // [width, height, pixel data of size width * height], and the total bytes | 43 // [width, height, pixel data of size width * height], and the total bytes |
33 // number of |data| is |size|. And here we are picking the largest icon. | 44 // number of |data| is |size|. And here we are picking the largest icon. |
34 int width = 0; | 45 int width = 0; |
35 int height = 0; | 46 int height = 0; |
36 int start = 0; | 47 int start = 0; |
(...skipping 18 matching lines...) Expand all Loading... | |
55 for (long y = 0; y < height; ++y) { | 66 for (long y = 0; y < height; ++y) { |
56 for (long x = 0; x < width; ++x) { | 67 for (long x = 0; x < width; ++x) { |
57 pixels_data[result.rowBytesAsPixels() * y + x] = | 68 pixels_data[result.rowBytesAsPixels() * y + x] = |
58 static_cast<uint32_t>(data[start + width * y + x]); | 69 static_cast<uint32_t>(data[start + width * y + x]); |
59 } | 70 } |
60 } | 71 } |
61 | 72 |
62 XFree(data); | 73 XFree(data); |
63 return gfx::ImageSkia::CreateFrom1xBitmap(result); | 74 return gfx::ImageSkia::CreateFrom1xBitmap(result); |
64 } | 75 } |
OLD | NEW |