Add AccessibilityController class, which dynamically registers (#2024)

and unregisters the google-docs.js content script
This commit is contained in:
toasted-nutbread 2021-11-23 22:23:14 -05:00 committed by GitHub
parent d454b52a18
commit f2102ade4c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
7 changed files with 104 additions and 24 deletions

View File

@ -182,6 +182,7 @@
"files": [ "files": [
"ext/js/core.js", "ext/js/core.js",
"ext/js/yomichan.js", "ext/js/yomichan.js",
"ext/js/accessibility/accessibility-controller.js",
"ext/js/background/backend.js", "ext/js/background/backend.js",
"ext/js/background/environment.js", "ext/js/background/environment.js",
"ext/js/background/profile-conditions-util.js", "ext/js/background/profile-conditions-util.js",

View File

@ -63,18 +63,6 @@
"js/script/dynamic-loader.js", "js/script/dynamic-loader.js",
"js/app/content-script-main.js" "js/app/content-script-main.js"
] ]
},
{
"run_at": "document_start",
"matches": [
"http://docs.google.com/*",
"https://docs.google.com/*"
],
"match_about_blank": true,
"all_frames": true,
"js": [
"js/accessibility/google-docs.js"
]
} }
], ],
"minimum_chrome_version": "57.0.0.0", "minimum_chrome_version": "57.0.0.0",

View File

@ -23,6 +23,7 @@
<script src="/js/yomichan.js"></script> <script src="/js/yomichan.js"></script>
<script src="/js/accessibility/accessibility-controller.js"></script>
<script src="/js/background/backend.js"></script> <script src="/js/background/backend.js"></script>
<script src="/js/background/environment.js"></script> <script src="/js/background/environment.js"></script>
<script src="/js/background/profile-conditions-util.js"></script> <script src="/js/background/profile-conditions-util.js"></script>

View File

@ -0,0 +1,97 @@
/*
* Copyright (C) 2021 Yomichan Authors
*
* This program is free software: you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by
* the Free Software Foundation, either version 3 of the License, or
* (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with this program. If not, see <https://www.gnu.org/licenses/>.
*/
/**
* This class controls the registration of accessibility handlers.
*/
class AccessibilityController {
/**
* Creates a new instance.
* @param {ScriptManager} scriptManager An instance of the `ScriptManager` class.
*/
constructor(scriptManager) {
this._scriptManager = scriptManager;
this._updateGoogleDocsAccessibilityToken = null;
this._updateGoogleDocsAccessibilityPromise = null;
this._forceGoogleDocsHtmlRenderingAny = false;
}
/**
* Updates the accessibility handlers.
* @param {object} fullOptions The full options object from the `Backend` instance.
* The value is treated as read-only and is not modified.
*/
async update(fullOptions) {
let forceGoogleDocsHtmlRenderingAny = false;
for (const {options} of fullOptions.profiles) {
if (options.accessibility.forceGoogleDocsHtmlRendering) {
forceGoogleDocsHtmlRenderingAny = true;
break;
}
}
await this._updateGoogleDocsAccessibility(forceGoogleDocsHtmlRenderingAny);
}
// Private
async _updateGoogleDocsAccessibility(forceGoogleDocsHtmlRenderingAny) {
// Reentrant token
const token = {};
this._updateGoogleDocsAccessibilityToken = token;
// Wait for previous
let promise = this._updateGoogleDocsAccessibilityPromise;
if (promise !== null) { await promise; }
// Reentrant check
if (this._updateGoogleDocsAccessibilityToken !== token) { return; }
// Update
promise = this._updateGoogleDocsAccessibilityInner(forceGoogleDocsHtmlRenderingAny);
this._updateGoogleDocsAccessibilityPromise = promise;
await promise;
this._updateGoogleDocsAccessibilityPromise = null;
}
async _updateGoogleDocsAccessibilityInner(forceGoogleDocsHtmlRenderingAny) {
if (this._forceGoogleDocsHtmlRenderingAny === forceGoogleDocsHtmlRenderingAny) { return; }
this._forceGoogleDocsHtmlRenderingAny = forceGoogleDocsHtmlRenderingAny;
const id = 'googleDocsAccessibility';
try {
if (forceGoogleDocsHtmlRenderingAny) {
if (await this._scriptManager.isContentScriptRegistered(id)) { return; }
const details = {
allFrames: true,
matchAboutBlank: true,
matches: ['*://docs.google.com/*'],
urlMatches: '^[^:]*://docs.google.com/[\\w\\W]*$',
runAt: 'document_start',
js: ['js/accessibility/google-docs.js']
};
await this._scriptManager.registerContentScript(id, details);
} else {
await this._scriptManager.unregisterContentScript(id);
}
} catch (e) {
log.error(e);
}
}
}

View File

@ -16,6 +16,7 @@
*/ */
/* global /* global
* AccessibilityController
* AnkiConnect * AnkiConnect
* AnkiUtil * AnkiUtil
* AudioDownloader * AudioDownloader
@ -69,6 +70,7 @@ class Backend {
}); });
this._optionsUtil = new OptionsUtil(); this._optionsUtil = new OptionsUtil();
this._scriptManager = new ScriptManager(); this._scriptManager = new ScriptManager();
this._accessibilityController = new AccessibilityController(this._scriptManager);
this._searchPopupTabId = null; this._searchPopupTabId = null;
this._searchPopupTabCreatePromise = null; this._searchPopupTabCreatePromise = null;
@ -988,6 +990,8 @@ class Backend {
this._clipboardMonitor.stop(); this._clipboardMonitor.stop();
} }
this._accessibilityController.update(this._getOptionsFull(false));
this._sendMessageAllTabsIgnoreResponse('Yomichan.optionsUpdated', {source}); this._sendMessageAllTabsIgnoreResponse('Yomichan.optionsUpdated', {source});
} }

View File

@ -62,18 +62,6 @@
"js/script/dynamic-loader.js", "js/script/dynamic-loader.js",
"js/app/content-script-main.js" "js/app/content-script-main.js"
] ]
},
{
"run_at": "document_start",
"matches": [
"http://docs.google.com/*",
"https://docs.google.com/*"
],
"match_about_blank": true,
"all_frames": true,
"js": [
"js/accessibility/google-docs.js"
]
} }
], ],
"minimum_chrome_version": "57.0.0.0", "minimum_chrome_version": "57.0.0.0",

View File

@ -20,6 +20,7 @@ self.importScripts(
'/lib/wanakana.min.js', '/lib/wanakana.min.js',
'/js/core.js', '/js/core.js',
'/js/yomichan.js', '/js/yomichan.js',
'/js/accessibility/accessibility-controller.js',
'/js/background/backend.js', '/js/background/backend.js',
'/js/background/environment.js', '/js/background/environment.js',
'/js/background/profile-conditions-util.js', '/js/background/profile-conditions-util.js',