Oilpan: Fix weak processing for IntersectionObserver::m_root

Oilpan: Fix weak processing for IntersectionObserver::m_root

This is a follow-up fix for https://codereview.chromium.org/1591763003/.
IntersectionObserver::m_root must be cleared in the weak callback when the m_root is dead.

BUG=

Committed: https://crrev.com/2760f06a3c43c8919f4ecae43f460ada20d0990b
Cr-Commit-Position: refs/heads/master@{#369956}

[haraken, 2016-01-16 14:11:07]

haraken@chromium.org changed reviewers:
+ sigbjornf@opera.com, szager@chromium.org

[haraken, 2016-01-16 14:11:07]

PTAL

[sof, 2016-01-16 14:16:36]

lgtm, thanks.

[haraken, 2016-01-16 14:16:56]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-01-16 14:17:02]

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

[commit-bot: I haz the power, 2016-01-16 14:20:25]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-01-16 14:20:25]

Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[haraken, 2016-01-16 14:24:32]

The CQ bit was checked by haraken@chromium.org

[haraken, 2016-01-16 14:24:32]

The patchset sent to the CQ was uploaded after l-g-t-m from sigbjornf@opera.com
Link to the patchset: https://codereview.chromium.org/1594813002/#ps20001
(title: " ")

[commit-bot: I haz the power, 2016-01-16 14:24:41]

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

[sof, 2016-01-16 14:26:12]

https://codereview.chromium.org/1594813002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/heap/Heap.h (right):

https://codereview.chromium.org/1594813002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/heap/Heap.h:121: static inline bool
isHeapObjectAlive(const WeakPtr<T>& ptr)
Do we really want this?

[haraken, 2016-01-16 14:28:59]

Added #if ENABLE(OILPAN) to the call site.

[sof, 2016-01-16 14:32:13]

On 2016/01/16 14:28:59, haraken wrote:
> Added #if ENABLE(OILPAN) to the call site.

lgtm, that aligns with other similar weak callbacks.

[haraken, 2016-01-16 15:36:55]

The CQ bit was checked by haraken@chromium.org

[commit-bot: I haz the power, 2016-01-16 15:36:57]

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

[commit-bot: I haz the power, 2016-01-16 16:26:01]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-01-16 16:26:16]

Failed to apply the patch.

[commit-bot: I haz the power, 2016-01-16 16:26:51]

Description was changed from

==========
Oilpan: Fix weak processing for IntersectionObserver::m_root

This is a follow-up fix for https://codereview.chromium.org/1591763003/.
IntersectionObserver::m_root must be cleared in the weak callback when the
m_root is dead.

BUG=
==========

to

==========
Oilpan: Fix weak processing for IntersectionObserver::m_root

This is a follow-up fix for https://codereview.chromium.org/1591763003/.
IntersectionObserver::m_root must be cleared in the weak callback when the
m_root is dead.

BUG=

Committed: https://crrev.com/2760f06a3c43c8919f4ecae43f460ada20d0990b
Cr-Commit-Position: refs/heads/master@{#369956}
==========

[commit-bot: I haz the power, 2016-01-16 16:26:51]

Patchset 3 (id:??) landed as
https://crrev.com/2760f06a3c43c8919f4ecae43f460ada20d0990b
Cr-Commit-Position: refs/heads/master@{#369956}

[sof, 2016-01-17 21:25:38]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.chromium.org/1596333002/ by sigbjornf@opera.com.

The reason for reverting is: This and the parent change introduced some
failures,


https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Oilpan...

backing out for now..