Revert of Restrict use of two app-launching command line flags

Revert of Restrict use of two app-launching command line flags (patchset #8 id:140001 of https://codereview.chromium.org/2108853002/ )

Reason for revert:
Breaks official builds.
chrome/browser/extensions/unpacked_installer.cc:43:12: error: unused variable 'kUnpackedExtensionInsteadOfAppError'

Original issue's description:
> Restrict use of two app-launching command line flags
>
> Mitigate abuse of the load-and-launch-app and load-apps flags by
> disallowing loading of non-app extensions from these flags.
>
> For the official build, the flags no-op instead of showing an error popup
> to avoid giving users with hijacked shortcuts a bad time (the error
> message is shown then Chrome shuts down)
>
> BUG=624098
>
> Committed: https://crrev.com/93c60cd3ef5908984e173112dc09345bb5714036
> Cr-Commit-Position: refs/heads/master@{#407824}

TBR=rdevlin.cronin@chromium.org,asargent@chromium.org,msw@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=624098
Committed: https://crrev.com/f50203526865893d6dc176e69f6a073116e5872b
Cr-Commit-Position: refs/heads/master@{#407830}

[proberge, 2016-07-26 16:54:55]

The CQ bit was checked by proberge@chromium.org

[proberge, 2016-07-26 16:54:55]

Created Revert of Restrict use of two app-launching command line flags

[commit-bot: I haz the power, 2016-07-26 16:55:05]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-26 16:55:06]

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

[commit-bot: I haz the power, 2016-07-26 16:55:07]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[Cait (Slow), 2016-07-26 16:56:08]

The CQ bit was checked by caitkp@chromium.org

[Cait (Slow), 2016-07-26 16:56:20]

lgtm

[commit-bot: I haz the power, 2016-07-26 16:56:26]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-26 16:58:00]

Description was changed from

==========
Revert of Restrict use of two app-launching command line flags (patchset #8
id:140001 of https://codereview.chromium.org/2108853002/ )

Reason for revert:
Breaks official builds.
chrome/browser/extensions/unpacked_installer.cc:43:12: error: unused variable
'kUnpackedExtensionInsteadOfAppError' 

Original issue's description:
> Restrict use of two app-launching command line flags
> 
> Mitigate abuse of the load-and-launch-app and load-apps flags by
> disallowing loading of non-app extensions from these flags.
> 
> For the official build, the flags no-op instead of showing an error popup
> to avoid giving users with hijacked shortcuts a bad time (the error
> message is shown then Chrome shuts down)
> 
> BUG=624098
> 
> Committed: https://crrev.com/93c60cd3ef5908984e173112dc09345bb5714036
> Cr-Commit-Position: refs/heads/master@{#407824}

TBR=rdevlin.cronin@chromium.org,asargent@chromium.org,msw@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=624098
==========

to

==========
Revert of Restrict use of two app-launching command line flags (patchset #8
id:140001 of https://codereview.chromium.org/2108853002/ )

Reason for revert:
Breaks official builds.
chrome/browser/extensions/unpacked_installer.cc:43:12: error: unused variable
'kUnpackedExtensionInsteadOfAppError' 

Original issue's description:
> Restrict use of two app-launching command line flags
> 
> Mitigate abuse of the load-and-launch-app and load-apps flags by
> disallowing loading of non-app extensions from these flags.
> 
> For the official build, the flags no-op instead of showing an error popup
> to avoid giving users with hijacked shortcuts a bad time (the error
> message is shown then Chrome shuts down)
> 
> BUG=624098
> 
> Committed: https://crrev.com/93c60cd3ef5908984e173112dc09345bb5714036
> Cr-Commit-Position: refs/heads/master@{#407824}

TBR=rdevlin.cronin@chromium.org,asargent@chromium.org,msw@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=624098
==========

[commit-bot: I haz the power, 2016-07-26 16:58:01]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-26 17:00:30]

Description was changed from

==========
Revert of Restrict use of two app-launching command line flags (patchset #8
id:140001 of https://codereview.chromium.org/2108853002/ )

Reason for revert:
Breaks official builds.
chrome/browser/extensions/unpacked_installer.cc:43:12: error: unused variable
'kUnpackedExtensionInsteadOfAppError' 

Original issue's description:
> Restrict use of two app-launching command line flags
> 
> Mitigate abuse of the load-and-launch-app and load-apps flags by
> disallowing loading of non-app extensions from these flags.
> 
> For the official build, the flags no-op instead of showing an error popup
> to avoid giving users with hijacked shortcuts a bad time (the error
> message is shown then Chrome shuts down)
> 
> BUG=624098
> 
> Committed: https://crrev.com/93c60cd3ef5908984e173112dc09345bb5714036
> Cr-Commit-Position: refs/heads/master@{#407824}

TBR=rdevlin.cronin@chromium.org,asargent@chromium.org,msw@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=624098
==========

to

==========
Revert of Restrict use of two app-launching command line flags (patchset #8
id:140001 of https://codereview.chromium.org/2108853002/ )

Reason for revert:
Breaks official builds.
chrome/browser/extensions/unpacked_installer.cc:43:12: error: unused variable
'kUnpackedExtensionInsteadOfAppError'

Original issue's description:
> Restrict use of two app-launching command line flags
>
> Mitigate abuse of the load-and-launch-app and load-apps flags by
> disallowing loading of non-app extensions from these flags.
>
> For the official build, the flags no-op instead of showing an error popup
> to avoid giving users with hijacked shortcuts a bad time (the error
> message is shown then Chrome shuts down)
>
> BUG=624098
>
> Committed: https://crrev.com/93c60cd3ef5908984e173112dc09345bb5714036
> Cr-Commit-Position: refs/heads/master@{#407824}

TBR=rdevlin.cronin@chromium.org,asargent@chromium.org,msw@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=624098

Committed: https://crrev.com/f50203526865893d6dc176e69f6a073116e5872b
Cr-Commit-Position: refs/heads/master@{#407830}
==========

[commit-bot: I haz the power, 2016-07-26 17:00:31]

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