Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

[Android] Add disposed check on FastRenderers.VisualElementRenderer #888

Merged
merged 1 commit into from Apr 28, 2017

Conversation

bentmar
Copy link
Contributor

@bentmar bentmar commented Apr 27, 2017

Description of Change

Added disposed check on UpdateBackgroundColor()

Bugs Fixed

None

API Changes

None

Behavioral Changes

Prevents crashes when element is disposed

PR Checklist

  • Has tests (if omitted, state reason in description)
  • [ X] Rebased on top of master at time of PR
  • [ X] Changes adhere to coding standard
  • [X ] Consolidate commits as makes sense

@dnfclas
Copy link

dnfclas commented Apr 27, 2017

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@bentmar bentmar changed the title Add disposed check on FastRenderers.VisualElementRenderer [Android] Add disposed check on FastRenderers.VisualElementRenderer Apr 27, 2017
@rmarinho rmarinho merged commit e843a9a into xamarin:master Apr 28, 2017
assemhakmeh added a commit to assemhakmeh/Xamarin.Forms that referenced this pull request May 12, 2017
* 2.3.5:
  [C] fix BP declaring Types for A11y (xamarin#913)
  [All] Rename Accessibility -> AutomationProperties (xamarin#912)
  Fix build
  [Core] Accessibility works with XamlC (xamarin#911)
  Fix bugzilla55542 (xamarin#898)
  [Android] Add null check before creating and drawing canvas bugzilla 55559 & 45602 (xamarin#899)
  Give layouts a chance to handle touch events after passing though immediate children (xamarin#902)
  Add disposed check on FastRenderer.VisualElemen... (xamarin#888)
  [Android] Add disposed check on FastRenderers.LabelRenderer (xamarin#886)
  add disposed check on FastRenderers.FrameRenderer (xamarin#887)
  [iOS] Null Element before disposing trial renderers in uneven ListViews (xamarin#894)
  [Xaml] move ValueConverterProvider to Core (xamarin#890)
@samhouts samhouts added this to the 2.3.5 milestone Jun 27, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants