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

Issue 10332267: Revert 138055 - gdata: Add requestDirectoryRefresh to file_browser_private. (Closed)

Created:
8 years, 7 months ago by satorux1
Modified:
8 years, 7 months ago
Reviewers:
satorux1
CC:
chromium-reviews, mihaip-chromium-reviews_chromium.org, nkostylev+watch_chromium.org, achuith+watch_chromium.org, Aaron Boodman, rginda+watch_chromium.org, stevenjb+watch_chromium.org, davemoore+watch_chromium.org
Visibility:
Public.

Description

Revert 138055 - gdata: Add requestDirectoryRefresh to file_browser_private. Broke unit_tests on Debug build. The new API is used to fetch the metadata of files in a particular directory. More specifically, the new API is used to fix the stale thumbnail URL issue. BUG=127697 TEST=added a unit test. Review URL: https://chromiumcodereview.appspot.com/10386226 TBR=satorux@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=138056

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -188 lines) Patch
M chrome/browser/chromeos/extensions/file_browser_private_api.h View 1 chunk +0 lines, -9 lines 0 comments Download
MM chrome/browser/chromeos/extensions/file_browser_private_api.cc View 1 chunk +0 lines, -16 lines 0 comments Download
M chrome/browser/chromeos/gdata/gdata_file_system.h View 3 chunks +0 lines, -22 lines 0 comments Download
M chrome/browser/chromeos/gdata/gdata_file_system.cc View 1 chunk +0 lines, -104 lines 0 comments Download
M chrome/browser/chromeos/gdata/gdata_file_system_unittest.cc View 1 chunk +0 lines, -13 lines 0 comments Download
M chrome/browser/chromeos/gdata/gdata_files.h View 1 chunk +1 line, -3 lines 0 comments Download
M chrome/browser/chromeos/gdata/gdata_files.cc View 2 chunks +1 line, -7 lines 0 comments Download
M chrome/browser/chromeos/gdata/mock_gdata_file_system.h View 1 chunk +0 lines, -2 lines 0 comments Download
M chrome/browser/extensions/extension_function_registry.cc View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/common/extensions/api/file_browser_private.json View 1 chunk +0 lines, -11 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
satorux1
8 years, 7 months ago (2012-05-20 06:27:09 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698