Update old syntax for action default_icon to populate 19 and 38px icons

chrome/common/extensions/api/extension_action/action_info.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/common/extensions/api/extension_action/action_info.h"
 
 #include "base/memory/scoped_ptr.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/common/extensions/api/commands/commands_handler.h"
 #include "extensions/common/constants.h"
@@ skipping 85 matching lines @@
               extension_misc::kExtensionActionIconSizes,
               extension_misc::kNumExtensionActionIconSizes,
               &result->default_icon,
               error)) {
         return scoped_ptr<ActionInfo>();
       }
     } else if (dict->GetString(keys::kPageActionDefaultIcon, &default_icon) &&
                manifest_handler_helpers::NormalizeAndValidatePath(
                    &default_icon)) {
       result->default_icon.Add(extension_misc::EXTENSION_ICON_ACTION,
                                default_icon);

[not at google - send to devlin, 2014/08/26 22:25:41]

The default icon logic will automatically scale down
(extension_misc::EXTENSION_ICON_ACTION * 2) to
extension_misc::EXTENSION_ICON_ACTION, right? So you shouldn't even need this
first case anymore.

[gpdavis, 2014/08/26 23:24:43]

Done.

+      result->default_icon.Add(extension_misc::EXTENSION_ICON_ACTION * 2,
+                               default_icon);
     } else {
       *error = base::ASCIIToUTF16(errors::kInvalidPageActionIconPath);
       return scoped_ptr<ActionInfo>();
     }
   }
 
   // Read the page action title from |default_title| if present, |name| if not
   // (both optional).
   if (dict->HasKey(keys::kPageActionDefaultTitle)) {
     if (!dict->GetString(keys::kPageActionDefaultTitle,
@@ skipping 96 matching lines @@
 
 // static
 bool ActionInfo::IsVerboseInstallMessage(const Extension* extension) {
   const ActionInfo* page_action_info = GetPageActionInfo(extension);
   return page_action_info &&
       (CommandsInfo::GetPageActionCommand(extension) ||
        !page_action_info->default_icon.empty());
 }
 
 }  // namespace extensions