From: joogab yun <joogab.yun@samsung.com>
Date: Thu, 27 Dec 2018 10:00:51 +0000 (+0000)
Subject: Revert "Fixed an incorrect ellipsis"
X-Git-Tag: dali_1.4.1~5^2
X-Git-Url: http://review.tizen.org/git/?p=platform%2Fcore%2Fuifw%2Fdali-toolkit.git;a=commitdiff_plain;h=cce53acd094585426f1aebbaf02169141f63b51d

Revert "Fixed an incorrect ellipsis"

This reverts commit 30758c617dee3352070497133b24818674d5d646.

Change-Id: I8ddde73a3c13d15af24525ba70714a17df8d244d
---

diff --git a/automated-tests/src/dali-toolkit-internal/utc-Dali-Text-ViewModel.cpp b/automated-tests/src/dali-toolkit-internal/utc-Dali-Text-ViewModel.cpp
old mode 100755
new mode 100644
index fc87193..49d36e0
--- a/automated-tests/src/dali-toolkit-internal/utc-Dali-Text-ViewModel.cpp
+++ b/automated-tests/src/dali-toolkit-internal/utc-Dali-Text-ViewModel.cpp
@@ -71,7 +71,6 @@ struct ElideData
   unsigned int numberOfLines;
   unsigned int numberOfGlyphs;
   float*       positions;
-  unsigned int ignoreOfGlyphs;
 };
 
 bool ElideTest( const ElideData& data )
@@ -131,7 +130,7 @@ bool ElideTest( const ElideData& data )
   if( numberOfLines != 0u )
   {
     const LineRun& lastLine = *( model->GetLines() + numberOfLines - 1u );
-    const Length numberOfLastLineGlyphs = data.numberOfGlyphs - lastLine.glyphRun.glyphIndex + data.ignoreOfGlyphs;
+    const Length numberOfLastLineGlyphs = data.numberOfGlyphs - lastLine.glyphRun.glyphIndex;
 
     std::cout << "  last line alignment offset : " << lastLine.alignmentOffset << std::endl;
 
@@ -595,9 +594,6 @@ int UtcDaliTextViewModelElideText02(void)
   Size textSize04( 80.f, 10.f );
   float positions04[] = { 2.f };
 
-  Size textSize05( 180.f, 100.f );
-  float positions05[] = { 0.f, 0.f };
-
   struct ElideData data[] =
   {
     {
@@ -606,8 +602,7 @@ int UtcDaliTextViewModelElideText02(void)
       textSize00,
       0u,
       0u,
-      NULL,
-      0u
+      NULL
     },
     {
       "Latin script",
@@ -615,8 +610,7 @@ int UtcDaliTextViewModelElideText02(void)
       textSize01,
       5u,
       42u,
-      positions01,
-      0u
+      positions01
     },
     {
       "Hebrew script",
@@ -624,8 +618,7 @@ int UtcDaliTextViewModelElideText02(void)
       textSize02,
       5u,
       49u,
-      positions02,
-      0u
+      positions02
     },
     {
       "Arabic script",
@@ -633,8 +626,7 @@ int UtcDaliTextViewModelElideText02(void)
       textSize03,
       5u,
       79u,
-      positions03,
-      0u
+      positions03
     },
     {
       "Small control size, no line fits.",
@@ -642,20 +634,10 @@ int UtcDaliTextViewModelElideText02(void)
       textSize04,
       1u,
       1u,
-      positions04,
-      0u
-    },
-    {
-      "Include newline character",
-      "<font family='TizenSans'>yesterday\n all\n my troubles\n seemed so far\n\n away now it looks</font>",
-      textSize05,
-      5u,
-      40,
-      positions05,
-      5u
-    },
+      positions04
+    }
   };
-  const unsigned int numberOfTests = 6u;
+  const unsigned int numberOfTests = 5u;
 
   for( unsigned int index = 0u; index < numberOfTests; ++index )
   {
diff --git a/dali-toolkit/internal/text/rendering/view-model.cpp b/dali-toolkit/internal/text/rendering/view-model.cpp
index 27932b2..3543fce 100755
--- a/dali-toolkit/internal/text/rendering/view-model.cpp
+++ b/dali-toolkit/internal/text/rendering/view-model.cpp
@@ -349,11 +349,6 @@ void ViewModel::ElideGlyphs()
           {
             if( index > 0u )
             {
-              // If the index decreases to the previous line, firstPenX must be recalculated.
-              if( numberOfLaidOutGlyphs - index == lastLine.glyphRun.numberOfGlyphs)
-              {
-                firstPenSet = false;
-              }
               --index;
             }
             else