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

Unified Diff: chrome/browser/resources/settings/internet_page/network_proxy_exclusions.js

Issue 1219753007: Add network proxy input to md-settings internet details (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@issue_470445_onc_proxy
Patch Set: . Created 5 years, 5 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: chrome/browser/resources/settings/internet_page/network_proxy_exclusions.js
diff --git a/chrome/browser/resources/settings/internet_page/network_proxy_exclusions.js b/chrome/browser/resources/settings/internet_page/network_proxy_exclusions.js
new file mode 100644
index 0000000000000000000000000000000000000000..c420935467f8759fe2e1dcce103d3a44befb86cb
--- /dev/null
+++ b/chrome/browser/resources/settings/internet_page/network_proxy_exclusions.js
@@ -0,0 +1,42 @@
+// Copyright 2015 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+/**
+ * @fileoverview Polymer element for displaying a list of proxy exclusions.
+ * Includes UI for adding, changing, and removing entries.
+ */
+
+(function() {
+
+Polymer({
+ is: 'network-proxy-exclusions',
+
+ properties: {
+ /**
+ * The list of exclusions.
+ * @type {!Array<string>}
+ */
+ exclusions: {
+ type: Array,
+ value: function() { return []; },
+ notify: true,
+ observer: 'exclusionsChanged_'
+ }
+ },
+
+ exclusionsChanged_: function() {
+ console.log('exclusions: ' + this.exclusions);
+ },
+
+ /**
+ * Event triggered when an item is removed.
+ * @private
+ */
+ removeItem_: function(event) {
+ var index = event.model.index;
+ console.debug('network-proxy-exclusions: removed: ' + index);
+ this.fire('removed', index);
michaelpg 2015/07/10 00:38:30 Can you do the splice here instead?
stevenjb 2015/07/10 17:31:59 Done.
+ }
+});
+})();

Powered by Google App Engine
This is Rietveld 408576698