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

Issue 29323439: Issue 2877 - Element Hiding Helper causes tab to crash when selecting some elements (Closed)

Created:
Aug. 10, 2015, 9:22 p.m. by Wladimir Palant
Modified:
Aug. 11, 2015, 1:57 p.m.
Reviewers:
saroyanm
Visibility:
Public.

Description

Issue 2877 - Element Hiding Helper causes tab to crash when selecting some elements

Patch Set 1 #

Total comments: 5

Patch Set 2 : Removed extra newline #

Unified diffs Side-by-side diffs Delta from patch set Stats (+30 lines, -13 lines) Patch
M chrome/content/frameScript.js View 1 chunk +2 lines, -1 line 0 comments Download
M lib/aardvark.js View 1 2 chunks +28 lines, -12 lines 0 comments Download

Messages

Total messages: 5
Wladimir Palant
Aug. 10, 2015, 9:22 p.m. (2015-08-10 21:22:25 UTC) #1
saroyanm
https://codereview.adblockplus.org/29323439/diff/29323440/chrome/content/frameScript.js File chrome/content/frameScript.js (right): https://codereview.adblockplus.org/29323439/diff/29323440/chrome/content/frameScript.js#newcode1 chrome/content/frameScript.js:1: /* Did this file exist ? I can't find ...
Aug. 11, 2015, 8:41 a.m. (2015-08-11 08:41:50 UTC) #2
Wladimir Palant
https://codereview.adblockplus.org/29323439/diff/29323440/chrome/content/frameScript.js File chrome/content/frameScript.js (right): https://codereview.adblockplus.org/29323439/diff/29323440/chrome/content/frameScript.js#newcode1 chrome/content/frameScript.js:1: /* On 2015/08/11 08:41:50, saroyanm wrote: > Did this ...
Aug. 11, 2015, 10:05 a.m. (2015-08-11 10:05:18 UTC) #3
saroyanm
LGTM with small nit. https://codereview.adblockplus.org/29323439/diff/29323440/chrome/content/frameScript.js File chrome/content/frameScript.js (right): https://codereview.adblockplus.org/29323439/diff/29323440/chrome/content/frameScript.js#newcode1 chrome/content/frameScript.js:1: /* On 2015/08/11 10:05:18, Wladimir ...
Aug. 11, 2015, 1:50 p.m. (2015-08-11 13:50:15 UTC) #4
Wladimir Palant
Aug. 11, 2015, 1:57 p.m. (2015-08-11 13:57:06 UTC) #5
https://codereview.adblockplus.org/29323439/diff/29323440/lib/aardvark.js
File lib/aardvark.js (right):

https://codereview.adblockplus.org/29323439/diff/29323440/lib/aardvark.js#new...
lib/aardvark.js:615: 
On 2015/08/11 13:50:14, saroyanm wrote:
> Nit: please remove new line here.

Done.

Powered by Google App Engine
This is Rietveld