[GenerateVisitDOMWrapper] should use SetReferenceFromGroup instead of SetObjectGroupId

Given:

  [GenerateVisitDOMWrapper=ownerNode] interface HTMLOptionsCollection { ... }

This means that HTMLOptionsCollection's wrapper should be kept alive if the object group of HTMLOptionsCollection::ownerNode() is alive. On the other hand, this does *not* mean that the object group of HTMLOptionsCollection::ownerNode() should be kept alive if HTMLOptionsCollection's wrapper is alive.

The above fact means that we should use SetReferenceFromGroup instead of SetObjectGroupId. SetReferenceFromGroup adds a reference from the object group to the wrapper, whereas SetObjectGroupId adds bi-directional references between the object group and the wrapper.

(In this particular case of HTMLOptionsCollection and other DOM objects that are using [GenerateVisitDOMWrapper], I don't see any harm in adding bi-directional references. However, it's not nice to add unnecessary lifetime dependency. So this CL fixes them.)

Review URL: https://codereview.chromium.org/116093008

git-svn-id: svn://svn.chromium.org/blink/trunk@165324 bbb929c8-8fbe-4397-9dbb-9b2b20218538