Fixed crash when GetClientRect fails....

app/gfx/gl/gl_context_egl.cc

 // Copyright (c) 2010 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 <EGL/egl.h>
 
 #include "build/build_config.h"
 #if defined(OS_LINUX)
 #include "app/x11_util.h"
 #define EGL_HAS_PBUFFERS 1
@@ skipping 161 matching lines @@
   return false;
 }
 
 bool NativeViewEGLContext::SwapBuffers() {
   return eglSwapBuffers(g_display, surface_) == EGL_TRUE;
 }
 
 gfx::Size NativeViewEGLContext::GetSize() {
 #if defined(OS_WIN)
   RECT rect;
-  CHECK(GetClientRect(static_cast<HWND>(window_), &rect));
+  if (!GetClientRect(static_cast<HWND>(window_), &rect)) {
+    DCHECK(false) << "GetClientRect failed.";

[Ken Russell (switch to Gerrit), 2010/10/06 17:53:49]

Given this fix, do you still want to assert in debug builds?

+    return gfx::Size();
+  }
+
   return gfx::Size(rect.right - rect.left, rect.bottom - rect.top);
 #else
   // TODO(piman): This doesn't work correctly on Windows yet, the size doesn't
   // get updated on resize. When it does, we can share the code.
   EGLint width;
   EGLint height;
   CHECK(eglQuerySurface(g_display, surface_, EGL_WIDTH, &width));
   CHECK(eglQuerySurface(g_display, surface_, EGL_HEIGHT, &height));
   return gfx::Size(width, height);
 #endif
@@ skipping 108 matching lines @@
 
 void* SecondaryEGLContext::GetHandle() {
   return context_;
 }
 
 EGLSurface SecondaryEGLContext::GetSurface() {
   return surface_;
 }
 
 }  // namespace gfx