Change h5vccAudioConnectors attribute in html_media_element.idl

b/330929788
youtube · Mar 22, 2024 · bf17afe · bf17afe
1 parent 62f49e9
commit bf17afe
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 11 deletions.
diff --git a/cobalt/dom/html_media_element.cc b/cobalt/dom/html_media_element.cc
@@ -642,21 +642,16 @@ void HTMLMediaElement::ScheduleEvent(const scoped_refptr<web::Event>& event) {
   event_queue_.Enqueue(event);
 }
 
-std::string HTMLMediaElement::h5vcc_audio_connectors(
-    script::ExceptionState* exception_state) const {
+std::string HTMLMediaElement::h5vcc_audio_connectors() const {
 #if SB_API_VERSION >= 15
   if (!player_) {
-    web::DOMException::Raise(web::DOMException::kInvalidStateErr,
-                             exception_state);
-    return std::string();
+    return "NOT_SUPPORTED";
   }
 
   std::vector<std::string> configs = player_->GetAudioConnectors();
   return base::JoinString(configs, ";");
 #else   // SB_API_VERSION >= 15
-  web::DOMException::Raise(web::DOMException::kNotSupportedErr,
-                           exception_state);
-  return std::string();
+  return "NOT_SUPPORTED";
 #endif  // SB_API_VERSION >= 15
 }
 

diff --git a/cobalt/dom/html_media_element.h b/cobalt/dom/html_media_element.h
@@ -149,8 +149,7 @@ class HTMLMediaElement : public HTMLElement,
   // Returns semicolon separated names of audio connectors, like
   // "hdmi;bluetooth".
   // TODO(b/267678497): The current interface is tentative, to be refined.
-  std::string h5vcc_audio_connectors(
-      script::ExceptionState* exception_state) const;
+  std::string h5vcc_audio_connectors() const;
 
   // Set max video capabilities.
   void SetMaxVideoCapabilities(const std::string& max_video_capabilities,

diff --git a/cobalt/dom/html_media_element.idl b/cobalt/dom/html_media_element.idl
@@ -66,5 +66,5 @@ interface HTMLMediaElement : HTMLElement {
   // "hdmi;bluetooth".  It raises `NotSupportedError` on apps doesn't support
   // this feature, or `InvalidStateError` if there isn't an active playback.
   // TODO(b/267678497): The current interface is tentative, to be refined.
-  [RaisesException] readonly attribute DOMString h5vccAudioConnectors;
+  readonly attribute DOMString h5vccAudioConnectors;
 };