Update Blink GC plugin to reflect that Oilpan is now always enabled.

Update Blink GC plugin to reflect that Oilpan is now always enabled.

With the Blink codebase having migrated to an Oilpan only state, update
the GC plugin accordingly:

 - drop handling of RawPtr<T>, it no longer exists.
 - remove !ENABLE(OILPAN) specific checks.

R=haraken
BUG=604463
Committed: https://crrev.com/1465e995b89191c56885c93c0ca43543e9d8cad4
Cr-Commit-Position: refs/heads/master@{#388188}

[sof, 2016-04-18 21:23:14]

sigbjornf@opera.com changed reviewers:
+ hans@chromium.org, oilpan-reviews@chromium.org, thakis@chromium.org

[sof, 2016-04-18 21:23:15]

please take a look.

We'll have to back out enable-oilpan support & usage by Blink in three steps,
this being the first one.

https://codereview.chromium.org/1895943002/diff/1/tools/clang/blink_gc_plugin...
File tools/clang/blink_gc_plugin/CheckFieldsVisitor.cpp (right):

https://codereview.chromium.org/1895943002/diff/1/tools/clang/blink_gc_plugin...
tools/clang/blink_gc_plugin/CheckFieldsVisitor.cpp:90: (Parent()->IsRefPtr() &&
!edge->value()->IsGCRefCounted()) ||
Note: this RefPtr<T> check wasn't previously done with enable-oilpan, but now
is. (There is test coverage for this, but it implicitly relied on the test being
run without enable-oilpan..)

I think it has merit to catch&report such misuse of RefPtr<> as a plugin error,
but code using it will fail for other reasons as well. (e.g., no ref()/deref()
present on T.)

[haraken, 2016-04-19 00:02:56]

LGTM

[sof, 2016-04-19 12:20:25]

Description was changed from

==========
Update Blink GC plugin to reflect that Oilpan is now always enabled.

With the Blink codebase having migrated to an Oilpan only state, update
the GC plugin accordingly:

 - drop handling of RawPtr<T>, it no longer exists.
 - remove !ENABLE(OILPAN) specific checks.

R=
BUG=604463
==========

to

==========
Update Blink GC plugin to reflect that Oilpan is now always enabled.

With the Blink codebase having migrated to an Oilpan only state, update
the GC plugin accordingly:

 - drop handling of RawPtr<T>, it no longer exists.
 - remove !ENABLE(OILPAN) specific checks.

R=haraken
BUG=604463
==========

[sof, 2016-04-19 12:20:32]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2016-04-19 12:20:45]

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

[commit-bot: I haz the power, 2016-04-19 12:24:49]

Description was changed from

==========
Update Blink GC plugin to reflect that Oilpan is now always enabled.

With the Blink codebase having migrated to an Oilpan only state, update
the GC plugin accordingly:

 - drop handling of RawPtr<T>, it no longer exists.
 - remove !ENABLE(OILPAN) specific checks.

R=haraken
BUG=604463
==========

to

==========
Update Blink GC plugin to reflect that Oilpan is now always enabled.

With the Blink codebase having migrated to an Oilpan only state, update
the GC plugin accordingly:

 - drop handling of RawPtr<T>, it no longer exists.
 - remove !ENABLE(OILPAN) specific checks.

R=haraken
BUG=604463
==========

[commit-bot: I haz the power, 2016-04-19 12:24:50]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-19 12:26:07]

Description was changed from

==========
Update Blink GC plugin to reflect that Oilpan is now always enabled.

With the Blink codebase having migrated to an Oilpan only state, update
the GC plugin accordingly:

 - drop handling of RawPtr<T>, it no longer exists.
 - remove !ENABLE(OILPAN) specific checks.

R=haraken
BUG=604463
==========

to

==========
Update Blink GC plugin to reflect that Oilpan is now always enabled.

With the Blink codebase having migrated to an Oilpan only state, update
the GC plugin accordingly:

 - drop handling of RawPtr<T>, it no longer exists.
 - remove !ENABLE(OILPAN) specific checks.

R=haraken
BUG=604463

Committed: https://crrev.com/1465e995b89191c56885c93c0ca43543e9d8cad4
Cr-Commit-Position: refs/heads/master@{#388188}
==========

[commit-bot: I haz the power, 2016-04-19 12:26:08]

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