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

Issue 29828604: Issue 6790 - Implement uabinject-defuser snippet (Closed)

Created:
July 12, 2018, 4:31 p.m. by Manish Jethani
Modified:
July 24, 2018, 2:02 p.m.
Reviewers:
kzar, hub
CC:
Felix Dahlke, Sebastian Noack
Base URL:
https://hg.adblockplus.org/adblockpluscore/
Visibility:
Public.

Description

Based on https://codereview.adblockplus.org/29829569/

Patch Set 1 #

Patch Set 2 : Fix ESLint error #

Patch Set 3 : Use strict mode in injected code #

Patch Set 4 : Add attribution #

Patch Set 5 : Rebase on patch #29829569 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -0 lines) Patch
M lib/content/snippets.js View 1 2 3 4 1 chunk +13 lines, -0 lines 0 comments Download

Messages

Total messages: 10
Manish Jethani
July 12, 2018, 4:31 p.m. (2018-07-12 16:31:20 UTC) #1
Manish Jethani
Patch Set 1
July 12, 2018, 4:34 p.m. (2018-07-12 16:34:09 UTC) #2
Manish Jethani
Patch Set 2: Fix ESLint error
July 13, 2018, 10:38 a.m. (2018-07-13 10:38:15 UTC) #3
Manish Jethani
Patch Set 3: Use strict mode in injected code
July 13, 2018, 10:44 a.m. (2018-07-13 10:44:36 UTC) #4
Manish Jethani
Patch Set 4: Add attribution
July 13, 2018, 12:59 p.m. (2018-07-13 12:59:42 UTC) #5
Manish Jethani
Patch Set 5: Rebase on patch #29829569
July 13, 2018, 2:35 p.m. (2018-07-13 14:35:31 UTC) #6
kzar
Code itself LGTM. However, before you push this, I'd like to see Kai/Felix confirm about ...
July 16, 2018, 2:25 p.m. (2018-07-16 14:25:30 UTC) #7
Manish Jethani
On 2018/07/16 14:25:30, kzar wrote: > Code itself LGTM. > > However, before you push ...
July 17, 2018, 10 a.m. (2018-07-17 10:00:01 UTC) #8
kzar
Well, like I say I defer to Sebastian/Felix regarding attribution and related legal stuff.
July 17, 2018, 10:25 a.m. (2018-07-17 10:25:07 UTC) #9
Manish Jethani
July 24, 2018, 2:02 p.m. (2018-07-24 14:02:27 UTC) #10
Thanks!

Powered by Google App Engine
This is Rietveld