Fix issue with setUserVisibleHint() being called too early.

Fix issue with setUserVisibleHint() being called too early.

This caused a crash bug in FRE code on some versions of
Android where setUserVisibleHint() gets called before
onViewCreated().

This was broken by my CL from this morning:
https://codereview.chromium.org/2561433002

BUG=632199
NOTRY=true

Review-Url: https://codereview.chromium.org/2628943002
Cr-Commit-Position: refs/heads/master@{#443026}
Committed: https://chromium.googlesource.com/chromium/src/+/f4f7c520efaa03663cd8c19662f915bf313bfe7b

[Alexei Svitkine (slow), 2017-01-11 19:34:30]

PTAL

[Alexei Svitkine (slow), 2017-01-11 19:35:55]

Description was changed from

==========
Fix issue with setUserVisibleHint() being called too early.

This caused a crash bug in FRE code on some versions of
Android where setUserVisibleHint() before onViewCreated().

This was broken by my CL from this morning:
https://codereview.chromium.org/2561433002

BUG=632199
TBR=tedchoc@chromium.org
==========

to

==========
Fix issue with setUserVisibleHint() being called too early.

This caused a crash bug in FRE code on some versions of
Android where setUserVisibleHint() gets called before
onViewCreated().

This was broken by my CL from this morning:
https://codereview.chromium.org/2561433002

BUG=632199
TBR=tedchoc@chromium.org
==========

[Ted C, 2017-01-11 19:36:48]

lgtm

[Alexei Svitkine (slow), 2017-01-11 19:36:58]

Description was changed from

==========
Fix issue with setUserVisibleHint() being called too early.

This caused a crash bug in FRE code on some versions of
Android where setUserVisibleHint() gets called before
onViewCreated().

This was broken by my CL from this morning:
https://codereview.chromium.org/2561433002

BUG=632199
TBR=tedchoc@chromium.org
==========

to

==========
Fix issue with setUserVisibleHint() being called too early.

This caused a crash bug in FRE code on some versions of
Android where setUserVisibleHint() gets called before
onViewCreated().

This was broken by my CL from this morning:
https://codereview.chromium.org/2561433002

BUG=632199
==========

[Alexei Svitkine (slow), 2017-01-11 19:37:01]

The CQ bit was checked by asvitkine@chromium.org

[commit-bot: I haz the power, 2017-01-11 19:37:53]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-11 19:52:40]

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

[commit-bot: I haz the power, 2017-01-11 19:52:41]

Try jobs failed on following builders:
  android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)

[Alexei Svitkine (slow), 2017-01-11 20:05:44]

The CQ bit was checked by asvitkine@chromium.org

[commit-bot: I haz the power, 2017-01-11 20:06:43]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-11 20:17:51]

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

[commit-bot: I haz the power, 2017-01-11 20:17:52]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[Alexei Svitkine (slow), 2017-01-11 20:20:07]

Description was changed from

==========
Fix issue with setUserVisibleHint() being called too early.

This caused a crash bug in FRE code on some versions of
Android where setUserVisibleHint() gets called before
onViewCreated().

This was broken by my CL from this morning:
https://codereview.chromium.org/2561433002

BUG=632199
==========

to

==========
Fix issue with setUserVisibleHint() being called too early.

This caused a crash bug in FRE code on some versions of
Android where setUserVisibleHint() gets called before
onViewCreated().

This was broken by my CL from this morning:
https://codereview.chromium.org/2561433002

BUG=632199
NOTRY=true
==========

[Alexei Svitkine (slow), 2017-01-11 20:20:13]

The CQ bit was checked by asvitkine@chromium.org

[Alexei Svitkine (slow), 2017-01-11 20:20:20]

The CQ bit was unchecked by asvitkine@chromium.org

[Alexei Svitkine (slow), 2017-01-11 20:28:23]

The CQ bit was checked by asvitkine@chromium.org

[commit-bot: I haz the power, 2017-01-11 20:28:54]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Alexei Svitkine (slow), 2017-01-11 20:29:29]

Setting NOTRY=true because cq is not landing it due failing "compile (without
patch) " step due to lint error in some translated grit strings. :(

[commit-bot: I haz the power, 2017-01-11 22:44:22]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484166503199630, "parent_rev":
"829fa5d5abc060c3fca83c2324bb8ce47a43361c", "commit_rev":
"f4f7c520efaa03663cd8c19662f915bf313bfe7b"}

[commit-bot: I haz the power, 2017-01-11 22:44:52]

Description was changed from

==========
Fix issue with setUserVisibleHint() being called too early.

This caused a crash bug in FRE code on some versions of
Android where setUserVisibleHint() gets called before
onViewCreated().

This was broken by my CL from this morning:
https://codereview.chromium.org/2561433002

BUG=632199
NOTRY=true
==========

to

==========
Fix issue with setUserVisibleHint() being called too early.

This caused a crash bug in FRE code on some versions of
Android where setUserVisibleHint() gets called before
onViewCreated().

This was broken by my CL from this morning:
https://codereview.chromium.org/2561433002

BUG=632199
NOTRY=true

Review-Url: https://codereview.chromium.org/2628943002
Cr-Commit-Position: refs/heads/master@{#443026}
Committed:
https://chromium.googlesource.com/chromium/src/+/f4f7c520efaa03663cd8c19662f9...
==========

[commit-bot: I haz the power, 2017-01-11 22:44:53]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/f4f7c520efaa03663cd8c19662f9...