Revert "tdf#107204: Make sure we always do Unicode Bidi"

This reverts commit e3b7ef45, as it
breaks the testTableKeep part of CppunitTest_sw_ww8export on the
Jenkins_Callgrind tinderbox (and also for me locally) and there is no
fix in progress to address that.

See the bug for a comparision of the old/new layout for the problematic
testcase.

Khaled says it's OK to push this revert till it's clear how to fix the
bug without side-effects.

Change-Id: I669de98ffa8b519024e332ae468808d7e7243df2

sw/source/core/text/porfld.cxx

@@ -228,6 +228,8 @@ void SwFieldPortion::CheckScript( const SwTextSizeInfo &rInf )
                              rSI.GetDefaultDir() :
                              rSI.DirType( IsFollow() ? rInf.GetIdx() - 1 : rInf.GetIdx() );
 
+        bool bPerformUBA = UBIDI_LTR != nFieldDir || i18n::ScriptType::COMPLEX == nScript;
+        if (bPerformUBA)
         {
             UErrorCode nError = U_ZERO_ERROR;
             UBiDi* pBidi = ubidi_openSized( aText.getLength(), 0, &nError );

sw/source/core/text/porlay.cxx

@@ -1184,6 +1184,16 @@ void SwScriptInfo::InitScriptInfo( const SwTextNode& rNode, bool bRTL )
     aDirectionChanges.clear();
 
     // Perform Unicode Bidi Algorithm for text direction information
+    bool bPerformUBA = UBIDI_LTR != nDefaultDir;
+    nCnt = 0;
+    while( !bPerformUBA && nCnt < CountScriptChg() )
+    {
+        if ( i18n::ScriptType::COMPLEX == GetScriptType( nCnt++ ) )
+            bPerformUBA = true;
+    }
+
+    // do not call the unicode bidi algorithm if not required
+    if ( bPerformUBA )
     {
         UpdateBidiInfo( rText );