chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js - Issue 2617633004: Fix AccessibilityExtensionLoader's handling of locked screen

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js

Issue 2617633004: Fix AccessibilityExtensionLoader's handling of locked screen (Closed)

Patch Set: Test removals. Created 3 years, 11 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 side-by-side diff with in-line comments

Download patch

« no previous file with comments | « chrome/browser/guest_view/web_view/chrome_web_view_guest_delegate.cc ('k') | chrome/browser/resources/chromeos/chromevox/cvox2/background/keyboard_handler.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js

diff --git a/chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js b/chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js

index b1f96568adefb43fa50001d7eaf87e2d2f88473a..8c0c9d5038661c83cafecf629a86644f5abba24d 100644

--- a/chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js

+++ b/chrome/browser/resources/chromeos/chromevox/cvox2/background/background.js

@@ -88,12 +88,6 @@ Background = function() {

this.currentRange_ = null;

- /**

- * @type {cursors.Range}

- * @private

- */

- this.savedRange_ = null;

// Manually bind all functions to |this|.

for (var func in this) {

if (typeof(this[func]) == 'function')

@@ -156,9 +150,6 @@ Background = function() {

/** @type {!LiveRegions} @private */

this.liveRegions_ = new LiveRegions(this);

- /** @type {boolean} @private */

- this.inExcursion_ = false;

/**

* Stores the mode as computed the last time a current range was set.

* @type {?ChromeVoxMode}

@@ -397,9 +388,6 @@ Background.prototype = {

* @override

setCurrentRange: function(newRange) {

- if (!this.inExcursion_ && newRange)

- this.savedRange_ = new cursors.Range(newRange.start, newRange.end);

if (newRange && !newRange.isValid())

return;