Use a RefPtr to store implementation in QualifiedName

Use a RefPtr to store implementation in QualifiedName

Use a RefPtr to store implementation in QualifiedName instead of handling
ref-counting manually. This makes the code shorter / simpler.

R=eseidel, esprehn
BUG=None

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=164998

[esprehn, 2014-01-13 08:51:27]

Lgtm, looks reasonable to me. Any idea why it didn't before?

To unsubscribe from this group and stop receiving emails from it, send an email
to blink-reviews+unsubscribe@chromium.org.

[Inactive, 2014-01-13 14:31:42]

On 2014/01/13 08:51:27, esprehn wrote:
> Lgtm, looks reasonable to me. Any idea why it didn't before?

I looked at the git history, it looks like ref-counting has always been handled
manually for that class (since the khtml days).
Based on https://bugs.webkit.org/attachment.cgi?id=2873&action=review, it looks
like QualifiedName::deref() used to take care of updating the
QualifiedNameCache. However, nowadays, this is handled by the QualifiedNameImpl
destructor and QualifiedName::deref() is a mere proxy to RefCounted::deref().

[commit-bot: I haz the power, 2014-01-13 19:55:15]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/136063002/1

[commit-bot: I haz the power, 2014-01-13 20:01:01]

Change committed as 164998