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

Side by Side Diff: chrome/browser/resources/options/cookies_view.js

Issue 410293004: Split OptionsPage into Page and PageManager (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed feedback Created 6 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 cr.define('options', function() { 5 cr.define('options', function() {
6 6
7 var OptionsPage = options.OptionsPage; 7 var Page = cr.ui.pageManager.Page;
8 var PageManager = cr.ui.pageManager.PageManager;
8 9
9 ///////////////////////////////////////////////////////////////////////////// 10 /////////////////////////////////////////////////////////////////////////////
10 // CookiesView class: 11 // CookiesView class:
11 12
12 /** 13 /**
13 * Encapsulated handling of the cookies and other site data page. 14 * Encapsulated handling of the cookies and other site data page.
14 * @constructor 15 * @constructor
15 */ 16 */
16 function CookiesView(model) { 17 function CookiesView(model) {
17 OptionsPage.call(this, 'cookies', 18 Page.call(this, 'cookies',
18 loadTimeData.getString('cookiesViewPageTabTitle'), 19 loadTimeData.getString('cookiesViewPageTabTitle'),
19 'cookies-view-page'); 20 'cookies-view-page');
20 } 21 }
21 22
22 cr.addSingletonGetter(CookiesView); 23 cr.addSingletonGetter(CookiesView);
23 24
24 CookiesView.prototype = { 25 CookiesView.prototype = {
25 __proto__: OptionsPage.prototype, 26 __proto__: Page.prototype,
26 27
27 /** 28 /**
28 * The timer id of the timer set on search query change events. 29 * The timer id of the timer set on search query change events.
29 * @type {number} 30 * @type {number}
30 * @private 31 * @private
31 */ 32 */
32 queryDelayTimerId_: 0, 33 queryDelayTimerId_: 0,
33 34
34 /** 35 /**
35 * The most recent search query, empty string if the query is empty. 36 * The most recent search query, empty string if the query is empty.
36 * @type {string} 37 * @type {string}
37 * @private 38 * @private
38 */ 39 */
39 lastQuery_: '', 40 lastQuery_: '',
40 41
41 initializePage: function() { 42 initializePage: function() {
42 OptionsPage.prototype.initializePage.call(this); 43 Page.prototype.initializePage.call(this);
43 44
44 var searchBox = this.pageDiv.querySelector('.cookies-search-box'); 45 var searchBox = this.pageDiv.querySelector('.cookies-search-box');
45 searchBox.addEventListener( 46 searchBox.addEventListener(
46 'search', this.handleSearchQueryChange_.bind(this)); 47 'search', this.handleSearchQueryChange_.bind(this));
47 searchBox.onkeydown = function(e) { 48 searchBox.onkeydown = function(e) {
48 // Prevent the overlay from handling this event. 49 // Prevent the overlay from handling this event.
49 if (e.keyIdentifier == 'Enter') 50 if (e.keyIdentifier == 'Enter')
50 e.stopPropagation(); 51 e.stopPropagation();
51 }; 52 };
52 53
53 this.pageDiv.querySelector('.remove-all-cookies-button').onclick = 54 this.pageDiv.querySelector('.remove-all-cookies-button').onclick =
54 function(e) { 55 function(e) {
55 chrome.send('removeAllCookies'); 56 chrome.send('removeAllCookies');
56 }; 57 };
57 58
58 var cookiesList = this.pageDiv.querySelector('.cookies-list'); 59 var cookiesList = this.pageDiv.querySelector('.cookies-list');
59 options.CookiesList.decorate(cookiesList); 60 options.CookiesList.decorate(cookiesList);
60 61
61 this.addEventListener('visibleChange', this.handleVisibleChange_); 62 this.addEventListener('visibleChange', this.handleVisibleChange_);
62 63
63 this.pageDiv.querySelector('.cookies-view-overlay-confirm').onclick = 64 this.pageDiv.querySelector('.cookies-view-overlay-confirm').onclick =
64 OptionsPage.closeOverlay.bind(OptionsPage); 65 PageManager.closeOverlay.bind(PageManager);
65 }, 66 },
66 67
67 /** @override */ 68 /** @override */
68 didShowPage: function() { 69 didShowPage: function() {
69 this.pageDiv.querySelector('.cookies-search-box').value = ''; 70 this.pageDiv.querySelector('.cookies-search-box').value = '';
70 this.lastQuery_ = ''; 71 this.lastQuery_ = '';
71 }, 72 },
72 73
73 /** 74 /**
74 * Search cookie using text in |cookies-search-box|. 75 * Search cookie using text in |cookies-search-box|.
(...skipping 20 matching lines...) Expand all
95 if (this.queryDelayTimerId_) 96 if (this.queryDelayTimerId_)
96 window.clearTimeout(this.queryDelayTimerId_); 97 window.clearTimeout(this.queryDelayTimerId_);
97 98
98 this.queryDelayTimerId_ = window.setTimeout( 99 this.queryDelayTimerId_ = window.setTimeout(
99 this.searchCookie.bind(this), 500); 100 this.searchCookie.bind(this), 500);
100 }, 101 },
101 102
102 initialized_: false, 103 initialized_: false,
103 104
104 /** 105 /**
105 * Handler for OptionsPage's visible property change event. 106 * Handler for Page's visible property change event.
106 * @param {Event} e Property change event. 107 * @param {Event} e Property change event.
107 * @private 108 * @private
108 */ 109 */
109 handleVisibleChange_: function(e) { 110 handleVisibleChange_: function(e) {
110 if (!this.visible) 111 if (!this.visible)
111 return; 112 return;
112 113
113 chrome.send('reloadCookies'); 114 chrome.send('reloadCookies');
114 115
115 if (!this.initialized_) { 116 if (!this.initialized_) {
(...skipping 19 matching lines...) Expand all
135 CookiesView.loadChildren = function(args) { 136 CookiesView.loadChildren = function(args) {
136 $('cookies-list').loadChildren(args[0], args[1]); 137 $('cookies-list').loadChildren(args[0], args[1]);
137 }; 138 };
138 139
139 // Export 140 // Export
140 return { 141 return {
141 CookiesView: CookiesView 142 CookiesView: CookiesView
142 }; 143 };
143 144
144 }); 145 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698