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

Unified Diff: chrome/browser/resources/file_manager/js/file_manager.js

Issue 7658001: file manager: remove download folder warning. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/file_manager/js/file_manager.js
diff --git a/chrome/browser/resources/file_manager/js/file_manager.js b/chrome/browser/resources/file_manager/js/file_manager.js
index b82846bff1dfb3284ee6131110fab1dc13251743..09803f11800e4baf53dafb8800c03f445a7ac483 100644
--- a/chrome/browser/resources/file_manager/js/file_manager.js
+++ b/chrome/browser/resources/file_manager/js/file_manager.js
@@ -183,11 +183,6 @@ FileManager.prototype = {
const DOWNLOADS_DIRECTORY = '/Downloads';
/**
- * Height of the downloads folder warning, in px.
- */
- const DOWNLOADS_WARNING_HEIGHT = '57px';
-
- /**
* Location of the FAQ about the downloads directory.
*/
const DOWNLOADS_FAQ_URL = 'http://www.google.com/support/chromeos/bin/' +
@@ -576,15 +571,6 @@ FileManager.prototype = {
this.copyButton_ = this.dialogDom_.querySelector('.clipboard-copy');
this.pasteButton_ = this.dialogDom_.querySelector('.clipboard-paste');
- this.downloadsWarning_ =
- this.dialogDom_.querySelector('.downloads-warning');
- var html = util.htmlUnescape(strf('DOWNLOADS_DIRECTORY_WARNING',
- DOWNLOADS_FAQ_URL));
- // TODO(rginda): Fix this string in the grd file to include the target
- // attribute, post R14.
- html = html.replace('<a href=', '<a target=new_ href=');
- this.downloadsWarning_.lastElementChild.innerHTML = html;
-
this.document_.addEventListener('keydown', this.onKeyDown_.bind(this));
this.descriptionTable_ =
@@ -2507,21 +2493,6 @@ FileManager.prototype = {
* @param {cr.Event} event The directory-changed event.
*/
FileManager.prototype.onDirectoryChanged_ = function(event) {
- if (this.dialogType_ == FileManager.DialogType.FULL_PAGE &&
- this.currentDirEntry_.fullPath.substr(0, DOWNLOADS_DIRECTORY.length) ==
- DOWNLOADS_DIRECTORY) {
- if (this.downloadsWarning_.style.height != DOWNLOADS_WARNING_HEIGHT) {
- // Current path starts with DOWNLOADS_DIRECTORY, show the warning.
- this.downloadsWarning_.style.height = DOWNLOADS_WARNING_HEIGHT;
- this.requestResize_(100);
- }
- } else {
- if (this.downloadsWarning_.style.height != '0') {
- this.downloadsWarning_.style.height = '0';
- this.requestResize_(100);
- }
- }
-
this.updateCommands_();
this.updateOkButton_();
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698