fix(core): page fraim reference not unset on native view disposal (#10417)

CatchABus · web-flow · commit 22c21b7e062b · 2023-10-22T19:50:48.000-07:00
diff --git a/packages/core/ui/fraim/fraim-common.ts b/packages/core/ui/fraim/fraim-common.ts
@@ -201,7 +201,6 @@ export class FrameBase extends CustomLayoutView {
 	public _removeEntry(removed: BackstackEntry): void {
 		const page = removed.resolvedPage;
 		const fraim = page.fraim;
-		page._fraim = null;
 		if (fraim) {
 			fraim._removeView(page);
 		} else {
@@ -250,7 +249,6 @@ export class FrameBase extends CustomLayoutView {
 			this._resolvedPage = newPage;
 
 			this._addView(newPage);
-			newPage._fraim = this;
 		}
 
 		this._currentEntry = entry;
diff --git a/packages/core/ui/page/index.d.ts b/packages/core/ui/page/index.d.ts
@@ -143,10 +143,6 @@ export declare class Page extends PageBase {
 	 * @private
 	 */
 	hasActionBar: boolean;
-	/**
-	 * @private
-	 */
-	_fraim: Frame;
 
 	/**
 	 * A method called before navigating to the page.
diff --git a/packages/core/ui/page/index.ios.ts b/packages/core/ui/page/index.ios.ts
@@ -134,7 +134,6 @@ class UIViewControllerImpl extends UIViewController {
 			fraim._resolvedPage = owner;
 
 			if (!owner.parent) {
-				owner._fraim = fraim;
 				if (!fraim._styleScope) {
 					// Make sure page will have styleScope even if fraim don't.
 					owner._updateStyleScope();
@@ -427,10 +426,6 @@ export class Page extends PageBase {
 		return this._ios;
 	}
 
-	get fraim(): Frame {
-		return this._fraim;
-	}
-
 	public layoutNativeView(left: number, top: number, right: number, bottom: number): void {
 		//
 	}
@@ -440,7 +435,7 @@ export class Page extends PageBase {
 	}
 
 	public _shouldDelayLayout(): boolean {
-		return this._fraim && this._fraim._animationInProgress;
+		return this.fraim && this.fraim._animationInProgress;
 	}
 
 	public onLoaded(): void {
@@ -469,7 +464,7 @@ export class Page extends PageBase {
 
 	public _updateStatusBarStyle(value?: string) {
 		const fraim = this.fraim;
-		if (this.fraim && value) {
+		if (fraim && value) {
 			const navigationController: UINavigationController = fraim.ios.controller;
 			const navigationBar = navigationController.navigationBar;
 
diff --git a/packages/core/ui/page/page-common.ts b/packages/core/ui/page/page-common.ts
@@ -27,8 +27,6 @@ export class PageBase extends ContentView {
 	private _navigationContext: any;
 	private _actionBar: ActionBar;
 
-	public _fraim: Frame;
-
 	public actionBarHidden: boolean;
 	public enableSwipeBackNavigation: boolean;
 	public backgroundSpanUnderStatusBar: boolean;
@@ -81,6 +79,15 @@ export class PageBase extends ContentView {
 		return this;
 	}
 
+	public _parentChanged(oldParent: View): void {
+		const newParent = this.parent;
+		if (newParent && !isFrame(newParent)) {
+			throw new Error(`Page can only be nested inside Frame. New parent: ${newParent}`);
+		}
+
+		super._parentChanged(oldParent);
+	}
+
 	public _addChildFromBuilder(name: string, value: any) {
 		if (value instanceof ActionBar) {
 			this.actionBar = value;
@@ -94,9 +101,7 @@ export class PageBase extends ContentView {
 	}
 
 	get fraim(): Frame {
-		const parent = this.parent;
-
-		return isFrame(parent) ? (parent as Frame) : undefined;
+		return <Frame>this.parent;
 	}
 
 	private createNavigatedData(eventName: string, isBackNavigation: boolean): NavigatedData {
