tabs/tracing-support: Refactorings for uniffi_parse_rs

components/support/tracing/src/layer.rs

@@ -14,7 +14,7 @@ use tracing_subscriber::{
     Layer,
 };
 
-use crate::{EventSink, Level};
+use crate::{EventSink, TracingLevel};
 use tracing::field::{Field, Visit};
 
 static SINKS: RwLock<Vec<RegisteredEventSink>> = const_rwlock(Vec::new());
@@ -52,13 +52,13 @@ pub struct EventSinkSpecification {
     pub targets: Vec<EventTarget>,
     // Send events have a `min_level` or above.
     #[uniffi(default)]
-    pub min_level: Option<Level>,
+    pub min_level: Option<TracingLevel>,
 }
 
 #[derive(uniffi::Record, Debug)]
 pub struct EventTarget {
     pub target: String,
-    pub level: Level,
+    pub level: TracingLevel,
 }
 
 #[uniffi::export]
@@ -124,7 +124,7 @@ fn find_sinks_for_event(event: &tracing::Event<'_>) -> Vec<Arc<dyn EventSink>> {
         Some(index) => &target[..index],
         None => target,
     };
-    let level = Level::from(*event.metadata().level());
+    let level = TracingLevel::from(*event.metadata().level());
 
     // This requires a iterating through the entire SINKS vec, which could have performance impacts
     // if we have many sinks registered.  In practice, there should only be a handful of these so

components/support/tracing/src/lib.rs

@@ -12,8 +12,8 @@ pub use filters::{
 };
 
 pub use layer::{
-    register_event_sink, simple_event_layer, unregister_event_sink, EventSinkId,
-    EventSinkSpecification, EventTarget,
+    register_event_sink, register_event_sink_box, simple_event_layer, unregister_event_sink,
+    EventSinkId, EventSinkSpecification, EventTarget,
 };
 // Re-export tracing so that our dependencies can use it.
 pub use tracing;
@@ -152,7 +152,7 @@ pub struct TracingEvent {
     pub target: String,
     pub name: String,
     pub message: String,
-    pub fields: serde_json::Value,
+    pub fields: TracingJsonValue,
 }
 
 #[uniffi::export(callback_interface)]

components/tabs/src/lib.rs

@@ -30,7 +30,9 @@ uniffi::custom_type!(TabsGuid, String, {
     lower: |obj| obj.into(),
 });
 
-pub use crate::storage::{ClientRemoteTabs, LocalTabsInfo, RemoteTabRecord, TabsDeviceType};
+pub use crate::storage::{
+    ClientRemoteTabs, LocalTabsInfo, RemoteTabRecord, TabGroup, TabsDeviceType, Window, WindowType,
+};
 pub use crate::store::{RemoteCommandStore, TabsStore};
 pub use error::{ApiResult, Error, Result, TabsApiError};
 use sync15::DeviceType;