Use SkCanvas::SkCanvas(SkBitmap) instead of going thru SkSurface

chrome/browser/ui/webui/app_launcher_login_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 "chrome/browser/ui/webui/app_launcher_login_handler.h"
 
 #include <stddef.h>
 
 #include <string>
 
@@ skipping 41 matching lines @@
 using content::Referrer;
 
 namespace {
 
 SkBitmap GetGAIAPictureForNTP(const gfx::Image& image) {
   // This value must match the width and height value of login-status-icon
   // in new_tab.css.
   const int kLength = 27;
   SkBitmap bmp = skia::ImageOperations::Resize(*image.ToSkBitmap(),
       skia::ImageOperations::RESIZE_BEST, kLength, kLength);
+  SkCanvas canvas(bmp);
 
   // Draw a gray border on the inside of the icon.
-  sk_sp<SkSurface> surface =
-      SkSurface::MakeRasterDirect(bmp.info(), bmp.getPixels(), bmp.rowBytes());
   SkPaint paint;
   paint.setColor(SkColorSetARGB(83, 0, 0, 0));
   paint.setStyle(SkPaint::kStroke_Style);
-  surface->getCanvas()->drawRect(
-      gfx::RectToSkRect(gfx::Rect(kLength - 1, kLength - 1)), paint);
+  canvas.drawRect(gfx::RectToSkRect(gfx::Rect(kLength - 1, kLength - 1)),
+                  paint);
   return bmp;
 }
 
 // Puts the |content| into an element with the given CSS class.
 base::string16 CreateElementWithClass(const base::string16& content,
                                       const std::string& tag_name,
                                       const std::string& css_class,
                                       const std::string& extends_tag) {
   base::string16 start_tag = base::ASCIIToUTF16("<" + tag_name +
       " class='" + css_class + "' is='" + extends_tag + "'>");
@@ skipping 175 matching lines @@
   values->SetString("login_status_message", message);
   values->SetString("login_status_url",
       hide_sync ? std::string() : chrome::kSyncLearnMoreURL);
   values->SetString("login_status_advanced",
       hide_sync ? base::string16() :
       l10n_util::GetStringUTF16(IDS_SYNC_PROMO_NTP_BUBBLE_ADVANCED));
   values->SetString("login_status_dismiss",
       hide_sync ? base::string16() :
       l10n_util::GetStringUTF16(IDS_SYNC_PROMO_NTP_BUBBLE_OK));
 }