More pre-MD odds and ends.

chrome/common/extensions/api/extension_action/action_info.cc

Index: chrome/common/extensions/api/extension_action/action_info.cc
diff --git a/chrome/common/extensions/api/extension_action/action_info.cc b/chrome/common/extensions/api/extension_action/action_info.cc
index fb14e9f1958b145213f0ab98e43378e61e886d54..a17e5c008a7501cdca64c74dea76a6105a797da8 100644
--- a/chrome/common/extensions/api/extension_action/action_info.cc
+++ b/chrome/common/extensions/api/extension_action/action_info.cc
@@ -73,7 +73,9 @@ std::unique_ptr<ActionInfo> ActionInfo::Load(const Extension* extension,
         *error = base::ASCIIToUTF16(errors::kInvalidPageActionIconPath);
         return std::unique_ptr<ActionInfo>();
       }
-      result->default_icon.Add(extension_misc::EXTENSION_ICON_ACTION, path);
+      // 19 is the old size for extension icons (the size that was in use at the
+      // time of kPageActionIcons).
+      result->default_icon.Add(19, path);

[Evan Stade, 2016/10/26 00:36:00]

I could alternatively make this a constant (E_I_ACTION_LEGACY) in constants.h,
but it needs to stop being part of the enum for the purpose of switch statements
not wanting to handle it. Since this is the only place it's used (in production
code) I didn't bother adding to constants.h.

     }
 
     std::string id;