Oilpan: remove now-unused thread affinity macros.

Oilpan: remove now-unused thread affinity macros.

Explicit USED_FROM_MULTIPLE_THREADS*() declarations haven't been used for
some time; remove the unused macros.

R=haraken
BUG=420515
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=184402

[sof, 2014-10-24 12:45:07]

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

[sof, 2014-10-24 12:45:08]

Please take a look.

[ Don't think I will miss them much :-) ]

[haraken, 2014-10-24 13:29:44]

LGTM.

We need to have a better way to indicate that a given class hierarchy is
MainThreadOnly. I uploaded a CL here
(https://codereview.chromium.org/661603005/) but the solution was not complete
(See Slava's comment).

[sof, 2014-10-24 13:36:01]

On 2014/10/24 13:29:44, haraken wrote:
> LGTM.
> 
> We need to have a better way to indicate that a given class hierarchy is
> MainThreadOnly. I uploaded a CL here
> (https://codereview.chromium.org/661603005/) but the solution was not complete
> (See Slava's comment).

Oh, sorry, that CL passed me by. Taking a look now.

[haraken, 2014-10-24 13:40:55]

On 2014/10/24 13:36:01, sof wrote:
> On 2014/10/24 13:29:44, haraken wrote:
> > LGTM.
> > 
> > We need to have a better way to indicate that a given class hierarchy is
> > MainThreadOnly. I uploaded a CL here
> > (https://codereview.chromium.org/661603005/) but the solution was not
complete
> > (See Slava's comment).
> 
> Oh, sorry, that CL passed me by. Taking a look now.

Either way, this CL looks good.

It turned out that it's not realistic to make MainThreadOnly by default with
UsedByMultipleThread annotations. We need to invent a way to make
UsedByMultipleThread by default with MainThreadOnly annotations.

[sof, 2014-10-25 07:06:13]

On 2014/10/24 13:40:55, haraken wrote:
> On 2014/10/24 13:36:01, sof wrote:
> > On 2014/10/24 13:29:44, haraken wrote:
> > > LGTM.
> > > 
> > > We need to have a better way to indicate that a given class hierarchy is
> > > MainThreadOnly. I uploaded a CL here
> > > (https://codereview.chromium.org/661603005/) but the solution was not
> complete
> > > (See Slava's comment).
> > 
> > Oh, sorry, that CL passed me by. Taking a look now.
> 
> Either way, this CL looks good.
> 
> It turned out that it's not realistic to make MainThreadOnly by default with
> UsedByMultipleThread annotations. We need to invent a way to make
> UsedByMultipleThread by default with MainThreadOnly annotations.

Yes, I see how that CL's approach runs into problems with the persistent on
RefCountedGarbageCollected. Hmm.

[sof, 2014-10-25 07:06:55]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2014-10-25 07:07:58]

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

[commit-bot: I haz the power, 2014-10-25 09:51:47]

Committed patchset #1 (id:1) as 184402