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

Issue 5749479856668672: issue #764 - Adblock Warning opt in message isn't triggered (Closed)

Created:
July 9, 2014, 6:39 p.m. by saroyanm
Modified:
July 10, 2014, 7:40 a.m.
Reviewers:
Wladimir Palant
Visibility:
Public.

Description

Related ticket: https://issues.adblockplus.org/ticket/764

Patch Set 1 #

Total comments: 2

Patch Set 2 : #

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

Messages

Total messages: 5
saroyanm
Wladimir can you please have a look, Should be a quick one.
July 9, 2014, 6:43 p.m. (2014-07-09 18:43:09 UTC) #1
Wladimir Palant
http://codereview.adblockplus.org/5749479856668672/diff/5629499534213120/lib/ui.js File lib/ui.js (right): http://codereview.adblockplus.org/5749479856668672/diff/5629499534213120/lib/ui.js#newcode1890 lib/ui.js:1890: if ("links" in notification) How about |if (notification.links)| just ...
July 9, 2014, 6:51 p.m. (2014-07-09 18:51:46 UTC) #2
saroyanm
http://codereview.adblockplus.org/5749479856668672/diff/5629499534213120/lib/ui.js File lib/ui.js (right): http://codereview.adblockplus.org/5749479856668672/diff/5629499534213120/lib/ui.js#newcode1890 lib/ui.js:1890: if ("links" in notification) On 2014/07/09 18:51:46, Wladimir Palant ...
July 9, 2014, 6:57 p.m. (2014-07-09 18:57:25 UTC) #3
saroyanm
Updated.
July 9, 2014, 7:01 p.m. (2014-07-09 19:01:28 UTC) #4
Wladimir Palant
July 10, 2014, 5:42 a.m. (2014-07-10 05:42:27 UTC) #5
LGTM

Powered by Google App Engine
This is Rietveld