Fix a crash in the fullscreen handler on Windows which occurs due to CHECK for success in creating …

ui/views/win/fullscreen_handler.cc

 // Copyright (c) 2012 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 "ui/views/win/fullscreen_handler.h"
 
 #include <memory>
 
 #include "base/logging.h"
 #include "base/win/win_util.h"
@@ skipping 72 matching lines @@
     gfx::Rect new_rect(saved_window_info_.window_rect);
     SetWindowPos(hwnd_, NULL, new_rect.x(), new_rect.y(), new_rect.width(),
                  new_rect.height(),
                  SWP_NOZORDER | SWP_NOACTIVATE | SWP_FRAMECHANGED);
   }
 
   if (!task_bar_list_) {
     HRESULT hr = ::CoCreateInstance(CLSID_TaskbarList, NULL,
                                     CLSCTX_INPROC_SERVER,
                                     IID_PPV_ARGS(&task_bar_list_));
-    CHECK(SUCCEEDED(hr));
+    if (SUCCEEDED(hr) && FAILED(task_bar_list_->HrInit()))
+      task_bar_list_ = nullptr;

[msw, 2017/07/10 18:53:31]

Maybe we should have a flag to avoid retrying creation/init if it failed on the
first attempt? Otherwise, this block will retry each time it's hit.

   }
 
   // As per MSDN marking the window as fullscreen should ensure that the
   // taskbar is moved to the bottom of the Z-order when the fullscreen window
   // is activated. If the window is not fullscreen, the Shell falls back to
   // heuristics to determine how the window should be treated, which means
   // that it could still consider the window as fullscreen. :(
-  task_bar_list_->MarkFullscreenWindow(hwnd_, !!fullscreen);
+  if (task_bar_list_)
+    task_bar_list_->MarkFullscreenWindow(hwnd_, !!fullscreen);
 }
 
 }  // namespace views