X11Window::initialize remove a small code redundancy

BUG= Change-Id: Id5f98332129b3cc854254187b494bf6a97921dbd Reviewed-on: https://chromium-review.googlesource.com/289272Reviewed-by: Geoff Lang <geofflang@chromium.org> Reviewed-by: Jamie Madill <jmadill@chromium.org> Tested-by: Corentin Wallez <cwallez@chromium.org>

X11Window::initialize remove a small code redundancy
b2e33b90 · Corentin Wallez · 23a2ae0c · b2e33b90
Commit b2e33b90 authored Jul 28, 2015 by Corentin Wallez
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

X11Window.cpp util/x11/X11Window.cpp +2 -2

No files found.
--- a/util/x11/X11Window.cpp
+++ b/util/x11/X11Window.cpp
@@ -176,9 +176,9 @@ bool X11Window::initialize(const std::string &name, size_t width, size_t height)
        int screen = DefaultScreen(mDisplay);
        Window root = RootWindow(mDisplay, screen);

-        Colormap colormap = XCreateColormap(mDisplay, root, DefaultVisual(mDisplay, screen), AllocNone);
-        int depth = DefaultDepth(mDisplay, screen);
        Visual *visual = DefaultVisual(mDisplay, screen);
+        int depth = DefaultDepth(mDisplay, screen);
+        Colormap colormap = XCreateColormap(mDisplay, root, visual, AllocNone);

        XSetWindowAttributes attributes;
        unsigned long attributeMask = CWBorderPixel | CWColormap | CWEventMask;