<?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>154530</bug_id>
          
          <creation_ts>2016-02-22 02:20:52 -0800</creation_ts>
          <short_desc>New intl-numberformat.js test fails on many Linux platforms</short_desc>
          <delta_ts>2016-06-06 05:20:08 -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>JavaScriptCore</component>
          <version>Other</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=158066</see_also>
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>158417</dependson>
          <blocked>147605</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Csaba Osztrogonác">ossy</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>andy</cc>
    
    <cc>bugs-noreply</cc>
    
    <cc>cgarcia</cc>
    
    <cc>clopez</cc>
    
    <cc>commit-queue</cc>
    
    <cc>darin</cc>
    
    <cc>gustavo</cc>
    
    <cc>gyuyoung.kim</cc>
    
    <cc>jh718.park</cc>
    
    <cc>keith_miller</cc>
    
    <cc>mark.lam</cc>
    
    <cc>mcatanzaro</cc>
    
    <cc>msaboff</cc>
    
    <cc>ossy</cc>
    
    <cc>saam</cc>
    
    <cc>sukolsak</cc>
    
    <cc>ysuzuki</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1166973</commentid>
    <comment_count>0</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2016-02-22 02:20:52 -0800</bug_when>
    <thetext>intl-numberformat.js introduced in http://trac.webkit.org/changeset/196850,
but fails on EFL ARM and GTK x86_64 buildbots:
- GTK bot: https://build.webkit.org/builders/GTK%20Linux%2064-bit%20Release%20%28Tests%29/builds/13917
- EFL ARM bot: https://build.webkit.org/builders/EFL%20Linux%20ARMv7%20Thumb2%20Release/builds/17225

It&apos;s strange that it passes on the EFL x86_64 bot. I don&apos;t know anything about this code path.
Is it possible that some 3rd party libraries with different version cause this failure?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1167031</commentid>
    <comment_count>1</comment_count>
    <who name="Sukolsak Sakshuwong">sukolsak</who>
    <bug_when>2016-02-22 08:54:40 -0800</bug_when>
    <thetext>This is likely the same issue as Bug 152448. It is because Intl uses POSIX locale.

    $ LANG=en_US.UTF-8 ./jsc -e &quot;print(Infinity.toLocaleString())&quot;
    ∞
    $ LANG=en_US_POSIX.UTF-8 ./jsc -e &quot;print(Infinity.toLocaleString())&quot;
    INF

The proposed patch in Bug 152448 should fix it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1168932</commentid>
    <comment_count>2</comment_count>
    <who name="Andy VanWagoner">andy</who>
    <bug_when>2016-02-27 17:30:33 -0800</bug_when>
    <thetext>The patch for Bug 152448 likely hasn&apos;t resolved this issue, because it only provided a hook for defaultLanguage that is implemented in WebCore.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1168936</commentid>
    <comment_count>3</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2016-02-27 17:45:24 -0800</bug_when>
    <thetext>There are two separate questions here:

One is how to make WebKit and JavaScriptCore properly choose a default locale for each platform. I don’t know what the right way to do that is for Linux in general. Fixing that for WebCore is what bug 152448 is all about. But it almost certainly needs to be fixed for direct JavaScriptCore use too. At least I know it does need to be done on the Apple platforms. So we may need a bug to track that.

