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

Issue 29345734: Issue 4141 - Safari: Fixed blocking of video ads on YouTube (Closed)

Created:
June 9, 2016, 5:29 p.m. by Wladimir Palant
Modified:
June 11, 2016, 9 p.m.
Reviewers:
Sebastian Noack, kzar
Visibility:
Public.

Description

Issue 4141 - Safari: Fixed blocking of video ads on YouTube Repository: hg.adblockplus.org/adblockpluschrome

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -0 lines) Patch
M safari/include.youtube.js View 1 chunk +2 lines, -0 lines 2 comments Download

Messages

Total messages: 4
Wladimir Palant
June 9, 2016, 5:29 p.m. (2016-06-09 17:29:43 UTC) #1
kzar
LGTM
June 9, 2016, 7:46 p.m. (2016-06-09 19:46:10 UTC) #2
Sebastian Noack
https://codereview.adblockplus.org/29345734/diff/29345735/safari/include.youtube.js File safari/include.youtube.js (right): https://codereview.adblockplus.org/29345734/diff/29345735/safari/include.youtube.js#newcode25 safari/include.youtube.js:25: var beforeLoadEvent = document.createEvent("Event"); I wonder whether we couldn't ...
June 11, 2016, 8:56 p.m. (2016-06-11 20:56:35 UTC) #3
Sebastian Noack
June 11, 2016, 9 p.m. (2016-06-11 21:00:30 UTC) #4
Message was sent while issue was closed.
https://codereview.adblockplus.org/29345734/diff/29345735/safari/include.yout...
File safari/include.youtube.js (right):

https://codereview.adblockplus.org/29345734/diff/29345735/safari/include.yout...
safari/include.youtube.js:25: var beforeLoadEvent =
document.createEvent("Event");
On 2016/06/11 20:56:34, Sebastian Noack wrote:
> I wonder whether we couldn't simply use ext.backgroundPage.sendMessage() here.

sendMessageSync() I meant.

Powered by Google App Engine
This is Rietveld