Separate gfx_ipc into skia-dependent and non-skia-dependent parts

chrome/browser/extensions/extension_action.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/extensions/extension_action.h"
 
 #include <algorithm>
 #include <utility>
 
 #include "base/base64.h"
@@ skipping 17 matching lines @@
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/animation/animation_delegate.h"
 #include "ui/gfx/canvas.h"
 #include "ui/gfx/color_utils.h"
 #include "ui/gfx/geometry/rect.h"
 #include "ui/gfx/geometry/size.h"
 #include "ui/gfx/image/image.h"
 #include "ui/gfx/image/image_skia.h"
 #include "ui/gfx/image/image_skia_source.h"
 #include "ui/gfx/ipc/gfx_param_traits.h"
+#include "ui/gfx/ipc/gfx_skia_param_traits.h"

[Devlin, 2016/03/28 21:01:51]

ditto

[Mark Dittmer, 2016/03/29 17:41:24]

This file needs IPC::ParamTraits<SkBitmap>.

[Mark Dittmer, 2016/03/30 22:04:41]

Removed unnecessary non-skia include.

 #include "ui/gfx/skbitmap_operations.h"
 #include "url/gurl.h"
 
 namespace {
 
 // Returns the default icon image for extensions.
 gfx::Image GetDefaultIcon() {
   return ui::ResourceBundle::GetSharedInstance().GetImageNamed(
       IDR_EXTENSIONS_FAVICON);
 }
@@ skipping 278 matching lines @@
   // If there is a default icon, the icon width will be set depending on our
   // action type.
   if (default_icon_)
     return ActionIconSize();
 
   // If no icon has been set and there is no default icon, we need favicon
   // width.
   return ui::ResourceBundle::GetSharedInstance().GetImageNamed(
           IDR_EXTENSIONS_FAVICON).Width();
 }