Display Window Icon In Picker UI

chrome/browser/media/window_icon_util_x11.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/media/window_icon_util.h"
 
 #include <X11/Xatom.h>
 #include <X11/Xutil.h>
 
+#include "ui/gfx/x/x11_error_tracker.h"
 #include "ui/gfx/x/x11_types.h"
 
 gfx::ImageSkia GetWindowIcon(content::DesktopMediaID id) {
   DCHECK(id.type == content::DesktopMediaID::TYPE_WINDOW);
 
   Display* display = gfx::GetXDisplay();
   Atom property = XInternAtom(display, "_NET_WM_ICON", True);
   Atom actual_type;
   int actual_format;
   unsigned long bytes_after;  // NOLINT: type required by XGetWindowProperty
   unsigned long size;
   long* data;
 
+  std::unique_ptr<gfx::X11ErrorTracker> error_tracker(

[sky, 2016/08/26 16:40:35]

Please add a comment here, something similar to Dan's explanation here.

[qiangchen, 2016/08/26 17:30:01]

Done.

+      new gfx::X11ErrorTracker());
   int status = XGetWindowProperty(display, id.id, property, 0L, ~0L, False,
                                   AnyPropertyType, &actual_type, &actual_format,
                                   &size, &bytes_after,
                                   reinterpret_cast<unsigned char**>(&data));
+  error_tracker.reset();
+
   if (status != Success) {
     return gfx::ImageSkia();
   }
 
   // The format of |data| is concatenation of sections like
   // [width, height, pixel data of size width * height], and the total bytes
   // number of |data| is |size|. And here we are picking the largest icon.
   int width = 0;
   int height = 0;
   int start = 0;
@@ skipping 18 matching lines @@
   for (long y = 0; y < height; ++y) {
     for (long x = 0; x < width; ++x) {
       pixels_data[result.rowBytesAsPixels() * y + x] =
           static_cast<uint32_t>(data[start + width * y + x]);
     }
   }
 
   XFree(data);
   return gfx::ImageSkia::CreateFrom1xBitmap(result);
 }