Remove unnecessary clone from ServiceWorkerManager::prepare_activation

[wdv4758h]

remove unnecessary for scope_url

• ./mach build -d does not report any errors
• ./mach test-tidy does not report any errors
• These changes fix #12999
• These changes do not require tests because it should works when it compiles

---

This change is

[highfive]

Thanks for the pull request, and welcome! The Servo team is excited to review your changes, and you should hear from @asajeffrey (or someone else) soon.

[highfive]

Heads up! This PR modifies the following files:

• @KiChjang: components/script/serviceworker_manager.rs

[highfive]

Warning

• These commits modify script code, but no tests are modified. Please consider adding a test!

[jdm]

@bors-servo: r+
Thanks!

[bors-servo]

📌 Commit 10cf2ee has been approved by jdm

[bors-servo]

⌛ Testing commit 10cf2ee with merge d93597c...

[bors-servo]

💔 Test failed - mac-rel-wpt

[izgzhen]

I don't see why this will fail... @jdm idea?

{"status": "TIMEOUT", "line": 20933, "subtest": null, "action": "test_result", "test": "/XMLHttpRequest/send-usp.worker", "message": null, "stack": null, "expected": "OK"}

[KiChjang]

Ran 6850 tests finished in 2126.0 seconds.
  • 6849 ran as expected. 1660 tests skipped.
  • 1 tests timed out unexpectedly

Tests with unexpected results:
  ▶ TIMEOUT [expected OK] /XMLHttpRequest/send-usp.worker
  │ 
  │ Stack trace for thread "ScriptThread PipelineId { namespace_id: PipelineNamespaceId(0), index: PipelineIndex(0) }"
  │ stack backtrace:
  │    0:        0x110b5469e - backtrace::backtrace::trace::h22a738598d75d447
  │    1:        0x110b549ac - backtrace::capture::Backtrace::new::h1a277804460ccff5
  │    2:        0x10f2a0898 - servo::install_crash_handler::handler::h7d01d896c8f0d15d
  │    3:     0x7fff9852df19 - _sigtramp
  │    4:        0x110197e3f - 13JSCompartment4wrapEP9JSContextN2JS13MutableHandleIP8JSObjectEENS2_6HandleIS5_E
  │    5:        0x1101757dd - 13JSCompartment4wrapEP9JSContextN2JS13MutableHandleINS2_5ValueEEENS2_6HandleIP8JSObjectE
  │    6:        0x11015c644 - _Z12JS_WrapValueP9JSContextN2JS13MutableHandleINS1_5ValueEEE
  │    7:        0x10f7a8ac8 - std::panicking::try::do_call::h7f1663ddafd7ad43
  │    8:        0x110beb22a - __rust_maybe_catch_panic
  │    9:        0x10fa849d7 - script::dom::bindings::codegen::Bindings::MessageEventBinding::MessageEventBinding::get_data::h9dcb7acd957cbcbf
  │ Stack trace for thread "ScriptThread PipelineId { namespace_id: PipelineNamespaceId(0), index: PipelineIndex(0) }"
  │ stack backtrace:
  │    0:        0x110b5469e - backtrace::backtrace::trace::h22a738598d75d447
  │    1:        0x110b549ac - backtrace::capture::Backtrace::new::h1a277804460ccff5
  │    2:        0x10f2a0898 - servo::install_crash_handler::handler::h7d01d896c8f0d15d
  │    3:     0x7fff9852df19 - _sigtramp
  │    4:        0x10f2a091c - servo::install_crash_handler::handler::h7d01d896c8f0d15d
  │    5:     0x7fff9852df19 - _sigtramp
  │    6:        0x110197e3f - 13JSCompartment4wrapEP9JSContextN2JS13MutableHandleIP8JSObjectEENS2_6HandleIS5_E
  │    7:        0x1101757dd - 13JSCompartment4wrapEP9JSContextN2JS13MutableHandleINS2_5ValueEEENS2_6HandleIP8JSObjectE
  │    8:        0x11015c644 - _Z12JS_WrapValueP9JSContextN2JS13MutableHandleINS1_5ValueEEE
  │    9:        0x10f7a8ac8 - std::panicking::try::do_call::h7f1663ddafd7ad43
  │   10:        0x110beb22a - __rust_maybe_catch_panic
  └   11:        0x10fa849d7 - script::dom::bindings::codegen::Bindings::MessageEventBinding::MessageEventBinding::get_data::h9dcb7acd957cbcbf

[KiChjang]

@bors-servo retry

• #12654

[bors-servo]

⌛ Testing commit 10cf2ee with merge a0f45c6...

[bors-servo]

☀️ Test successful - arm32, arm64, linux-dev, linux-rel, mac-dev-unit, mac-rel-css, mac-rel-wpt, windows-dev