The other is how to make the test harness set a consistent language so that tests are reliable, rather than being sensitive to the language the user running the tests happens to prefer. That&apos;s something we need to address separately. There are lots of calls to setenv in various test tools, so it could be as simple as doing a setenv(&quot;LANG&quot;) in the right place for the appropriate platforms in some of the test tools.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1169143</commentid>
    <comment_count>4</comment_count>
    <who name="Carlos Alberto Lopez Perez">clopez</who>
    <bug_when>2016-02-29 07:34:45 -0800</bug_when>
    <thetext>(In reply to comment #3)
&gt; There are two separate questions here:
&gt; 
&gt; One is how to make WebKit and JavaScriptCore properly choose a default
&gt; locale for each platform. I don’t know what the right way to do that is for
&gt; Linux in general. Fixing that for WebCore is what bug 152448 is all about.
&gt; But it almost certainly needs to be fixed for direct JavaScriptCore use too.
&gt; At least I know it does need to be done on the Apple platforms. So we may
&gt; need a bug to track that.
&gt; 


With glibc (GNU/Linux) the default locale is defined on the environment variable LANG. If LANG is not defined, then the locale defaults to &apos;C&apos; (also known as &apos;POSIX&apos;) which is the default portable locale.

This default locale can be overridden with the environment variable LC_ALL.

So perhaps we can just do setenv(&quot;LC_ALL&quot;, &quot;C&quot;) before starting the tests. This will override any locale defined by the user, and it will be portable across operating systems

Some quick tests:

# Linux
$ LANG=es_ES.UTF-8 WebKitBuild/Release/bin/jsc -e &quot;print(Infinity.toLocaleString())&quot;
∞
$ LC_ALL=POSIX LANG=es_ES.UTF-8 WebKitBuild/Release/bin/jsc -e &quot;print(Infinity.toLocaleString())&quot;
INF
$ LC_ALL=C LANG=es_ES.UTF-8 WebKitBuild/Release/bin/jsc -e &quot;print(Infinity.toLocaleString())&quot;
INF


# MacOS
$ LANG=es_ES.UTF-8 WebKitBuild//Release/JavaScriptCore.framework/Versions/A/Resources/jsc -e &quot;print(Infinity.toLocaleString())&quot;
∞
$ LC_ALL=POSIX LANG=es_ES.UTF-8 WebKitBuild/Release/JavaScriptCore.framework/Versions/A/Resources/jsc -e &quot;print(Infinity.toLocaleString())&quot;
INF
$ LC_ALL=C LANG=es_ES.UTF-8 WebKitBuild/Release/JavaScriptCore.framework/Versions/A/Resources/jsc -e &quot;print(Infinity.toLocaleString())&quot;
INF


&gt; The other is how to make the test harness set a consistent language so that
&gt; tests are reliable, rather than being sensitive to the language the user
&gt; running the tests happens to prefer. That&apos;s something we need to address
&gt; separately. There are lots of calls to setenv in various test tools, so it
&gt; could be as simple as doing a setenv(&quot;LANG&quot;) in the right place for the
&gt; appropriate platforms in some of the test tools.

As explained above, just doing setenv(&quot;LC_ALL&quot;, &quot;C&quot;) before starting the tests should work for all use cases.

More info: http://man7.org/linux/man-pages/man3/setlocale.3.html and https://developer.apple.com/library/mac/documentation/Darwin/Reference/ManPages/man1/locale.1.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1169179</commentid>
    <comment_count>5</comment_count>
    <who name="Carlos Alberto Lopez Perez">clopez</who>
    <bug_when>2016-02-29 09:52:56 -0800</bug_when>
    <thetext>Seems that my previous suggestion of setting LC_ALL is not going to help.

This test js/intl-numberformat.html is manually setting the locales and trying some numeric and monetary conversions and checking that it returns the right values.

For example see the follow examples:

linux # WebKitBuild/Release/bin/jsc -e &quot;print(Intl.NumberFormat(&apos;es&apos;).format(1234.567))&quot;
1 234,567
linux # $ WebKitBuild/Release/bin/jsc -e &quot;print(Intl.NumberFormat(&apos;en&apos;).format(1234.567))&quot;
1,234.567


macos # WebKitBuild/Release/JavaScriptCore.framework/Versions/A/Resources/jsc -e &quot;print(Intl.NumberFormat(&apos;es&apos;).format(1234.567))&quot;
1.234,567
macos # WebKitBuild/Release/JavaScriptCore.framework/Versions/A/Resources/jsc -e &quot;print(Intl.NumberFormat(&apos;en&apos;).format(1234.567))&quot;
1,234.567


So it seems there is fundamental difference here.
In MacOS we are getting a &apos;.&apos; in the thousands decimal mark for the spanish locale (right), meanwhile in Linux we are getting an space (wrong).

The above checks are part of the test js/intl-numberformat.html therefore on Linux we are getting:

-PASS Intl.NumberFormat(&apos;es&apos;).format(1234.567) is &apos;1.234,567&apos;
+FAIL Intl.NumberFormat(&apos;es&apos;).format(1234.567) should be 1.234,567. Was 1 234,567.


Not sure what is wrong, at least with printf on Linux I get the right values

LC_NUMERIC=en_US.UTF8 /usr/bin/printf &quot;%&apos;.2f\n&quot; &apos;1234.567&apos;
1,234.57
LC_NUMERIC=es_ES.UTF8 /usr/bin/printf &quot;%&apos;.2f\n&quot; &apos;1234.567&apos;
1.234,57

So this don&apos;t seems a problem on the Linux glibc, but at first it looks that something on jsc is not picking/setting the right values for the locales when running on Linux


This is the full text difference for the test js/intl-numberformat.html with WebKitGTK+ (Linux):

--- /home/clopez/webkit/webkit/layout-test-results/js/intl-numberformat-expected.txt
+++ /home/clopez/webkit/webkit/layout-test-results/js/intl-numberformat-actual.txt
@@ -140,7 +140,7 @@
 PASS Intl.NumberFormat(&apos;en&apos;).format(Number.MIN_VALUE) is &apos;0&apos;
 PASS Intl.NumberFormat(&apos;en&apos;).format(Number.MAX_VALUE) is &apos;179,769,313,486,232,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000,000&apos;
 PASS Intl.NumberFormat(&apos;en&apos;).format(1234.567) is &apos;1,234.567&apos;
-PASS Intl.NumberFormat(&apos;es&apos;).format(1234.567) is &apos;1.234,567&apos;
+FAIL Intl.NumberFormat(&apos;es&apos;).format(1234.567) should be 1.234,567. Was 1 234,567.
 PASS Intl.NumberFormat(&apos;fr&apos;).format(1234.567) is &apos;1 234,567&apos;
 PASS Intl.NumberFormat(&apos;en-u-nu-latn&apos;).format(1234.567) is &apos;1,234.567&apos;
 PASS Intl.NumberFormat(&apos;en-u-nu-fullwide&apos;).format(1234.567) is &apos;１,２３４.５６７&apos;
@@ -156,14 +156,14 @@
 PASS Intl.NumberFormat(&apos;en&apos;, {style: &apos;currency&apos;, currency: &apos;USD&apos;}).format(Infinity) is &apos;$∞&apos;
 PASS Intl.NumberFormat(&apos;en&apos;, {style: &apos;currency&apos;, currency: &apos;JPY&apos;}).format(4.2) is &apos;¥4&apos;
 PASS Intl.NumberFormat(&apos;en&apos;, {style: &apos;currency&apos;, currency: &apos;xXx&apos;}).format(4.2) is &apos;XXX4.20&apos;
-PASS Intl.NumberFormat(&apos;en&apos;, {style: &apos;currency&apos;, currency: &apos;USD&apos;, currencyDisplay: &apos;code&apos;}).format(4) is &apos;USD4.00&apos;
+FAIL Intl.NumberFormat(&apos;en&apos;, {style: &apos;currency&apos;, currency: &apos;USD&apos;, currencyDisplay: &apos;code&apos;}).format(4) should be USD4.00. Threw exception Error: Failed to format a number.
 PASS Intl.NumberFormat(&apos;en&apos;, {style: &apos;currency&apos;, currency: &apos;USD&apos;, currencyDisplay: &apos;symbol&apos;}).format(4) is &apos;$4.00&apos;
-PASS Intl.NumberFormat(&apos;en&apos;, {style: &apos;currency&apos;, currency: &apos;USD&apos;, currencyDisplay: &apos;name&apos;}).format(4) is &apos;4.00 US dollars&apos;
-PASS Intl.NumberFormat(&apos;en&apos;, {style: &apos;currency&apos;, currency: &apos;JPY&apos;, currencyDisplay: &apos;code&apos;}).format(-4.2) is &apos;-JPY4&apos;
+FAIL Intl.NumberFormat(&apos;en&apos;, {style: &apos;currency&apos;, currency: &apos;USD&apos;, currencyDisplay: &apos;name&apos;}).format(4) should be 4.00 US dollars. Threw exception Error: Failed to format a number.
+FAIL Intl.NumberFormat(&apos;en&apos;, {style: &apos;currency&apos;, currency: &apos;JPY&apos;, currencyDisplay: &apos;code&apos;}).format(-4.2) should be -JPY4. Threw exception Error: Failed to format a number.
 PASS Intl.NumberFormat(&apos;en&apos;, {style: &apos;currency&apos;, currency: &apos;JPY&apos;, currencyDisplay: &apos;symbol&apos;}).format(-4.2) is &apos;-¥4&apos;
-PASS Intl.NumberFormat(&apos;en&apos;, {style: &apos;currency&apos;, currency: &apos;JPY&apos;, currencyDisplay: &apos;name&apos;}).format(-4.2) is &apos;-4 Japanese yen&apos;
-PASS Intl.NumberFormat(&apos;fr&apos;, {style: &apos;currency&apos;, currency: &apos;USD&apos;, currencyDisplay: &apos;name&apos;}).format(4) is &apos;4,00 dollars des États-Unis&apos;
-PASS Intl.NumberFormat(&apos;fr&apos;, {style: &apos;currency&apos;, currency: &apos;JPY&apos;, currencyDisplay: &apos;name&apos;}).format(4) is &apos;4 yens japonais&apos;
+FAIL Intl.NumberFormat(&apos;en&apos;, {style: &apos;currency&apos;, currency: &apos;JPY&apos;, currencyDisplay: &apos;name&apos;}).format(-4.2) should be -4 Japanese yen. Threw exception Error: Failed to format a number.
+FAIL Intl.NumberFormat(&apos;fr&apos;, {style: &apos;currency&apos;, currency: &apos;USD&apos;, currencyDisplay: &apos;name&apos;}).format(4) should be 4,00 dollars des États-Unis. Threw exception Error: Failed to format a number.
+FAIL Intl.NumberFormat(&apos;fr&apos;, {style: &apos;currency&apos;, currency: &apos;JPY&apos;, currencyDisplay: &apos;name&apos;}).format(4) should be 4 yens japonais. Threw exception Error: Failed to format a number.
 PASS Intl.NumberFormat(&apos;en&apos;, {minimumIntegerDigits: 4}).format(12) is &apos;0,012&apos;
 PASS Intl.NumberFormat(&apos;en&apos;, {minimumIntegerDigits: 4}).format(12345) is &apos;12,345&apos;
 PASS Intl.NumberFormat(&apos;en&apos;, {minimumFractionDigits: 3}).format(1) is &apos;1.000&apos;
@@ -184,7 +184,7 @@
 PASS Intl.NumberFormat(&apos;en&apos;, {maximumSignificantDigits: 4}).format(0.1234567) is &apos;0.1235&apos;
 PASS Intl.NumberFormat(&apos;en&apos;, {maximumSignificantDigits: 4}).format(1234567) is &apos;1,235,000&apos;
 PASS Intl.NumberFormat(&apos;en&apos;, {useGrouping: true}).format(1234567.123) is &apos;1,234,567.123&apos;
-PASS Intl.NumberFormat(&apos;es&apos;, {useGrouping: true}).format(1234567.123) is &apos;1.234.567,123&apos;
+FAIL Intl.NumberFormat(&apos;es&apos;, {useGrouping: true}).format(1234567.123) should be 1.234.567,123. Was 1 234 567,123.
 PASS Intl.NumberFormat(&apos;fr&apos;, {useGrouping: true}).format(1234567.123) is &apos;1 234 567,123&apos;
 PASS Intl.NumberFormat(&apos;en&apos;, {useGrouping: false}).format(1234567.123) is &apos;1234567.123&apos;
 PASS Intl.NumberFormat(&apos;es&apos;, {useGrouping: false}).format(1234567.123) is &apos;1234567,123&apos;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1196284</commentid>
    <comment_count>6</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2016-05-25 04:13:13 -0700</bug_when>
    <thetext>Many other tests started to fail after r201066, but the problem seems to be the same. run-javascriptcore-tests does $ENV{LANG}=&quot;en_US.UTF-8&quot;; but we are not actually honoring the environment variables at all when using jsc binary. We are using setlocale with a nullptr locale to get the current one, but the current one is always &quot;C&quot;, because to set the locale according to the environment variables we need to call setlocale with an empty string as locale. That&apos;s done by  gtk_init() which is called by all our binaries (web process, network process, etc.), but not by jsc (because jsc doesn&apos;t depend on GTK+). The reason why it has always worked for EFL is because they call ecore_init() in jsc that calls setlocale. I&apos;ll submit a patch to fix this and the new tests failing.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1196291</commentid>
    <comment_count>7</comment_count>
      <attachid>279758</attachid>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2016-05-25 04:22:34 -0700</bug_when>
    <thetext>Created attachment 279758
Patch

I&apos;m not sure this is also needed for jsc-only port. In that case it would be a matter of updating the ifdef</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1196292</commentid>
    <comment_count>8</comment_count>
      <attachid>279758</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2016-05-25 04:26:20 -0700</bug_when>
    <thetext>Comment on attachment 279758
Patch

OK. Not sure this is the right way to do things. Seems unfortunate to have significant platform dependencies in a command line tool. Unpleasant to have iOS, EFL, Windows, and GTK specifics. I suppose I should be grateful we don’t have anything like this for Mac yet?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1196294</commentid>
    <comment_count>9</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2016-05-25 05:20:25 -0700</bug_when>
    <thetext>hmm, I was wrong, even with this patch intl-numberformat.js is filing, but all others that started to fail after r201066 are passing now, so I&apos;ll submit a separate bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1196295</commentid>
    <comment_count>10</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2016-05-25 05:22:19 -0700</bug_when>
    <thetext>(In reply to comment #7)
&gt; Created attachment 279758 [details]
&gt; Patch
&gt; 
&gt; I&apos;m not sure this is also needed for jsc-only port. In that case it would be
&gt; a matter of updating the ifdef

I&apos;ll check if it fixes failures on jsc-only ARM bots. 
( without guard, of course :) )

Could you wait ~2-3 hours for my testing before landing?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1196298</commentid>
    <comment_count>11</comment_count>
      <attachid>279758</attachid>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2016-05-25 05:27:14 -0700</bug_when>
    <thetext>Comment on attachment 279758
Patch

Clearing flags, I moved the patch to bug #158066</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1196299</commentid>
    <comment_count>12</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2016-05-25 05:28:01 -0700</bug_when>
    <thetext>(In reply to comment #10)
&gt; (In reply to comment #7)
&gt; &gt; Created attachment 279758 [details]
&gt; &gt; Patch
&gt; &gt; 
&gt; &gt; I&apos;m not sure this is also needed for jsc-only port. In that case it would be
&gt; &gt; a matter of updating the ifdef
&gt; 
&gt; I&apos;ll check if it fixes failures on jsc-only ARM bots. 
&gt; ( without guard, of course :) )
&gt; 
&gt; Could you wait ~2-3 hours for my testing before landing?

Sure</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1196300</commentid>
    <comment_count>13</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2016-05-25 05:31:22 -0700</bug_when>
    <thetext>intl-numberformat.js works for me locally, but not in the bots</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1196305</commentid>
    <comment_count>14</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2016-05-25 05:57:05 -0700</bug_when>
    <thetext>In my laptop:

$ LANG=en_US.UTF-8 bin/jsc -e &quot;print(Intl.NumberFormat(&apos;es&apos;, {useGrouping: true}).format(1234567.123))&quot;
1.234.567,123

In the bot:

$ LANG=en_US.UTF-8 bin/jsc -e &quot;print(Intl.NumberFormat(&apos;es&apos;, {useGrouping: true}).format(1234567.123))&quot;
1 234 567,123

There must be something else, doesn&apos;t look like a WebKit problem.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1196306</commentid>
    <comment_count>15</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2016-05-25 06:00:28 -0700</bug_when>
    <thetext>Are we using ICU for this? could it be the ICU version? it&apos;s the only (relevant) thing that looks different between my laptop and the bot</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1196705</commentid>
    <comment_count>16</comment_count>
      <attachid>279882</attachid>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2016-05-26 06:32:54 -0700</bug_when>
    <thetext>Created attachment 279882
Patch for GTK+

So it was indeed the ICU version installed in the bots. Confirmed that using a newer version in the bots fixes the tests.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1196710</commentid>
    <comment_count>17</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2016-05-26 07:43:03 -0700</bug_when>
    <thetext>I tried the patch on a clean build in a tests bot we have and it worked, so I&apos;m not sure about he EWS failure, maybe a clan build would fix it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1196729</commentid>
    <comment_count>18</comment_count>
      <attachid>279882</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2016-05-26 08:47:29 -0700</bug_when>
    <thetext>Comment on attachment 279882
Patch for GTK+

Please look at the compile failure in GTK EWS.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1196730</commentid>
    <comment_count>19</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2016-05-26 08:51:23 -0700</bug_when>
    <thetext>Darin, I think it&apos;s a clean build issue, because I&apos;ve tried in a test bot and it worked. I&apos;ll trigger clean builds in the bots and watch them after landing.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1196993</commentid>
    <comment_count>20</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2016-05-27 00:17:40 -0700</bug_when>
    <thetext>Committed r201449: &lt;http://trac.webkit.org/changeset/201449&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1197304</commentid>
    <comment_count>21</comment_count>
    <who name="Yusuke Suzuki">ysuzuki</who>
    <bug_when>2016-05-27 20:14:23 -0700</bug_when>
    <thetext>While your change solves the JSC tests (Great!!), the layout-tests in GTK ports pose many new failing tests instead. Do you already know the reason?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1197307</commentid>
    <comment_count>22</comment_count>
    <who name="Yusuke Suzuki">ysuzuki</who>
    <bug_when>2016-05-27 20:43:29 -0700</bug_when>
    <thetext>Ah, no, these tests are previously failing. Sorry.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1197474</commentid>
    <comment_count>23</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2016-05-29 03:52:11 -0700</bug_when>
    <thetext>It broke the GTK ARM bot.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1197550</commentid>
    <comment_count>24</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2016-05-29 23:11:46 -0700</bug_when>
    <thetext>(In reply to comment #23)
&gt; It broke the GTK ARM bot.

I already talked to Gustavo, who was going to look at it. For some reason wget is failing to download from sourceforge in the ARM bot. In any case, it&apos;s a problem of the bot itself, not the patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1197756</commentid>
    <comment_count>25</comment_count>
    <who name="Carlos Alberto Lopez Perez">clopez</who>
    <bug_when>2016-05-31 03:33:20 -0700</bug_when>
    <thetext>Committed r201509: &lt;http://trac.webkit.org/changeset/201509&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>279758</attachid>
            <date>2016-05-25 04:22:34 -0700</date>
            <delta_ts>2016-05-25 05:27:38 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>jsc-intl-setlocale.diff</filename>
            <type>text/plain</type>
            <size>2043</size>
            <attacher name="Carlos Garcia Campos">cgarcia</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cgYi9Tb3VyY2UvSmF2
YVNjcmlwdENvcmUvQ2hhbmdlTG9nCmluZGV4IDg3MGU2MzkuLmUxMjFlMTEgMTAwNjQ0Ci0tLSBh
L1NvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL0phdmFTY3JpcHRD
b3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDIxIEBACisyMDE2LTA1LTI1ICBDYXJsb3MgR2FyY2lh
IENhbXBvcyAgPGNnYXJjaWFAaWdhbGlhLmNvbT4KKworICAgICAgICBOZXcgaW50bC1udW1iZXJm
b3JtYXQuanMgdGVzdCBmYWlscyBvbiBtYW55IExpbnV4IHBsYXRmb3JtcworICAgICAgICBodHRw
czovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTU0NTMwCisKKyAgICAgICAgUmV2
aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgcnVuLWphdmFzY3JpcHRjb3JlLXRl
c3RzIGRvZXMgJEVOVntMQU5HfT0iZW5fVVMuVVRGLTgiOyBidXQgd2UgYXJlIG5vdCBhY3R1YWxs
eSBob25vcmluZyB0aGUgZW52aXJvbm1lbnQKKyAgICAgICAgdmFyaWFibGVzIGF0IGFsbCB3aGVu
IHVzaW5nIGpzYyBiaW5hcnkuIFdlIGFyZSB1c2luZyBzZXRsb2NhbGUoKSB3aXRoIGEgbnVsbHB0
ciBsb2NhbGUgdG8gZ2V0IHRoZSBjdXJyZW50IG9uZSwgYnV0CisgICAgICAgIHRoZSBjdXJyZW50
IG9uZSBpcyBhbHdheXMgIkMiLCBiZWNhdXNlIHRvIHNldCB0aGUgbG9jYWxlIGFjY29yZGluZyB0
byB0aGUgZW52aXJvbm1lbnQgdmFyaWFibGVzIHdlIG5lZWQgdG8gY2FsbAorICAgICAgICBzZXRs
b2NhbGUgd2l0aCBhbiBlbXB0eSBzdHJpbmcgYXMgbG9jYWxlLiBUaGF0J3MgZG9uZSBieSBndGtf
aW5pdCgpLCB3aGljaCBpcyBjYWxsZWQgYnkgYWxsIG91ciBiaW5hcmllcyAod2ViCisgICAgICAg
IHByb2Nlc3MsIG5ldHdvcmsgcHJvY2VzcywgZXRjLiksIGJ1dCBub3QgYnkganNjIChiZWNhdXNl
IGpzYyBkb2Vzbid0IGRlcGVuZCBvbiBHVEsrKS4gVGhlIHJlYXNvbiB3aHkgaXQgaGFzCisgICAg
ICAgIGFsd2F5cyB3b3JrZWQgZm9yIEVGTCBpcyBiZWNhdXNlIHRoZXkgY2FsbCBlY29yZV9pbml0
KCkgaW4ganNjIHRoYXQgY2FsbHMgc2V0bG9jYWxlLiBUaGlzIGZpeGVzIG90aGVyIGludGwgdGVz
dHMKKyAgICAgICAgdGhhdCBzdGFydGVkIHRvIGZhaWwgYWZ0ZXIgcjIwMTA2NiB0b28uCisKKyAg
ICAgICAgKiBqc2MuY3BwOgorICAgICAgICAobWFpbik6IENhbGwgc2V0bG9jYWxlKExDX0FMTCwg
IiIpIG9uIEdUSysuCisKIDIwMTYtMDUtMjQgIFNhYW0gYmFyYXRpICA8c2JhcmF0aUBhcHBsZS5j
b20+IGFuZCBZdXN1a2UgU3V6dWtpIDx1dGF0YW5lLnRlYUBnbWFpbC5jb20+CiAKICAgICAgICAg
VGhpc1REWk1vZGUgaXMgbm8gbG9uZ2VyIG5lZWRlZApkaWZmIC0tZ2l0IGEvU291cmNlL0phdmFT
Y3JpcHRDb3JlL2pzYy5jcHAgYi9Tb3VyY2UvSmF2YVNjcmlwdENvcmUvanNjLmNwcAppbmRleCA0
MDk3Mzg0Li5iMDIxZDEwIDEwMDY0NAotLS0gYS9Tb3VyY2UvSmF2YVNjcmlwdENvcmUvanNjLmNw
cAorKysgYi9Tb3VyY2UvSmF2YVNjcmlwdENvcmUvanNjLmNwcApAQCAtMTkzMSw2ICsxOTMxLDEx
IEBAIGludCBtYWluKGludCBhcmdjLCBjaGFyKiogYXJndikKICAgICBlY29yZV9pbml0KCk7CiAj
ZW5kaWYKIAorI2lmIFBMQVRGT1JNKEdUSykKKyAgICBpZiAoIXNldGxvY2FsZShMQ19BTEwsICIi
KSkKKyAgICAgICAgV1RGTG9nQWx3YXlzKCJMb2NhbGUgbm90IHN1cHBvcnRlZCBieSBDIGxpYnJh
cnkuXG5cdFVzaW5nIHRoZSBmYWxsYmFjayAnQycgbG9jYWxlLiIpOworI2VuZGlmCisKICAgICAv
LyBOZWVkIHRvIGluaXRpYWxpemUgV1RGIHRocmVhZGluZyBiZWZvcmUgd2Ugc3RhcnQgYW55IHRo
cmVhZHMuIENhbm5vdCBpbml0aWFsaXplIEpTQwogICAgIC8vIHRocmVhZGluZyB5ZXQsIHNpbmNl
IHRoYXQgd291bGQgZG8gc29tZXRoaW5ncyB0aGF0IHdlJ2QgbGlrZSB0byBkZWZlciB1bnRpbCBh
ZnRlciB3ZQogICAgIC8vIGhhdmUgYSBjaGFuY2UgdG8gcGFyc2Ugb3B0aW9ucy4K
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>279882</attachid>
            <date>2016-05-26 06:32:54 -0700</date>
            <delta_ts>2016-05-26 16:30:52 -0700</delta_ts>
            <desc>Patch for GTK+</desc>
            <filename>icu.diff</filename>
            <type>text/plain</type>
            <size>2595</size>
            <attacher name="Carlos Garcia Campos">cgarcia</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCBiOWFi
MzU5Li4wZmU4YWIzIDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hh
bmdlTG9nCkBAIC0xLDMgKzEsMTYgQEAKKzIwMTYtMDUtMjYgIENhcmxvcyBHYXJjaWEgQ2FtcG9z
ICA8Y2dhcmNpYUBpZ2FsaWEuY29tPgorCisgICAgICAgIE5ldyBpbnRsLW51bWJlcmZvcm1hdC5q
cyB0ZXN0IGZhaWxzIG9uIG1hbnkgTGludXggcGxhdGZvcm1zCisgICAgICAgIGh0dHBzOi8vYnVn
cy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xNTQ1MzAKKworICAgICAgICBSZXZpZXdlZCBi
eSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBUaGUgdGVzdCBpcyBhY3R1YWxseSBmYWlsaW5n
IGJlY2F1c2Ugb2YgYSBidWcgaW4gdGhlIGljdSB2ZXJzaW9uIGluc3RhbGxlZCBpbiB0aGUgYm90
cywgdXNpbmcgYSBuZXdlciB2ZXJzaW9uIG9mCisgICAgICAgIGljdSBtYWtlcyB0aGUgdGVzdHMg
cGFzcyBhbmQgZXhwbGFpbnMgd2h5IGl0IHdvcmtlZCBmb3Igc29tZSBwZW9wbGUuIFNvLCBsZXQn
cyBhZGQgaWN1IDU1LjEgdG8gdGhlIGludGVybmFsCisgICAgICAgIGpoYnVpbGQgdG8gZW5zdXJl
IEpTQyB0ZXN0cyBwYXNzIGluIHRoZSBib3RzIG5vIG1hdHRlciB3aGF0IHRoZSBpY3UgdmVyc2lv
biBpbnN0YWxsZWQgaXMuCisKKyAgICAgICAgKiBndGsvamhidWlsZC5tb2R1bGVzOgorCiAyMDE2
LTA1LTI0ICBBYWthc2ggSmFpbiAgPGFha2FzaF9qYWluQGFwcGxlLmNvbT4KIAogICAgICAgICBS
ZW9yZ2FuaXplIGRhc2hib2FyZCBjb2RlOiBtb3ZlIGNvZGUgZnJvbSBfdGltZUludGVydmFsU3Ry
aW5nIHRvIGJhc2UgY2xhc3MKZGlmZiAtLWdpdCBhL1Rvb2xzL2d0ay9qaGJ1aWxkLm1vZHVsZXMg
Yi9Ub29scy9ndGsvamhidWlsZC5tb2R1bGVzCmluZGV4IGQ2NGNlZjYuLjhkMjg5MGUgMTAwNjQ0
Ci0tLSBhL1Rvb2xzL2d0ay9qaGJ1aWxkLm1vZHVsZXMKKysrIGIvVG9vbHMvZ3RrL2poYnVpbGQu
bW9kdWxlcwpAQCAtMjAsNiArMjAsNyBAQAogICAgICAgPGRlcCBwYWNrYWdlPSJnbm9tZS1pY29u
LXRoZW1lLXN5bWJvbGljIi8+CiAgICAgICA8ZGVwIHBhY2thZ2U9Imdub21lLXRoZW1lcy1zdGFu
ZGFyZCIvPgogICAgICAgPGRlcCBwYWNrYWdlPSJndGstZG9jIi8+CisgICAgICA8ZGVwIHBhY2th
Z2U9ImljdSIvPgogICAgICAgPGRlcCBwYWNrYWdlPSJsaWJzb3VwIi8+CiAgICAgICA8ZGVwIHBh
Y2thZ2U9ImF0ayIvPgogICAgICAgPGRlcCBwYWNrYWdlPSJnc3RyZWFtZXIiLz4KQEAgLTY5LDYg
KzcwLDggQEAKICAgICAgIGhyZWY9Imh0dHA6Ly9wZW9wbGUuZnJlZWRlc2t0b3Aub3JnIi8+CiAg
IDxyZXBvc2l0b3J5IHR5cGU9InRhcmJhbGwiIG5hbWU9Imxsdm0ub3JnIgogICAgICAgaHJlZj0i
aHR0cDovL2xsdm0ub3JnIi8+CisgIDxyZXBvc2l0b3J5IHR5cGU9InRhcmJhbGwiIG5hbWU9Imlj
dSIKKyAgICAgIGhyZWY9Imh0dHA6Ly9kb3dubG9hZC5pY3UtcHJvamVjdC5vcmcvZmlsZXMvaWN1
NGMvIi8+CiAKICAgPGF1dG90b29scyBpZD0iY2Fpcm8iPgogICAgIDxpZiBjb25kaXRpb24tc2V0
PSJsaW51eCI+CkBAIC0xMTksNiArMTIyLDkgQEAKICAgPC9hdXRvdG9vbHM+CiAKICAgPGF1dG90
b29scyBpZD0iaGFyZmJ1enoiIGF1dG9nZW4tc2g9ImNvbmZpZ3VyZSI+CisgICAgPGRlcGVuZGVu
Y2llcz4KKyAgICAgIDxkZXAgcGFja2FnZT0iaWN1Ii8+CisgICAgPC9kZXBlbmRlbmNpZXM+CiAg
ICAgPGJyYW5jaCBtb2R1bGU9InNvZnR3YXJlL2hhcmZidXp6L3JlbGVhc2UvaGFyZmJ1enotMC45
LjM1LnRhci5iejIiCiAgICAgICAgICAgICB2ZXJzaW9uPSIwLjkuMzUiCiAgICAgICAgICAgICBj
aGVja291dGRpcj0iaGFyZmJ1enotMC45LjM1IgpAQCAtNDgzLDcgKzQ4OSwxNiBAQAogICAgICAg
PHBhdGNoIGZpbGU9InNoYXJlZC1taW1lLWluZm8teGh0bWwtbWFnaWMucGF0Y2giIHN0cmlwPSIx
Ii8+CiAgICAgPC9icmFuY2g+CiAgIDwvYXV0b3Rvb2xzPgotICAKKworICA8YXV0b3Rvb2xzIGlk
PSJpY3UiCisgICAgICAgICAgICAgYXV0b2dlbi1zaD0iLi9zb3VyY2UvY29uZmlndXJlIgorICAg
ICAgICAgICAgIGF1dG9nZW5hcmdzPSItLWRpc2FibGUtc2FtcGxlcyAtLWVuYWJsZS13ZWFrLXRo
cmVhZHMiPgorICAgIDxicmFuY2ggbW9kdWxlPSI1NS4xL2ljdTRjLTU1XzEtc3JjLnRneiIgdmVy
c2lvbj0iNTUuMSIgY2hlY2tvdXRkaXI9ImljdSIKKyAgICAgICAgICAgIHJlcG89ImljdSIKKyAg
ICAgICAgICAgIGhhc2g9InNoYTI1NjplMTZiMjJjYmVmZGQzNTRiZWMxMTQ1NDFmNzg0OWExMmY4
ZmMyMDE1MzIwY2E1MjgyZWU0ZmQ3ODc1NzE0NTdiIj4KKyAgICA8L2JyYW5jaD4KKyAgPC9hdXRv
dG9vbHM+CisKICAgPCEtLSBEZXBlbmRlbmNpZXMgbGlzdGVkIGJlbG93IHRoaXMgcG9pbnQgYXJl
IG5vdCB0aG91Z2h0IHRvIGFmZmVjdCB0ZXN0IHJlc3VsdHMsIGFuZCBhcmUgb25seQogICAgICAg
IGluY2x1ZGVkIGJlY2F1c2UgdGhleSB0aGVtc2VsdmVzIGRlcGVuZCBvbiBvdGhlciBkZXBlbmRl
bmNpZXMgYnVpbHQgYnkgamhidWlsZC4gLS0+CiAK
</data>
<flag name="review"
          id="303905"
          type_id="1"
          status="+"
          setter="darin"
    />
          </attachment>
      

    </bug>

</bugzilla>