Chromium Code Reviews| Index: chrome/browser/extensions/extension_install_prompt_unittest.cc |
| diff --git a/chrome/browser/extensions/extension_install_prompt_unittest.cc b/chrome/browser/extensions/extension_install_prompt_unittest.cc |
| index e1a671caf1f0c42452f6cd6888f08456e61a17ea..a318b4926e2cea34a841fb19b2c61f7b4b0bc526 100644 |
| --- a/chrome/browser/extensions/extension_install_prompt_unittest.cc |
| +++ b/chrome/browser/extensions/extension_install_prompt_unittest.cc |
| @@ -82,8 +82,7 @@ TEST(ExtensionInstallPromptUnittest, PromptShowsWithheldPermissions) { |
| .Set("permissions", |
| ListBuilder().Append("http://*/*") |
| .Append("http://www.google.com/") |
| - .Append("tabs"))) |
| - .Build(); |
| + .Append("tabs"))).Build(); |
| ExtensionInstallPrompt prompt(nullptr /* no web contents in this test */); |
| base::RunLoop run_loop; |
| @@ -107,10 +106,10 @@ TEST(ExtensionInstallPromptUnittest, DelegatedPromptShowsOptionalPermissions) { |
| .Set("version", "1.0") |
| .Set("manifest_version", 2) |
| .Set("description", "Random Ext") |
| - .Set("permissions", ListBuilder().Append("tabs")) |
| + .Set("permissions", |
| + ListBuilder().Append("clipboardRead")) |
| .Set("optional_permissions", |
| - ListBuilder().Append("location"))) |
|
Marc Treib
2015/03/09 14:05:59
For some reason, "location" does not result in any
not at google - send to devlin
2015/03/09 16:33:53
Well that's weird. Location is supposed to be a va
Marc Treib
2015/03/09 17:30:40
Yup, especially as it works fine on all the other
|
| - .Build(); |
| + ListBuilder().Append("tabs"))).Build(); |
| ExtensionInstallPrompt prompt(nullptr /* no web contents in this test */); |
| base::RunLoop run_loop; |
| prompt.set_callback_for_test( |