Fix permission regression where warnings aren't shown in the dialog

Fix permission regression where warnings aren't shown in the dialog

A regression was introduced in r275142 causing the permissions dialog to not
show permissions. Obviously very bad. Fixed, and added a regression test.

BUG=397900
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=288365

[Yoyo Zhou, 2014-08-06 23:03:03]

LGTM

[Devlin, 2014-08-07 13:29:55]

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

[commit-bot: I haz the power, 2014-08-07 13:30:24]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rdevlin.cronin@chromium.org/448883002/1

[commit-bot: I haz the power, 2014-08-07 16:48:34]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_triggered_tests on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_dbg_tri...)

[commit-bot: I haz the power, 2014-08-07 18:06:38]

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

[commit-bot: I haz the power, 2014-08-07 18:06:40]

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

[Devlin, 2014-08-07 18:11:31]

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

[commit-bot: I haz the power, 2014-08-07 18:17:21]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rdevlin.cronin@chromium.org/448883002/1

[commit-bot: I haz the power, 2014-08-07 18:42:37]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_triggered_tests on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_dbg_tri...)

[commit-bot: I haz the power, 2014-08-07 20:51:33]

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

[commit-bot: I haz the power, 2014-08-07 20:51:34]

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

[Devlin, 2014-08-07 20:59:18]

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

[commit-bot: I haz the power, 2014-08-07 21:07:19]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rdevlin.cronin@chromium.org/448883002/1

[commit-bot: I haz the power, 2014-08-07 21:48:40]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_triggered_tests on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_dbg_tri...)

[commit-bot: I haz the power, 2014-08-08 00:44:38]

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

[commit-bot: I haz the power, 2014-08-08 00:44:48]

Failed to request the patch to try. Please note that binary files are still
unsupported at the moment, this is being worked on.

Thanks for your patience.

Transient error: <urlopen error [Errno 104] Connection reset by peer>

[Devlin, 2014-08-08 15:45:18]

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

[commit-bot: I haz the power, 2014-08-08 15:46:49]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rdevlin.cronin@chromium.org/448883002/1

[commit-bot: I haz the power, 2014-08-08 15:48:27]

Change committed as 288365