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

Side by Side Diff: chrome/browser/resources/uber/uber_frame.js

Issue 18593003: Disable mouse-focus of buttons in some WebUI pages (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Add a public function to FocusManager Created 7 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 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 // This file contains the navigation controls that are visible on the left side 5 // This file contains the navigation controls that are visible on the left side
6 // of the uber page. It exists separately from uber.js so that it may be loaded 6 // of the uber page. It exists separately from uber.js so that it may be loaded
7 // in an iframe. Iframes can be layered on top of each other, but not mixed in 7 // in an iframe. Iframes can be layered on top of each other, but not mixed in
8 // with page content, so all overlapping content on uber must be framed. 8 // with page content, so all overlapping content on uber must be framed.
9 9
10 <include src="../../../../ui/webui/resources/js/util.js"></include> 10 <include src="../../../../ui/webui/resources/js/util.js"></include>
11 <include src="uber_utils.js"></include> 11 <include src="uber_utils.js"></include>
12 12
13 cr.define('uber_frame', function() { 13 cr.define('uber_frame', function() {
14 14
15 /** 15 /**
16 * Handles page initialization. 16 * Handles page initialization.
17 */ 17 */
18 function onLoad() { 18 function onLoad() {
19 var navigationItems = document.querySelectorAll('li'); 19 var navigationItems = document.querySelectorAll('li');
20 20
21 for (var i = 0; i < navigationItems.length; ++i) { 21 for (var i = 0; i < navigationItems.length; ++i) {
22 navigationItems[i].addEventListener('click', onNavItemClicked); 22 navigationItems[i].addEventListener('click', onNavItemClicked);
23 } 23 }
24 24
25 window.addEventListener('message', handleWindowMessage); 25 window.addEventListener('message', handleWindowMessage);
26 uber.invokeMethodOnParent('navigationControlsLoaded'); 26 uber.invokeMethodOnParent('navigationControlsLoaded');
27 27
28 document.documentElement.addEventListener('mousewheel', onMouseWheel); 28 document.documentElement.addEventListener('mousewheel', onMouseWheel);
29 cr.ui.FocusManager.disableMouseFocusOnButtons();
29 } 30 }
30 31
31 /** 32 /**
32 * Handles clicks on the navigation controls (switches the page and updates 33 * Handles clicks on the navigation controls (switches the page and updates
33 * the URL). 34 * the URL).
34 * @param {Event} e The click event. 35 * @param {Event} e The click event.
35 */ 36 */
36 function onNavItemClicked(e) { 37 function onNavItemClicked(e) {
37 // Though pointer-event: none; is applied to the .selected nav item, users 38 // Though pointer-event: none; is applied to the .selected nav item, users
38 // can still tab to them and press enter/space which simulates a click. 39 // can still tab to them and press enter/space which simulates a click.
(...skipping 106 matching lines...) Expand 10 before | Expand all | Expand 10 after
145 } 146 }
146 147
147 return { 148 return {
148 onLoad: onLoad, 149 onLoad: onLoad,
149 setNavigationOverride: setNavigationOverride, 150 setNavigationOverride: setNavigationOverride,
150 }; 151 };
151 152
152 }); 153 });
153 154
154 document.addEventListener('DOMContentLoaded', uber_frame.onLoad); 155 document.addEventListener('DOMContentLoaded', uber_frame.onLoad);
OLDNEW
« no previous file with comments | « chrome/browser/resources/uber/uber_frame.html ('k') | ui/webui/resources/js/cr/ui/focus_manager.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698