Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(389)

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

Can't Edit
Can't Publish+Mail
Start Review
Created:
5 years, 3 months ago by Sebastian Noack
Modified:
5 years, 2 months ago
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
5 years, 3 months ago (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 ...
5 years, 3 months ago (2014-11-26 12:00:08 UTC) #2
Wladimir Palant
5 years, 2 months ago (2014-12-08 16:25:44 UTC) #3
LGTM, good to see more AdThwart code become sane.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 87257f5