Problems with compressing named anonymous functions

[Wiktor Walc]

When compressing the following code:

function a() {
	return function b(){};
};

it is changed into:

function a(){return function b(){b=;};;};

note that in the final result there is a javascript error: b=;

[Wiktor Walc]

It was causing problems in CKFinder and it may be a matter of time when we experience this bug in CKEditor.

[Garry Yao]

Proposing a simplified patch.

[Wiktor Walc]

Using function type looks smart. Please check the third patch I have uploaded - it works in the same way (at least I hope so). I'd prefer using there constants instead of numbers (FUNCTION_EXPRESSION vs 2, we don't need to keep the CKPackager code as short as possible) + I made one conditional statement simpler around L377.

So R+ from me for the second patch, if the third one looks good to you, go ahead ad commit it.

[Garry Yao]

Fixed with [6330:6331].