Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2)

Issue 22361002: Add the code to delete the unused folder-shortcut data (Closed)

Created:
7 years, 4 months ago by yoshiki
Modified:
7 years, 4 months ago
Reviewers:
mtomasz
CC:
chromium-reviews, rginda+watch_chromium.org, arv+watch_chromium.org
Visibility:
Public.

Description

Add the code to delete the unused folder-shortcut data The data in chrome.storage.local had been used but is now no longer used. This patch adds the code to remove them, in order to save the limited user storage. BUG=268784 TEST=Files.app launches without error in console. R=mtomasz@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=215856

Patch Set 1 #

Patch Set 2 : . #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -0 lines) Patch
M chrome/browser/resources/file_manager/js/file_manager.js View 1 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
yoshiki
@mtomasz: PTAL. Thanks.
7 years, 4 months ago (2013-08-06 05:15:39 UTC) #1
mtomasz
lgtm. Please make sure that it doesn't throw an error if the item is not ...
7 years, 4 months ago (2013-08-06 05:19:50 UTC) #2
yoshiki
On 2013/08/06 05:19:50, mtomasz wrote: > lgtm. Please make sure that it doesn't throw an ...
7 years, 4 months ago (2013-08-06 05:25:37 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/yoshiki@chromium.org/22361002/3001
7 years, 4 months ago (2013-08-06 05:26:44 UTC) #4
yoshiki
7 years, 4 months ago (2013-08-06 08:38:19 UTC) #5
Message was sent while issue was closed.
Committed patchset #2 manually as r215856 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698