Auto merge of #29931 - mrobinson:update-uuid, r=jdm

Update uuid

This is required to use the new version of gleam and update ipc-channels
without duplicating the dependency in the future.

<!-- Please describe your changes on the following line: -->

---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: -->
- [x] `./mach build -d` does not report any errors
- [x] `./mach test-tidy` does not report any errors
- [x] There are tests for these changes

<!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.-->

<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->

Cargo.toml

@@ -75,7 +75,7 @@ tokio = "1"
 unicode-bidi = "0.3.4"
 unicode-script = "0.5"
 url = "2.0"
-uuid = { version = "0.8", features = ["v4"] }
+uuid = { version = "1.3.4", features = ["v4"] }
 webdriver = "0.48.0"
 winapi = "0.3"
 xi-unicode = "0.1.0"

components/net/http_loader.rs

@@ -1683,7 +1683,7 @@ async fn http_network_fetch(
     let request_id = context
         .devtools_chan
         .as_ref()
-        .map(|_| uuid::Uuid::new_v4().to_simple().to_string());
+        .map(|_| uuid::Uuid::new_v4().simple().to_string());
 
     if log_enabled!(log::Level::Info) {
         info!("{:?} request for {}", request.method, url);

components/net/tests/fetch.rs

@@ -198,7 +198,7 @@ fn test_fetch_blob() {
         true,
         "http://www.example.org".into(),
     );
-    let url = ServoUrl::parse(&format!("blob:{}{}", origin.as_str(), id.to_simple())).unwrap();
+    let url = ServoUrl::parse(&format!("blob:{}{}", origin.as_str(), id.simple())).unwrap();
 
     let mut request = Request::new(
         url,

components/rand/lib.rs

@@ -27,7 +27,7 @@ use std::cell::RefCell;
 use std::rc::Rc;
 use std::sync::Mutex;
 use std::u64;
-use uuid::{Builder, Uuid, Variant, Version};
+use uuid::{Builder, Uuid};
 
 // The shared RNG which may hold on to a file descriptor
 lazy_static! {
@@ -197,8 +197,5 @@ where
 pub fn random_uuid() -> Uuid {
     let mut bytes = [0; 16];
     thread_rng().fill_bytes(&mut bytes);
-    Builder::from_bytes(bytes)
-        .set_variant(Variant::RFC4122)
-        .set_version(Version::Random)
-        .build()
+    Builder::from_random_bytes(bytes).into_uuid()
 }

components/script/dom/baseaudiocontext.rs

@@ -447,7 +447,7 @@ impl BaseAudioContextMethods for BaseAudioContext {
         if audio_data.len() > 0 {
             // Step 2.
             // XXX detach array buffer.
-            let uuid = Uuid::new_v4().to_simple().to_string();
+            let uuid = Uuid::new_v4().simple().to_string();
             let uuid_ = uuid.clone();
             self.decode_resolvers.borrow_mut().insert(
                 uuid.clone(),

components/script/dom/node.rs

@@ -1078,15 +1078,15 @@ impl Node {
 
         if rare_data.unique_id.is_none() {
             let id = UniqueId::new();
-            ScriptThread::save_node_id(id.borrow().to_simple().to_string());
+            ScriptThread::save_node_id(id.borrow().simple().to_string());
             rare_data.unique_id = Some(id);
         }
         rare_data
             .unique_id
             .as_ref()
             .unwrap()
             .borrow()
-            .to_simple()
+            .simple()
             .to_string()
     }
 

servo-tidy.toml

@@ -72,6 +72,9 @@ packages = [
 
   # Duplicated by winit/surfman update.
   "raw-window-handle",
+
+  # Temporarily duplicated until ipc-channels and gleam can be upgrded.
+  "uuid",
 ]
 # Files that are ignored for all tidy and lint checks.
 files = [