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

Unified Diff: composer.postload.js

Issue 29452181: Noissue - Merge current tip to Edge bookmark (Closed)
Patch Set: Created May 30, 2017, 3:49 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 | « composer.js ('k') | dependencies » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: composer.postload.js
===================================================================
--- a/composer.postload.js
+++ b/composer.postload.js
@@ -1,6 +1,6 @@
/*
* This file is part of Adblock Plus <https://adblockplus.org/>,
- * Copyright (C) 2006-2016 Eyeo GmbH
+ * Copyright (C) 2006-2017 eyeo GmbH
*
* Adblock Plus is free software: you can redistribute it and/or modify
* it under the terms of the GNU General Public License version 3 as
@@ -15,6 +15,8 @@
* along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>.
*/
+/* globals checkCollapse, elemhide, getURLsFromElement, typeMap */
+
"use strict";
// The page ID for the popup filter selection dialog (top frame only).
@@ -43,8 +45,7 @@
function getFiltersForElement(element, callback)
{
let src = element.getAttribute("src");
- ext.backgroundPage.sendMessage(
- {
+ ext.backgroundPage.sendMessage({
type: "composer.getFilters",
tagName: element.localName,
id: element.id,
@@ -52,7 +53,7 @@
style: element.getAttribute("style"),
classes: Array.prototype.slice.call(element.classList),
urls: getURLsFromElement(element),
- mediatype: typeMap[element.localName],
+ mediatype: typeMap.get(element.localName),
baseURL: document.location.href
},
response =>
@@ -388,21 +389,15 @@
if (currentlyPickingElement)
stopPickingElement();
- ext.backgroundPage.sendMessage(
- {
+ ext.backgroundPage.sendMessage({
type: "composer.openDialog"
},
popupId =>
{
- ext.backgroundPage.sendMessage(
- {
+ ext.backgroundPage.sendMessage({
type: "forward",
targetPageId: popupId,
- payload:
- {
- type: "composer.dialog.init",
- filters: filters
- }
+ payload: {type: "composer.dialog.init", filters}
});
// Only the top frame keeps a record of the popup window's ID,
@@ -413,14 +408,9 @@
}
else
{
- ext.backgroundPage.sendMessage(
- {
+ ext.backgroundPage.sendMessage({
type: "forward",
- payload:
- {
- type: "composer.content.dialogOpened",
- popupId: popupId
- }
+ payload: {type: "composer.content.dialogOpened", popupId}
});
}
});
@@ -461,8 +451,7 @@
if (blockelementPopupId != null)
{
- ext.backgroundPage.sendMessage(
- {
+ ext.backgroundPage.sendMessage({
type: "forward",
targetPageId: blockelementPopupId,
payload:
@@ -504,8 +493,7 @@
lastRightClickEvent = event;
lastRightClickEventIsMostRecent = true;
- ext.backgroundPage.sendMessage(
- {
+ ext.backgroundPage.sendMessage({
type: "forward",
payload:
{
@@ -520,9 +508,11 @@
{
case "composer.content.getState":
if (window == window.top)
+ {
sendResponse({
active: currentlyPickingElement || blockelementPopupId != null
});
+ }
break;
case "composer.content.startPickingElement":
if (window == window.top)
@@ -572,8 +562,7 @@
// to be careful here. (This is not perfect, but best we can do.)
if (window == window.top && blockelementPopupId == msg.popupId)
{
- ext.backgroundPage.sendMessage(
- {
+ ext.backgroundPage.sendMessage({
type: "forward",
payload:
{
« no previous file with comments | « composer.js ('k') | dependencies » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld