Make image set insertion work in more cases

Make image set insertion work in more cases

In case there was a background with multiple layers (background: url(...),
url(...);), code would just replace first URL occurrence with -webkit-image-set
rule. Handles multiple layers in case there are only url() functions. Still
won't work if value contains more values than just url()s.

Additionally, use \s for matching white space, so that Windows line endings are
also handled.

And allow line breaks between 'content|background[...]:' and the following
value. Code would previously stop matching if there was a line break between
the colon and the value.

[Joao da Silva, 2013-09-03 08:20:23]

Rob is the right reviewer for this.

[flackr, 2013-09-03 15:37:30]

LGTM

[Rafał Chłodnicki, 2013-09-03 15:43:18]

On 2013/09/03 15:37:30, flackr wrote:
> LGTM

Thanks.

If you want you can also trigger bots (I can't do that myself). I've run them as
part of the upload hook of course but who knows, maybe there would be some
differences on other platforms (thinking about CRLF test).

[flackr, 2013-09-03 18:46:12]

On 2013/09/03 15:43:18, Rafał Chłodnicki wrote:
> On 2013/09/03 15:37:30, flackr wrote:
> > LGTM
> 
> Thanks.
> 
> If you want you can also trigger bots (I can't do that myself). I've run them
as
> part of the upload hook of course but who knows, maybe there would be some
> differences on other platforms (thinking about CRLF test).

Nevermind the earlier message, I found some try server guides for subprojects
but it looks like this doesn't apply on top of grit's ToT. Can you merge and
upload another patch set?

[Rafał Chłodnicki, 2013-09-03 18:52:39]

On 2013/09/03 18:46:12, flackr wrote:
> > If you want you can also trigger bots (I can't do that myself). I've run
them
> as
> > part of the upload hook of course but who knows, maybe there would be some
> > differences on other platforms (thinking about CRLF test).
> 
> Nevermind the earlier message, I found some try server guides for subprojects
> but it looks like this doesn't apply on top of grit's ToT. Can you merge and
> upload another patch set?

I don't understand what you want me to do. This is based on r134 which is the
ToT according to https://code.google.com/p/grit-i18n/source/list

[flackr, 2013-09-03 19:13:05]

On 2013/09/03 18:52:39, Rafał Chłodnicki wrote:
> On 2013/09/03 18:46:12, flackr wrote:
> > > If you want you can also trigger bots (I can't do that myself). I've run
> them
> > as
> > > part of the upload hook of course but who knows, maybe there would be some
> > > differences on other platforms (thinking about CRLF test).
> > 
> > Nevermind the earlier message, I found some try server guides for
subprojects
> > but it looks like this doesn't apply on top of grit's ToT. Can you merge and
> > upload another patch set?
> 
> I don't understand what you want me to do. This is based on r134 which is the
> ToT according to https://code.google.com/p/grit-i18n/source/list

Looks like it was a problem on my end with my local master being behind, sorry
about that. Anyways, failing being able to try in advance of landing in grit I
can land it and we can try against chrome on changing the DEPS file.

[Rafał Chłodnicki, 2013-09-03 19:14:59]

On 2013/09/03 19:13:05, flackr wrote:
> On 2013/09/03 18:52:39, Rafał Chłodnicki wrote:
> > On 2013/09/03 18:46:12, flackr wrote:
> > > > If you want you can also trigger bots (I can't do that myself). I've run
> > them
> > > as
> > > > part of the upload hook of course but who knows, maybe there would be
some
> > > > differences on other platforms (thinking about CRLF test).
> > > 
> > > Nevermind the earlier message, I found some try server guides for
> subprojects
> > > but it looks like this doesn't apply on top of grit's ToT. Can you merge
and
> > > upload another patch set?
> > 
> > I don't understand what you want me to do. This is based on r134 which is
the
> > ToT according to https://code.google.com/p/grit-i18n/source/list
> 
> Looks like it was a problem on my end with my local master being behind, sorry
> about that. Anyways, failing being able to try in advance of landing in grit I
> can land it and we can try against chrome on changing the DEPS file.

Sure, sounds good.

[flackr, 2013-09-03 21:04:18]

Committed manually as r135.

[flackr, 2013-09-03 21:05:25]

Now that this is in grit, you can put up a CL for chromium to pull grit r135 in
the DEPS file.