commit aecb834e3a3f8cc004d9c4dbb328b6959ecea89e
author Dake Gu <dake@google.com> Mon Apr 11 18:42:25 2016 -0700
committer Dake Gu <dake@google.com> Mon Apr 11 18:47:13 2016 -0700
tree bd4929dacc9eee8bc4ec7e3aa3a93d44f363906e
parent 8cbc65a17b99e7f7c93d20ab3fc74de07f38ee3d

Fix NPE in notifyViewCreated()

Change-Id: I9ad6f3dc641fc37f86aa0bbd388968ecb309e51f

samples/SupportLeanbackDemos/src/com/example/android/leanback/BrowseFragment.java

diff --git a/samples/SupportLeanbackDemos/src/com/example/android/leanback/BrowseFragment.java b/samples/SupportLeanbackDemos/src/com/example/android/leanback/BrowseFragment.java
index afd4038..737d2f0 100644
--- a/samples/SupportLeanbackDemos/src/com/example/android/leanback/BrowseFragment.java
+++ b/samples/SupportLeanbackDemos/src/com/example/android/leanback/BrowseFragment.java
@@ -327,13 +327,10 @@
         }
 
         @Override
-        public void onStart() {
-            super.onStart();
-            mMainFragmentAdapter.getFragmentHost().notifyViewCreated(mMainFragmentAdapter);
-        }
-
-        @Override
         public void onViewCreated(View view, Bundle savedInstanceState) {
+            if (mMainFragmentAdapter.getFragmentHost() != null) {
+                mMainFragmentAdapter.getFragmentHost().notifyViewCreated(mMainFragmentAdapter);
+            }
             setEntranceTransitionState(mEntranceTransitionState);
         }
 

samples/SupportLeanbackDemos/src/com/example/android/leanback/BrowseSupportFragment.java

diff --git a/samples/SupportLeanbackDemos/src/com/example/android/leanback/BrowseSupportFragment.java b/samples/SupportLeanbackDemos/src/com/example/android/leanback/BrowseSupportFragment.java
index 02da218..1291a7f 100644
--- a/samples/SupportLeanbackDemos/src/com/example/android/leanback/BrowseSupportFragment.java
+++ b/samples/SupportLeanbackDemos/src/com/example/android/leanback/BrowseSupportFragment.java
@@ -330,6 +330,9 @@
 
         @Override
         public void onViewCreated(View view, Bundle savedInstanceState) {
+            if (mMainFragmentAdapter.getFragmentHost() != null) {
+                mMainFragmentAdapter.getFragmentHost().notifyViewCreated(mMainFragmentAdapter);
+            }
             setEntranceTransitionState(mEntranceTransitionState);
         }
 

samples/SupportLeanbackDemos/src/com/example/android/leanback/PlaybackOverlaySupportActivity.java

diff --git a/samples/SupportLeanbackDemos/src/com/example/android/leanback/PlaybackOverlaySupportActivity.java b/samples/SupportLeanbackDemos/src/com/example/android/leanback/PlaybackOverlaySupportActivity.java
index f965b37..e1b01e1 100644
--- a/samples/SupportLeanbackDemos/src/com/example/android/leanback/PlaybackOverlaySupportActivity.java
+++ b/samples/SupportLeanbackDemos/src/com/example/android/leanback/PlaybackOverlaySupportActivity.java
@@ -48,6 +48,6 @@
     }
 
     public interface PictureInPictureListener {
-        void onPictureInPictureModeChanged(boolean isInPictureInPicture);
+        void onPictureInPictureModeChanged(boolean isInPictureInPictureMode);
     }
 }