Disallow reentrance of FrameView::updateLifecyclePhasesInternal()

Disallow reentrance of FrameView::updateLifecyclePhasesInternal()

Reentrance of FrameView::updateLifecyclePhasesInternal() is bad.
For example, when we are invalidating paint, we call a function which
looks like a non-side-affecting getter which unexpectedly initiates
another lifecycle update which may unexpectedly delete the object
that the caller is processing.

This CL checks reentrance of FrameView::updateLifecyclePhasesInternal(),
and crash with NOTREACHED() if DCHECK is enabled or early return
otherwise.

BUG=590856, 621360
Review-Url: https://codereview.chromium.org/2310453002
Cr-Commit-Position: refs/heads/master@{#416712}

[Xianzhu, 2016-09-07 22:51:29]

Committed patchset #1 (id:1) to pending queue manually as
1c038161ca3bb2ee8118ede9d051ac134253d834.