Don't reload an svg <use> element's xlink:href if only the fragment changes

Don't reload an svg <use> element's xlink:href if only the fragment changes

BUG=580809
TEST=http/tests/cache/svg-use-revalidate.html

Committed: https://crrev.com/332761c7f4da3aa451028a6a3d16aafd607b69be
Cr-Commit-Position: refs/heads/master@{#373371}

[Nate Chapin, 2016-02-03 20:55:29]

japhet@chromium.org changed reviewers:
+ pdr@chromium.org

[Nate Chapin, 2016-02-03 20:55:30]

It appears that any time a svg <use> element's href changed, we used to
re-request the resource, even if only the fragment changed. That seems wrong?

I can fix this by making revalidating work exactly the way it used to, but even
with Cache-Control headers, this doesn't seem like an appropriate time to
re-request.

[pdr., 2016-02-03 21:02:34]

On 2016/02/03 at 20:55:30, japhet wrote:
> It appears that any time a svg <use> element's href changed, we used to
re-request the resource, even if only the fragment changed. That seems wrong?
> 
> I can fix this by making revalidating work exactly the way it used to, but
even with Cache-Control headers, this doesn't seem like an appropriate time to
re-request.

LGTM

[Nate Chapin, 2016-02-03 21:39:35]

The CQ bit was checked by japhet@chromium.org

[commit-bot: I haz the power, 2016-02-03 21:41:44]

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

[commit-bot: I haz the power, 2016-02-03 22:49:10]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-03 22:50:28]

Description was changed from

==========
Don't reload an svg <use> element's xlink:href if only the fragment changes

BUG=580809
TEST=http/tests/cache/svg-use-revalidate.html
==========

to

==========
Don't reload an svg <use> element's xlink:href if only the fragment changes

BUG=580809
TEST=http/tests/cache/svg-use-revalidate.html

Committed: https://crrev.com/332761c7f4da3aa451028a6a3d16aafd607b69be
Cr-Commit-Position: refs/heads/master@{#373371}
==========

[commit-bot: I haz the power, 2016-02-03 22:50:29]

Patchset 1 (id:??) landed as
https://crrev.com/332761c7f4da3aa451028a6a3d16aafd607b69be
Cr-Commit-Position: refs/heads/master@{#373371}