[BUG] Destruction error

[charsleysa]

Expected Behavior

Chart should be successfully destroyed.

Current Behavior

Chart throws error during destruction.

Possible Solution

Error is thrown on this line with the context being undefined.
Issue is caused on this line due the context being set to undefined

Steps to Reproduce (for bugs)

1. Create a Chart
2. Destroy the chart calling .destroy()

Context

For SPA this can cause issues with state corruption.

Environment

• Chart.js version: 2.4.0 (latest master)
• Browser name and version: Chrome 54
• Link to your project: JSBin

[etimberg]

@charsleysa which version of 2.4 are you using? the latest from master?

[etimberg]

Also, do you have a test case that can reproduce this?

[charsleysa]

@etimberg the issue only occurs on high density screen (such as the Surface Pro 4 screen).
I've added a link to a JSBin which recreates this issue.
I am using the latest from master.

[simonbrunel]

Introduced by that PR, I will fix it for 2.4. Thank you @charsleysa for reporting this issue before we release it.