[ChromeOS] Implement collected cookies in webui.

chrome/browser/resources/collected_cookies.js

+// Copyright (c) 2011 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.
+
+function closeDialog() {
+  chrome.send('DialogClose', ['']);
+}
+
+function setInfobarLabel(text) {
+  $('info-banner').textContent = text;
+  $('info-banner').hidden = !text.length;
+}
+
+function updateControlState() {
+  var allowedCookies = $('allowed-cookies');
+  $('block-button').disabled = !allowedCookies.children.length ||
+      !allowedCookies.selectedItem ||
+      allowedCookies.selectedItem.data.type != 'origin';
+
+  var blockedCookies = $('blocked-cookies');
+  $('allow-button').disabled =
+  $('allow-this-session-button').disabled = !blockedCookies.children.length ||
+      !blockedCookies.selectedItem ||
+      blockedCookies.selectedItem.data.type != 'origin';;

[arv (Not doing code reviews), 2011/03/10 18:03:00]

too many semicolons

[xiyuan, 2011/03/10 19:23:59]

Done.

+}
+
+function handleCookiesTreeChange(e) {
+  updateControlState();
+}
+
+function handleBlockButtonClick(e) {
+  var selected = $('allowed-cookies').selectedItem;
+  if (!selected)

[arv (Not doing code reviews), 2011/03/10 18:03:00]

negate and remove return?

[xiyuan, 2011/03/10 19:23:59]

Done.

+    return;
+
+  chrome.send('Block', [selected.pathId]);
+}
+
+function handleAllowButtonClick(e) {
+  var selected = $('blocked-cookies').selectedItem;
+  if (!selected)
+    return;
+
+  chrome.send('Allow', [selected.pathId]);
+}
+
+function handleAllowThisSessionButtonClick(e) {
+  var selected = $('blocked-cookies').selectedItem;
+  if (!selected)
+    return;
+
+  chrome.send('AllowThisSession', [selected.pathId]);
+}
+
+window.onkeydown = function(e) {

[arv (Not doing code reviews), 2011/03/10 18:03:00]

document.onkeydown

[xiyuan, 2011/03/10 19:23:59]

Done. And moved it into load().

+  if (e.keyCode == 27)  // Esc
+    closeDialog();
+}
+
+function load() {
+  ui.CookiesTree.decorate($('allowed-cookies'));
+  ui.CookiesTree.decorate($('blocked-cookies'));
+
+  chrome.send('BindCookiesTreeModel', []);
+
+  $('allowed-cookies').addEventListener('change', handleCookiesTreeChange);
+  $('blocked-cookies').addEventListener('change', handleCookiesTreeChange);
+
+  $('block-button').addEventListener('click', handleBlockButtonClick);
+  $('allow-button').addEventListener('click', handleAllowButtonClick);
+  $('allow-this-session-button').addEventListener('click',
+      handleAllowThisSessionButtonClick);
+
+  $('close-button').addEventListener('click', function(e) {

[arv (Not doing code reviews), 2011/03/10 18:03:00]

$('close-button').addEventListener('click', closeDialog);

[xiyuan, 2011/03/10 19:23:59]

Done.

+    closeDialog();
+  });
+
+  cr.defineProperty($('info-banner'), 'hidden', cr.PropertyKind.BOOL_ATTR);

[arv (Not doing code reviews), 2011/03/10 18:03:00]

You don't need this line. hidden is already available on all elements

[xiyuan, 2011/03/10 19:23:59]

Removed.

+
+  updateControlState();
+}
+
+document.addEventListener('DOMContentLoaded', load);