<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>71445</bug_id>
          
          <creation_ts>2011-11-03 01:22:10 -0700</creation_ts>
          <short_desc>[GTK] [WK2] ttf-liberation fonts moved to a new location (in Debian)</short_desc>
          <delta_ts>2011-11-03 03:08:54 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKitGTK</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Philippe Normand">pnormand</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>pnormand</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>495256</commentid>
    <comment_count>0</comment_count>
    <who name="Philippe Normand">pnormand</who>
    <bug_when>2011-11-03 01:22:10 -0700</bug_when>
    <thetext>+++ This bug was initially created as a clone of Bug #71359 +++

The fonts-liberation package (previously ttf-liberation) installs the fonts to /usr/share/fonts/truetype/liberation

We already added a case for Fedora font paths one year ago. For each font filename there were 2 possible paths. Adding a new search path involves adding a new dimension in the fontPaths variable in initializeFonts(). This is rather painful, I&apos;d propose to have two separate arrays, one for the directories and one for the font filenames we need to register in fontconfig.

While this involves a speed penalization I think it&apos;s easier to maintain.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>495285</commentid>
    <comment_count>1</comment_count>
      <attachid>113446</attachid>
    <who name="Philippe Normand">pnormand</who>
    <bug_when>2011-11-03 02:36:14 -0700</bug_when>
    <thetext>Created attachment 113446
proposed patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>495290</commentid>
    <comment_count>2</comment_count>
      <attachid>113446</attachid>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2011-11-03 02:53:12 -0700</bug_when>
    <thetext>Comment on attachment 113446
proposed patch

View in context: https://bugs.webkit.org/attachment.cgi?id=113446&amp;action=review

Thanks for the fix! Please fix the nits below before landing.

&gt; Tools/WebKitTestRunner/InjectedBundle/gtk/ActivateFontsGtk.cpp:109
&gt; +    GOwnPtr&lt;gchar&gt; directoriesDescription;
&gt; +    for (size_t path = 0; path &lt; G_N_ELEMENTS(fontDirectories); path++)
&gt; +        directoriesDescription.set(g_strjoin(&quot;:&quot;, directoriesDescription.release(), fontDirectories[path], NULL));

Perhaps only calculate this if found is false below?

