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

Issue 29362481: Issue 4600 - Change CSS property matching to be case insensitive (Closed)

Created:
Nov. 11, 2016, 4:52 p.m. by Felix Dahlke
Modified:
Nov. 14, 2016, 6:54 p.m.
Reviewers:
kzar, Wladimir Palant
Base URL:
https://bitbucket.org/fhd/adblockpluscore
Visibility:
Public.

Description

Issue 4600 - Change CSS property matching to be case insensitive

Patch Set 1 #

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

Messages

Total messages: 3
Felix Dahlke
Nov. 11, 2016, 4:52 p.m. (2016-11-11 16:52:27 UTC) #1
kzar
LGTM assuming we don't care about $match-case here.
Nov. 11, 2016, 6:35 p.m. (2016-11-11 18:35:28 UTC) #2
Wladimir Palant
Nov. 13, 2016, 9:32 a.m. (2016-11-13 09:32:39 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld