Commit be173095 authored by andresantoso's avatar andresantoso Committed by Commit bot

Revert of [fsp] Enable the File System Provider API for extensions. (patchset...

Revert of [fsp] Enable the File System Provider API for extensions. (patchset #2 id:20001 of https://codereview.chromium.org/527663002/)

Reason for revert:
Speculative revert, failed ChromiumOS browser_tests.

[1422/1422] FileSystemProviderApiTest.Extension (11555 ms)
1 test failed:
    FileSystemProviderApiTest.Extension

http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Tests%20%28dbg%29%282%29/builds/11812

Original issue's description:
> [fsp] Enable the File System Provider API for extensions.
>
> Previously, only apps were allowed to use this api, however most clients will
> be actually written as extensions.
>
> TEST=browser_tests: *FileSystemProvider*Extension*
> BUG=409487
>
> Committed: https://chromium.googlesource.com/chromium/src/+/4acb869459901c6fa627473bb15b8a0f6190cc11

TBR=benwells@chromium.org,kalman@chromium.org,mtomasz@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=409487

Review URL: https://codereview.chromium.org/537603002

Cr-Commit-Position: refs/heads/master@{#293148}
parent aa68220c
......@@ -120,10 +120,4 @@ IN_PROC_BROWSER_TEST_F(FileSystemProviderApiTest, WriteFile) {
<< message_;
}
IN_PROC_BROWSER_TEST_F(FileSystemProviderApiTest, Extension) {
ASSERT_TRUE(RunPlatformAppTestWithFlags("file_system_provider/extension",
kFlagLoadAsComponent))
<< message_;
}
} // namespace extensions
......@@ -559,7 +559,7 @@
}],
"fileSystemProvider": {
"channel": "dev",
"extension_types": ["extension", "platform_app"]
"extension_types": ["platform_app"]
},
"firstRunPrivate": {
"channel": "stable",
......
{{+partials.standard_extensions_api api:apis.extensions.fileSystemProvider intro:intros.fileSystemProvider/}}
{
"key": "MIGdMA0GCSqGSIb3DQEBAQUAA4GLADCBhwKBgQDOuXEIuoK1kAkBe0SKiJn/N9oNn3oUxGa4dwj40MnJqPn+w0aR2vuyocm0R4Drp67aYwtLjOVPF4CICRq6ICP6eU07gGwQxGdZ7HJASXV8hm0tab5I70oJmRLfFJyVAMCeWlFaOGq05v2i6EbifZM0qO5xALKNGQt+yjXi5INM5wIBIw==",
"name": "chrome.fileSystemProvider.mount in extensions",
"version": "0.1",
"manifest_version": 2,
"description": "Test for chrome.fileSystemProvider.mount() in an extension.",
"permissions": ["fileSystemProvider", "fileBrowserPrivate"],
"background": {
"persistent": false,
"scripts": [
"chrome-extension://gfnblenhaahcnmfdbebgincjohfkbnch/test_util.js",
"test.js"
]
}
}
// Copyright 2014 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
'use strict';
chrome.test.runTests([
// Tests if mounting succeeds when invoked from an extension. Note that all
// other tests are implemented as apps.
function mount() {
var onTestSuccess = chrome.test.callbackPass();
chrome.fileSystemProvider.mount(
{fileSystemId: 'file-system-id', displayName: 'file-system-name'},
function() {
onTestSuccess();
},
function(error) {
chrome.test.fail();
});
},
]);
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment