Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: ext/background.js

Issue 29531677: Issue 5599 - ext needs to be defined in the postload script (Closed) Base URL: https://hg.adblockplus.org/adblockpluschrome/
Patch Set: New approach as discussed in the bug Created Sept. 1, 2017, 8:37 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | lib/composer.js » ('j') | metadata.chrome » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ext/background.js
===================================================================
--- a/ext/background.js
+++ b/ext/background.js
@@ -741,17 +741,17 @@
// https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/10276332/
let optionsUrl = "options.html";
let fullOptionsUrl = ext.getURL(optionsUrl);
chrome.tabs.query({}, tabs =>
{
// We find a tab ourselves because Edge has a bug when quering tabs
// with extension URL protocol:
- // https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/8094141/
+ // https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/8094141/
hub 2017/09/01 20:41:36 only here to pass eslint (trailing space)
// https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/8604703/
let tab = tabs.find(element => element.url == fullOptionsUrl);
if (tab)
{
chrome.windows.update(tab.windowId, {focused: true});
chrome.tabs.update(tab.id, {active: true});
if (callback)
« no previous file with comments | « no previous file | lib/composer.js » ('j') | metadata.chrome » ('J')

Powered by Google App Engine
This is Rietveld