Eliminate HICON leaks caused by creating icons from bitmap image.

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

Index: chrome/browser/ui/views/frame/glass_browser_frame_view.cc
diff --git a/chrome/browser/ui/views/frame/glass_browser_frame_view.cc b/chrome/browser/ui/views/frame/glass_browser_frame_view.cc
index 7168d7c672257d63b1507c009f496039fd1d72ba..9f8551a343c165b1d4fecc2e5ff8817dae1e98be 100644
--- a/chrome/browser/ui/views/frame/glass_browser_frame_view.cc
+++ b/chrome/browser/ui/views/frame/glass_browser_frame_view.cc
@@ -585,6 +585,10 @@ void GlassBrowserFrameView::StopThrobber() {
             icon, GetSystemMetrics(SM_CXSMICON), GetSystemMetrics(SM_CYSMICON));
         big_icon = CreateHICONFromSkBitmapSizedTo(
             icon, GetSystemMetrics(SM_CXICON), GetSystemMetrics(SM_CYICON));
+
+        // Take responsibility for eventually destroying the created icons.
+        small_window_icon_.Set(small_icon);

[sky, 2015/11/02 16:41:38]

Won't this be problematic with the code on 618-623?

Also, you should make CreateHICONFromSkBitmapSizedTo return a scopedHICON to
make ownership clear. I suspect this means adding move semantics to ScopedHICon,
but that's for the better.

+        big_window_icon_.Set(big_icon);
       }
     }