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

Side by Side Diff: lib/cssProperties.js

Issue 29338928: Issue 3853 - Use new messaging API for the first-run page (Closed)
Patch Set: Using Symbol for internal property Created April 18, 2016, 3:28 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 /*
2 * This file is part of Adblock Plus <https://adblockplus.org/>,
3 * Copyright (C) 2006-2016 Eyeo GmbH
4 *
5 * Adblock Plus is free software: you can redistribute it and/or modify
6 * it under the terms of the GNU General Public License version 3 as
7 * published by the Free Software Foundation.
8 *
9 * Adblock Plus is distributed in the hope that it will be useful,
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of
11 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12 * GNU General Public License for more details.
13 *
14 * You should have received a copy of the GNU General Public License
15 * along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>.
16 */
17
18 /**
19 * @fileOverview This is merely forwarding messages from the content script to
20 * message responder, these will hopefully be received directly soon.
21 */
22
23 "use strict";
24
25 let {port} = require("messaging");
26 let {onMessage} = require("ext_background");
27
28 port.on("cssPropertiesRequest", ({payload, frames}) =>
29 {
30 let result = undefined;
31
32 // HACK: Message responder doesn't care about sender.page but it passes
33 // sender.frame to whitelisting.checkWhitelisted(). Instead of converting
34 // our frame list into the format used in Chrome we keep it as is, then our
35 // whitelisting.checkWhitelisted() implementation won't need to convert it
36 // back. We merely have to set frames.url, message responder needs it.
37 frames.url = new URL(frames[0].location);
38 let sender = {
39 page: null,
40 frame: frames
41 };
42
43 onMessage._dispatch(payload, sender, data => {
44 result = data;
45 });
46 return result;
47 });
OLDNEW
« ext/common.js ('K') | « lib/child/cssProperties.js ('k') | lib/main.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld