Use <string> serialization rules for computed value of reference filter

Use <string> serialization rules for computed value of reference filter

According to [1] the filter reference is a <url> type. For <url> types
the spec says to use <string>s [2]. And <string>s serialize with double
quotes.

Also remove the unused quoteCSSURLIfNeeded.

Behavior matches Firefox.

[1] https://drafts.fxtf.org/filters/#typedef-filter-function-list
[2] https://www.w3.org/TR/css-values/#urls
[3] https://drafts.csswg.org/cssom/#serialize-a-string

Committed: https://crrev.com/b2be3f36e2f4ba3db981809d6252b4148295d238
Cr-Commit-Position: refs/heads/master@{#380517}

[rwlbuis, 2016-03-10 20:56:52]

Description was changed from

==========
584999 WIP

BUG=

patch from issue 1778743003 at patchset 40001
(http://crrev.com/1778743003#ps40001)
==========

to

==========
Use <string> serialization rules for computed value of reference filter


https://drafts.fxtf.org/filters/#typedef-filter-function-list

https://www.w3.org/TR/css-values/#urls

https://drafts.csswg.org/cssom/#serialize-a-string
==========

[rwlbuis, 2016-03-10 23:07:37]

Description was changed from

==========
Use <string> serialization rules for computed value of reference filter


https://drafts.fxtf.org/filters/#typedef-filter-function-list

https://www.w3.org/TR/css-values/#urls

https://drafts.csswg.org/cssom/#serialize-a-string
==========

to

==========
Use <string> serialization rules for computed value of reference filter

According to [1] the filter reference is a <url> type. For <url> types
the spec says to use <string>s [2]. And <string>s serialize with double
quotes.

Behavior matches Firefox.

[1] https://drafts.fxtf.org/filters/#typedef-filter-function-list
[2] https://www.w3.org/TR/css-values/#urls
[3] https://drafts.csswg.org/cssom/#serialize-a-string
==========

[rwlbuis, 2016-03-10 23:08:18]

Description was changed from

==========
Use <string> serialization rules for computed value of reference filter

According to [1] the filter reference is a <url> type. For <url> types
the spec says to use <string>s [2]. And <string>s serialize with double
quotes.

Behavior matches Firefox.

[1] https://drafts.fxtf.org/filters/#typedef-filter-function-list
[2] https://www.w3.org/TR/css-values/#urls
[3] https://drafts.csswg.org/cssom/#serialize-a-string
==========

to

==========
Use <string> serialization rules for computed value of reference filter

According to [1] the filter reference is a <url> type. For <url> types
the spec says to use <string>s [2]. And <string>s serialize with double
quotes.

Also remove the unused quoteCSSURLIfNeeded.

Behavior matches Firefox.

[1] https://drafts.fxtf.org/filters/#typedef-filter-function-list
[2] https://www.w3.org/TR/css-values/#urls
[3] https://drafts.csswg.org/cssom/#serialize-a-string
==========

[rwlbuis, 2016-03-10 23:21:03]

rob.buis@samsung.com changed reviewers:
+ timloh@chromium.org

[rwlbuis, 2016-03-10 23:21:04]

PTAL. Not sure if this needs a bug, I discovered this while starting to work on
bug 584999.

[Timothy Loh, 2016-03-10 23:24:13]

lgtm

[rwlbuis, 2016-03-10 23:35:22]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2016-03-10 23:36:10]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1783693006/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1783693006/20001

[commit-bot: I haz the power, 2016-03-11 00:09:35]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-11 00:09:36]

Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[rwlbuis, 2016-03-11 00:25:32]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2016-03-11 00:26:09]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1783693006/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1783693006/20001

[commit-bot: I haz the power, 2016-03-11 00:59:12]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-11 00:59:13]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[rwlbuis, 2016-03-11 01:08:24]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2016-03-11 01:08:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1783693006/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1783693006/20001

[commit-bot: I haz the power, 2016-03-11 01:30:36]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-11 01:30:37]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[rwlbuis, 2016-03-11 02:30:50]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2016-03-11 02:31:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1783693006/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1783693006/20001

[commit-bot: I haz the power, 2016-03-11 03:07:41]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-11 03:07:42]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[rwlbuis, 2016-03-11 03:43:43]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2016-03-11 03:44:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1783693006/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1783693006/20001

[commit-bot: I haz the power, 2016-03-11 04:26:06]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-03-11 04:26:07]

Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[rwlbuis, 2016-03-11 04:27:08]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2016-03-11 04:27:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1783693006/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1783693006/20001

[commit-bot: I haz the power, 2016-03-11 05:16:43]

Description was changed from

==========
Use <string> serialization rules for computed value of reference filter

According to [1] the filter reference is a <url> type. For <url> types
the spec says to use <string>s [2]. And <string>s serialize with double
quotes.

Also remove the unused quoteCSSURLIfNeeded.

Behavior matches Firefox.

[1] https://drafts.fxtf.org/filters/#typedef-filter-function-list
[2] https://www.w3.org/TR/css-values/#urls
[3] https://drafts.csswg.org/cssom/#serialize-a-string
==========

to

==========
Use <string> serialization rules for computed value of reference filter

According to [1] the filter reference is a <url> type. For <url> types
the spec says to use <string>s [2]. And <string>s serialize with double
quotes.

Also remove the unused quoteCSSURLIfNeeded.

Behavior matches Firefox.

[1] https://drafts.fxtf.org/filters/#typedef-filter-function-list
[2] https://www.w3.org/TR/css-values/#urls
[3] https://drafts.csswg.org/cssom/#serialize-a-string
==========

[commit-bot: I haz the power, 2016-03-11 05:16:44]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-11 05:18:11]

Description was changed from

==========
Use <string> serialization rules for computed value of reference filter

According to [1] the filter reference is a <url> type. For <url> types
the spec says to use <string>s [2]. And <string>s serialize with double
quotes.

Also remove the unused quoteCSSURLIfNeeded.

Behavior matches Firefox.

[1] https://drafts.fxtf.org/filters/#typedef-filter-function-list
[2] https://www.w3.org/TR/css-values/#urls
[3] https://drafts.csswg.org/cssom/#serialize-a-string
==========

to

==========
Use <string> serialization rules for computed value of reference filter

According to [1] the filter reference is a <url> type. For <url> types
the spec says to use <string>s [2]. And <string>s serialize with double
quotes.

Also remove the unused quoteCSSURLIfNeeded.

Behavior matches Firefox.

[1] https://drafts.fxtf.org/filters/#typedef-filter-function-list
[2] https://www.w3.org/TR/css-values/#urls
[3] https://drafts.csswg.org/cssom/#serialize-a-string

Committed: https://crrev.com/b2be3f36e2f4ba3db981809d6252b4148295d238
Cr-Commit-Position: refs/heads/master@{#380517}
==========

[commit-bot: I haz the power, 2016-03-11 05:18:12]

Patchset 2 (id:??) landed as
https://crrev.com/b2be3f36e2f4ba3db981809d6252b4148295d238
Cr-Commit-Position: refs/heads/master@{#380517}