Don't morph frames flagged with "refresh=morph" as part of the full p…

…age refresh

We will refresh those frames with morphing as part of the page refresh, so it does
not make sense to morph them also as part of the full page refresh. If we do, we'll
trigger a manual reload because the complete attribute will get removed. This aligns
the upstreamed version with the private gem we've been using internally.

This also fixes a couple of issues:

- We don't want to manually reload all the remote turbo-frames, only those that are
flagged with "refresh=morph". Regular remote frames will get reloaded automatically
when removing their complete attribute during regular page refreshes.

- Using idiomorph's "innerHTML" was resulting in a turbo-frame nested inside the
target turbo-frame. I think its semantics is not morphing inner contents from both
currentElement and newElement, but morphing newElement as the inner contents of currentElement.

See #1019 (comment)

src/core/drive/morph_renderer.js

@@ -26,6 +26,8 @@ export class MorphRenderer extends Renderer {
   }
 
   #morphElements(currentElement, newElement, morphStyle = "outerHTML") {
+    this.isMorphingTurboFrame = this.isFrameReloadedWithMorph(currentElement)
+
     Idiomorph.morph(currentElement, newElement, {
       morphStyle: morphStyle,
       callbacks: {
@@ -40,8 +42,8 @@ export class MorphRenderer extends Renderer {
     this.#remoteFrames().forEach((frame) => {
       if (this.#isFrameReloadedWithMorph(frame)) {
         this.#renderFrameWithMorph(frame)
+        frame.reload()
       }
-      frame.reload()
     })
   }
 
@@ -56,7 +58,7 @@ export class MorphRenderer extends Renderer {
       target: currentElement,
       detail: { currentElement, newElement }
     })
-    this.#morphElements(currentElement, newElement, "innerHTML")
+    this.#morphElements(currentElement, newElement)
   }
 
   #shouldRemoveElement = (node) => {
@@ -65,7 +67,7 @@ export class MorphRenderer extends Renderer {
 
   #shouldMorphElement = (node) => {
     if (node instanceof HTMLElement) {
-      return !node.hasAttribute("data-turbo-permanent")
+      return !node.hasAttribute("data-turbo-permanent") && (this.isMorphingTurboFrame || !this.#isFrameReloadedWithMorph(node))
     } else {
       return true
     }