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

Issue 5040954746601472: Issue 186 - disabled ABP icon in tool bar re-appears after seamonkey restart (Closed)

Created:
May 6, 2014, 12:05 p.m. by saroyanm
Modified:
May 9, 2014, 7:05 a.m.
Reviewers:
Wladimir Palant
Visibility:
Public.

Description

This issue is related to current ticket: https://issues.adblockplus.org/ticket/186

Patch Set 1 #

Total comments: 2

Patch Set 2 : Check for localname #

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

Messages

Total messages: 4
saroyanm
Wladimir can you please have a look when you have time, I guess this review ...
May 6, 2014, 12:19 p.m. (2014-05-06 12:19:46 UTC) #1
Wladimir Palant
http://codereview.adblockplus.org/5040954746601472/diff/5629499534213120/lib/customizableUI.js File lib/customizableUI.js (right): http://codereview.adblockplus.org/5040954746601472/diff/5629499534213120/lib/customizableUI.js#newcode187 lib/customizableUI.js:187: if (node && node.parentNode.id != "BrowserToolbarPalette") Please check |node.parentNode.localName ...
May 8, 2014, 1:40 p.m. (2014-05-08 13:40:21 UTC) #2
saroyanm
Uploaded a new patch with your mentioned change Wladimir. http://codereview.adblockplus.org/5040954746601472/diff/5629499534213120/lib/customizableUI.js File lib/customizableUI.js (right): http://codereview.adblockplus.org/5040954746601472/diff/5629499534213120/lib/customizableUI.js#newcode187 lib/customizableUI.js:187: ...
May 8, 2014, 3:13 p.m. (2014-05-08 15:13:13 UTC) #3
Wladimir Palant
May 8, 2014, 6:03 p.m. (2014-05-08 18:03:32 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld