Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(70)

Issue 65530043: allow css background-size and resize (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years ago by felix8a
Modified:
12 years ago
Reviewers:
kpreid2
CC:
google-caja-discuss_googlegroups.com
Base URL:
http://google-caja.googlecode.com/svn/trunk/
Visibility:
Public.

Description

this will also be merged into the es53 branch. background-size: was already defined, and it's implicitly allowed by the background: property, but it was inadvertently left off the whitelist. resize: is new. I've also added a test for text-overflow: which was in a previous CL.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+88 lines, -0 lines) Patch
src/com/google/caja/lang/css/css3-defs.json View 1 chunk +8 lines, -0 lines 0 comments Download
src/com/google/caja/lang/css/css3-whitelist.json View 2 chunks +2 lines, -0 lines 0 comments Download
tests/com/google/caja/plugin/sanitizecss_test.js View 1 chunk +78 lines, -0 lines 0 comments Download

Messages

Total messages: 3
felix8a
12 years ago (2014-02-18 18:30:37 UTC) #1
kpreid2
LGTM
12 years ago (2014-02-18 18:54:12 UTC) #2
felix8a
12 years ago (2014-02-18 19:39:32 UTC) #3
@r5666 trunk
@r5667 es53 branch
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b