Remove some pre-MD code from toolbar/tabstrip/frame.

chrome/browser/themes/browser_theme_pack.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/themes/browser_theme_pack.h"
 
 #include <limits.h>
 #include <stddef.h>
 
 #include <limits>
@@ skipping 137 matching lines @@
     // The rest of these entries have no key because they can't be overridden
     // from the json manifest.
     {15, IDR_BACK, NULL},
     {16, IDR_BACK_D, NULL},
     {17, IDR_BACK_H, NULL},
     {18, IDR_BACK_P, NULL},
     {19, IDR_FORWARD, NULL},
     {20, IDR_FORWARD_D, NULL},
     {21, IDR_FORWARD_H, NULL},
     {22, IDR_FORWARD_P, NULL},
+#if defined(OS_MACOSX)
     {23, IDR_HOME, NULL},
     {24, IDR_HOME_H, NULL},
     {25, IDR_HOME_P, NULL},
+#endif

[Peter Kasting, 2016/09/22 23:27:24]

So we still use the back/forward/reload/stop bitmaps etc. somehow on non-Mac,
but not home?

[Evan Stade, 2016/09/23 00:31:07]

yea unfortunately there are still a couple places in secondary ui that use
these.

     {26, IDR_RELOAD, NULL},
     {27, IDR_RELOAD_H, NULL},
     {28, IDR_RELOAD_P, NULL},
     {29, IDR_STOP, NULL},
     {30, IDR_STOP_D, NULL},
     {31, IDR_STOP_H, NULL},
     {32, IDR_STOP_P, NULL},
     {33, IDR_TOOLS, NULL},
     {34, IDR_TOOLS_H, NULL},
     {35, IDR_TOOLS_P, NULL},
@@ skipping 1370 matching lines @@
                                            false,
                                            &bitmap_data)) {
       NOTREACHED() << "Unable to encode theme image for prs_id="
                    << prs_id << " for scale_factor=" << scale_factors_[i];
       break;
     }
     image_memory_[scaled_raw_id] =
         base::RefCountedBytes::TakeVector(&bitmap_data);
   }
 }