<?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>77533</bug_id>
          
          <creation_ts>2012-02-01 01:53:47 -0800</creation_ts>
          <short_desc>[Qt] JSC build should handle --no-webkit2 option to avoid unwanted clean-builds</short_desc>
          <delta_ts>2012-04-17 09:27:09 -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>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>Qt, QtTriaged</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Nandor Huszka">hnandor</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>dbates</cc>
    
    <cc>eric</cc>
    
    <cc>ossy</cc>
    
    <cc>vestbo</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>546920</commentid>
    <comment_count>0</comment_count>
    <who name="Nandor Huszka">hnandor</who>
    <bug_when>2012-02-01 01:53:47 -0800</bug_when>
    <thetext>This bug is made because of https://bugs.webkit.org/show_bug.cgi?id=74519
If we want to build only JavaScriptCore there may be an unwanted clean-build,
because there may be defines that can be found after building WebKit, 
but cannot found when it is built with the --no-webkit2 option.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>546923</commentid>
    <comment_count>1</comment_count>
      <attachid>124910</attachid>
    <who name="Nandor Huszka">hnandor</who>
    <bug_when>2012-02-01 01:58:56 -0800</bug_when>
    <thetext>Created attachment 124910
Modifications in build-jsc

As far as I know, we do not use the --no-webkit2 option on bots, so passing the option to build-jsc by hand is solvable.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>546953</commentid>
    <comment_count>2</comment_count>
      <attachid>124910</attachid>
    <who name="Tor Arne Vestbø">vestbo</who>
    <bug_when>2012-02-01 02:39:41 -0800</bug_when>
    <thetext>Comment on attachment 124910
Modifications in build-jsc

I&apos;d rather we silently pick up BUILD_WEBKIT_ARGS, like in build-webkit, then introduce a  new option to build-jsc that does not make sense for a user passing --help</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>547030</commentid>
    <comment_count>3</comment_count>
      <attachid>124936</attachid>
    <who name="Nandor Huszka">hnandor</who>
    <bug_when>2012-02-01 05:38:30 -0800</bug_when>
    <thetext>Created attachment 124936
Modifications in build-jsc

(In reply to comment #2)
This patch picks up the option from BUILD_WEBKIT_ARGS, as you suggested. With this, if I understand it correctly, before want to  build JSC without wk2, we have to put the --no-webkit2 option to the mentioned environment variable.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>547043</commentid>
    <comment_count>4</comment_count>
      <attachid>124936</attachid>
    <who name="Tor Arne Vestbø">vestbo</who>
    <bug_when>2012-02-01 06:14:41 -0800</bug_when>
    <thetext>Comment on attachment 124936
Modifications in build-jsc

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

&gt; Tools/Scripts/build-jsc:79
&gt; +    if ($ENV{&apos;BUILD_WEBKIT_ARGS&apos;}) {

How about an unconditional

push @ARGV, split(/ /, $ENV{&apos;BUILD_WEBKIT_ARGS&apos;}) if ($ENV{&apos;BUILD_WEBKIT_ARGS&apos;});

&gt; Tools/Scripts/build-jsc:85
&gt; +        push @buildWebKitArgs, split(/ /, $ENV{&apos;BUILD_WEBKIT_ARGS&apos;});
&gt; +        foreach (@buildWebKitArgs) {
&gt; +            if ($_ eq &apos;--no-webkit2&apos;) {
&gt; +                push @ARGV, &quot;--qmakearg=CONFIG+=no_webkit2&quot;;
&gt; +            }

And then a

push @ARGV, &quot;--qmakearg=CONFIG+=no_webkit2&quot; if checkForArgumentAndRemoveFromARGV(&quot;--no-webkit2&quot;);</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>547055</commentid>
    <comment_count>5</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-02-01 06:43:43 -0800</bug_when>
    <thetext>Or what if making isWK2() handle --no-webkit2, not only -2? And then we can use it instead of foreach or checkForArgumentAndRemoveFromARGV. It would good for build-webkit script too.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>547076</commentid>
    <comment_count>6</comment_count>
    <who name="Tor Arne Vestbø">vestbo</who>
    <bug_when>2012-02-01 07:34:53 -0800</bug_when>
    <thetext>(In reply to comment #5)
&gt; Or what if making isWK2() handle --no-webkit2, not only -2? And then we can use it instead of foreach or checkForArgumentAndRemoveFromARGV. It would good for build-webkit script too.

That would be an option. Though it would change the default from 0 to 1.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>547724</commentid>
    <comment_count>7</comment_count>
      <attachid>125095</attachid>
    <who name="Nandor Huszka">hnandor</who>
    <bug_when>2012-02-02 01:02:29 -0800</bug_when>
    <thetext>Created attachment 125095
Patch

(In reply to comment #5)
I tried it, but it is impossible to indicate 3 cases with one bool variable.
I mean if we put this into isWK2():

if (checkForArgumentAndRemoveFromARGV(&quot;-2&quot;)) {
  $isWK2 = 0;
} elsif (checkForArgumentAndRemoveFromARGV(&quot;--no-webkit2&quot;)){
  $isWK2 = 1;
} else {
  $isWK2 = 0;
}

or other combination of ifs, it would be incorrect, I think.
I tried to write an isNotWK2() function, but if in it we use in it the $isWK2 too, its value depends on whether isWK2() or isNotWK2() function was called sooner. But if we use a new, e.g. $isNotWK2 variable in it, it would be incomprehensible why we use $isWK2 and $isNotWK2 too.

(In reply to comment #4)
Ok, I have done that, but I do not understand why can we pass on all the other unchecked options in BUILD_WEBKIT_ARGS to the buildQMakeProjects.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>554177</commentid>
    <comment_count>8</comment_count>
      <attachid>125095</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2012-02-10 10:29:03 -0800</bug_when>
    <thetext>Comment on attachment 125095
Patch

Do we normally do this transition in the main script like this?   don&apos;t we have helper functions in one of our perl libraries for this?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>558026</commentid>
    <comment_count>9</comment_count>
    <who name="Nandor Huszka">hnandor</who>
    <bug_when>2012-02-16 02:46:25 -0800</bug_when>
    <thetext>(In reply to comment #8)
&gt; (From update of attachment 125095 [details])
&gt; Do we normally do this transition in the main script like this?   don&apos;t we have helper functions in one of our perl libraries for this?

build-webkit does similar thing in line ~578:
    push @options, &quot;--qmakearg=CONFIG+=no_webkit2&quot; if $noWebKit2;

but it does not use helper function for it. I did not find any other method which can be used here.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>592015</commentid>
    <comment_count>10</comment_count>
    <who name="Nandor Huszka">hnandor</who>
    <bug_when>2012-03-30 03:43:05 -0700</bug_when>
    <thetext>(In reply to comment #9)
Is there any reaction related to my patch, is it OK? I couldn&apos;t find a simpler way to solve it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>592065</commentid>
    <comment_count>11</comment_count>
      <attachid>125095</attachid>
    <who name="Tor Arne Vestbø">vestbo</who>
    <bug_when>2012-03-30 05:03:31 -0700</bug_when>
    <thetext>Comment on attachment 125095
Patch

I think there&apos;s other ways to solve this, but those would be more intrusive and require more time, so this is good for now.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>592070</commentid>
    <comment_count>12</comment_count>
      <attachid>125095</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-03-30 05:07:03 -0700</bug_when>
    <thetext>Comment on attachment 125095
Patch

Rejecting attachment 125095 from commit-queue.

Failed to run &quot;[&apos;/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch&apos;, &apos;--status-host=queues.webkit.org&apos;, &apos;-...&quot; exit_code: 2

Last 500 characters of output:
&apos;, u&apos;--force&apos;, u&apos;--reviewer&apos;, u&apos;Tor Arne V...&quot; exit_code: 1 cwd: /mnt/git/webkit-commit-queue/

Parsed 2 diffs from patch file(s).
patching file Tools/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file Tools/Scripts/build-jsc
Hunk #1 FAILED at 74.
1 out of 1 hunk FAILED -- saving rejects to file Tools/Scripts/build-jsc.rej

Failed to run &quot;[u&apos;/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply&apos;, u&apos;--force&apos;, u&apos;--reviewer&apos;, u&apos;Tor Arne V...&quot; exit_code: 1 cwd: /mnt/git/webkit-commit-queue/

Full output: http://queues.webkit.org/results/12288197</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>593215</commentid>
    <comment_count>13</comment_count>
      <attachid>135032</attachid>
    <who name="Nandor Huszka">hnandor</who>
    <bug_when>2012-04-02 00:19:19 -0700</bug_when>
    <thetext>Created attachment 135032
Patch

The build-jsc is modified, in my opinion that is why the review bot could not run. I redo the modifications with the latest revision.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>603953</commentid>
    <comment_count>14</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-04-17 05:46:43 -0700</bug_when>
    <thetext>ping review?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>604075</commentid>
    <comment_count>15</comment_count>
      <attachid>135032</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-04-17 09:27:03 -0700</bug_when>
    <thetext>Comment on attachment 135032
Patch

Clearing flags on attachment: 135032

Committed r114387: &lt;http://trac.webkit.org/changeset/114387&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>604076</commentid>
    <comment_count>16</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-04-17 09:27:09 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>124910</attachid>
            <date>2012-02-01 01:58:56 -0800</date>
            <delta_ts>2012-02-01 05:38:30 -0800</delta_ts>
            <desc>Modifications in build-jsc</desc>
            <filename>buildjsc.diff</filename>
            <type>text/plain</type>
            <size>1724</size>
            <attacher name="Nandor Huszka">hnandor</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCAxYzY3
Njc1Li5iMDg3MzViIDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hh
bmdlTG9nCkBAIC0xLDMgKzEsMTYgQEAKKzIwMTItMDItMDEgIE7DoW5kb3IgSHVzemthICA8aHVz
emthLm5hbmRvckBzdHVkLnUtc3plZ2VkLmh1PgorCisgICAgICAgIFtRdF0gSlNDIGJ1aWxkIHNo
b3VsZCBoYW5kbGUgLS1uby13ZWJraXQyIG9wdGlvbiB0byBhdm9pZCB1bndhbnRlZCBjbGVhbi1i
dWlsZHMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTc3
NTMzCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgUGFz
cyAtLW5vLXdlYmtpdDIgb3B0aW9uIHRvIGJ1aWxkUU1ha2VQcm9qZWN0cyB0byBhdm9pZCB1bndh
bnRlZCBjbGVhbiBidWlsZAorICAgICAgICBpZiBXZWJLaXQgd2FzIGJ1aWxkIGJ5IHRoZSBzYW1l
IG9wdGlvbi4gQXMgYSByZXN1bHQgaXQgaXMgbmVlZGVkCisgICAgICAgIHRvIGNhbGwgYnVpbGQt
anNjIHdpdGggdGhpcyBvcHRpb24gaWYgd2UgcGFzc2VkIGl0IHRvIGJ1aWxkLXdlYmtpdC4KKwor
ICAgICAgICAqIHFtYWtlL21rc3BlY3MvZmVhdHVyZXMvZmVhdHVyZXMucHJmOgorCiAyMDEyLTAy
LTAxICBQaGlsaXBwZSBOb3JtYW5kICA8cG5vcm1hbmRAaWdhbGlhLmNvbT4gYW5kIFNlcmdpbyBW
aWxsYXIgU2VuaW4gIDxzdmlsbGFyQGlnYWxpYS5jb20+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkg
TWFydGluIFJvYmluc29uLgpkaWZmIC0tZ2l0IGEvVG9vbHMvU2NyaXB0cy9idWlsZC1qc2MgYi9U
b29scy9TY3JpcHRzL2J1aWxkLWpzYwppbmRleCAyZGE5ZWU5Li5kYjEyNzQwIDEwMDc1NQotLS0g
YS9Ub29scy9TY3JpcHRzL2J1aWxkLWpzYworKysgYi9Ub29scy9TY3JpcHRzL2J1aWxkLWpzYwpA
QCAtMzcsNiArMzcsNyBAQCB1c2UgUE9TSVg7CiAKIG15ICRjb3ZlcmFnZVN1cHBvcnQgPSAwOwog
bXkgJHNob3dIZWxwID0gMDsKK215ICRub1dlYktpdDIgPSAwOwogCiBteSAkcHJvZ3JhbU5hbWUg
PSBiYXNlbmFtZSgkMCk7CiBteSAkdXNhZ2UgPSA8PEVPRjsKQEAgLTQ3LDcgKzQ4LDggQEAgRU9G
CiAKIEdldE9wdGlvbnMoCiAgICAgJ2NvdmVyYWdlIScgPT4gXCRjb3ZlcmFnZVN1cHBvcnQsCi0g
ICAgJ2hlbHAnID0+IFwkc2hvd0hlbHAKKyAgICAnaGVscCcgPT4gXCRzaG93SGVscCwKKyAgICAn
bm8td2Via2l0MicgPT4gXCRub1dlYktpdDIKICk7CiAKIGlmICgkc2hvd0hlbHApIHsKQEAgLTc0
LDYgKzc2LDcgQEAgaWYgKGlzQXBwbGVNYWNXZWJLaXQoKSkgewogICAgICMgUmVtb3ZlIGR1cGxp
Y2F0ZWQgLS1xdCBvcHRpb25zIHRvIGF2b2lkIHBhc3NpbmcgdGhlbSB0byBxbWFrZQogICAgIGNo
ZWNrRm9yQXJndW1lbnRBbmRSZW1vdmVGcm9tQVJHVigiLS1xdCIpOwogICAgIG15IEBwcm9qZWN0
cyA9ICgiV1RGIiwgIkphdmFTY3JpcHRDb3JlIik7CisgICAgcHVzaCBAQVJHViwgIi0tcW1ha2Vh
cmc9Q09ORklHKz1ub193ZWJraXQyIiBpZiAkbm9XZWJLaXQyOwogICAgICRyZXN1bHQgPSBidWls
ZFFNYWtlUHJvamVjdHMoXEBwcm9qZWN0cywgMCwgQEFSR1YpOwogfSBlbHNpZiAoaXNXeCgpKSB7
CiAgICAgIyBCdWlsZHMgZXZlcnl0aGluZyBpbiBvbmUtc2hvdC4gTm8gbmVlZCB0byBidWlsZCBh
bnl0aGluZyBoZXJlLgo=
</data>
<flag name="review"
          id="126298"
          type_id="1"
          status="-"
          setter="vestbo"
    />
    <flag name="commit-queue"
          id="126299"
          type_id="3"
          status="-"
          setter="vestbo"
    />
          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>124936</attachid>
            <date>2012-02-01 05:38:30 -0800</date>
            <delta_ts>2012-02-02 01:02:29 -0800</delta_ts>
            <desc>Modifications in build-jsc</desc>
            <filename>buildjsc.diff</filename>
            <type>text/plain</type>
            <size>1684</size>
            <attacher name="Nandor Huszka">hnandor</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCAwYmFh
YWZiLi43OGQ2OGEzIDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hh
bmdlTG9nCkBAIC0xLDUgKzEsMTcgQEAKIDIwMTItMDItMDEgIE7DoW5kb3IgSHVzemthICA8aHVz
emthLm5hbmRvckBzdHVkLnUtc3plZ2VkLmh1PgogCisgICAgICAgIFtRdF0gSlNDIGJ1aWxkIHNo
b3VsZCBoYW5kbGUgLS1uby13ZWJraXQyIG9wdGlvbiB0byBhdm9pZCB1bndhbnRlZCBjbGVhbi1i
dWlsZHMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTc3
NTMzCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgUGlj
ayB1cCB0aGUgLS1uby13ZWJraXQyIG9wdGlvbiBmcm9tIEJVSUxEX1dFQktJVF9BUkdTIGFuZCBw
YXNzIG9uCisgICAgICAgIHRoZSBjb3JyZXNwb25kaW5nIHBhcmFtZXRlciB0byB0aGUgYnVpbGRR
TWFrZVByb2plY3RzIGZ1bmN0aW9uLgorCisgICAgICAgICogU2NyaXB0cy9idWlsZC1qc2M6CisK
KzIwMTItMDItMDEgIE7DoW5kb3IgSHVzemthICA8aHVzemthLm5hbmRvckBzdHVkLnUtc3plZ2Vk
Lmh1PgorCiAgICAgICAgIFtRdF0gQXV0b21hdGljIGNsZWFuIGJ1aWxkIGZlYXR1cmUgYWx3YXlz
IGRvIGNsZWFuIGJ1aWxkIHdpdGggLS1uby13ZWJraXQyCiAgICAgICAgIGh0dHBzOi8vYnVncy53
ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD03NDUxOQogCmRpZmYgLS1naXQgYS9Ub29scy9TY3Jp
cHRzL2J1aWxkLWpzYyBiL1Rvb2xzL1NjcmlwdHMvYnVpbGQtanNjCmluZGV4IDJkYTllZTkuLjZl
NWUzMGYgMTAwNzU1Ci0tLSBhL1Rvb2xzL1NjcmlwdHMvYnVpbGQtanNjCisrKyBiL1Rvb2xzL1Nj
cmlwdHMvYnVpbGQtanNjCkBAIC03NCw2ICs3NCwxOCBAQCBpZiAoaXNBcHBsZU1hY1dlYktpdCgp
KSB7CiAgICAgIyBSZW1vdmUgZHVwbGljYXRlZCAtLXF0IG9wdGlvbnMgdG8gYXZvaWQgcGFzc2lu
ZyB0aGVtIHRvIHFtYWtlCiAgICAgY2hlY2tGb3JBcmd1bWVudEFuZFJlbW92ZUZyb21BUkdWKCIt
LXF0Iik7CiAgICAgbXkgQHByb2plY3RzID0gKCJXVEYiLCAiSmF2YVNjcmlwdENvcmUiKTsKKwor
ICAgICMgUGljayB1cCB0aGUgLS1uby13ZWJraXQyIG9wdGlvbiBmcm9tIEJVSUxEX1dFQktJVF9B
UkdTIGlmIGl0IGlzIG5lZWRlZAorICAgIGlmICgkRU5WeydCVUlMRF9XRUJLSVRfQVJHUyd9KSB7
CisgICAgICAgIG15IEBidWlsZFdlYktpdEFyZ3MgPSAoKTsKKyAgICAgICAgcHVzaCBAYnVpbGRX
ZWJLaXRBcmdzLCBzcGxpdCgvIC8sICRFTlZ7J0JVSUxEX1dFQktJVF9BUkdTJ30pOworICAgICAg
ICBmb3JlYWNoIChAYnVpbGRXZWJLaXRBcmdzKSB7CisgICAgICAgICAgICBpZiAoJF8gZXEgJy0t
bm8td2Via2l0MicpIHsKKyAgICAgICAgICAgICAgICBwdXNoIEBBUkdWLCAiLS1xbWFrZWFyZz1D
T05GSUcrPW5vX3dlYmtpdDIiOworICAgICAgICAgICAgfQorICAgICAgICB9CisgICAgfQorCiAg
ICAgJHJlc3VsdCA9IGJ1aWxkUU1ha2VQcm9qZWN0cyhcQHByb2plY3RzLCAwLCBAQVJHVik7CiB9
IGVsc2lmIChpc1d4KCkpIHsKICAgICAjIEJ1aWxkcyBldmVyeXRoaW5nIGluIG9uZS1zaG90LiBO
byBuZWVkIHRvIGJ1aWxkIGFueXRoaW5nIGhlcmUuCg==
</data>
<flag name="review"
          id="126318"
          type_id="1"
          status="-"
          setter="vestbo"
    />
    <flag name="commit-queue"
          id="126319"
          type_id="3"
          status="-"
          setter="vestbo"
    />
          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>125095</attachid>
            <date>2012-02-02 01:02:29 -0800</date>
            <delta_ts>2012-04-02 00:19:19 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>buildjscv2.diff</filename>
            <type>text/plain</type>
            <size>1502</size>
            <attacher name="Nandor Huszka">hnandor</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCBiMDBi
ZjFkLi41ZWYzNmE2IDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hh
bmdlTG9nCkBAIC0xLDMgKzEsMTUgQEAKKzIwMTItMDItMDIgIE7DoW5kb3IgSHVzemthICA8aHVz
emthLm5hbmRvckBzdHVkLnUtc3plZ2VkLmh1PgorCisgICAgICAgIFtRdF0gSlNDIGJ1aWxkIHNo
b3VsZCBoYW5kbGUgLS1uby13ZWJraXQyIG9wdGlvbiB0byBhdm9pZCB1bndhbnRlZCBjbGVhbi1i
dWlsZHMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTc3
NTMzCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgUGlj
ayB1cCB0aGUgLS1uby13ZWJraXQyIG9wdGlvbiBmcm9tIEJVSUxEX1dFQktJVF9BUkdTIGFuZCBw
YXNzIG9uCisgICAgICAgIHRoZSBjb3JyZXNwb25kaW5nIHBhcmFtZXRlciB0byB0aGUgYnVpbGRR
TWFrZVByb2plY3RzIGZ1bmN0aW9uLgorCisgICAgICAgICogU2NyaXB0cy9idWlsZC1qc2M6CisK
IDIwMTItMDItMDIgIFJ5b3N1a2UgTml3YSAgPHJuaXdhQHdlYmtpdC5vcmc+CiAKICAgICAgICAg
UmUtbGFuZCByMTA2NDQyIHBlciBBZGFtJ3Mgc3VnZ2VzdGlvbi4gV2UnbGwgZG8gYm90aCByZXN0
YXJ0aW5nIERSVCBhbmQgbG9hZGluZyBhIGZha2UgdGVzdCBmaXJzdC4KZGlmZiAtLWdpdCBhL1Rv
b2xzL1NjcmlwdHMvYnVpbGQtanNjIGIvVG9vbHMvU2NyaXB0cy9idWlsZC1qc2MKaW5kZXggMmRh
OWVlOS4uMDA3NGJjZCAxMDA3NTUKLS0tIGEvVG9vbHMvU2NyaXB0cy9idWlsZC1qc2MKKysrIGIv
VG9vbHMvU2NyaXB0cy9idWlsZC1qc2MKQEAgLTc0LDYgKzc0LDkgQEAgaWYgKGlzQXBwbGVNYWNX
ZWJLaXQoKSkgewogICAgICMgUmVtb3ZlIGR1cGxpY2F0ZWQgLS1xdCBvcHRpb25zIHRvIGF2b2lk
IHBhc3NpbmcgdGhlbSB0byBxbWFrZQogICAgIGNoZWNrRm9yQXJndW1lbnRBbmRSZW1vdmVGcm9t
QVJHVigiLS1xdCIpOwogICAgIG15IEBwcm9qZWN0cyA9ICgiV1RGIiwgIkphdmFTY3JpcHRDb3Jl
Iik7CisgICAgIyBQaWNrIHVwIHRoZSAtLW5vLXdlYmtpdDIgb3B0aW9uIGZyb20gQlVJTERfV0VC
S0lUX0FSR1MgaWYgaXQgaXMgbmVlZGVkCisgICAgcHVzaCBAQVJHViwgc3BsaXQoLyAvLCAkRU5W
eydCVUlMRF9XRUJLSVRfQVJHUyd9KSBpZiAoJEVOVnsnQlVJTERfV0VCS0lUX0FSR1MnfSk7Cisg
ICAgcHVzaCBAQVJHViwgIi0tcW1ha2Vhcmc9Q09ORklHKz1ub193ZWJraXQyIiBpZiBjaGVja0Zv
ckFyZ3VtZW50QW5kUmVtb3ZlRnJvbUFSR1YoIi0tbm8td2Via2l0MiIpOwogICAgICRyZXN1bHQg
PSBidWlsZFFNYWtlUHJvamVjdHMoXEBwcm9qZWN0cywgMCwgQEFSR1YpOwogfSBlbHNpZiAoaXNX
eCgpKSB7CiAgICAgIyBCdWlsZHMgZXZlcnl0aGluZyBpbiBvbmUtc2hvdC4gTm8gbmVlZCB0byBi
dWlsZCBhbnl0aGluZyBoZXJlLgo=
</data>
<flag name="review"
          id="126525"
          type_id="1"
          status="+"
          setter="vestbo"
    />
    <flag name="commit-queue"
          id="126526"
          type_id="3"
          status="-"
          setter="webkit.review.bot"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>135032</attachid>
            <date>2012-04-02 00:19:19 -0700</date>
            <delta_ts>2012-04-17 09:27:03 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>nowkFinal.diff</filename>
            <type>text/plain</type>
            <size>1419</size>
            <attacher name="Nandor Huszka">hnandor</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCAyMDhi
NDNlLi44MDY5OGE0IDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hh
bmdlTG9nCkBAIC0xLDMgKzEsMTUgQEAKKzIwMTItMDQtMDIgIE5hbmRvciBIdXN6a2EgIDxobmFu
ZG9yQGluZi51LXN6ZWdlZC5odT4KKworICAgICAgICBbUXRdIEpTQyBidWlsZCBzaG91bGQgaGFu
ZGxlIC0tbm8td2Via2l0MiBvcHRpb24gdG8gYXZvaWQgdW53YW50ZWQgY2xlYW4tYnVpbGRzCisg
ICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD03NzUzMworCisg
ICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFBpY2sgdXAgdGhl
IC0tbm8td2Via2l0MiBvcHRpb24gZnJvbSBCVUlMRF9XRUJLSVRfQVJHUyBhbmQgcGFzcyBvbgor
ICAgICAgICB0aGUgY29ycmVzcG9uZGluZyBwYXJhbWV0ZXIgdG8gdGhlIGJ1aWxkUU1ha2VQcm9q
ZWN0cyBmdW5jdGlvbi4KKworICAgICAgICAqIFNjcmlwdHMvYnVpbGQtanNjOgorCiAyMDEyLTA0
LTAxICBSeW9zdWtlIE5pd2EgIDxybml3YUB3ZWJraXQub3JnPgogCiAgICAgICAgIHBlcmYtby1t
YXRpYyBzaG91bGQgc3RvcmUgdGVzdCByZXN1bHRzJyB1bml0cwpkaWZmIC0tZ2l0IGEvVG9vbHMv
U2NyaXB0cy9idWlsZC1qc2MgYi9Ub29scy9TY3JpcHRzL2J1aWxkLWpzYwppbmRleCBhNGM0MmIy
Li4xNDQ3YmJhIDEwMDc1NQotLS0gYS9Ub29scy9TY3JpcHRzL2J1aWxkLWpzYworKysgYi9Ub29s
cy9TY3JpcHRzL2J1aWxkLWpzYwpAQCAtNjQsNiArNjQsOSBAQCBteSBAY292ZXJhZ2VTdXBwb3J0
T3B0aW9ucyA9ICgkY292ZXJhZ2VTdXBwb3J0KSA/IFhjb2RlQ292ZXJhZ2VTdXBwb3J0T3B0aW9u
cygpCiBpZiAoaXNRdCgpKSB7CiAgICAgY2hlY2tGb3JBcmd1bWVudEFuZFJlbW92ZUZyb21BUkdW
KCItLXF0Iik7CiAgICAgbXkgQHByb2plY3RzID0gKCJXVEYiLCAiSmF2YVNjcmlwdENvcmUiKTsK
KyAgICAjIFBpY2sgdXAgdGhlIC0tbm8td2Via2l0MiBvcHRpb24gZnJvbSBCVUlMRF9XRUJLSVRf
QVJHUyBpZiBpdCBpcyBuZWVkZWQKKyAgICBwdXNoIEBBUkdWLCBzcGxpdCgvIC8sICRFTlZ7J0JV
SUxEX1dFQktJVF9BUkdTJ30pIGlmICgkRU5WeydCVUlMRF9XRUJLSVRfQVJHUyd9KTsKKyAgICBw
dXNoIEBBUkdWLCAiLS1xbWFrZWFyZz1DT05GSUcrPW5vX3dlYmtpdDIiIGlmIGNoZWNrRm9yQXJn
dW1lbnRBbmRSZW1vdmVGcm9tQVJHVigiLS1uby13ZWJraXQyIik7CiAgICAgbXkgJHJlc3VsdCA9
IGJ1aWxkUU1ha2VQcm9qZWN0cyhcQHByb2plY3RzLCAwLCBAQVJHVik7CiAgICAgZXhpdCBleGl0
U3RhdHVzKCRyZXN1bHQpOwogfSBlbHNpZiAoY21ha2VCYXNlZFBvcnROYW1lKCkpIHsK
</data>

          </attachment>
      

    </bug>

</bugzilla>