🔥Not all methods could|should be updated

[theKashey]

Each component consist of 2 different pieces:

1. Class methods. They exists on prototype and could be updated just by replacing the old prototype by a new one.
2. Class properties. Their initiation occurs in class constructor.

To update second ones RHL creates 2 components - old and new, then compares all descriptors.
Comparison ignores "objects", thus keeps state untouched, and updates functions and simple variables.

But that does mean "updates functions"?
It literally cast function toString and then eval that code in the context of Class.

The problem

Casting function toString defines what is it, not how it was created.
Before, RHL did update, only if "text" got changes, but it was changed #949 and now they always got updated.

This causes #995, #978, #969, #984

RHL have rights only to update "own" functions, but could not detect them.

Solution

1. Revert v4 does not use updated non-React modules #949
2. Update "arrow" functions only if text got changed. Or if "this" keysword is found inside (probably should fix v4 does not use updated non-React modules #949 back)
3. Update bound class methods. Ie those ones, which names exists on prototype.

[theKashey]

Look like it is impossible to properly "hot" update class, and even heavy babel magic will not work.
We should not try to find a way to fix the problem, which does not exists.

The right way:

1. Create a new class
2. Copy state and all variables you have to keep from old component to the new one
3. Switch to the new component.

Not the change current component, but adopt a new one

This require a one line change in React-Fiber - https://github.com/facebook/react/blob/dd5fad29616f706f484938663e93aaadd2a5e594/packages/react-reconciler/src/ReactChildFiber.js#L375
The similar changes as (not yet) was made to Preact - preactjs/preact#1120

[theKashey]

fixed in v4.3.0

[xinggangling]

nice