CSS quotes output quotes when depth is negative

• Component: Layout and Rendering
• Status: RESOLVED FIXED

[Elliott Sprehn]

q::before { content: close-quote }

<q></q>

This outputs two quotes, but should output no quotes.

[Elliott Sprehn]

Created attachment 157063 [details]
Patch

Fix quotes when they go negative and add more tests.

[Eric Seidel (no email)]

Comment on attachment 157063 [details]
Patch

LGTM.

[WebKit Review Bot]

Comment on attachment 157063 [details]
Patch

Rejecting attachment 157063 [details] from commit-queue.

Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2

Last 500 characters of output:
st/css-generated-content/close-quote-negative-depth-expected.html
patching file LayoutTests/fast/css-generated-content/close-quote-negative-depth.html
patching file LayoutTests/fast/css-generated-content/nested-quote-more-than-pairs-expected.html
patching file LayoutTests/fast/css-generated-content/nested-quote-more-than-pairs.html

Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--force', u'--reviewer', u'Eric Seidel']" exit_code: 1 cwd: /mnt/git/webkit-commit-queue/

Full output: http://queues.webkit.org/results/13459080

[Elliott Sprehn]

(In reply to comment #3)
> (From update of attachment 157063 [details])
> Rejecting attachment 157063 [details] from commit-queue.
> 
> Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2
> 

Eric do you know what happened here? This isn't a test failure, it's svn-apply failing? I don't understand what happened.

[Eric Seidel (no email)]

Yes, it's a horrible error message from a failed apply.  You'll want to rebase the patch and re-upload (or have someone do so for you).  I can help tomorrow if you like.

[Elliott Sprehn]

This conflicts with WKBug 93056 so one or the other needs to land first.

[Elliott Sprehn]

Created attachment 157607 [details]
Patch for landing

[WebKit Review Bot]

Comment on attachment 157607 [details]
Patch for landing

Rejecting attachment 157607 [details] from commit-queue.

Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2

Last 500 characters of output:
webkit-commit-queue/Source/WebKit/chromium/webkit --revision 150169 --non-interactive --force --accept theirs-conflict --ignore-externals' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium'
49>At revision 150169.

________ running '/usr/bin/python tools/clang/scripts/update.py --mac-only' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium'

________ running '/usr/bin/python gyp_webkit' in '/mnt/git/webkit-commit-queue/Source/WebKit/chromium'
Updating webkit projects from gyp files...

Full output: http://queues.webkit.org/results/13477056

[Eric Seidel (no email)]

Comment on attachment 157607 [details]
Patch for landing

This appears to be missing a ChangeLog.  I suspect that's what the queue was complaining about.

[Elliott Sprehn]

(In reply to comment #9)
> (From update of attachment 157607 [details])
> This appears to be missing a ChangeLog.  I suspect that's what the queue was complaining about.

Oh weird, when I ran update-webkit and then reuploaded the patch it stripped off the ChangeLog...

[Elliott Sprehn]

Created attachment 158059 [details]
Patch for landing

[Elliott Sprehn]

Created attachment 158078 [details]
Patch

Finally get some ChangeLogs.

[Elliott Sprehn]

Created attachment 158080 [details]
Patch for landing

[WebKit Review Bot]

Comment on attachment 158080 [details]
Patch for landing

Clearing flags on attachment: 158080

Committed r125448: <http://trac.webkit.org/changeset/125448>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.