Once issue 535386 is resolved, move the Chrome OS and Mac-specific vector icon files into their own subdirectories of ui/gfx/vector_icons, and modify ui/gfx/BUILD.gn to only include the vector icons needed for specific build configurations (e.g., only ship the Ash-specific icons when use_ash is true).
Hey,
The change in #15 has broken chromium os builds:
obj/chrome/browser/ui/libui.a(browser_header_painter_ash.o):../../chrome/browser/ui/views/frame/browser_header_painter_ash.cc:function BrowserHeaderPainterAsh::UpdateCaptionButtons(): error: undefined reference to 'ash::kWindowControlMinimizeIcon'
obj/chrome/browser/ui/libui.a(browser_header_painter_ash.o):../../chrome/browser/ui/views/frame/browser_header_painter_ash.cc:function BrowserHeaderPainterAsh::UpdateCaptionButtons(): error: undefined reference to 'ash::kWindowControlCloseIcon'
obj/chrome/browser/ui/libui.a(browser_header_painter_ash.o):../../chrome/browser/ui/views/frame/browser_header_painter_ash.cc:function BrowserHeaderPainterAsh::UpdateCaptionButtons(): error: undefined reference to 'ash::kWindowControlLeftSnappedIcon'
obj/chrome/browser/ui/libui.a(browser_header_painter_ash.o):../../chrome/browser/ui/views/frame/browser_header_painter_ash.cc:function BrowserHeaderPainterAsh::UpdateCaptionButtons(): error: undefined reference to 'ash::kWindowControlRightSnappedIcon'
obj/chrome/browser/ui/libui.a(browser_header_painter_ash.o):../../chrome/browser/ui/views/frame/browser_header_painter_ash.cc:function BrowserHeaderPainterAsh::UpdateCaptionButtons(): error: undefined reference to 'ash::kWindowControlRestoreIcon'
obj/chrome/browser/ui/libui.a(browser_header_painter_ash.o):../../chrome/browser/ui/views/frame/browser_header_painter_ash.cc:function BrowserHeaderPainterAsh::UpdateCaptionButtons(): error: undefined reference to 'ash::kWindowControlMaximizeIcon'
The new icon constants are not checked in. I will revert to unblock the build.
Comment 1 by tdander...@chromium.org
, Jul 9 2016