<?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>117275</bug_id>
          
          <creation_ts>2013-06-05 13:35:27 -0700</creation_ts>
          <short_desc>[GTK][EFL] Use function jhbuildWrapperPrefixIfNeeded to run launcher</short_desc>
          <delta_ts>2014-01-07 04:48:34 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Tools / Tests</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>Enhancement</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Diego Pino">dpino</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>berto</cc>
    
    <cc>cgarcia</cc>
    
    <cc>mrobinson</cc>
    
    <cc>xan.lopez</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>897531</commentid>
    <comment_count>0</comment_count>
    <who name="Diego Pino">dpino</who>
    <bug_when>2013-06-05 13:35:27 -0700</bug_when>
    <thetext>The code that runs the jhbuild-wrapper in &apos;./Tools/Scripts/run-launcher&apos; composes the path and parameters by itself. There&apos;s a function, jhbuildWrapperPrefixIfNeeded(), in the Webkitdirs module that does that.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>897532</commentid>
    <comment_count>1</comment_count>
      <attachid>203879</attachid>
    <who name="Diego Pino">dpino</who>
    <bug_when>2013-06-05 13:42:53 -0700</bug_when>
    <thetext>Created attachment 203879
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>897586</commentid>
    <comment_count>2</comment_count>
      <attachid>203879</attachid>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2013-06-05 15:40:32 -0700</bug_when>
    <thetext>Comment on attachment 203879
Patch

Nice.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>897655</commentid>
    <comment_count>3</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2013-06-05 23:51:19 -0700</bug_when>
    <thetext>*** Bug 116095 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>897663</commentid>
    <comment_count>4</comment_count>
      <attachid>203879</attachid>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2013-06-06 00:08:03 -0700</bug_when>
    <thetext>Comment on attachment 203879
Patch

This patch doesn&apos;t work when not using jhbuild, see:

$ Tools/Scripts/run-launcher --gtk
Starting webkit launcher.
Use of uninitialized value $launcherPath in exec at Tools/Scripts/run-launcher line 87.
Can&apos;t exec &quot;&quot;: No existe el archivo o el directorio at Tools/Scripts/run-launcher line 87.
Died at Tools/Scripts/run-launcher line 87.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>897668</commentid>
    <comment_count>5</comment_count>
    <who name="Diego Pino">dpino</who>
    <bug_when>2013-06-06 00:35:59 -0700</bug_when>
    <thetext>OK, I think the reason is because the function jhbuildWrapperPrefixIfNeeded() tests first if the jhbuildPath exists. If not, returns an empty path.

What I don&apos;t understand is that running the script using jhbuildWrapperPrefixIfNeeded() and not using it builds the same path:

$ ./Tools/Scripts/run-launcher --gtk
without using jhbuildWrapperPrefixIfNeeded: /home/dpino/workspace/WebKit/Tools/jhbuild/jhbuild-wrapper
using jhbuildWrapperPrefixIfNeeded: /home/dpino/workspace/WebKit/Tools/jhbuild/jhbuild-wrapper

