Re-enabled a couple of ExtensionCrxInstallerTests:

Re-enabled a couple of ExtensionCrxInstallerTests:

- Whitelisting is disabled on ChromeOS by design;
- PackAndInstallExtension passes on a local ChromeOS build;
- AllowOffStore needed a fix in its waiting logic and now passes too.

BUG=404789, 136397, 140893
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=290805

[Joao da Silva, 2014-08-19 14:05:39]

PTAL, thanks!

[not at google - send to devlin, 2014-08-19 14:22:55]

lgtm

https://codereview.chromium.org/486233002/diff/20001/chrome/browser/extension...
File chrome/browser/extensions/crx_installer_browsertest.cc (right):

https://codereview.chromium.org/486233002/diff/20001/chrome/browser/extension...
chrome/browser/extensions/crx_installer_browsertest.cc:237:
IN_PROC_BROWSER_TEST_F(ExtensionCrxInstallerTest, MAYBE_Whitelisting) {
Remove the MAYBE_?

[Joao da Silva, 2014-08-20 07:37:11]

https://codereview.chromium.org/486233002/diff/20001/chrome/browser/extension...
File chrome/browser/extensions/crx_installer_browsertest.cc (right):

https://codereview.chromium.org/486233002/diff/20001/chrome/browser/extension...
chrome/browser/extensions/crx_installer_browsertest.cc:237:
IN_PROC_BROWSER_TEST_F(ExtensionCrxInstallerTest, MAYBE_Whitelisting) {
On 2014/08/19 14:22:55, kalman wrote:
> Remove the MAYBE_?

Done.

[Joao da Silva, 2014-08-20 07:37:15]

The CQ bit was checked by joaodasilva@chromium.org

[commit-bot: I haz the power, 2014-08-20 07:38:27]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/joaodasilva@chromium.org/486233002/40001

[commit-bot: I haz the power, 2014-08-20 10:45:22]

Committed patchset #3 (40001) as 290805