From c995aa2cfd62f90baae9672e80d78b6554503e82 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Alexandre=20K=C3=BChn?= <aku@odoo.com>
Date: Thu, 16 Aug 2018 13:31:02 +0200
Subject: [FIX] mail: tests: remove phantomjs hack with discuss

Revision on https://github.com/odoo/odoo/commit/e0a51edd856b36bfdf1cacb4d7e8d70fad7884fa

In the commit above, we added an option to Discuss widget
to render threads specifically for PhantomJS. The reason
for this is that a test was failing, because it needs full
support of CSS flexbox to pass.

However, PhantomJS does not support flexbox very well. As
a result, it was messing with the rendering of threads.
We couldn't make the hack just in the test itself, because
it is already at the moment of rendering the thread that
PhantomJS messes the whole process of the test.

PhantomJS has been replaced with Chrome headless [1], So
the hack is no longer be necessary, as Chrome headless
fully supports CSS flexbox.

[1] https://github.com/odoo/odoo/commit/de84ad9c1f4853cb8a748ee1d0c880081dae9fe2
---
 addons/mail/static/src/xml/thread.xml | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

(limited to 'addons/mail/static/src/xml/thread.xml')

diff --git a/addons/mail/static/src/xml/thread.xml b/addons/mail/static/src/xml/thread.xml
index 2ec6826b51b..55e6d250600 100644
--- a/addons/mail/static/src/xml/thread.xml
+++ b/addons/mail/static/src/xml/thread.xml
@@ -156,9 +156,6 @@
         @param {boolean} [options.displayLoadMore=false]
         @param {string|integer} [options.messagesSeparatorPosition] 'top' or
             message ID, the separator is placed just after this message.
-        @param {boolean} [options.withPhantomJS=false] phantomJS is a mess with flexbox
-            so it may break some features like auto-fetch message on scroll.
-            TODO: remove this option when we no longer use phantomJS
 
                      _____________
                     |             |
@@ -172,7 +169,7 @@
 
     -->
     <t t-name="mail.widget.Thread.Content.ASC">
-        <div t-att-class="options.withPhantomJS ? '' : 'o_mail_thread_content_asc'">
+        <div class="o_mail_thread_content_asc">
             <t t-if="options.messagesSeparatorPosition == 'top'" t-call="mail.MessagesSeparator"/>
             <t t-set="messages" t-value="messages.slice().reverse()"/>
             <t t-if="thread.hasTypingNotification()" t-call="mail.widget.Thread.TypingNotificationBar"/>
@@ -186,9 +183,6 @@
         @param {Object} options
         @param {boolean} [options.displayBottomThreadFreeSpace=false]
         @param {boolean} [options.displayLoadMore=false]
-        @param {boolean} [options.withPhantomJS=false] phantomJS is a mess with flexbox
-            so it may break some features like auto-fetch message on scroll.
-            TODO: remove this option when we no longer use phantomJS
 
                      _____________
                     |             |
@@ -201,7 +195,7 @@
                       DESC Order
     -->
     <t t-name="mail.widget.Thread.Content.DESC">
-        <div t-att-class="options.withPhantomJS ? '' : 'o_mail_thread_content_desc'">
+        <div class="o_mail_thread_content_desc">
             <t t-if="options.displayLoadMore" t-call="mail.widget.Thread.LoadMore"/>
             <t t-call="mail.widget.Thread.Messages"/>
             <t t-if="options.displayBottomThreadFreeSpace">
-- 
cgit v1.2.3