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

Issue 29337878: Issue 3736 - Fixed: Checks for share resources failed (Closed)

Created:
March 4, 2016, 5:19 p.m. by Thomas Greiner
Modified:
March 14, 2016, 4:27 p.m.
Reviewers:
kzar
CC:
saroyanm
Visibility:
Public.

Description

Issue 3736 - Fixed: Checks for share resources failed

Patch Set 1 #

Patch Set 2 : Rebased and added RegExpFilter mock #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -3 lines) Patch
M background.js View 1 1 chunk +3 lines, -1 line 0 comments Download
M messageResponder.js View 1 1 chunk +3 lines, -2 lines 0 comments Download

Messages

Total messages: 4
Thomas Greiner
March 4, 2016, 5:24 p.m. (2016-03-04 17:24:55 UTC) #1
kzar
LGTM
March 7, 2016, 4:48 p.m. (2016-03-07 16:48:04 UTC) #2
Thomas Greiner
I noticed that a mock for `RegExpFilter` was required for it to work in the ...
March 14, 2016, 3:53 p.m. (2016-03-14 15:53:54 UTC) #3
kzar
March 14, 2016, 3:58 p.m. (2016-03-14 15:58:28 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld