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

Unified Diff: ui/file_manager/file_manager/foreground/js/ui/file_table.js

Issue 657253004: Fix suspicious code detected by closure compiler. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Reflect review comments. Created 6 years, 2 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
Index: ui/file_manager/file_manager/foreground/js/ui/file_table.js
diff --git a/ui/file_manager/file_manager/foreground/js/ui/file_table.js b/ui/file_manager/file_manager/foreground/js/ui/file_table.js
index 263ba49bdf20f840e78f45828fc66449c0943b06..13491d52e26f0cae14666b533991afffd53d1520 100644
--- a/ui/file_manager/file_manager/foreground/js/ui/file_table.js
+++ b/ui/file_manager/file_manager/foreground/js/ui/file_table.js
@@ -422,10 +422,10 @@ FileTable.prototype.fitColumn = function(index) {
* @param {boolean} use12hourClock True if 12 hours clock, False if 24 hours.
*/
FileTable.prototype.setDateTimeFormat = function(use12hourClock) {
- this.timeFormatter_ = Intl.DateTimeFormat(
+ this.timeFormatter_ = new Intl.DateTimeFormat(
[] /* default locale */,
{hour: 'numeric', minute: 'numeric', hour12: use12hourClock});
- this.dateFormatter_ = Intl.DateTimeFormat(
+ this.dateFormatter_ = new Intl.DateTimeFormat(
[] /* default locale */,
{
year: 'numeric', month: 'short', day: 'numeric',

Powered by Google App Engine
This is Rietveld 408576698