Vector stores Predicate object as OwnPtr instead of raw pointer in XPath.

Vector stores Predicate object as OwnPtr instead of raw pointer in XPath.

Vector can handle OwnPtr, so we don't have to delete an element in Vector
manually.

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

[dshwang, 2013-10-17 08:53:36]

It is follow-up of https://codereview.chromium.org/26763004/

I mentioned Vector has a bug : Vector<OwnPtr<> > copy constructor.
My mention was wrong, because OwnPtr<> must not have copy constructor.

So, In this CL, there are following codes:
Vector<OwnPtr<T> > v1;
Vector<OwnPtr<T> > v2;
v1.swap(v2);
I checked all swap code is intended.

[abarth-chromium, 2013-10-17 18:45:13]

lgtm

Yeah, swap works well here.  The odd copy constructor and assignment operator
are a bit unfortunate, but the rest of the CL seems beneficial enough to be
worthwhile.  Thanks!

[commit-bot: I haz the power, 2013-10-17 18:46:11]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dongseong.hwang@intel.com/26709019/1

[dshwang, 2013-10-17 19:06:15]

On 2013/10/17 18:45:13, abarth wrote:
> lgtm
> 
> Yeah, swap works well here.  The odd copy constructor and assignment operator
> are a bit unfortunate, but the rest of the CL seems beneficial enough to be
> worthwhile.  Thanks!

Thank you for kind review! :)

[commit-bot: I haz the power, 2013-10-17 22:26:29]

Change committed as 159908