Skip to content

Commit f570aac

Browse files
maniac103firebird11
authored andcommitted
Fix NPE in dock mode.
Add the call recording button also to the landscape layout. Also properly add (and hide) button spacers. Change-Id: Idc4c7669a78a7445a421bc28bd527f6d423f5c6a
1 parent f70ae4d commit f570aac

4 files changed

Lines changed: 27 additions & 1 deletion

File tree

res/layout-land/call_button_fragment.xml

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -87,6 +87,18 @@
8787
<!-- Separator between 4th (or 5th) button and right padding -->
8888
<View style="@style/VerticalSeparator" />
8989

90+
<com.android.incallui.CallRecordingButton
91+
android:id="@+id/recordButton"
92+
style="@style/InCallButton"
93+
android:src="@drawable/ic_record_holo_dark"
94+
android:contentDescription="@string/onscreenRecordText"
95+
android:visibility="gone"
96+
/>
97+
98+
<View android:id="@+id/recordSpacer"
99+
style="@style/VerticalSeparator"
100+
android:visibility="gone" />
101+
90102
<!-- "Blacklist" -->
91103
<!-- The button used for toggling blackist. -->
92104
<ImageButton android:id="@+id/addBlacklistButton"

res/layout/call_button_fragment.xml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -197,6 +197,10 @@
197197
android:contentDescription="@string/onscreenBlacklistText"
198198
/>
199199

200+
<View android:id="@+id/recordSpacer"
201+
style="@style/VerticalSeparator"
202+
android:visibility="gone" />
203+
200204
<com.android.incallui.CallRecordingButton
201205
android:id="@+id/recordButton"
202206
style="@style/InCallButton"

src/com/android/incallui/CallButtonFragment.java

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,9 @@ public class CallButtonFragment
5959
private ImageButton mModifyCallButton;
6060
private CallRecordingButton mRecordButton;
6161

62+
private View mBlacklistSpacer;
63+
private View mRecordSpacer;
64+
6265
private PopupMenu mAudioModePopup;
6366
private boolean mAudioModePopupVisible;
6467
private View mEndCallButton;
@@ -158,18 +161,21 @@ public void onClick(View v) {
158161

159162
// "Add to black list" button
160163
mBlacklistButton = (ImageButton) parent.findViewById(R.id.addBlacklistButton);
164+
mBlacklistSpacer = parent.findViewById(R.id.blacklistSpacer);
161165
if (BlacklistUtils.isBlacklistEnabled(getActivity())) {
162166
mBlacklistButton.setVisibility(View.VISIBLE);
163167
mBlacklistButton.setOnClickListener(this);
168+
mBlacklistSpacer.setVisibility(View.VISIBLE);
164169
} else {
165170
mBlacklistButton.setVisibility(View.GONE);
171+
mBlacklistSpacer.setVisibility(View.GONE);
166172
}
167173

168174
mModifyCallButton = (ImageButton) parent.findViewById(R.id.modifyCallButton);
169175
mModifyCallButton.setOnClickListener(this);
170176

171177
mRecordButton = (CallRecordingButton) parent.findViewById(R.id.recordButton);
172-
mRecordButton.setOnClickListener(mRecordButton);
178+
mRecordSpacer = parent.findViewById(R.id.recordSpacer);
173179

174180
return parent;
175181
}
@@ -285,6 +291,7 @@ public void showSwap(boolean show) {
285291
@Override
286292
public void showRecording(boolean show) {
287293
mRecordButton.setVisibility(show ? View.VISIBLE : View.GONE);
294+
mRecordSpacer.setVisibility(show ? View.VISIBLE : View.GONE);
288295
}
289296

290297
@Override

src/com/android/incallui/CallRecordingButton.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,14 +28,17 @@ public class CallRecordingButton extends ImageButton
2828

2929
public CallRecordingButton(Context context) {
3030
super(context);
31+
setOnClickListener(this);
3132
}
3233

3334
public CallRecordingButton(Context context, AttributeSet attrs) {
3435
super(context, attrs);
36+
setOnClickListener(this);
3537
}
3638

3739
public CallRecordingButton(Context context, AttributeSet attrs, int defStyle) {
3840
super(context, attrs, defStyle);
41+
setOnClickListener(this);
3942
}
4043

4144
@Override

0 commit comments

Comments
 (0)