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

Unified Diff: chrome/browser/resources/bluetooth_internals/sidebar.js

Issue 2568283003: bluetooth: Add notification system to internals page. (Closed)
Patch Set: Change handleInspect, danger->error, fix text color Created 4 years 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/bluetooth_internals/sidebar.js
diff --git a/chrome/browser/resources/bluetooth_internals/sidebar.js b/chrome/browser/resources/bluetooth_internals/sidebar.js
index 81fcb2ee368ca53f90d27d460c80b8a544f59671..f060e713979238a4cd899271606dd828771254f6 100644
--- a/chrome/browser/resources/bluetooth_internals/sidebar.js
+++ b/chrome/browser/resources/bluetooth_internals/sidebar.js
@@ -45,6 +45,7 @@ cr.define('sidebar', function() {
close: function() {
this.sidebarDiv_.classList.remove('open');
document.body.style.overflow = '';
+ document.dispatchEvent(new CustomEvent('contentfocus'));
Dan Beam 2016/12/16 08:42:58 why is this not named something like 'sidebar-clos
mbrunson 2016/12/16 21:42:30 The main purpose of these 2 events (contentfocus/c
},
/**
@@ -53,6 +54,7 @@ cr.define('sidebar', function() {
open: function() {
document.body.style.overflow = 'hidden';
this.sidebarDiv_.classList.add('open');
+ document.dispatchEvent(new CustomEvent('contentblur'));
},
/**

Powered by Google App Engine
This is Rietveld 408576698