Remove unused branch in native theme win painting

ui/native_theme/native_theme_win.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/native_theme/native_theme_win.h"
 
 #include <windows.h>
 #include <stddef.h>
 #include <uxtheme.h>
 #include <vsstyle.h>
@@ skipping 261 matching lines @@
       PaintMenuBackground(canvas, rect);
       return;
     case kMenuItemBackground:
       CommonThemePaintMenuItemBackground(this, canvas, state, rect,
                                          extra.menu_item);
       return;
     default:
       break;
   }
 
-  HDC surface = skia::GetNativeDrawingContext(canvas);
-
-  // When drawing the task manager or the bookmark editor, we draw into an
-  // offscreen buffer, where we can use OS-specific drawing routines for
-  // UI features like scrollbars. However, we need to set up that buffer,
-  // and then read it back when it's done and blit it onto the screen.
-
-  if (surface)
-    PaintDirect(canvas, surface, part, state, rect, extra);
-  else
-    PaintIndirect(canvas, part, state, rect, extra);
+  PaintIndirect(canvas, part, state, rect, extra);

[Peter Kasting, 2017/02/24 23:10:10]

Nit: Now that this call is so simple, I suggest moving this up into the
"default:" case above and terminating that with a return (instead of break).

[enne (OOO), 2017/02/24 23:23:12]

Done.

 }
 
 NativeThemeWin::NativeThemeWin()
     : draw_theme_(NULL),
       draw_theme_ex_(NULL),
       get_theme_color_(NULL),
       get_theme_content_rect_(NULL),
       get_theme_part_size_(NULL),
       open_theme_(NULL),
       close_theme_(NULL),
@@ skipping 1798 matching lines @@
     break;
   case LAST:
     NOTREACHED();
     break;
   }
   theme_handles_[theme_name] = handle;
   return handle;
 }
 
 }  // namespace ui