SVG: Move/rename reference management to SVGElement

SVG: Move/rename reference management to SVGElement

No change in logic, but rename and move.

I used different naming convensions based on suggestions by fs@opera.com :
"referencingElements" is renamed "outgoingReferences", and
"referencedElements" is renamed "incomingReferences".

- |SVGDocumentExtensions::setOfElementsReferencingTarget|
  moved to |SVGElement::setOfIncomingReferences|
- |SVGDocumentExtensions::addElementReferencingTarget|
  moved to |SVGElement::addOutgoingReferenceTo|
- |SVGDocumentExtensions::removeAllTargetReferencesForElement|
  moved to |SVGElement::removeAllOutgoingReferences|
- |SVGDocumentExtensions::rebuildAllElementReferencesForTarget|
  moved to |SVGElement::rebuildAllIncomingReferences|
- |SVGDocumentExtensions::removeAllElementReferencesForTarget|
  moved to |SVGElement::removeAllIncomingReferences|

Changes to files SVG\w+Element.cpp made by a sed script:
s/document()\.accessSVGExtensions()\.addElementReferencingTarget(this, \([a-Z()\*]\+\))/addOutgoingReferenceTo(\1)/g
s/document()\.accessSVGExtensions()\.removeAllTargetReferencesForElement(this)/removeAllOutgoingReferences()/g
s/document()\.accessSVGExtensions()\.setOfElementsReferencingTarget(this)/setOfIncomingReferences()/g

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

[kouhei (in TOK), 2014-06-25 08:40:15]

Refactoring renaming/moving methods to SVGElement. Please take a look when you
have time.

The names "referencedElements" and "referencingElements" are too confusing to
me. :p

[fs, 2014-06-25 10:59:54]

On 2014/06/25 08:40:15, kouhei wrote:
> Refactoring renaming/moving methods to SVGElement. Please take a look when you
> have time.
> 
> The names "referencedElements" and "referencingElements" are too confusing to
> me. :p

Amen to that.

LGTM - but then I guess I'm biased =). I'd probably have let
"addOutgoingReferenceTo" be just "addReferenceTo" (since you'll never add an
incoming edge this way).

[pdr., 2014-06-25 16:02:44]

On 2014/06/25 10:59:54, fs wrote:
> On 2014/06/25 08:40:15, kouhei wrote:
> > Refactoring renaming/moving methods to SVGElement. Please take a look when
you
> > have time.
> > 
> > The names "referencedElements" and "referencingElements" are too confusing
to
> > me. :p
> 
> Amen to that.
> 
> LGTM - but then I guess I'm biased =). I'd probably have let
> "addOutgoingReferenceTo" be just "addReferenceTo" (since you'll never add an
> incoming edge this way).

LGTM2. I also think fs's idea of addReferenceTo would work well, but I'm fine
landing this just like it is too.

[kouhei (in TOK), 2014-06-26 23:24:12]

Thanks for review! renamed to addReferenceTo

[kouhei (in TOK), 2014-06-26 23:25:28]

The CQ bit was checked by kouhei@chromium.org

[commit-bot: I haz the power, 2014-06-26 23:26:08]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/kouhei@chromium.org/344883007/60001

[commit-bot: I haz the power, 2014-06-27 01:45:33]

Change committed as 177039