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

Issue 8818006: Revert 113152 - [filebrowser] Add left panel with roots. (Closed)

Created:
9 years ago by Jói
Modified:
9 years ago
Reviewers:
dgozman
CC:
chromium-reviews, jstritar+watch_chromium.org, achuith+watch_chromium.org, mihaip+watch_chromium.org, Aaron Boodman, rginda+watch_chromium.org, arv (Not doing code reviews)
Visibility:
Public.

Description

Revert 113152 - [filebrowser] Add left panel with roots. Not the final UI yet. Additional improvements: - file name is selected in save-as dialog at start; - new folder moved to context menu, button deleted. BUG=chromium-os:20168, chromium-os:22106, chromium-os:22105, chromium-os:22032, chromium-os:20547, chromium-os:20549 TEST=Manual Review URL: http://codereview.chromium.org/8554003 Reason for revert: http://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromeOS%20Aura/builds/1508 TBR=dgozman@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=113176

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+302 lines, -630 lines) Patch
M chrome/app/generated_resources.grd View 1 chunk +3 lines, -0 lines 0 comments Download
MM chrome/browser/extensions/extension_file_browser_private_api.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/resources/component_extension_resources.grd View 1 chunk +0 lines, -6 lines 0 comments Download
M chrome/browser/resources/file_manager/css/file_manager.css View 5 chunks +11 lines, -172 lines 0 comments Download
M chrome/browser/resources/file_manager/js/file_manager.js View 35 chunks +233 lines, -391 lines 0 comments Download
M chrome/browser/resources/file_manager/js/harness.js View 1 chunk +1 line, -5 lines 0 comments Download
M chrome/browser/resources/file_manager/js/mock_chrome.js View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/resources/file_manager/main.html View 1 chunk +44 lines, -54 lines 0 comments Download
M chrome/browser/resources/file_manager/manifest.json View 1 chunk +8 lines, -0 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
Jói
9 years ago (2011-12-06 10:15:57 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698