| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
total number of jobs.
* "added" type is added
* "num_total_jobs" member is added
* and this is for displaying the total number of jobs in Chrome OS file manager
BUG=400636
TEST=
manually
1. Copy the local file to drive directory.
2. Observe the status of syncing status.
Review URL: https://codereview.chromium.org/507293002
Cr-Commit-Position: refs/heads/master@{#293257}
|
|
|
|
|
|
|
|
|
|
|
| |
This keeps the "Visit Website" link from showing up in chrome://extensions,
which makes sense because shared modules are automatically installed.
BUG=408927
Review URL: https://codereview.chromium.org/532973002
Cr-Commit-Position: refs/heads/master@{#293203}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(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}
|
|
|
|
|
|
|
|
|
|
|
| |
This CL adds descriptions to dictionaries.
TEST=Tested manually with a test server.
BUG=248427
Review URL: https://codereview.chromium.org/527723002
Cr-Commit-Position: refs/heads/master@{#293121}
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
Review URL: https://codereview.chromium.org/527663002
Cr-Commit-Position: refs/heads/master@{#293100}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The length is already available as data.byteLength, where data is of
the ArrayBuffer type.
TBR=benwells@chromium.org
TEST=browser_tests, unit_tests: *FileSystemProvider*Write*
BUG=248427
Review URL: https://codereview.chromium.org/530373002
Cr-Commit-Position: refs/heads/master@{#293093}
|
|
|
|
|
|
|
|
|
|
|
|
| |
The read-only version is pretty much done, so we should let developers use it
on the dev channel for initial feedback.
TEST=browser_tests: *FileSystemProvider*
BUG=248427
Review URL: https://codereview.chromium.org/531463002
Cr-Commit-Position: refs/heads/master@{#293063}
|
|
|
|
|
|
| |
Review URL: https://codereview.chromium.org/531973003
Cr-Commit-Position: refs/heads/master@{#293051}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The example extension for the chrome.power API registered a
listener for windows.onCreated to work around an issue where
runtime.onStartup didn't fire on Chrome OS. The underlying
Chrome bug has been fixed now, so the windows.onCreated
workaround is no longer needed.
BUG=222473
TEST=install the extension on Chrome OS and check that
/var/log/power_manager/powerd.LATEST reports an
extension-driven policy change when logging in
Review URL: https://codereview.chromium.org/531953002
Cr-Commit-Position: refs/heads/master@{#293043}
|
|
|
|
|
|
|
|
| |
BUG=352293
Review URL: https://codereview.chromium.org/515563003
Cr-Commit-Position: refs/heads/master@{#293016}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This should enable the message receiver to identify a key to be used for
message decryption.
BUG=409099
(depends on https://codereview.chromium.org/513013003/)
Review URL: https://codereview.chromium.org/518643002
Cr-Commit-Position: refs/heads/master@{#293006}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is the first in a series of CLs intended to reduce the amount of caching
in docserver code. The @Cache annotation is used to indicate that compiled data
should be cached in CompiledFileSystem.
NOTRY=True
Review URL: https://codereview.chromium.org/521693003
Cr-Commit-Position: refs/heads/master@{#292994}
|
|
|
|
|
|
|
|
| |
NOTRY=True
Review URL: https://codereview.chromium.org/509253003
Cr-Commit-Position: refs/heads/master@{#292987}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The initializer for the Copresence permission is malformed. It specifies the
message id instead of the flag.
R=kalman@chromium.org
BUG=408843
Committed to pending queue: https://chromium.googlesource.com/chromium/src/+/c2a41e8
Review URL: https://codereview.chromium.org/523433002
Cr-Commit-Position: refs/heads/master@{#292952}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Analogically to CreateFile, we should just return an EXISTS error code if the
entry already exists. The EXISTS error code is later converted to OK if the
operation was requested to be not 'exclusive'.
TBR=benwells@chromium.org
TEST=browser_tests, unit_tests: *FileSystemProvider*CreateDirectory*
BUG=393141
Review URL: https://codereview.chromium.org/528683002
Cr-Commit-Position: refs/heads/master@{#292873}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This CL removes:
- "Move window to..." menu item from the gear menu.
- Remove APIs fileBrowserPrivate.onProfileAdded and fileBrowserPrivate.onDesktopChanged as they are used only for updating gear menu.
- Remove corresponding tests.
BUG=408965
TEST=run browser_test, unit_tests
TBR=benwells@chromium.org
# TBRing for just removing unused events from file_browser_private.idl
Review URL: https://codereview.chromium.org/530553006
Cr-Commit-Position: refs/heads/master@{#292860}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch makes the method providing additional metadata for entries available
for provided file systems too.
NOTRY=True
TEST=Tested manually. Files app works as before.
BUG=408017
Review URL: https://codereview.chromium.org/515033002
Cr-Commit-Position: refs/heads/master@{#292851}
|
|
|
|
|
|
|
|
|
|
|
|
| |
The field is not used, so it should be removed.
NOTRY=True
TEST=Tested manually that Files app works correctly.
BUG=408017
Review URL: https://codereview.chromium.org/522533003
Cr-Commit-Position: refs/heads/master@{#292846}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Previously, thumbnails were generated from full size images, which in case of
cloud providers would invoke downloading the entire file.
This CL adds an ability to provide a thumbnail as a data URI, so:
(1) We can have thumbnails for other file types than images.
(2) Thumbnails are shown fast.
(3) Resource consumption is significantly reduced.
Note, that the thumbnails are not wired to Files app yet. That will be done in
a separate patch.
TEST=unit_tests: *FileSystemProvider*GetMetadata*
BUG=407954
Review URL: https://codereview.chromium.org/513683002
Cr-Commit-Position: refs/heads/master@{#292840}
|
|
|
|
|
|
|
|
|
|
|
| |
Now that the Files.app uses default system header, the own profile badge is redundant.
BUG=408816
TEST=confirmed manually, ran browser_tests
Review URL: https://codereview.chromium.org/520623002
Cr-Commit-Position: refs/heads/master@{#292827}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
_GetFileVersionFromCache/_GetFileListingVersionFromCache, because they're
protected not public methods. More importantly, make them return Futures.
Plus a little more cleanup.
These are small changes in preparation to the more interesting patch which
addresses bug 402903.
BUG=402903
TBR=yoz@chromium.org
Review URL: https://codereview.chromium.org/522183002
Cr-Commit-Position: refs/heads/master@{#292797}
|
|
|
|
|
|
|
|
| |
BUG=404334
Review URL: https://codereview.chromium.org/522053002
Cr-Commit-Position: refs/heads/master@{#292757}
|
|
|
|
|
|
|
|
|
| |
BUG=395240
TBR=keybuk@chromium.org
Review URL: https://codereview.chromium.org/420663003
Cr-Commit-Position: refs/heads/master@{#292719}
|
|
|
|
|
|
|
|
| |
NOTRY=True
Review URL: https://codereview.chromium.org/521453003
Cr-Commit-Position: refs/heads/master@{#292708}
|
|
|
|
|
|
|
|
| |
BUG=370071
Review URL: https://codereview.chromium.org/343053002
Cr-Commit-Position: refs/heads/master@{#292700}
|
|
|
|
|
|
|
|
| |
BUG=392215
Review URL: https://codereview.chromium.org/508263003
Cr-Commit-Position: refs/heads/master@{#292660}
|
|
|
|
|
|
|
|
| |
BUG=352293
Review URL: https://codereview.chromium.org/494853004
Cr-Commit-Position: refs/heads/master@{#292640}
|
|
|
|
|
|
|
|
|
|
| |
- Move ImageDetails from chrome to extensions.
BUG=352290
Review URL: https://codereview.chromium.org/487533005
Cr-Commit-Position: refs/heads/master@{#292567}
|
|
|
|
|
|
|
|
| |
BUG=324071,374440,408737
Review URL: https://codereview.chromium.org/522463002
Cr-Commit-Position: refs/heads/master@{#292562}
|
|
|
|
|
|
|
|
| |
NOTRY=true
Review URL: https://codereview.chromium.org/524433003
Cr-Commit-Position: refs/heads/master@{#292549}
|
|
|
|
|
|
|
|
|
|
|
|
| |
The current warning is the generic "Read your browsing history"
warning. The new warning for topSites will be suppressed by
read only and read/write history warnings, and all hosts warning.
BUG=404334
Review URL: https://codereview.chromium.org/482953003
Cr-Commit-Position: refs/heads/master@{#292545}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
https://codereview.chromium.org/523433002/)
Reason for revert:
This change causes a test failure:
http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%20Tests%20%281%29/builds/48008/steps/unit_tests/logs/stdio
Error:
[ RUN ] PermissionsTest.PermissionMessages
../../chrome/common/extensions/permissions/permission_set_unittest.cc:781: Failure
Value of: permission_info->message_id()
Actual: 65
Expected: PermissionMessage::kNone
Which is: 1
unexpected message_id for copresence
Looks like this change is the cause.
Original issue's description:
> Fix the Copresence permission.
>
> The initializer for the Copresence permission is malformed. It specifies the
> message id instead of the flag.
>
> R=kalman@chromium.org
> BUG=408843
>
> Committed to pending queue: https://chromium.googlesource.com/chromium/src/+/c2a41e8
TBR=kalman@chromium.org,rkc@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=408843
Review URL: https://codereview.chromium.org/520493002
Cr-Commit-Position: refs/heads/master@{#292532}
|
|
|
|
|
|
|
|
| |
NOTRY=true
Review URL: https://codereview.chromium.org/522483002
Cr-Commit-Position: refs/heads/master@{#292519}
|
|
|
|
|
|
|
|
|
|
|
|
| |
The initializer for the Copresence permission is malformed. It specifies the
message id instead of the flag.
R=kalman@chromium.org
BUG=None.
Review URL: https://codereview.chromium.org/523433002
Cr-Commit-Position: refs/heads/master@{#292495}
|
|
|
|
|
|
|
|
| |
BUG=338449
Review URL: https://codereview.chromium.org/503503003
Cr-Commit-Position: refs/heads/master@{#292474}
|
|
|
|
|
|
| |
Review URL: https://codereview.chromium.org/512663002
Cr-Commit-Position: refs/heads/master@{#292448}
|
|
|
|
|
|
|
|
|
|
| |
BUG=408429
TEST=none
TBR=miket
Review URL: https://codereview.chromium.org/515313002
Cr-Commit-Position: refs/heads/master@{#292444}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
webrequest api, to extensions.
This cl resolve the dependencies of ExtensionsWarningSet of chrome strings:
- move the declarations of IDS_EXTENSION_WARNINGS_XXX to extensions_strings.grd;
- replace the usage of "IDS_PRODUCT_NAME" by creating GetProductName() function in ExtensionsClient and its subclasses.
BUG=352293
Review URL: https://codereview.chromium.org/503033002
Cr-Commit-Position: refs/heads/master@{#292378}
|
|
|
|
|
|
|
|
| |
BUG=390086
Review URL: https://codereview.chromium.org/467113004
Cr-Commit-Position: refs/heads/master@{#292338}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
reformatting.
BUG=none
R= mkearney@chromium.org, smain@chromium.org, binji@chromium.org, binji@google.com, kalman@chromium.org
TEST=none
NOTRY=true
(documentation only change)
Review URL: https://codereview.chromium.org/483163002
Cr-Commit-Position: refs/heads/master@{#292305}
|
|
|
|
|
|
|
|
| |
BUG=144491
Review URL: https://codereview.chromium.org/440813002
Cr-Commit-Position: refs/heads/master@{#292271}
|
|
|
|
|
|
|
|
| |
BUG=408276
Review URL: https://codereview.chromium.org/513973002
Cr-Commit-Position: refs/heads/master@{#292254}
|
|
|
|
|
|
|
|
| |
BUG=404572
Review URL: https://codereview.chromium.org/508783002
Cr-Commit-Position: refs/heads/master@{#292236}
|
|
|
|
|
|
|
|
|
|
|
| |
Since the code from the ledger* internal repos has now been open-sourced, we should get rid of this code from Chrome.
R=isherman@chromium.org, kalman@chromium.org, thakis@chromium.org
BUG=None.
Review URL: https://codereview.chromium.org/510893003
Cr-Commit-Position: refs/heads/master@{#292231}
|
|
|
|
|
|
|
|
| |
BUG=405226
Review URL: https://codereview.chromium.org/509143002
Cr-Commit-Position: refs/heads/master@{#292230}
|
|
|
|
|
|
|
|
| |
BUG=407833
Review URL: https://codereview.chromium.org/512543002
Cr-Commit-Position: refs/heads/master@{#292224}
|
|
|
|
|
|
|
|
|
| |
BUG=400760
NOTRY=True
Review URL: https://codereview.chromium.org/453713002
Cr-Commit-Position: refs/heads/master@{#292223}
|
|
|
|
|
|
|
|
|
| |
BUG=305280
NOTRY=True
Review URL: https://codereview.chromium.org/512453002
Cr-Commit-Position: refs/heads/master@{#292212}
|
|
|
|
|
|
|
|
| |
BUG=396340
Review URL: https://codereview.chromium.org/447783003
Cr-Commit-Position: refs/heads/master@{#292187}
|
|
|
|
|
|
|
|
| |
BUG=407693
Review URL: https://codereview.chromium.org/508743002
Cr-Commit-Position: refs/heads/master@{#292185}
|