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

Issue 6236888415338496: Issue 1611 - Fixed inconsistent behavior with "Block element" context menu item (Closed)

Created:
Nov. 26, 2014, 11:28 a.m. by Sebastian Noack
Modified:
Dec. 8, 2014, 5:29 p.m.
Reviewers:
kzar, Wladimir Palant
Visibility:
Public.

Description

Issue 1611 - Fixed inconsistent behavior with "Block element" context menu item

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -48 lines) Patch
M background.js View 1 chunk +2 lines, -3 lines 0 comments Download
M chrome/ext/background.js View 1 chunk +1 line, -1 line 0 comments Download
M include.postload.js View 1 chunk +2 lines, -40 lines 0 comments Download
M safari/ext/background.js View 2 chunks +1 line, -3 lines 0 comments Download
M safari/ext/content.js View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 3
Sebastian Noack
Nov. 26, 2014, 11:28 a.m. (2014-11-26 11:28:38 UTC) #1
kzar
I've checked through the patchset and I can't see anything that's obviously wrong. I think ...
Nov. 26, 2014, noon (2014-11-26 12:00:08 UTC) #2
Wladimir Palant
Dec. 8, 2014, 4:25 p.m. (2014-12-08 16:25:44 UTC) #3
LGTM, good to see more AdThwart code become sane.

Powered by Google App Engine
This is Rietveld