I could remove the checking of &quot;jhbuildPath exists&quot; in jhbuildWrapperPrefixIfNeeded() but that seems wrong.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>899460</commentid>
    <comment_count>6</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2013-06-12 00:27:14 -0700</bug_when>
    <thetext>(In reply to comment #5)
&gt; OK, I think the reason is because the function jhbuildWrapperPrefixIfNeeded() tests first if the jhbuildPath exists. If not, returns an empty path.

In case of empty path we need to use the launcher path directly.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>899469</commentid>
    <comment_count>7</comment_count>
    <who name="Diego Pino">dpino</who>
    <bug_when>2013-06-12 01:28:04 -0700</bug_when>
    <thetext>(In reply to comment #6)
&gt; (In reply to comment #5)
&gt; &gt; OK, I think the reason is because the function jhbuildWrapperPrefixIfNeeded() tests first if the jhbuildPath exists. If not, returns an empty path.
&gt; 
&gt; In case of empty path we need to use the launcher path directly.

OK, so regardless the path is empty or not, it seems the launcher path is always the same. If that&apos;s the case what I could do is to add a buildLauncherPath() method to webkitdirs.pm and make the function jhbuildWrapperPrefixIfNeeded() to use buildLauncherPath() if the path is not empty. The patch proposed will use buildLauncherPath() instead of jhbuildWrapperPrefixIfNeeded(). 

What do you think?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>899478</commentid>
    <comment_count>8</comment_count>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2013-06-12 02:26:09 -0700</bug_when>
    <thetext>(In reply to comment #7)
&gt; (In reply to comment #6)
&gt; &gt; (In reply to comment #5)
&gt; &gt; &gt; OK, I think the reason is because the function jhbuildWrapperPrefixIfNeeded() tests first if the jhbuildPath exists. If not, returns an empty path.
&gt; &gt; 
&gt; &gt; In case of empty path we need to use the launcher path directly.
&gt; 
&gt; OK, so regardless the path is empty or not, it seems the launcher path is always the same. If that&apos;s the case what I could do is to add a buildLauncherPath() method to webkitdirs.pm and make the function jhbuildWrapperPrefixIfNeeded() to use buildLauncherPath() if the path is not empty. The patch proposed will use buildLauncherPath() instead of jhbuildWrapperPrefixIfNeeded(). 
&gt; 
&gt; What do you think?

I don&apos;t think so, jhbuildWrapperPrefixIfNeeded() return the jhbuild prefix if needed. So we need to build the launcher path, Programs/GtkLauncher, for example, and if there&apos;s a jhbuild prefix, prepend it so what you run in the end is jhbuild run Programs/GtkLauncher and if there&apos;s not jhbuild prefix you run Programs/GtkLauncher directly.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>964955</commentid>
    <comment_count>9</comment_count>
      <attachid>220510</attachid>
    <who name="Alberto Garcia">berto</who>
    <bug_when>2014-01-07 04:20:45 -0800</bug_when>
    <thetext>Created attachment 220510
Patch

(In reply to comment #5)
&gt; OK, I think the reason is because the function
&gt; jhbuildWrapperPrefixIfNeeded() tests first if the jhbuildPath
&gt; exists. If not, returns an empty path.

That function returns an empty list, so when do you

($launcherPath, my @args) = jhbuildWrapperPrefixIfNeeded();

you&apos;re turning $launcherPath into an undefined variable, which you
then are trying to exec.

I reworked the script a bit, now there&apos;s a scalar variable for the
launcher command, an array for the jhbuild wrapper and its options and
@ARGV, with the command-line parameters set by the user, which I
decided to leave untouched.

Hope it&apos;s a bit clearer now.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>964956</commentid>
    <comment_count>10</comment_count>
      <attachid>220511</attachid>
    <who name="Alberto Garcia">berto</who>
    <bug_when>2014-01-07 04:30:03 -0800</bug_when>
    <thetext>Created attachment 220511
Patch

There was actually a duplicate variable in that script, I updated the
patch to remove it as well.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>964957</commentid>
    <comment_count>11</comment_count>
      <attachid>220512</attachid>
    <who name="Alberto Garcia">berto</who>
    <bug_when>2014-01-07 04:31:13 -0800</bug_when>
    <thetext>Created attachment 220512
Patch

Today is not my day. This is the correct one.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>964960</commentid>
    <comment_count>12</comment_count>
      <attachid>220512</attachid>
    <who name="Carlos Garcia Campos">cgarcia</who>
    <bug_when>2014-01-07 04:37:16 -0800</bug_when>
    <thetext>Comment on attachment 220512
Patch

Thank you very much!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>964962</commentid>
    <comment_count>13</comment_count>
    <who name="Alberto Garcia">berto</who>
    <bug_when>2014-01-07 04:48:34 -0800</bug_when>
    <thetext>Committed r161419: &lt;http://trac.webkit.org/changeset/161419&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>203879</attachid>
            <date>2013-06-05 13:42:53 -0700</date>
            <delta_ts>2014-01-07 04:20:45 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-117275-20130605224121.patch</filename>
            <type>text/plain</type>
            <size>1680</size>
            <attacher name="Diego Pino">dpino</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTUwOTUzCmRpZmYgLS1naXQgYS9Ub29scy9DaGFuZ2VMb2cg
Yi9Ub29scy9DaGFuZ2VMb2cKaW5kZXggNDFlOTMzOWJiN2U2NDczYzY5MjFiNzNhZTRlYWNiNDMw
ZTA1NjBhNi4uNDhhODIzZTk4ZjhiMzE1YjIwNmRkNzEzZmFmNjhlYzk4ZjAwMzk4NCAxMDA2NDQK
LS0tIGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBiL1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDEz
IEBACisyMDEzLTA2LTA1ICBEaWVnbyBQaW5vIEdhcmPDrWEgIDxkcGlub0BpZ2FsaWEuY29tPgor
CisgICAgICAgIFtHVEtdIFVzZSBmdW5jdGlvbiBqaGJ1aWxkV3JhcHBlclByZWZpeElmTmVlZGVk
IHRvIHJ1biBsYXVuY2hlcgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1
Zy5jZ2k/aWQ9MTE3Mjc1CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisK
KyAgICAgICAgKiBTY3JpcHRzL3J1bi1sYXVuY2hlcjogdXNlIGZ1bmN0aW9uIGpoYnVpbGRXcmFw
cGVyUHJlZml4SWZOZWVkZWQoKSB0bworICAgICAgICBydW4gbGF1bmNoZXIKKwogMjAxMy0wNS0z
MCAgUnlvc3VrZSBOaXdhICA8cm5pd2FAd2Via2l0Lm9yZz4KIAogICAgICAgICBSZW1vdmUgcmVx
dWlyZXNfaHR0cF9zZXJ2ZXIgYXMgQ2hyb21pdW0gQW5kcm9pZCB3YXMgdGhlIG9ubHkgcG9ydCB0
aGF0IG5lZWRlZCB0aGlzCmRpZmYgLS1naXQgYS9Ub29scy9TY3JpcHRzL3J1bi1sYXVuY2hlciBi
L1Rvb2xzL1NjcmlwdHMvcnVuLWxhdW5jaGVyCmluZGV4IDY2ODg2ZmUwYjE5NzRiMjMxOTM4NDFk
YzdjOTdlMzgxOThiODk5YzcuLjE2YzYzMTU4M2E3ZGI4NTc5YTY0MGU2OTNlMDJmYjczOGNmMWFh
YzggMTAwNzU1Ci0tLSBhL1Rvb2xzL1NjcmlwdHMvcnVuLWxhdW5jaGVyCisrKyBiL1Rvb2xzL1Nj
cmlwdHMvcnVuLWxhdW5jaGVyCkBAIC02OSw4ICs2OSw2IEBAIGlmIChpc1F0KCkpIHsKICAgICAg
ICAgfSBlbHNlIHsKICAgICAgICAgICAgIHVuc2hpZnQoQEFSR1YsIGNhdGRpcigkbGF1bmNoZXJQ
YXRoLCAiUHJvZ3JhbXMiLCAiR3RrTGF1bmNoZXIiKSk7CiAgICAgICAgIH0KLSAgICAgICAgJGxh
dW5jaGVyUGF0aCA9IGNhdGRpcihzb3VyY2VEaXIoKSwgIlRvb2xzIiwgImpoYnVpbGQiLCAiamhi
dWlsZC13cmFwcGVyIik7Ci0gICAgICAgIHVuc2hpZnQoQEFSR1YsICgiLS1ndGsiLCAicnVuIikp
OwogICAgIH0KIAogICAgIGlmIChpc0VmbCgpKSB7CkBAIC03OSwxMiArNzcsMTEgQEAgaWYgKGlz
UXQoKSkgewogICAgICAgICB9IGVsc2UgewogICAgICAgICAgICAgdW5zaGlmdChAQVJHViwgY2F0
ZGlyKCRsYXVuY2hlclBhdGgsICJiaW4iLCAiRVdlYkxhdW5jaGVyIikpOwogICAgICAgICB9Ci0g
ICAgICAgICRsYXVuY2hlclBhdGggPSBjYXRkaXIoc291cmNlRGlyKCksICJUb29scyIsICJqaGJ1
aWxkIiwgImpoYnVpbGQtd3JhcHBlciIpOwotICAgICAgICB1bnNoaWZ0KEBBUkdWLCAoIi0tZWZs
IiwgInJ1biIpKTsKICAgICB9CisgICAgKCRsYXVuY2hlclBhdGgsIG15IEBhcmdzKSA9IGpoYnVp
bGRXcmFwcGVyUHJlZml4SWZOZWVkZWQoKTsKKyAgICB1bnNoaWZ0KEBBUkdWLCBAYXJncyk7CiAK
ICAgICBwcmludCAiU3RhcnRpbmcgd2Via2l0IGxhdW5jaGVyLlxuIjsKIH0KIAogZXhlYyAkbGF1
bmNoZXJQYXRoLCBAQVJHViBvciBkaWU7Ci0K
</data>
<flag name="review"
          id="225339"
          type_id="1"
          status="+"
          setter="mrobinson"
    />
    <flag name="commit-queue"
          id="225373"
          type_id="3"
          status="-"
          setter="cgarcia"
    />
          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>220510</attachid>
            <date>2014-01-07 04:20:45 -0800</date>
            <delta_ts>2014-01-07 04:30:03 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>117275.diff</filename>
            <type>text/plain</type>
            <size>2371</size>
            <attacher name="Alberto Garcia">berto</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCBhYmZj
MDdjLi5lYzI1MDA1IDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hh
bmdlTG9nCkBAIC0xLDMgKzEsMTYgQEAKKzIwMTQtMDEtMDcgIEFsYmVydG8gR2FyY2lhICA8YmVy
dG9AaWdhbGlhLmNvbT4KKworICAgICAgICBbR1RLXVtFRkxdIFVzZSBmdW5jdGlvbiBqaGJ1aWxk
V3JhcHBlclByZWZpeElmTmVlZGVkIHRvIHJ1biBsYXVuY2hlcgorICAgICAgICBodHRwczovL2J1
Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTE3Mjc1CisKKyAgICAgICAgUmV2aWV3ZWQg
YnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgSWYgd2UgYXJlIG5vdCB1c2luZyB0aGUgaW50
ZXJuYWwgamhidWlsZCB3ZSBtdXN0IG5vdCB1c2UgdGhlCisgICAgICAgIGpoYnVpbGQtd3JhcHBl
ciBzY3JpcHQsIGVsc2UgaXQgd2lsbCB0cnkgdG8gZG93bmxvYWQgYW5kIGluc3RhbGwKKyAgICAg
ICAgaXQuCisKKyAgICAgICAgKiBTY3JpcHRzL3J1bi1sYXVuY2hlcjoKKwogMjAxNC0wMS0wNiAg
TWFyayBSb3dlICA8bXJvd2VAYXBwbGUuY29tPgogCiAgICAgICAgIDxodHRwczovL3dlYmtpdC5v
cmcvYi8xMjY1NjA+IEZpeCBpbmNvcnJlY3RuZXNzIGluIHVzZSBvZiBzb21lIEFwcEtpdCBlbnVt
cwpkaWZmIC0tZ2l0IGEvVG9vbHMvU2NyaXB0cy9ydW4tbGF1bmNoZXIgYi9Ub29scy9TY3JpcHRz
L3J1bi1sYXVuY2hlcgppbmRleCBiODk2MTFmLi5hNDljNzlmIDEwMDc1NQotLS0gYS9Ub29scy9T
Y3JpcHRzL3J1bi1sYXVuY2hlcgorKysgYi9Ub29scy9TY3JpcHRzL3J1bi1sYXVuY2hlcgpAQCAt
MzksNiArMzksOCBAQCB1c2Ugd2Via2l0ZGlyczsKIHNldENvbmZpZ3VyYXRpb24oKTsKIG15ICRw
cm9kdWN0RGlyID0gcHJvZHVjdERpcigpOwogbXkgJGxhdW5jaGVyUGF0aCA9IHByb2R1Y3REaXIo
KTsKK215ICRsYXVuY2hlckNtZDsKK215IEBqaGJ1aWxkV3JhcHBlcjsKIAogIyBDaGVjayB0byBz
ZWUgdGhhdCBhbGwgdGhlIGZyYW1ld29ya3MgYXJlIGJ1aWx0LgogY2hlY2tGcmFtZXdvcmtzKCk7
CkBAIC00NiwyNSArNDgsMjUgQEAgY2hlY2tGcmFtZXdvcmtzKCk7CiAjIFNldCBwYXRocyBhY2Nv
cmRpbmcgdG8gdGhlIGJ1aWxkIHN5c3RlbSB1c2VkCiBpZiAoaXNHdGsoKSkgewogICAgIGlmIChp
c1dLMigpKSB7Ci0gICAgICAgIHVuc2hpZnQoQEFSR1YsIGNhdGRpcigkbGF1bmNoZXJQYXRoLCAi
UHJvZ3JhbXMiLCAiTWluaUJyb3dzZXIiKSk7CisgICAgICAgICRsYXVuY2hlckNtZCA9IGNhdGRp
cigkbGF1bmNoZXJQYXRoLCAiUHJvZ3JhbXMiLCAiTWluaUJyb3dzZXIiKTsKICAgICB9IGVsc2Ug
ewotICAgICAgICB1bnNoaWZ0KEBBUkdWLCBjYXRkaXIoJGxhdW5jaGVyUGF0aCwgIlByb2dyYW1z
IiwgIkd0a0xhdW5jaGVyIikpOworICAgICAgICAkbGF1bmNoZXJDbWQgPSBjYXRkaXIoJGxhdW5j
aGVyUGF0aCwgIlByb2dyYW1zIiwgIkd0a0xhdW5jaGVyIik7CiAgICAgfQotICAgICRsYXVuY2hl
clBhdGggPSBjYXRkaXIoc291cmNlRGlyKCksICJUb29scyIsICJqaGJ1aWxkIiwgImpoYnVpbGQt
d3JhcHBlciIpOwotICAgIHVuc2hpZnQoQEFSR1YsICgiLS1ndGsiLCAicnVuIikpOwogfQogCiBp
ZiAoaXNFZmwoKSkgewogICAgIGlmIChpc1dLMigpKSB7Ci0gICAgICAgIHVuc2hpZnQoQEFSR1Ys
IGNhdGRpcigkbGF1bmNoZXJQYXRoLCAiYmluIiwgIk1pbmlCcm93c2VyIikpOworICAgICAgICAk
bGF1bmNoZXJDbWQgPSBjYXRkaXIoJGxhdW5jaGVyUGF0aCwgImJpbiIsICJNaW5pQnJvd3NlciIp
OwogICAgIH0gZWxzZSB7Ci0gICAgICAgIHVuc2hpZnQoQEFSR1YsIGNhdGRpcigkbGF1bmNoZXJQ
YXRoLCAiYmluIiwgIkVXZWJMYXVuY2hlciIpKTsKKyAgICAgICAgJGxhdW5jaGVyQ21kID0gY2F0
ZGlyKCRsYXVuY2hlclBhdGgsICJiaW4iLCAiRVdlYkxhdW5jaGVyIik7CiAgICAgfQotICAgICRs
YXVuY2hlclBhdGggPSBjYXRkaXIoc291cmNlRGlyKCksICJUb29scyIsICJqaGJ1aWxkIiwgImpo
YnVpbGQtd3JhcHBlciIpOwotICAgIHVuc2hpZnQoQEFSR1YsICgiLS1lZmwiLCAicnVuIikpOwog
fQogCiskbGF1bmNoZXJDbWQgJiYgLXggJGxhdW5jaGVyQ21kIG9yIGRpZSAiQ2FuJ3QgZmluZCB0
aGUgbGF1bmNoZXIgY29tbWFuZCI7CisKK0BqaGJ1aWxkV3JhcHBlciA9IGpoYnVpbGRXcmFwcGVy
UHJlZml4SWZOZWVkZWQoKTsKKwogcHJpbnQgIlN0YXJ0aW5nIHdlYmtpdCBsYXVuY2hlci5cbiI7
CiAKLWV4ZWMgJGxhdW5jaGVyUGF0aCwgQEFSR1Ygb3IgZGllOworZXhlYyBAamhidWlsZFdyYXBw
ZXIsICRsYXVuY2hlckNtZCwgQEFSR1Ygb3IgZGllOwogCg==
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>220511</attachid>
            <date>2014-01-07 04:30:03 -0800</date>
            <delta_ts>2014-01-07 04:31:13 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>117275.diff</filename>
            <type>text/plain</type>
            <size>2467</size>
            <attacher name="Alberto Garcia">berto</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCBhYmZj
MDdjLi5lYzI1MDA1IDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hh
bmdlTG9nCkBAIC0xLDMgKzEsMTYgQEAKKzIwMTQtMDEtMDcgIEFsYmVydG8gR2FyY2lhICA8YmVy
dG9AaWdhbGlhLmNvbT4KKworICAgICAgICBbR1RLXVtFRkxdIFVzZSBmdW5jdGlvbiBqaGJ1aWxk
V3JhcHBlclByZWZpeElmTmVlZGVkIHRvIHJ1biBsYXVuY2hlcgorICAgICAgICBodHRwczovL2J1
Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTE3Mjc1CisKKyAgICAgICAgUmV2aWV3ZWQg
YnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgSWYgd2UgYXJlIG5vdCB1c2luZyB0aGUgaW50
ZXJuYWwgamhidWlsZCB3ZSBtdXN0IG5vdCB1c2UgdGhlCisgICAgICAgIGpoYnVpbGQtd3JhcHBl
ciBzY3JpcHQsIGVsc2UgaXQgd2lsbCB0cnkgdG8gZG93bmxvYWQgYW5kIGluc3RhbGwKKyAgICAg
ICAgaXQuCisKKyAgICAgICAgKiBTY3JpcHRzL3J1bi1sYXVuY2hlcjoKKwogMjAxNC0wMS0wNiAg
TWFyayBSb3dlICA8bXJvd2VAYXBwbGUuY29tPgogCiAgICAgICAgIDxodHRwczovL3dlYmtpdC5v
cmcvYi8xMjY1NjA+IEZpeCBpbmNvcnJlY3RuZXNzIGluIHVzZSBvZiBzb21lIEFwcEtpdCBlbnVt
cwpkaWZmIC0tZ2l0IGEvVG9vbHMvU2NyaXB0cy9ydW4tbGF1bmNoZXIgYi9Ub29scy9TY3JpcHRz
L3J1bi1sYXVuY2hlcgppbmRleCBiODk2MTFmLi5lNDlmZjUxIDEwMDc1NQotLS0gYS9Ub29scy9T
Y3JpcHRzL3J1bi1sYXVuY2hlcgorKysgYi9Ub29scy9TY3JpcHRzL3J1bi1sYXVuY2hlcgpAQCAt
MzgsNyArMzgsOCBAQCB1c2Ugd2Via2l0ZGlyczsKIAogc2V0Q29uZmlndXJhdGlvbigpOwogbXkg
JHByb2R1Y3REaXIgPSBwcm9kdWN0RGlyKCk7Ci1teSAkbGF1bmNoZXJQYXRoID0gcHJvZHVjdERp
cigpOworbXkgJGxhdW5jaGVyUGF0aDsKK215IEBqaGJ1aWxkV3JhcHBlcjsKIAogIyBDaGVjayB0
byBzZWUgdGhhdCBhbGwgdGhlIGZyYW1ld29ya3MgYXJlIGJ1aWx0LgogY2hlY2tGcmFtZXdvcmtz
KCk7CkBAIC00NiwyNSArNDcsMjggQEAgY2hlY2tGcmFtZXdvcmtzKCk7CiAjIFNldCBwYXRocyBh
Y2NvcmRpbmcgdG8gdGhlIGJ1aWxkIHN5c3RlbSB1c2VkCiBpZiAoaXNHdGsoKSkgewogICAgIGlm
IChpc1dLMigpKSB7Ci0gICAgICAgIHVuc2hpZnQoQEFSR1YsIGNhdGRpcigkbGF1bmNoZXJQYXRo
LCAiUHJvZ3JhbXMiLCAiTWluaUJyb3dzZXIiKSk7CisgICAgICAgICRsYXVuY2hlclBhdGggPSBj
YXRkaXIoJHByb2R1Y3REaXIsICJQcm9ncmFtcyIsICJNaW5pQnJvd3NlciIpOwogICAgIH0gZWxz
ZSB7Ci0gICAgICAgIHVuc2hpZnQoQEFSR1YsIGNhdGRpcigkbGF1bmNoZXJQYXRoLCAiUHJvZ3Jh
bXMiLCAiR3RrTGF1bmNoZXIiKSk7CisgICAgICAgICRsYXVuY2hlclBhdGggPSBjYXRkaXIoJHBy
b2R1Y3REaXIsICJQcm9ncmFtcyIsICJHdGtMYXVuY2hlciIpOwogICAgIH0KLSAgICAkbGF1bmNo
ZXJQYXRoID0gY2F0ZGlyKHNvdXJjZURpcigpLCAiVG9vbHMiLCAiamhidWlsZCIsICJqaGJ1aWxk
LXdyYXBwZXIiKTsKLSAgICB1bnNoaWZ0KEBBUkdWLCAoIi0tZ3RrIiwgInJ1biIpKTsKIH0KIAog
aWYgKGlzRWZsKCkpIHsKICAgICBpZiAoaXNXSzIoKSkgewotICAgICAgICB1bnNoaWZ0KEBBUkdW
LCBjYXRkaXIoJGxhdW5jaGVyUGF0aCwgImJpbiIsICJNaW5pQnJvd3NlciIpKTsKKyAgICAgICAg
JGxhdW5jaGVyUGF0aCA9IGNhdGRpcigkcHJvZHVjdERpciwgImJpbiIsICJNaW5pQnJvd3NlciIp
OwogICAgIH0gZWxzZSB7Ci0gICAgICAgIHVuc2hpZnQoQEFSR1YsIGNhdGRpcigkbGF1bmNoZXJQ
YXRoLCAiYmluIiwgIkVXZWJMYXVuY2hlciIpKTsKKyAgICAgICAgJGxhdW5jaGVyUGF0aCA9IGNh
dGRpcigkcHJvZHVjdERpciwgImJpbiIsICJFV2ViTGF1bmNoZXIiKTsKICAgICB9Ci0gICAgJGxh
dW5jaGVyUGF0aCA9IGNhdGRpcihzb3VyY2VEaXIoKSwgIlRvb2xzIiwgImpoYnVpbGQiLCAiamhi
dWlsZC13cmFwcGVyIik7Ci0gICAgdW5zaGlmdChAQVJHViwgKCItLWVmbCIsICJydW4iKSk7CiB9
CiAKKyRsYXVuY2hlclBhdGggJiYgLXggJGxhdW5jaGVyUGF0aCBvciBkaWUgIkNhbid0IGZpbmQg
dGhlIGxhdW5jaGVyIGNvbW1hbmQiOworCitAamhidWlsZFdyYXBwZXIgPSBqaGJ1aWxkV3JhcHBl
clByZWZpeElmTmVlZGVkKCk7CisKIHByaW50ICJTdGFydGluZyB3ZWJraXQgbGF1bmNoZXIuXG4i
OwogCi1leGVjICRsYXVuY2hlclBhdGgsIEBBUkdWIG9yIGRpZTsKK3N1YiBmb28geyBmb3JlYWNo
IChAXykgeyBwcmludCAiLT4kXzwtXG4iOyB9IH0KK2ZvbyBAamhidWlsZFdyYXBwZXIsICRsYXVu
Y2hlclBhdGgsIEBBUkdWOworCitleGVjIEBqaGJ1aWxkV3JhcHBlciwgJGxhdW5jaGVyUGF0aCwg
QEFSR1Ygb3IgZGllOwogCg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>220512</attachid>
            <date>2014-01-07 04:31:13 -0800</date>
            <delta_ts>2014-01-07 04:37:15 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>117275.diff</filename>
            <type>text/plain</type>
            <size>2373</size>
            <attacher name="Alberto Garcia">berto</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCBhYmZj
MDdjLi5lYzI1MDA1IDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hh
bmdlTG9nCkBAIC0xLDMgKzEsMTYgQEAKKzIwMTQtMDEtMDcgIEFsYmVydG8gR2FyY2lhICA8YmVy
dG9AaWdhbGlhLmNvbT4KKworICAgICAgICBbR1RLXVtFRkxdIFVzZSBmdW5jdGlvbiBqaGJ1aWxk
V3JhcHBlclByZWZpeElmTmVlZGVkIHRvIHJ1biBsYXVuY2hlcgorICAgICAgICBodHRwczovL2J1
Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTE3Mjc1CisKKyAgICAgICAgUmV2aWV3ZWQg
YnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgSWYgd2UgYXJlIG5vdCB1c2luZyB0aGUgaW50
ZXJuYWwgamhidWlsZCB3ZSBtdXN0IG5vdCB1c2UgdGhlCisgICAgICAgIGpoYnVpbGQtd3JhcHBl
ciBzY3JpcHQsIGVsc2UgaXQgd2lsbCB0cnkgdG8gZG93bmxvYWQgYW5kIGluc3RhbGwKKyAgICAg
ICAgaXQuCisKKyAgICAgICAgKiBTY3JpcHRzL3J1bi1sYXVuY2hlcjoKKwogMjAxNC0wMS0wNiAg
TWFyayBSb3dlICA8bXJvd2VAYXBwbGUuY29tPgogCiAgICAgICAgIDxodHRwczovL3dlYmtpdC5v
cmcvYi8xMjY1NjA+IEZpeCBpbmNvcnJlY3RuZXNzIGluIHVzZSBvZiBzb21lIEFwcEtpdCBlbnVt
cwpkaWZmIC0tZ2l0IGEvVG9vbHMvU2NyaXB0cy9ydW4tbGF1bmNoZXIgYi9Ub29scy9TY3JpcHRz
L3J1bi1sYXVuY2hlcgppbmRleCBiODk2MTFmLi5jYTA1MjYxIDEwMDc1NQotLS0gYS9Ub29scy9T
Y3JpcHRzL3J1bi1sYXVuY2hlcgorKysgYi9Ub29scy9TY3JpcHRzL3J1bi1sYXVuY2hlcgpAQCAt
MzgsNyArMzgsOCBAQCB1c2Ugd2Via2l0ZGlyczsKIAogc2V0Q29uZmlndXJhdGlvbigpOwogbXkg
JHByb2R1Y3REaXIgPSBwcm9kdWN0RGlyKCk7Ci1teSAkbGF1bmNoZXJQYXRoID0gcHJvZHVjdERp
cigpOworbXkgJGxhdW5jaGVyUGF0aDsKK215IEBqaGJ1aWxkV3JhcHBlcjsKIAogIyBDaGVjayB0
byBzZWUgdGhhdCBhbGwgdGhlIGZyYW1ld29ya3MgYXJlIGJ1aWx0LgogY2hlY2tGcmFtZXdvcmtz
KCk7CkBAIC00NiwyNSArNDcsMjUgQEAgY2hlY2tGcmFtZXdvcmtzKCk7CiAjIFNldCBwYXRocyBh
Y2NvcmRpbmcgdG8gdGhlIGJ1aWxkIHN5c3RlbSB1c2VkCiBpZiAoaXNHdGsoKSkgewogICAgIGlm
IChpc1dLMigpKSB7Ci0gICAgICAgIHVuc2hpZnQoQEFSR1YsIGNhdGRpcigkbGF1bmNoZXJQYXRo
LCAiUHJvZ3JhbXMiLCAiTWluaUJyb3dzZXIiKSk7CisgICAgICAgICRsYXVuY2hlclBhdGggPSBj
YXRkaXIoJHByb2R1Y3REaXIsICJQcm9ncmFtcyIsICJNaW5pQnJvd3NlciIpOwogICAgIH0gZWxz
ZSB7Ci0gICAgICAgIHVuc2hpZnQoQEFSR1YsIGNhdGRpcigkbGF1bmNoZXJQYXRoLCAiUHJvZ3Jh
bXMiLCAiR3RrTGF1bmNoZXIiKSk7CisgICAgICAgICRsYXVuY2hlclBhdGggPSBjYXRkaXIoJHBy
b2R1Y3REaXIsICJQcm9ncmFtcyIsICJHdGtMYXVuY2hlciIpOwogICAgIH0KLSAgICAkbGF1bmNo
ZXJQYXRoID0gY2F0ZGlyKHNvdXJjZURpcigpLCAiVG9vbHMiLCAiamhidWlsZCIsICJqaGJ1aWxk
LXdyYXBwZXIiKTsKLSAgICB1bnNoaWZ0KEBBUkdWLCAoIi0tZ3RrIiwgInJ1biIpKTsKIH0KIAog
aWYgKGlzRWZsKCkpIHsKICAgICBpZiAoaXNXSzIoKSkgewotICAgICAgICB1bnNoaWZ0KEBBUkdW
LCBjYXRkaXIoJGxhdW5jaGVyUGF0aCwgImJpbiIsICJNaW5pQnJvd3NlciIpKTsKKyAgICAgICAg
JGxhdW5jaGVyUGF0aCA9IGNhdGRpcigkcHJvZHVjdERpciwgImJpbiIsICJNaW5pQnJvd3NlciIp
OwogICAgIH0gZWxzZSB7Ci0gICAgICAgIHVuc2hpZnQoQEFSR1YsIGNhdGRpcigkbGF1bmNoZXJQ
YXRoLCAiYmluIiwgIkVXZWJMYXVuY2hlciIpKTsKKyAgICAgICAgJGxhdW5jaGVyUGF0aCA9IGNh
dGRpcigkcHJvZHVjdERpciwgImJpbiIsICJFV2ViTGF1bmNoZXIiKTsKICAgICB9Ci0gICAgJGxh
dW5jaGVyUGF0aCA9IGNhdGRpcihzb3VyY2VEaXIoKSwgIlRvb2xzIiwgImpoYnVpbGQiLCAiamhi
dWlsZC13cmFwcGVyIik7Ci0gICAgdW5zaGlmdChAQVJHViwgKCItLWVmbCIsICJydW4iKSk7CiB9
CiAKKyRsYXVuY2hlclBhdGggJiYgLXggJGxhdW5jaGVyUGF0aCBvciBkaWUgIkNhbid0IGZpbmQg
dGhlIGxhdW5jaGVyIGNvbW1hbmQiOworCitAamhidWlsZFdyYXBwZXIgPSBqaGJ1aWxkV3JhcHBl
clByZWZpeElmTmVlZGVkKCk7CisKIHByaW50ICJTdGFydGluZyB3ZWJraXQgbGF1bmNoZXIuXG4i
OwogCi1leGVjICRsYXVuY2hlclBhdGgsIEBBUkdWIG9yIGRpZTsKK2V4ZWMgQGpoYnVpbGRXcmFw
cGVyLCAkbGF1bmNoZXJQYXRoLCBAQVJHViBvciBkaWU7CiAK
</data>
<flag name="review"
          id="244233"
          type_id="1"
          status="+"
          setter="cgarcia"
    />
          </attachment>
      

    </bug>

</bugzilla>