Revert of Change extension icon load errors to warnings

Revert of Change extension icon load errors to warnings (patchset #8 id:140001 of https://codereview.chromium.org/1537473003/ )

Reason for revert:
caused crbug.com/571701

Original issue's description:
> Change extension icon load errors to warnings
> 
> During the Extension parsing step, check if the icon file exists and if not, remove that entry from the dictionary.
> 
> Keep the same check during the validation phase and don't apply the workaround to unpacked extensions. This will more strongly discourage new extensions from making this mistake.
> 
> BUG=570249
> 
> Committed: https://crrev.com/6e8e7d1c49657e82d0e8f2518ad463794346321b
> Cr-Commit-Position: refs/heads/master@{#366253}

TBR=estade@chromium.org
BUG=570249

[Devlin, 2015-12-22 23:13:35]

Created Revert of Change extension icon load errors to warnings

[Devlin, 2015-12-22 23:13:42]

Description was changed from

==========
Revert of Change extension icon load errors to warnings (patchset #8 id:140001
of https://codereview.chromium.org/1537473003/ )

Reason for revert:
caused crbug.com/571701

Original issue's description:
> Change extension icon load errors to warnings
> 
> During the Extension parsing step, check if the icon file exists and if not,
remove that entry from the dictionary.
> 
> Keep the same check during the validation phase and don't apply the workaround
to unpacked extensions. This will more strongly discourage new extensions from
making this mistake.
> 
> BUG=570249
> 
> Committed: https://crrev.com/6e8e7d1c49657e82d0e8f2518ad463794346321b
> Cr-Commit-Position: refs/heads/master@{#366253}

TBR=estade@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=570249
==========

to

==========
Revert of Change extension icon load errors to warnings (patchset #8 id:140001
of https://codereview.chromium.org/1537473003/ )

Reason for revert:
caused crbug.com/571701

Original issue's description:
> Change extension icon load errors to warnings
> 
> During the Extension parsing step, check if the icon file exists and if not,
remove that entry from the dictionary.
> 
> Keep the same check during the validation phase and don't apply the workaround
to unpacked extensions. This will more strongly discourage new extensions from
making this mistake.
> 
> BUG=570249
> 
> Committed: https://crrev.com/6e8e7d1c49657e82d0e8f2518ad463794346321b
> Cr-Commit-Position: refs/heads/master@{#366253}

TBR=estade@chromium.org
BUG=570249
==========

[Devlin, 2015-12-22 23:13:48]

The CQ bit was checked by rdevlin.cronin@chromium.org

[commit-bot: I haz the power, 2015-12-22 23:14:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1541113003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1541113003/1

[commit-bot: I haz the power, 2015-12-22 23:17:10]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-12-22 23:17:10]

Try jobs failed on following builders:
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Devlin, 2015-12-23 06:49:47]

The CQ bit was checked by rdevlin.cronin@chromium.org

[commit-bot: I haz the power, 2015-12-23 06:50:08]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1541113003/260001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1541113003/260001

[commit-bot: I haz the power, 2015-12-23 06:55:03]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-12-23 06:55:03]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Evan Stade, 2015-12-29 17:39:22]

On 2015/12/23 06:55:03, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

https://codereview.chromium.org/1537473003/

[Evan Stade, 2015-12-29 17:39:41]

On 2015/12/29 17:39:22, Evan Stade (ooo to Tues12-29) wrote:
> On 2015/12/23 06:55:03, commit-bot: I haz the power wrote:
> > Try jobs failed on following builders:
> >   chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
> >
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
> 
> https://codereview.chromium.org/1537473003/

err, revert here: https://codereview.chromium.org/1554583002/