fix-next: correct raising of layoutChanged event (NativeScript#6457)

tgpetrov · MartoYankov · commit f5cca13a7caa · 2018-10-26T17:03:23.000+03:00
Currently the layoutChanged event can be raised even when there is no change due to safe area calculations.
diff --git a/tns-core-modules/ui/core/view/view.ios.ts b/tns-core-modules/ui/core/view/view.ios.ts
@@ -154,7 +154,8 @@ export class View extends ViewCommon {
     }
 
     public _setNativeViewFrame(nativeView: UIView, frame: CGRect): void {
-        if (!CGRectEqualToRect(nativeView.frame, frame)) {
+        let oldFrame = this._cachedFrame || nativeView.frame;
+        if (!CGRectEqualToRect(oldFrame, frame)) {
             if (traceEnabled()) {
                 traceWrite(this + " :_setNativeViewFrame: " + JSON.stringify(ios.getPositionFromFrame(frame)), traceCategories.Layout);
             }

Original file line number	Diff line number	Diff line change
`@@ -154,7 +154,8 @@ export class View extends ViewCommon {`
`154`	`154`	`}`
`155`	`155`
`156`	`156`	`public _setNativeViewFrame(nativeView: UIView, frame: CGRect): void {`
`157`		`- if (!CGRectEqualToRect(nativeView.frame, frame)) {`
	`157`	`+ let oldFrame = this._cachedFrame \|\| nativeView.frame;`
	`158`	`+ if (!CGRectEqualToRect(oldFrame, frame)) {`
`158`	`159`	`if (traceEnabled()) {`
`159`	`160`	`traceWrite(this + " :_setNativeViewFrame: " + JSON.stringify(ios.getPositionFromFrame(frame)), traceCategories.Layout);`
`160`	`161`	`}`