Remove old Document Picture in Picture API entirely
Finish removal of the older document PiP API, since everybody has
migrated to the new one. All that was left of the old one is an
error message saying to use the new one.
Bug: 1361409
Change-Id: I461a7aa1be2a54741a3bbf30f4814439e240fa68
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3934464
Reviewed-by: Fr <beaufort.francois@gmail.com>
Commit-Queue: Fr <beaufort.francois@gmail.com>
Auto-Submit: Frank Liberato <liberato@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1055147}
diff --git a/third_party/blink/renderer/bindings/idl_in_modules.gni b/third_party/blink/renderer/bindings/idl_in_modules.gni
index 4b00617..e844cef5 100644
--- a/third_party/blink/renderer/bindings/idl_in_modules.gni
+++ b/third_party/blink/renderer/bindings/idl_in_modules.gni
@@ -212,7 +212,6 @@
"//third_party/blink/renderer/modules/document_picture_in_picture/document_picture_in_picture.idl",
"//third_party/blink/renderer/modules/document_picture_in_picture/document_picture_in_picture_session.idl",
"//third_party/blink/renderer/modules/document_picture_in_picture/navigator_document_picture_in_picture.idl",
- "//third_party/blink/renderer/modules/document_picture_in_picture/window_picture_in_picture.idl",
"//third_party/blink/renderer/modules/donottrack/navigator_do_not_track.idl",
"//third_party/blink/renderer/modules/encoding/text_decode_options.idl",
"//third_party/blink/renderer/modules/encoding/text_decoder.idl",
diff --git a/third_party/blink/renderer/modules/document_picture_in_picture/BUILD.gn b/third_party/blink/renderer/modules/document_picture_in_picture/BUILD.gn
index bd5a328..99d5ed3a 100644
--- a/third_party/blink/renderer/modules/document_picture_in_picture/BUILD.gn
+++ b/third_party/blink/renderer/modules/document_picture_in_picture/BUILD.gn
@@ -12,8 +12,6 @@
"document_picture_in_picture_session.h",
"picture_in_picture_controller_impl.cc",
"picture_in_picture_controller_impl.h",
- "window_picture_in_picture.cc",
- "window_picture_in_picture.h",
]
deps = [ "//third_party/blink/renderer/modules/picture_in_picture" ]
diff --git a/third_party/blink/renderer/modules/document_picture_in_picture/window_picture_in_picture.cc b/third_party/blink/renderer/modules/document_picture_in_picture/window_picture_in_picture.cc
deleted file mode 100644
index 118d0e1c..0000000
--- a/third_party/blink/renderer/modules/document_picture_in_picture/window_picture_in_picture.cc
+++ /dev/null
@@ -1,25 +0,0 @@
-// Copyright 2022 The Chromium Authors
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include "third_party/blink/renderer/modules/document_picture_in_picture/window_picture_in_picture.h"
-
-#include "third_party/blink/renderer/bindings/core/v8/script_promise_resolver.h"
-#include "third_party/blink/renderer/platform/bindings/exception_code.h"
-#include "third_party/blink/renderer/platform/bindings/exception_state.h"
-
-namespace blink {
-
-// static
-ScriptPromise WindowPictureInPicture::requestPictureInPictureWindow(
- ScriptState* script_state,
- LocalDOMWindow& window,
- PictureInPictureWindowOptions* options,
- ExceptionState& exception_state) {
- exception_state.ThrowDOMException(
- DOMExceptionCode::kAbortError,
- "Please use navigator.documentPictureInPicture.requestWindow() instead.");
- return ScriptPromise();
-}
-
-} // namespace blink
diff --git a/third_party/blink/renderer/modules/document_picture_in_picture/window_picture_in_picture.h b/third_party/blink/renderer/modules/document_picture_in_picture/window_picture_in_picture.h
deleted file mode 100644
index 8dff06b..0000000
--- a/third_party/blink/renderer/modules/document_picture_in_picture/window_picture_in_picture.h
+++ /dev/null
@@ -1,31 +0,0 @@
-// Copyright 2022 The Chromium Authors
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#ifndef THIRD_PARTY_BLINK_RENDERER_MODULES_DOCUMENT_PICTURE_IN_PICTURE_WINDOW_PICTURE_IN_PICTURE_H_
-#define THIRD_PARTY_BLINK_RENDERER_MODULES_DOCUMENT_PICTURE_IN_PICTURE_WINDOW_PICTURE_IN_PICTURE_H_
-
-#include "third_party/blink/renderer/core/dom/qualified_name.h"
-
-namespace blink {
-
-class ExceptionState;
-class LocalDOMWindow;
-class PictureInPictureWindowOptions;
-class ScriptPromise;
-class ScriptState;
-
-class WindowPictureInPicture {
- STATIC_ONLY(WindowPictureInPicture);
-
- public:
- static ScriptPromise requestPictureInPictureWindow(
- ScriptState*,
- LocalDOMWindow&,
- PictureInPictureWindowOptions*,
- ExceptionState&);
-};
-
-} // namespace blink
-
-#endif // THIRD_PARTY_BLINK_RENDERER_MODULES_DOCUMENT_PICTURE_IN_PICTURE_WINDOW_PICTURE_IN_PICTURE_H_
diff --git a/third_party/blink/renderer/modules/document_picture_in_picture/window_picture_in_picture.idl b/third_party/blink/renderer/modules/document_picture_in_picture/window_picture_in_picture.idl
deleted file mode 100644
index ca9f2ae..0000000
--- a/third_party/blink/renderer/modules/document_picture_in_picture/window_picture_in_picture.idl
+++ /dev/null
@@ -1,16 +0,0 @@
-// Copyright 2022 The Chromium Authors
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-[
- ImplementedAs=WindowPictureInPicture,
- RuntimeEnabled=DocumentPictureInPictureAPI,
- SecureContext
-]
-partial interface Window {
- // TODO(crbug.com/1361409): This call will be removed before
- // DocumentPictureInPictureAPI is enabled. It's just for backwards
- // compatibility temporarily, until the new API via
- // DocumentPictureInPicture is ready to replace it.
- [CallWith=ScriptState, RaisesException, Measure, NewObject] Promise<DocumentPictureInPictureSession> requestPictureInPictureWindow(optional PictureInPictureWindowOptions options = {});
-};
