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

Side by Side Diff: chrome/browser/resources/settings/downloads_page/downloads_page.js

Issue 988783002: Add an initial implementation of the downloads MD settings page. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Respond to comments. Created 5 years, 9 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 /**
6 * @fileoverview
7 * 'cr-settings-downloads-page' is the settings page containing all downloads
8 * settings.
9 *
10 * Example:
11 *
12 * <core-animated-pages>
13 * <cr-settings-downloads-page></cr-settings-downloads-page>
14 * ... other pages ...
15 * </core-animated-pages>
16 *
17 * @group Chrome Settings Elements
18 * @element cr-settings-downloads-page
19 */
20 Polymer('cr-settings-downloads-page', {
21 publish: {
22 /**
23 * Preferences state.
24 *
25 * @attribute prefs
26 * @type CrSettingsPrefsElement
27 * @default null
28 */
29 prefs: null,
30 },
31
32 selectDownloadLocation: function() {
33 // TODO(orenb): Communicate with the C++ to actually display a folder
34 // picker.
35 this.$.downloadsPath.value = '/Downloads';
36 },
37
38 updatePromptForDownload: function() {
39 this.prefs.settings.download.promptForDownload =
40 this.$.promptForDownload.checked;
michaelpg 2015/03/09 21:24:08 hmm I'm a little confused by this. It doesn't look
Oren Blasberg 2015/03/09 21:39:13 Oops -- yes, you're right. I'd added this method i
41 },
42 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698