cros: Fix browser header wrongly drawn frame image

chrome/browser/ui/views/frame/browser_header_painter_ash.cc

 // Copyright 2014 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/ui/views/frame/browser_header_painter_ash.h"
 
 #include "ash/common/ash_layout_constants.h"
 #include "ash/common/frame/caption_buttons/frame_caption_button_container_view.h"
 #include "ash/common/frame/header_painter_util.h"
 #include "ash/resources/vector_icons/vector_icons.h"
@@ skipping 61 matching lines @@
   gfx::ScopedCanvas scoped_save(canvas);
   canvas->ClipPath(frame_path, antialias);
 
   cc::PaintFlags flags;
   flags.setBlendMode(SkBlendMode::kPlus);
   flags.setAntiAlias(antialias);
 
   if (frame_image.isNull() && frame_overlay_image.isNull()) {
     flags.setColor(background_color);
     canvas->DrawRect(bounds, flags);
-  } else if (frame_overlay_image.isNull()) {
-    flags.setAlpha(alpha);
-    canvas->DrawImageInt(frame_image, -image_inset_x, 0, flags);
   } else {
     flags.setAlpha(alpha);
     canvas->SaveLayerWithFlags(flags);

[danakj, 2017/03/24 14:59:54]

SaveLayer isn't free, we should keep the middle branch IMO and and change it to
TileImageInt ?

[Qiang(Joe) Xu, 2017/03/24 16:22:05]

Done.

 
     if (frame_image.isNull()) {
       canvas->DrawColor(background_color);
     } else {
       canvas->TileImageInt(frame_image, image_inset_x, 0, 0, 0, bounds.width(),
                            bounds.height());
     }
     canvas->DrawImageInt(frame_overlay_image, 0, 0);
 
     canvas->Restore();
@@ skipping 200 matching lines @@
 }
 
 gfx::Rect BrowserHeaderPainterAsh::GetPaintedBounds() const {
   return gfx::Rect(view_->width(), painted_height_);
 }
 
 gfx::Rect BrowserHeaderPainterAsh::GetTitleBounds() const {
   return ash::HeaderPainterUtil::GetTitleBounds(window_icon_,
       caption_button_container_, BrowserFrame::GetTitleFontList());
 }