Skip to content

Commit ac126be

Browse files
adampAndroid (Google) Code Review
authored andcommitted
Merge "Fix bug 5226097 - App compatibility issues around new ListView fixes"
2 parents b3db5b7 + 247a0f0 commit ac126be

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

core/java/android/widget/ListView.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -264,7 +264,7 @@ public void addHeaderView(View v, Object data, boolean isSelectable) {
264264

265265
// in the case of re-adding a header view, or adding one later on,
266266
// we need to notify the observer
267-
if (mDataSetObserver != null) {
267+
if (mAdapter != null && mDataSetObserver != null) {
268268
mDataSetObserver.onChanged();
269269
}
270270
}
@@ -299,7 +299,7 @@ public int getHeaderViewsCount() {
299299
public boolean removeHeaderView(View v) {
300300
if (mHeaderViewInfos.size() > 0) {
301301
boolean result = false;
302-
if (((HeaderViewListAdapter) mAdapter).removeHeader(v)) {
302+
if (mAdapter != null && ((HeaderViewListAdapter) mAdapter).removeHeader(v)) {
303303
if (mDataSetObserver != null) {
304304
mDataSetObserver.onChanged();
305305
}
@@ -350,7 +350,7 @@ public void addFooterView(View v, Object data, boolean isSelectable) {
350350

351351
// in the case of re-adding a footer view, or adding one later on,
352352
// we need to notify the observer
353-
if (mDataSetObserver != null) {
353+
if (mAdapter != null && mDataSetObserver != null) {
354354
mDataSetObserver.onChanged();
355355
}
356356
}
@@ -384,7 +384,7 @@ public int getFooterViewsCount() {
384384
public boolean removeFooterView(View v) {
385385
if (mFooterViewInfos.size() > 0) {
386386
boolean result = false;
387-
if (((HeaderViewListAdapter) mAdapter).removeFooter(v)) {
387+
if (mAdapter != null && ((HeaderViewListAdapter) mAdapter).removeFooter(v)) {
388388
if (mDataSetObserver != null) {
389389
mDataSetObserver.onChanged();
390390
}

0 commit comments

Comments
 (0)