bug fix: When you click on action in secondary row its ignored for first time.

corochann · corochann · commit 4245b7287226 · 2016-05-09T22:26:37.000+08:00
diff --git a/app/src/main/java/com/corochann/androidtvapptutorial/ui/PlaybackOverlayFragment.java b/app/src/main/java/com/corochann/androidtvapptutorial/ui/PlaybackOverlayFragment.java
@@ -236,8 +236,6 @@ public void onActionClicked(Action action) {
                      * - HighQualityAction
                      * - ClosedCaptioningAction
                      */
-                    notifyChanged(action);
-
                     /* Change icon */
                     if (action instanceof PlaybackControlsRow.ThumbsUpAction ||
                             action instanceof PlaybackControlsRow.ThumbsDownAction ||
@@ -247,6 +245,8 @@ public void onActionClicked(Action action) {
                             action instanceof PlaybackControlsRow.ClosedCaptioningAction) {
                         ((PlaybackControlsRow.MultiAction) action).nextIndex();
                     }
+                    /* Note: notifyChanged must be called after action.nextIndex has been called */
+                    notifyChanged(action);
                 }
             }
         });

Original file line number	Diff line number	Diff line change
`@@ -236,8 +236,6 @@ public void onActionClicked(Action action) {`
`236`	`236`	`* - HighQualityAction`
`237`	`237`	`* - ClosedCaptioningAction`
`238`	`238`	`*/`
`239`		`- notifyChanged(action);`
`240`		`-`
`241`	`239`	`/* Change icon */`
`242`	`240`	`if (action instanceof PlaybackControlsRow.ThumbsUpAction \|\|`
`243`	`241`	`action instanceof PlaybackControlsRow.ThumbsDownAction \|\|`
`@@ -247,6 +245,8 @@ public void onActionClicked(Action action) {`
`247`	`245`	`action instanceof PlaybackControlsRow.ClosedCaptioningAction) {`
`248`	`246`	`((PlaybackControlsRow.MultiAction) action).nextIndex();`
`249`	`247`	`}`
	`248`	`+ /* Note: notifyChanged must be called after action.nextIndex has been called */`
	`249`	`+ notifyChanged(action);`
`250`	`250`	`}`
`251`	`251`	`}`
`252`	`252`	`});`