From 9b1bb1f64a7a559fbd4f71277b2b2a0ff9da07d0 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Alexandre=20K=C3=BChn?= <aku@odoo.com>
Date: Fri, 1 Apr 2022 15:21:46 +0000
Subject: [PATCH] [IMP] mail: introduce AttachmentViewer/onClickVideo

Task-2579306

Part-of: odoo/odoo#88152
---
 .../components/attachment_viewer/attachment_viewer.js | 11 -----------
 .../attachment_viewer/attachment_viewer.xml           |  2 +-
 addons/mail/static/src/models/attachment_viewer.js    | 11 ++++++++++-
 3 files changed, 11 insertions(+), 13 deletions(-)

diff --git a/addons/mail/static/src/components/attachment_viewer/attachment_viewer.js b/addons/mail/static/src/components/attachment_viewer/attachment_viewer.js
index 04be7d37ff5..858742fa8db 100644
--- a/addons/mail/static/src/components/attachment_viewer/attachment_viewer.js
+++ b/addons/mail/static/src/components/attachment_viewer/attachment_viewer.js
@@ -328,17 +328,6 @@ export class AttachmentViewer extends Component {
         this._rotate();
     }
 
-    /**
-     * Called when clicking on embed video player. Stop propagation to prevent
-     * closing the dialog.
-     *
-     * @private
-     * @param {MouseEvent} ev
-     */
-    _onClickVideo(ev) {
-        ev.stopPropagation();
-    }
-
     /**
      * Called when clicking on zoom in icon.
      *
diff --git a/addons/mail/static/src/components/attachment_viewer/attachment_viewer.xml b/addons/mail/static/src/components/attachment_viewer/attachment_viewer.xml
index e2786c1411e..cecc2e4d398 100644
--- a/addons/mail/static/src/components/attachment_viewer/attachment_viewer.xml
+++ b/addons/mail/static/src/components/attachment_viewer/attachment_viewer.xml
@@ -56,7 +56,7 @@
                         <iframe allow="autoplay; encrypted-media" class="o_AttachmentViewer_view o_AttachmentViewer_viewIframe o_AttachmentViewer_youtube w-75 h-100 bg-black border-0" t-att-src="attachmentViewer.attachment.defaultSource" height="315" width="560"/>
                     </t>
                     <t t-if="attachmentViewer.attachment.isVideo">
-                        <video class="o_AttachmentViewer_view o_AttachmentViewer_viewVideo w-75 h-75 bg-black" t-att-class="{ 'o-isMobile w-100 h-100': messaging.device.isMobile }" t-on-click="_onClickVideo" controls="controls">
+                        <video class="o_AttachmentViewer_view o_AttachmentViewer_viewVideo w-75 h-75 bg-black" t-att-class="{ 'o-isMobile w-100 h-100': messaging.device.isMobile }" t-on-click="attachmentViewer.onClickVideo" controls="controls">
                             <source t-att-data-type="attachmentViewer.attachment.mimetype" t-att-src="attachmentViewer.attachment.defaultSource"/>
                         </video>
                     </t>
diff --git a/addons/mail/static/src/models/attachment_viewer.js b/addons/mail/static/src/models/attachment_viewer.js
index 5ed2101536d..0e7b4928608 100644
--- a/addons/mail/static/src/models/attachment_viewer.js
+++ b/addons/mail/static/src/models/attachment_viewer.js
@@ -65,7 +65,7 @@ registerModel({
          * Called when clicking on image. Stop propagation of event to prevent
          * closing the dialog.
          *
-         * @param {MouseEvent} ev
+         * @param {MouseEvent} ev 
          */
         onClickImage(ev) {
             if (this.isDragging) {
@@ -73,6 +73,15 @@ registerModel({
             }
             ev.stopPropagation();
         },
+        /**
+         * Called when clicking on embed video player. Stop propagation to prevent
+         * closing the dialog.
+         *
+         * @param {MouseEvent} ev
+         */
+        onClickVideo(ev) {
+            ev.stopPropagation();
+        },
         /**
          * @private
          * @returns {string}