&gt; Tools/WebKitTestRunner/InjectedBundle/gtk/ActivateFontsGtk.cpp:128
&gt; +        if (!found)
&gt; +            g_error(&quot;Could not find font %s in %s. Either install this font or file a bug &quot;
&gt;                      &quot;at http://bugs.webkit.org if it is installed in another location.&quot;,

The curly brace should stay here because the g_error take multiple lines. This rule applies to the number of lines and not the number of statements. :/</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>495302</commentid>
    <comment_count>3</comment_count>
    <who name="Philippe Normand">pnormand</who>
    <bug_when>2011-11-03 03:03:25 -0700</bug_when>
    <thetext>(In reply to comment #2)
&gt; (From update of attachment 113446 [details])
&gt; View in context: https://bugs.webkit.org/attachment.cgi?id=113446&amp;action=review
&gt; 
&gt; Thanks for the fix! Please fix the nits below before landing.
&gt; 
&gt; &gt; Tools/WebKitTestRunner/InjectedBundle/gtk/ActivateFontsGtk.cpp:109
&gt; &gt; +    GOwnPtr&lt;gchar&gt; directoriesDescription;
&gt; &gt; +    for (size_t path = 0; path &lt; G_N_ELEMENTS(fontDirectories); path++)
&gt; &gt; +        directoriesDescription.set(g_strjoin(&quot;:&quot;, directoriesDescription.release(), fontDirectories[path], NULL));
&gt; 
&gt; Perhaps only calculate this if found is false below?
&gt; 

Hum right, good point! Will fix it in DRT as well.

&gt; &gt; Tools/WebKitTestRunner/InjectedBundle/gtk/ActivateFontsGtk.cpp:128
&gt; &gt; +        if (!found)
&gt; &gt; +            g_error(&quot;Could not find font %s in %s. Either install this font or file a bug &quot;
&gt; &gt;                      &quot;at http://bugs.webkit.org if it is installed in another location.&quot;,
&gt; 
&gt; The curly brace should stay here because the g_error take multiple lines. This rule applies to the number of lines and not the number of statements. :/

Damn me! :/ Will fix in DRT as well. Thanks! (BTW it&apos;s a pity to have to copy/paste code between WKTR and DRT :()</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>495309</commentid>
    <comment_count>4</comment_count>
    <who name="Philippe Normand">pnormand</who>
    <bug_when>2011-11-03 03:08:54 -0700</bug_when>
    <thetext>Committed r99158: &lt;http://trac.webkit.org/changeset/99158&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>113446</attachid>
            <date>2011-11-03 02:36:14 -0700</date>
            <delta_ts>2011-11-03 02:53:12 -0700</delta_ts>
            <desc>proposed patch</desc>
            <filename>proposed-patch.patch</filename>
            <type>text/plain</type>
            <size>7423</size>
            <attacher name="Philippe Normand">pnormand</attacher>
            
              <data encoding="base64">RnJvbSBmZDk4YjU0MGE1NmE3NTk4YmE1MmVjZTE3ZTA2ODgwNmJlOTQ4Mjg5IE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBQaGlsaXBwZSBOb3JtYW5kIDxwbm9ybWFuZEBpZ2FsaWEuY29t
PgpEYXRlOiBUaHUsIDMgTm92IDIwMTEgMDk6Mjc6MzMgKzAxMDAKU3ViamVjdDogW1BBVENIXSBw
cm9wb3NlZCBwYXRjaAoKLS0tCiBUb29scy9DaGFuZ2VMb2cgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICB8ICAgMTIgKysrCiAuLi4vSW5qZWN0ZWRCdW5kbGUvZ3RrL0FjdGl2YXRl
Rm9udHNHdGsuY3BwICAgICAgICB8ICAgOTUgKysrKysrKysrLS0tLS0tLS0tLQogMiBmaWxlcyBj
aGFuZ2VkLCA1NyBpbnNlcnRpb25zKCspLCA1MCBkZWxldGlvbnMoLSkKCmRpZmYgLS1naXQgYS9U
b29scy9DaGFuZ2VMb2cgYi9Ub29scy9DaGFuZ2VMb2cKaW5kZXggN2I3MGY0NS4uNTM4YWRiNCAx
MDA2NDQKLS0tIGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBiL1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSwz
ICsxLDE1IEBACisyMDExLTExLTAzICBQaGlsaXBwZSBOb3JtYW5kICA8cG5vcm1hbmRAaWdhbGlh
LmNvbT4KKworICAgICAgICBbR1RLXSBbV0syXSB0dGYtbGliZXJhdGlvbiBmb250cyBtb3ZlZCB0
byBhIG5ldyBsb2NhdGlvbiAoaW4gRGViaWFuKQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0
Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NzE0NDUKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkg
KE9PUFMhKS4KKworICAgICAgICAqIFdlYktpdFRlc3RSdW5uZXIvSW5qZWN0ZWRCdW5kbGUvZ3Rr
L0FjdGl2YXRlRm9udHNHdGsuY3BwOgorICAgICAgICAoV1RSOjppbml0aXRpYWxpemVGb250Q29u
ZmlnU2V0dGluZyk6IEFkZCB0aGUgbmV3IGZvbnQgcGF0aCBmb3IgRGViaWFuIGZvbnRzIGFuZAor
ICAgICAgICByZWZhY3RvcmVkIHRoZSBmb250IGZpbGVzIGxvYWRpbmcgY29kZSB0byBhdm9pZCBj
b3B5L3Bhc3RlcywgZWFzZQorICAgICAgICBtYWludGVuYW5jZSBhbmQgZnV0dXJlIGZvbnQgZGly
ZWN0b3JpZXMgYWRkaXRpb25zLgorCiAyMDExLTExLTAyICBQaGlsaXBwZSBOb3JtYW5kICA8cG5v
cm1hbmRAaWdhbGlhLmNvbT4KIAogICAgICAgICBbR1RLXSBbRFJUXSB0dGYtbGliZXJhdGlvbiBm
b250cyBtb3ZlZCB0byBhIG5ldyBsb2NhdGlvbiAoaW4gRGViaWFuKQpkaWZmIC0tZ2l0IGEvVG9v
bHMvV2ViS2l0VGVzdFJ1bm5lci9JbmplY3RlZEJ1bmRsZS9ndGsvQWN0aXZhdGVGb250c0d0ay5j
cHAgYi9Ub29scy9XZWJLaXRUZXN0UnVubmVyL0luamVjdGVkQnVuZGxlL2d0ay9BY3RpdmF0ZUZv
bnRzR3RrLmNwcAppbmRleCBjNzBjYmZkLi5iN2VlNDA5IDEwMDY0NAotLS0gYS9Ub29scy9XZWJL
aXRUZXN0UnVubmVyL0luamVjdGVkQnVuZGxlL2d0ay9BY3RpdmF0ZUZvbnRzR3RrLmNwcAorKysg
Yi9Ub29scy9XZWJLaXRUZXN0UnVubmVyL0luamVjdGVkQnVuZGxlL2d0ay9BY3RpdmF0ZUZvbnRz
R3RrLmNwcApAQCAtNzEsNjcgKzcxLDYyIEBAIHZvaWQgaW5pdGl0aWFsaXplRm9udENvbmZpZ1Nl
dHRpbmcoKQogICAgIGlmICghRmNDb25maWdQYXJzZUFuZExvYWQoY29uZmlnLCByZWludGVycHJl
dF9jYXN0PEZjQ2hhcjgqPihmb250Q29uZmlnRmlsZW5hbWUuZ2V0KCkpLCB0cnVlKSkKICAgICAg
ICAgZ19lcnJvcigiQ291bGRuJ3QgbG9hZCBmb250IGNvbmZpZ3VyYXRpb24gZmlsZSBmcm9tOiAl
cyIsIGZvbnRDb25maWdGaWxlbmFtZS5nZXQoKSk7CiAKLSAgICBzdGF0aWMgY29uc3QgY2hhciAq
Y29uc3QgZm9udFBhdGhzW11bMl0gPSB7Ci0gICAgICAgIHsgIi91c3Ivc2hhcmUvZm9udHMvdHJ1
ZXR5cGUvdHRmLWxpYmVyYXRpb24vTGliZXJhdGlvbk1vbm8tQm9sZEl0YWxpYy50dGYiLAotICAg
ICAgICAgICIvdXNyL3NoYXJlL2ZvbnRzL2xpYmVyYXRpb24vTGliZXJhdGlvbk1vbm8tQm9sZEl0
YWxpYy50dGYiLCB9LAotICAgICAgICB7ICIvdXNyL3NoYXJlL2ZvbnRzL3RydWV0eXBlL3R0Zi1s
aWJlcmF0aW9uL0xpYmVyYXRpb25Nb25vLUJvbGQudHRmIiwKLSAgICAgICAgICAiL3Vzci9zaGFy
ZS9mb250cy9saWJlcmF0aW9uL0xpYmVyYXRpb25Nb25vLUJvbGQudHRmIiwgfSwKLSAgICAgICAg
eyAiL3Vzci9zaGFyZS9mb250cy90cnVldHlwZS90dGYtbGliZXJhdGlvbi9MaWJlcmF0aW9uTW9u
by1JdGFsaWMudHRmIiwKLSAgICAgICAgICAiL3Vzci9zaGFyZS9mb250cy9saWJlcmF0aW9uL0xp
YmVyYXRpb25Nb25vLUl0YWxpYy50dGYiLCB9LAotICAgICAgICB7ICIvdXNyL3NoYXJlL2ZvbnRz
L3RydWV0eXBlL3R0Zi1saWJlcmF0aW9uL0xpYmVyYXRpb25Nb25vLVJlZ3VsYXIudHRmIiwKLSAg
ICAgICAgICAiL3Vzci9zaGFyZS9mb250cy9saWJlcmF0aW9uL0xpYmVyYXRpb25Nb25vLVJlZ3Vs
YXIudHRmIiwgfSwKLSAgICAgICAgeyAiL3Vzci9zaGFyZS9mb250cy90cnVldHlwZS90dGYtbGli
ZXJhdGlvbi9MaWJlcmF0aW9uU2Fucy1Cb2xkSXRhbGljLnR0ZiIsCi0gICAgICAgICAgIi91c3Iv
c2hhcmUvZm9udHMvbGliZXJhdGlvbi9MaWJlcmF0aW9uU2Fucy1Cb2xkSXRhbGljLnR0ZiIsIH0s
Ci0gICAgICAgIHsgIi91c3Ivc2hhcmUvZm9udHMvdHJ1ZXR5cGUvdHRmLWxpYmVyYXRpb24vTGli
ZXJhdGlvblNhbnMtQm9sZC50dGYiLAotICAgICAgICAgICIvdXNyL3NoYXJlL2ZvbnRzL2xpYmVy
YXRpb24vTGliZXJhdGlvblNhbnMtQm9sZC50dGYiLCB9LAotICAgICAgICB7ICIvdXNyL3NoYXJl
L2ZvbnRzL3RydWV0eXBlL3R0Zi1saWJlcmF0aW9uL0xpYmVyYXRpb25TYW5zLUl0YWxpYy50dGYi
LAotICAgICAgICAgICIvdXNyL3NoYXJlL2ZvbnRzL2xpYmVyYXRpb24vTGliZXJhdGlvblNhbnMt
SXRhbGljLnR0ZiIsIH0sCi0gICAgICAgIHsgIi91c3Ivc2hhcmUvZm9udHMvdHJ1ZXR5cGUvdHRm
LWxpYmVyYXRpb24vTGliZXJhdGlvblNhbnMtUmVndWxhci50dGYiLAotICAgICAgICAgICIvdXNy
L3NoYXJlL2ZvbnRzL2xpYmVyYXRpb24vTGliZXJhdGlvblNhbnMtUmVndWxhci50dGYiLCB9LAot
ICAgICAgICB7ICIvdXNyL3NoYXJlL2ZvbnRzL3RydWV0eXBlL3R0Zi1saWJlcmF0aW9uL0xpYmVy
YXRpb25TZXJpZi1Cb2xkSXRhbGljLnR0ZiIsCi0gICAgICAgICAgIi91c3Ivc2hhcmUvZm9udHMv
bGliZXJhdGlvbi9MaWJlcmF0aW9uU2VyaWYtQm9sZEl0YWxpYy50dGYiLCB9LAotICAgICAgICB7
ICIvdXNyL3NoYXJlL2ZvbnRzL3RydWV0eXBlL3R0Zi1saWJlcmF0aW9uL0xpYmVyYXRpb25TZXJp
Zi1Cb2xkLnR0ZiIsCi0gICAgICAgICAgIi91c3Ivc2hhcmUvZm9udHMvbGliZXJhdGlvbi9MaWJl
cmF0aW9uU2VyaWYtQm9sZC50dGYiLCB9LAotICAgICAgICB7ICIvdXNyL3NoYXJlL2ZvbnRzL3Ry
dWV0eXBlL3R0Zi1saWJlcmF0aW9uL0xpYmVyYXRpb25TZXJpZi1JdGFsaWMudHRmIiwKLSAgICAg
ICAgICAiL3Vzci9zaGFyZS9mb250cy9saWJlcmF0aW9uL0xpYmVyYXRpb25TZXJpZi1JdGFsaWMu
dHRmIiwgfSwKLSAgICAgICAgeyAiL3Vzci9zaGFyZS9mb250cy90cnVldHlwZS90dGYtbGliZXJh
dGlvbi9MaWJlcmF0aW9uU2VyaWYtUmVndWxhci50dGYiLAotICAgICAgICAgICIvdXNyL3NoYXJl
L2ZvbnRzL2xpYmVyYXRpb24vTGliZXJhdGlvblNlcmlmLVJlZ3VsYXIudHRmIiwgfSwKLSAgICAg
ICAgeyAiL3Vzci9zaGFyZS9mb250cy90cnVldHlwZS90dGYtZGVqYXZ1L0RlamFWdVNhbnMudHRm
IiwKLSAgICAgICAgICAiL3Vzci9zaGFyZS9mb250cy9kZWphdnUvRGVqYVZ1U2Fucy50dGYiLCB9
LAotICAgICAgICB7ICIvdXNyL3NoYXJlL2ZvbnRzL3RydWV0eXBlL3R0Zi1kZWphdnUvRGVqYVZ1
U2VyaWYudHRmIiwKLSAgICAgICAgICAiL3Vzci9zaGFyZS9mb250cy9kZWphdnUvRGVqYVZ1U2Vy
aWYudHRmIiwgfSwKKyAgICBzdGF0aWMgY29uc3QgY2hhciAqY29uc3QgZm9udERpcmVjdG9yaWVz
W10gPSB7CisgICAgICAgICIvdXNyL3NoYXJlL2ZvbnRzL3RydWV0eXBlL2xpYmVyYXRpb24iLAor
ICAgICAgICAiL3Vzci9zaGFyZS9mb250cy90cnVldHlwZS90dGYtbGliZXJhdGlvbiIsCisgICAg
ICAgICIvdXNyL3NoYXJlL2ZvbnRzL2xpYmVyYXRpb24iLAorICAgICAgICAiL3Vzci9zaGFyZS9m
b250cy90cnVldHlwZS90dGYtZGVqYXZ1IiwKKyAgICAgICAgIi91c3Ivc2hhcmUvZm9udHMvZGVq
YXZ1IiwKKyAgICAgICAgIi91c3Ivc2hhcmUvZm9udHMvb3BlbnR5cGUvc3RpeCIsCisgICAgICAg
ICIvdXNyL3NoYXJlL2ZvbnRzL3N0aXgiCisgICAgfTsKKworICAgIHN0YXRpYyBjb25zdCBjaGFy
ICpjb25zdCBmb250UGF0aHNbXSA9IHsKKyAgICAgICAgIkxpYmVyYXRpb25Nb25vLUJvbGRJdGFs
aWMudHRmIiwKKyAgICAgICAgIkxpYmVyYXRpb25Nb25vLUJvbGQudHRmIiwKKyAgICAgICAgIkxp
YmVyYXRpb25Nb25vLUl0YWxpYy50dGYiLAorICAgICAgICAiTGliZXJhdGlvbk1vbm8tUmVndWxh
ci50dGYiLAorICAgICAgICAiTGliZXJhdGlvblNhbnMtQm9sZEl0YWxpYy50dGYiLAorICAgICAg
ICAiTGliZXJhdGlvblNhbnMtQm9sZC50dGYiLAorICAgICAgICAiTGliZXJhdGlvblNhbnMtSXRh
bGljLnR0ZiIsCisgICAgICAgICJMaWJlcmF0aW9uU2Fucy1SZWd1bGFyLnR0ZiIsCisgICAgICAg
ICJMaWJlcmF0aW9uU2VyaWYtQm9sZEl0YWxpYy50dGYiLAorICAgICAgICAiTGliZXJhdGlvblNl
cmlmLUJvbGQudHRmIiwKKyAgICAgICAgIkxpYmVyYXRpb25TZXJpZi1JdGFsaWMudHRmIiwKKyAg
ICAgICAgIkxpYmVyYXRpb25TZXJpZi1SZWd1bGFyLnR0ZiIsCisgICAgICAgICJEZWphVnVTYW5z
LnR0ZiIsCisgICAgICAgICJEZWphVnVTZXJpZi50dGYiLAogCiAgICAgICAgIC8vIE1hdGhNTCB0
ZXN0cyByZXF1aXJlIHRoZSBTVElYIGZvbnRzLgotICAgICAgICB7ICIvdXNyL3NoYXJlL2ZvbnRz
L29wZW50eXBlL3N0aXgvU1RJWEdlbmVyYWwub3RmIiwKLSAgICAgICAgICAiL3Vzci9zaGFyZS9m
b250cy9zdGl4L1NUSVhHZW5lcmFsLm90ZiIgfSwKLSAgICAgICAgeyAiL3Vzci9zaGFyZS9mb250
cy9vcGVudHlwZS9zdGl4L1NUSVhHZW5lcmFsQm9sSXRhLm90ZiIsCi0gICAgICAgICAgIi91c3Iv
c2hhcmUvZm9udHMvc3RpeC9TVElYR2VuZXJhbEJvbEl0YS5vdGYiIH0sCi0gICAgICAgIHsgIi91
c3Ivc2hhcmUvZm9udHMvb3BlbnR5cGUvc3RpeC9TVElYR2VuZXJhbEJvbC5vdGYiLAotICAgICAg
ICAgICIvdXNyL3NoYXJlL2ZvbnRzL3N0aXgvU1RJWEdlbmVyYWxCb2wub3RmIiB9LAotICAgICAg
ICB7ICIvdXNyL3NoYXJlL2ZvbnRzL29wZW50eXBlL3N0aXgvU1RJWEdlbmVyYWxJdGFsaWMub3Rm
IiwKLSAgICAgICAgICAiL3Vzci9zaGFyZS9mb250cy9zdGl4L1NUSVhHZW5lcmFsSXRhbGljLm90
ZiIgfQorICAgICAgICAiU1RJWEdlbmVyYWwub3RmIiwKKyAgICAgICAgIlNUSVhHZW5lcmFsQm9s
SXRhLm90ZiIsCisgICAgICAgICJTVElYR2VuZXJhbEJvbC5vdGYiLAorICAgICAgICAiU1RJWEdl
bmVyYWxJdGFsaWMub3RmIgogICAgIH07CiAKKyAgICBHT3duUHRyPGdjaGFyPiBkaXJlY3Rvcmll
c0Rlc2NyaXB0aW9uOworICAgIGZvciAoc2l6ZV90IHBhdGggPSAwOyBwYXRoIDwgR19OX0VMRU1F
TlRTKGZvbnREaXJlY3Rvcmllcyk7IHBhdGgrKykKKyAgICAgICAgZGlyZWN0b3JpZXNEZXNjcmlw
dGlvbi5zZXQoZ19zdHJqb2luKCI6IiwgZGlyZWN0b3JpZXNEZXNjcmlwdGlvbi5yZWxlYXNlKCks
IGZvbnREaXJlY3Rvcmllc1twYXRoXSwgTlVMTCkpOworCiAgICAgLy8gVE9ETzogU29tZSB0ZXN0
cyB1c2UgTHVjaWRhLiBXZSBzaG91bGQgbG9hZCB0aGVzZSBhcyB3ZWxsLCBvbmNlIGl0IGJlY29t
ZXMKICAgICAvLyBjbGVhciBob3cgdG8gaW5zdGFsbCB0aGVzZSBmb250cyBlYXNpbHkgb24gRmVk
b3JhLgogICAgIGZvciAoc2l6ZV90IGZvbnQgPSAwOyBmb250IDwgR19OX0VMRU1FTlRTKGZvbnRQ
YXRocyk7IGZvbnQrKykgewogICAgICAgICBib29sIGZvdW5kID0gZmFsc2U7Ci0gICAgICAgIGZv
ciAoc2l6ZV90IHBhdGggPSAwOyBwYXRoIDwgMjsgcGF0aCsrKSB7Ci0gICAgICAgICAgICBpZiAo
IWdfZmlsZV90ZXN0KGZvbnRQYXRoc1tmb250XVtwYXRoXSwgR19GSUxFX1RFU1RfRVhJU1RTKSkK
LSAgICAgICAgICAgICAgICBjb250aW51ZTsKLQotICAgICAgICAgICAgZm91bmQgPSB0cnVlOwot
ICAgICAgICAgICAgaWYgKCFGY0NvbmZpZ0FwcEZvbnRBZGRGaWxlKGNvbmZpZywgcmVpbnRlcnBy
ZXRfY2FzdDxjb25zdCBGY0NoYXI4Kj4oZm9udFBhdGhzW2ZvbnRdW3BhdGhdKSkpCi0gICAgICAg
ICAgICAgICAgZ19lcnJvcigiQ291bGQgbm90IGxvYWQgZm9udCBhdCAlcyEiLCBmb250UGF0aHNb
Zm9udF1bcGF0aF0pOwotICAgICAgICAgICAgZWxzZQotICAgICAgICAgICAgICAgIGJyZWFrOwor
ICAgICAgICBmb3IgKHNpemVfdCBwYXRoID0gMDsgcGF0aCA8IEdfTl9FTEVNRU5UUyhmb250RGly
ZWN0b3JpZXMpOyBwYXRoKyspIHsKKyAgICAgICAgICAgIEdPd25QdHI8Z2NoYXI+IGZ1bGxQYXRo
KGdfYnVpbGRfZmlsZW5hbWUoZm9udERpcmVjdG9yaWVzW3BhdGhdLCBmb250UGF0aHNbZm9udF0s
IE5VTEwpKTsKKyAgICAgICAgICAgIGlmIChnX2ZpbGVfdGVzdChmdWxsUGF0aC5nZXQoKSwgR19G
SUxFX1RFU1RfRVhJU1RTKSkgeworICAgICAgICAgICAgICAgIGZvdW5kID0gdHJ1ZTsKKyAgICAg
ICAgICAgICAgICBpZiAoIUZjQ29uZmlnQXBwRm9udEFkZEZpbGUoY29uZmlnLCByZWludGVycHJl
dF9jYXN0PGNvbnN0IEZjQ2hhcjgqPihmdWxsUGF0aC5nZXQoKSkpKQorICAgICAgICAgICAgICAg
ICAgICBnX2Vycm9yKCJDb3VsZCBub3QgbG9hZCBmb250IGF0ICVzISIsIGZ1bGxQYXRoLmdldCgp
KTsKKyAgICAgICAgICAgICAgICBlbHNlCisgICAgICAgICAgICAgICAgICAgIGJyZWFrOworICAg
ICAgICAgICAgfQogICAgICAgICB9CiAKLSAgICAgICAgaWYgKCFmb3VuZCkgewotICAgICAgICAg
ICAgZ19lcnJvcigiQ291bGQgbm90IGZpbmQgZm9udCBhdCAlcy4gRWl0aGVyIGluc3RhbGwgdGhp
cyBmb250IG9yIGZpbGUgYSBidWcgIgorICAgICAgICBpZiAoIWZvdW5kKQorICAgICAgICAgICAg
Z19lcnJvcigiQ291bGQgbm90IGZpbmQgZm9udCAlcyBpbiAlcy4gRWl0aGVyIGluc3RhbGwgdGhp
cyBmb250IG9yIGZpbGUgYSBidWcgIgogICAgICAgICAgICAgICAgICAgICAiYXQgaHR0cDovL2J1
Z3Mud2Via2l0Lm9yZyBpZiBpdCBpcyBpbnN0YWxsZWQgaW4gYW5vdGhlciBsb2NhdGlvbi4iLAot
ICAgICAgICAgICAgICAgICAgICBmb250UGF0aHNbZm9udF1bMF0pOwotICAgICAgICB9CisgICAg
ICAgICAgICAgICAgICAgIGZvbnRQYXRoc1tmb250XSwgZGlyZWN0b3JpZXNEZXNjcmlwdGlvbi5n
ZXQoKSk7CiAgICAgfQogCiAgICAgLy8gQWhlbSBpcyB1c2VkIGJ5IG1hbnkgbGF5b3V0IHRlc3Rz
LgotLSAKMS43LjcuMQ==
</data>
<flag name="review"
          id="111818"
          type_id="1"
          status="+"
          setter="mrobinson"
    />
          </attachment>
      

    </bug>

</bugzilla>