Commit 93c4e7f3 authored by Erik Nordin's avatar Erik Nordin Committed by enordin@mozilla.com
Browse files

Bug 1820252 - Misc. TranslationsDocument Renames and Docs r=translations-reviewers,gregtatum 

This patch contains a collection of miscellaneous variable
renames and documentation comment updates that will lessen
the cognitive load on subsequent patches.

Differential Revision: https://phabricator.services.mozilla.com/D249968
parent ff944789

toolkit/components/translations/content/translations-document.sys.mjs

+424 −272

File changed.

Preview size limit exceeded, changes collapsed.

toolkit/components/translations/content/translations-engine.sys.mjs

+1 −1
Changes for toolkit/components/translations/content/translations-engine.sys.mjs: 1 added line, 1 removed line.
Original line number Diff line number Diff line
@@ -17,7 +17,7 @@ 
 * creating new engines and MessagePorts on the fly.

 *

 * The engine communicates directly with the content page via a MessagePort. Each end

 * of the port is transfered from the parent process to the content process, and this

 * of the port is transferred from the parent process to the content process, and this

 * engine process. This port is transitory, and may be closed at any time. Only when a

 * translation has been requested once (which is initiated by the parent process) can

 * the content process re-request translation ports. This ensures a rogue content process

toolkit/components/translations/tests/browser/browser_translations_full_page.js

+3 −1
Changes for toolkit/components/translations/tests/browser/browser_translations_full_page.js: 3 added lines, 1 removed line.
Original line number Diff line number Diff line
@@ -93,7 +93,9 @@ add_task(async function test_about_translations_enabled() { 
});



/**

 * Check that the full page translation feature works.

 * This test case ensures that the language is correctly identified as Spanish

 * when loading the NO_LANGUAGE_URL, and the page is translated just the same

 * as though it had a language tag specified in the markup.

 */

add_task(async function test_language_identification_for_page_translation() {

  await autoTranslatePage({