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

Issue 29984574: Noissue - Adapt for test pages changes running circumvention tests (Closed)

Created:
Jan. 17, 2019, 8:12 a.m. by Sebastian Noack
Modified:
Jan. 18, 2019, 3:22 p.m.
Reviewers:
tlucas, kzar, rossg
Visibility:
Public.

Description

Noissue - Adapt for test pages changes running circumvention tests

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+21 lines, -13 lines) Patch
M test/wrappers/pages.js View 5 chunks +21 lines, -13 lines 1 comment Download

Messages

Total messages: 3
Sebastian Noack
The related changes to the test pages we are adapting for here: https://codereview.adblockplus.org/29984560/ These changes ...
Jan. 17, 2019, 8:17 a.m. (2019-01-17 08:17:13 UTC) #1
Sebastian Noack
I went ahead pushing these changes, in order to avoid the test automation from breaking ...
Jan. 17, 2019, 10:48 p.m. (2019-01-17 22:48:27 UTC) #2
kzar
Jan. 18, 2019, 10:17 a.m. (2019-01-18 10:17:32 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld