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

Issue 29893555: Issue 6997 - Use createStyleSheet implementation from core (Closed)

Created:
Sept. 27, 2018, 5:10 a.m. by Manish Jethani
Modified:
Sept. 27, 2018, 10:04 p.m.
Reviewers:
Sebastian Noack
Base URL:
https://hg.adblockplus.org/adblockpluschrome/
Visibility:
Public.

Patch Set 1 #

Patch Set 2 : Update dependency to 0a316e60de4e #

Patch Set 3 : Revert to PS1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -30 lines) Patch
M dependencies View 2 1 chunk +1 line, -1 line 0 comments Download
M lib/contentFiltering.js View 2 chunks +1 line, -29 lines 0 comments Download

Messages

Total messages: 8
Manish Jethani
Sept. 27, 2018, 5:10 a.m. (2018-09-27 05:10:44 UTC) #1
Manish Jethani
Patch Set 1 This is on the next bookmark, includes a dependency update to core's ...
Sept. 27, 2018, 5:11 a.m. (2018-09-27 05:11:31 UTC) #2
Sebastian Noack
Looks good to me once there is an issue for the dependency update, and we ...
Sept. 27, 2018, 3:28 p.m. (2018-09-27 15:28:35 UTC) #3
Manish Jethani
On 2018/09/27 15:28:35, Sebastian Noack wrote: > Looks good to me once there is an ...
Sept. 27, 2018, 5:14 p.m. (2018-09-27 17:14:54 UTC) #4
Manish Jethani
Patch Set 2: Update dependency to 0a316e60de4e Landed one more change, might as well include ...
Sept. 27, 2018, 5:25 p.m. (2018-09-27 17:25:25 UTC) #5
Sebastian Noack
LGTM
Sept. 27, 2018, 8:03 p.m. (2018-09-27 20:03:48 UTC) #6
Manish Jethani
On 2018/09/27 20:03:48, Sebastian Noack wrote: > LGTM Thanks. Actually Patch Set 2 won't work ...
Sept. 27, 2018, 9:22 p.m. (2018-09-27 21:22:13 UTC) #7
Sebastian Noack
Sept. 27, 2018, 9:59 p.m. (2018-09-27 21:59:39 UTC) #8
LGTM

Powered by Google App Engine
This is Rietveld