Exclude zip-file loading from "Load Unpacked Extension"

Exclude zip-file loading from "Load Unpacked Extension"

This was only implemented half-way, and the file picker still limited
choices to directories.  So this code only got run (incorrectly)
by selecting a folder that ends in .zip (which happens on ChromeOS
for a mounted zip archive).

BUG=418753
Committed: https://crrev.com/15de820523bc885041fdebc72cf9b5d966a0c51b
Cr-Commit-Position: refs/heads/master@{#301141}

[elijahtaylor1, 2014-10-22 22:17:55]

elijahtaylor@chromium.org changed reviewers:
+ asargent@chromium.org

[elijahtaylor1, 2014-10-22 22:18:10]

PTAL

[asargent_no_longer_on_chrome, 2014-10-23 20:44:49]

lgtm

[elijahtaylor1, 2014-10-23 20:45:52]

The CQ bit was checked by elijahtaylor@chromium.org

[commit-bot: I haz the power, 2014-10-23 20:46:28]

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

[commit-bot: I haz the power, 2014-10-23 20:54:43]

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

[commit-bot: I haz the power, 2014-10-23 20:54:44]

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

[asargent_no_longer_on_chrome, 2014-10-23 20:58:33]

Oops, looks like you need someone
from chrome/browser/ui/webui/extensions/OWNERS to review.


On Thu, Oct 23, 2014 at 1:54 PM, <commit-bot@chromium.org> wrote:

> Try jobs failed on following builders:
>   chromium_presubmit on tryserver.chromium.linux
> (http://build.chromium.org/p/tryserver.chromium.linux/
> builders/chromium_presubmit/builds/19633)
>
> https://codereview.chromium.org/676503003/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[elijahtaylor1, 2014-10-23 21:05:53]

elijahtaylor@chromium.org changed reviewers:
+ miket@chromium.org

[elijahtaylor1, 2014-10-23 21:05:53]

elijahtaylor@chromium.org changed required reviewers:
+ miket@chromium.org

[elijahtaylor1, 2014-10-23 21:06:31]

PTAL, this time with actual OWNERS :)

[miket_OOO, 2014-10-24 17:44:56]

lgtm

[elijahtaylor1, 2014-10-24 17:46:10]

The CQ bit was checked by elijahtaylor@chromium.org

[commit-bot: I haz the power, 2014-10-24 17:47:35]

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

[commit-bot: I haz the power, 2014-10-24 17:52:31]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-10-24 17:53:25]

Patchset 1 (id:??) landed as
https://crrev.com/15de820523bc885041fdebc72cf9b5d966a0c51b
Cr-Commit-Position: refs/heads/master@{#301141}