Toggle [aria-busy] instead of [busy]

Since `<turbo-frame>` elements are custom elements, the framework has
total control over the names of the attributes.

There are existing semantics for what we've introduced as `[busy]`: the
ARIA guidelines suggest toggling [aria-busy="true"][aria-busy] when an
element is loading more content, and `aria-busy="false"` when the
content is loaded.

This provides an "interface" for loading styles through CSS attribute
selectors, and hints to assistive technologies the state of the frame.

As an alternative, we could continue to toggle the `[busy]` attribute,
and encourage consumer applications to monitor mutations to the `[busy]`
attribute (or listen to [`turbo:frame-visit` and `turbo:frame-load`
events][events]) to toggle it themselves.

[aria-busy]: https://www.w3.org/TR/wai-aria-1.1/#aria-busy
[events]: #59

src/core/frames/frame_controller.ts

@@ -129,7 +129,7 @@ export class FrameController implements AppearanceObserverDelegate, FetchRequest
   }
 
   requestStarted(request: FetchRequest) {
-    this.element.setAttribute("busy", "")
+    this.element.setAttribute("aria-busy", "true")
   }
 
   requestPreventedHandlingResponse(request: FetchRequest, response: FetchResponse) {
@@ -152,7 +152,7 @@ export class FrameController implements AppearanceObserverDelegate, FetchRequest
   }
 
   requestFinished(request: FetchRequest) {
-    this.element.removeAttribute("busy")
+    this.element.removeAttribute("aria-busy")
   }
 
   // Form submission delegate

src/tests/functional/frame_tests.ts

@@ -6,12 +6,12 @@ export class FrameTests extends FunctionalTestCase {
   }
 
   async "test following a link driving a frame toggles the [busy] attribute"() {
-    await this.observeAttributeChanges("frame", "busy")
+    await this.observeAttributeChanges("frame", "aria-busy")
     await this.clickSelector("#hello a")
     await this.nextBeat
 
-    this.assert.ok(await this.hasSelector("turbo-frame[id=frame][data-was-busy=true]"))
-    this.assert.notOk(await this.hasSelector("turbo-frame[id=frame][busy]"))
+    this.assert.ok(await this.hasSelector("turbo-frame[id=frame][data-was-aria-busy=true]"))
+    this.assert.notOk(await this.hasSelector("turbo-frame[id=frame][aria-busy]"))
   }
 
   async "test following a link to a page without a matching frame results in an empty frame"() {