<?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>98890</bug_id>
          
          <creation_ts>2012-10-10 05:45:34 -0700</creation_ts>
          <short_desc>EventHandler::handleGestureLongPress breaks compilation if CONTEXT_MENUS isn&apos;t set</short_desc>
          <delta_ts>2012-10-10 11:08:56 -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>UI Events</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="Luciano Wolf">luciano.wolf</reporter>
          <assigned_to name="Luciano Wolf">luciano.wolf</assigned_to>
          <cc>cmarcelo</cc>
    
    <cc>dglazkov</cc>
    
    <cc>peter+ews</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>738872</commentid>
    <comment_count>0</comment_count>
    <who name="Luciano Wolf">luciano.wolf</who>
    <bug_when>2012-10-10 05:45:34 -0700</bug_when>
    <thetext>If CONTEXT_MENU isn&apos;t enabled there will be a call to &quot;sendContextMenuEventForGesture(gestureEvent);&quot; that is wrapped with &quot;#if ENABLE(CONTEXT_MENU)&quot; and the code won&apos;t compile.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>738875</commentid>
    <comment_count>1</comment_count>
      <attachid>167984</attachid>
    <who name="Luciano Wolf">luciano.wolf</who>
    <bug_when>2012-10-10 05:53:46 -0700</bug_when>
    <thetext>Created attachment 167984
Proposed patch

Enable EventHandler::handleGestureLongPress only if CONTEXT_MENU is set</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>738877</commentid>
    <comment_count>2</comment_count>
      <attachid>167984</attachid>
    <who name="Build Bot">buildbot</who>
    <bug_when>2012-10-10 05:58:30 -0700</bug_when>
    <thetext>Comment on attachment 167984
Proposed patch

Attachment 167984 did not pass mac-ews (mac):
Output: http://queues.webkit.org/results/14238708</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>738878</commentid>
    <comment_count>3</comment_count>
      <attachid>167984</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-10-10 05:59:31 -0700</bug_when>
    <thetext>Comment on attachment 167984
Proposed patch

Attachment 167984 did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/14248495</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>738879</commentid>
    <comment_count>4</comment_count>
      <attachid>167984</attachid>
    <who name="Early Warning System Bot">webkit-ews</who>
    <bug_when>2012-10-10 06:00:42 -0700</bug_when>
    <thetext>Comment on attachment 167984
Proposed patch

Attachment 167984 did not pass qt-wk2-ews (qt):
Output: http://queues.webkit.org/results/14245514</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>738880</commentid>
    <comment_count>5</comment_count>
      <attachid>167984</attachid>
    <who name="Early Warning System Bot">webkit-ews</who>
    <bug_when>2012-10-10 06:00:52 -0700</bug_when>
    <thetext>Comment on attachment 167984
Proposed patch

Attachment 167984 did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/14252183</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>738883</commentid>
    <comment_count>6</comment_count>
      <attachid>167984</attachid>
    <who name="Peter Beverloo (cr-android ews)">peter+ews</who>
    <bug_when>2012-10-10 06:06:14 -0700</bug_when>
    <thetext>Comment on attachment 167984
Proposed patch

Attachment 167984 did not pass cr-android-ews (chromium-android):
Output: http://queues.webkit.org/results/14245517</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>738902</commentid>
    <comment_count>7</comment_count>
      <attachid>167984</attachid>
    <who name="Caio Marcelo de Oliveira Filho">cmarcelo</who>
    <bug_when>2012-10-10 06:31:59 -0700</bug_when>
    <thetext>Comment on attachment 167984
Proposed patch

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

&gt; Source/WebCore/page/EventHandler.cpp:2573
&gt; +#if ENABLE(CONTEXT_MENU)

This ifdef is in the wrong place. Shouldn&apos;t it be protecting the sendContextMenuEventForGesture() call?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>738948</commentid>
    <comment_count>8</comment_count>
      <attachid>167997</attachid>
    <who name="Luciano Wolf">luciano.wolf</who>
    <bug_when>2012-10-10 07:46:54 -0700</bug_when>
    <thetext>Created attachment 167997
Another proposed patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>738979</commentid>
    <comment_count>9</comment_count>
    <who name="Luciano Wolf">luciano.wolf</who>
    <bug_when>2012-10-10 08:37:11 -0700</bug_when>
    <thetext>(In reply to comment #7)
&gt; (From update of attachment 167984 [details])
&gt; View in context: https://bugs.webkit.org/attachment.cgi?id=167984&amp;action=review
&gt; 
&gt; &gt; Source/WebCore/page/EventHandler.cpp:2573
&gt; &gt; +#if ENABLE(CONTEXT_MENU)
&gt; 
&gt; This ifdef is in the wrong place. Shouldn&apos;t it be protecting the sendContextMenuEventForGesture() call?

But in this case we&apos;ve to return something. If you&apos;re not ANDROID the only line remaining is this sendContextMenuEventForGesture(). Should we just return &quot;false&quot; (after moving ENABLE(CONTEXT_MENUS) to the proper place)?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>738989</commentid>
    <comment_count>10</comment_count>
      <attachid>167984</attachid>
    <who name="Caio Marcelo de Oliveira Filho">cmarcelo</who>
    <bug_when>2012-10-10 08:48:22 -0700</bug_when>
    <thetext>Comment on attachment 167984
Proposed patch

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

&gt;&gt;&gt; Source/WebCore/page/EventHandler.cpp:2573
&gt;&gt;&gt; +#if ENABLE(CONTEXT_MENU)
&gt;&gt; 
&gt;&gt; This ifdef is in the wrong place. Shouldn&apos;t it be protecting the sendContextMenuEventForGesture() call?
&gt; 
&gt; But in this case we&apos;ve to return something. If you&apos;re not ANDROID the only line remaining is this sendContextMenuEventForGesture(). Should we just return &quot;false&quot; (after moving ENABLE(CONTEXT_MENUS) to the proper place)?

Yes. But sendContextMenuEventForGesture() is only declared if we have CONTEXT_MENUS (see EventHandler.h), so you need to guard it.

My understanding from the event handling is that if it&apos;s not defined you can simply return false here. So add an else case for the #if ENABLE(CONTEXT_MENUS) returning false should be enough.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>739005</commentid>
    <comment_count>11</comment_count>
      <attachid>168010</attachid>
    <who name="Luciano Wolf">luciano.wolf</who>
    <bug_when>2012-10-10 09:11:50 -0700</bug_when>
    <thetext>Created attachment 168010
Another proposed patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>739038</commentid>
    <comment_count>12</comment_count>
      <attachid>168010</attachid>
    <who name="Caio Marcelo de Oliveira Filho">cmarcelo</who>
    <bug_when>2012-10-10 09:48:17 -0700</bug_when>
    <thetext>Comment on attachment 168010
Another proposed patch

Let&apos;s wait the EWS just in case :-)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>739043</commentid>
    <comment_count>13</comment_count>
      <attachid>168020</attachid>
    <who name="Luciano Wolf">luciano.wolf</who>
    <bug_when>2012-10-10 09:54:22 -0700</bug_when>
    <thetext>Created attachment 168020
Patch for landing</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>739047</commentid>
    <comment_count>14</comment_count>
      <attachid>168022</attachid>
    <who name="Luciano Wolf">luciano.wolf</who>
    <bug_when>2012-10-10 10:03:48 -0700</bug_when>
    <thetext>Created attachment 168022
Patch for landing</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>739111</commentid>
    <comment_count>15</comment_count>
      <attachid>168022</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-10-10 10:50:01 -0700</bug_when>
    <thetext>Comment on attachment 168022
Patch for landing

Rejecting attachment 168022 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:
it2/ChangeLog
Auto-merging Source/WebKit2/win/WebKit2.def
Failed to merge in the changes.
Patch failed at 0001 Needs internal API to return distributed nodes for InsertionPoint

When you have resolved this problem run &quot;git rebase --continue&quot;.
If you would prefer to skip this patch, instead run &quot;git rebase --skip&quot;.
To restore the original branch and stop rebasing run &quot;git rebase --abort&quot;.

rebase refs/remotes/origin/master: command returned error: 1

Died at Tools/Scripts/update-webkit line 164.

Full output: http://queues.webkit.org/results/14260093</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>739142</commentid>
    <comment_count>16</comment_count>
    <who name="Caio Marcelo de Oliveira Filho">cmarcelo</who>
    <bug_when>2012-10-10 11:08:56 -0700</bug_when>
    <thetext>Committed r130930: &lt;http://trac.webkit.org/changeset/130930&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>167984</attachid>
            <date>2012-10-10 05:53:46 -0700</date>
            <delta_ts>2012-10-10 08:48:22 -0700</delta_ts>
            <desc>Proposed patch</desc>
            <filename>bug-98890-20121010095245.patch</filename>
            <type>text/plain</type>
            <size>1619</size>
            <attacher name="Luciano Wolf">luciano.wolf</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTMwODg5CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggYzY4Y2ViZjMwZjVkMWMy
MWQwZDg2M2E4MDhjMTE3MGE0OTEzNmI3MS4uNzFlY2JlYzgwYjNkZmI0OTBhZjQxZDZjMjEyMGRj
YTg5Yzk1YjA2YiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1IEBACisyMDEyLTEwLTEwICBMdWNp
YW5vIFdvbGYgIDxsdWNpYW5vLndvbGZAb3BlbmJvc3NhLm9yZz4KKworICAgICAgICBFdmVudEhh
bmRsZXI6OmhhbmRsZUdlc3R1cmVMb25nUHJlc3MgYnJlYWtzIGNvbXBpbGF0aW9uIGlmIENPTlRF
WFRfTUVOVSBpc24ndCBzZXQKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i
dWcuY2dpP2lkPTk4ODkwCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisK
KyAgICAgICAgRW5hYmxlIEV2ZW50SGFuZGxlcjo6aGFuZGxlR2VzdHVyZUxvbmdQcmVzcyBvbmx5
IGlmIENPTlRFWFRfTUVOVSBpcyBzZXQuCisKKyAgICAgICAgKiBwYWdlL0V2ZW50SGFuZGxlci5j
cHA6CisgICAgICAgIChXZWJDb3JlKToKKwogMjAxMi0xMC0xMCAgS2Vpc2hpIEhhdHRvcmkgIDxr
ZWlzaGlAd2Via2l0Lm9yZz4KIAogICAgICAgICBSRUdSRVNTSU9OIChyMTI5NzM4KTogQ2FsZW5k
YXIgcGlja2VyIGlzIHRvbyB3aWRlIHdoZW4gdGhlIGlucHV0IGlzIHJ0bApkaWZmIC0tZ2l0IGEv
U291cmNlL1dlYkNvcmUvcGFnZS9FdmVudEhhbmRsZXIuY3BwIGIvU291cmNlL1dlYkNvcmUvcGFn
ZS9FdmVudEhhbmRsZXIuY3BwCmluZGV4IDJkY2Q0MmMxZDU3YzFhNTU1MTNlZjI3ZGUxYTc5MWMx
NDQzYTQ1ODMuLjY1NjNlNzk4MmZkMmQ4NThiODNiZWIwM2YyYjI4YjdhOThiOTUyNmIgMTAwNjQ0
Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL3BhZ2UvRXZlbnRIYW5kbGVyLmNwcAorKysgYi9Tb3VyY2Uv
V2ViQ29yZS9wYWdlL0V2ZW50SGFuZGxlci5jcHAKQEAgLTI1NzAsNiArMjU3MCw3IEBAIGJvb2wg
RXZlbnRIYW5kbGVyOjpoYW5kbGVHZXN0dXJlVGFwKGNvbnN0IFBsYXRmb3JtR2VzdHVyZUV2ZW50
JiBnZXN0dXJlRXZlbnQpCiAgICAgcmV0dXJuIGRlZmF1bHRQcmV2ZW50ZWQ7CiB9CiAKKyNpZiBF
TkFCTEUoQ09OVEVYVF9NRU5VKQogYm9vbCBFdmVudEhhbmRsZXI6OmhhbmRsZUdlc3R1cmVMb25n
UHJlc3MoY29uc3QgUGxhdGZvcm1HZXN0dXJlRXZlbnQmIGdlc3R1cmVFdmVudCkKIHsKICNpZiBP
UyhBTkRST0lEKQpAQCAtMjU4NCw2ICsyNTg1LDcgQEAgYm9vbCBFdmVudEhhbmRsZXI6OmhhbmRs
ZUdlc3R1cmVMb25nUHJlc3MoY29uc3QgUGxhdGZvcm1HZXN0dXJlRXZlbnQmIGdlc3R1cmVFdmUK
ICNlbmRpZgogICAgIHJldHVybiBzZW5kQ29udGV4dE1lbnVFdmVudEZvckdlc3R1cmUoZ2VzdHVy
ZUV2ZW50KTsKIH0KKyNlbmRpZiAvLyBFTkFCTEUoQ09OVEVYVF9NRU5VKQogCiBib29sIEV2ZW50
SGFuZGxlcjo6aGFuZGxlR2VzdHVyZVNjcm9sbFVwZGF0ZShjb25zdCBQbGF0Zm9ybUdlc3R1cmVF
dmVudCYgZ2VzdHVyZUV2ZW50KQogewo=
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>167997</attachid>
            <date>2012-10-10 07:46:54 -0700</date>
            <delta_ts>2012-10-10 07:53:57 -0700</delta_ts>
            <desc>Another proposed patch</desc>
            <filename>bug-98890-20121010114553.patch</filename>
            <type>text/plain</type>
            <size>2750</size>
            <attacher name="Luciano Wolf">luciano.wolf</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTMwODg5CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggYzY4Y2ViZjMwZjVkMWMy
MWQwZDg2M2E4MDhjMTE3MGE0OTEzNmI3MS4uZDY2MGNmOWE1ZTM4OWE5YTI2ZWMwNDEzNjg4N2E2
M2M2MmM5MThhMSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE2IEBACisyMDEyLTEwLTEwICBMdWNp
YW5vIFdvbGYgIDxsdWNpYW5vLndvbGZAb3BlbmJvc3NhLm9yZz4KKworICAgICAgICBFdmVudEhh
bmRsZXI6OmhhbmRsZUdlc3R1cmVMb25nUHJlc3MgYnJlYWtzIGNvbXBpbGF0aW9uIGlmIENPTlRF
WFRfTUVOVSBpc24ndCBzZXQKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i
dWcuY2dpP2lkPTk4ODkwCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisK
KyAgICAgICAgKiBwYWdlL0V2ZW50SGFuZGxlci5jcHA6CisgICAgICAgIChXZWJDb3JlOjpFdmVu
dEhhbmRsZXI6OmhhbmRsZUdlc3R1cmVFdmVudCk6CisgICAgICAgIChXZWJDb3JlKToKKyAgICAg
ICAgKiBwYWdlL0V2ZW50SGFuZGxlci5oOgorICAgICAgICAoRXZlbnRIYW5kbGVyKToKKwogMjAx
Mi0xMC0xMCAgS2Vpc2hpIEhhdHRvcmkgIDxrZWlzaGlAd2Via2l0Lm9yZz4KIAogICAgICAgICBS
RUdSRVNTSU9OIChyMTI5NzM4KTogQ2FsZW5kYXIgcGlja2VyIGlzIHRvbyB3aWRlIHdoZW4gdGhl
IGlucHV0IGlzIHJ0bApkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvcGFnZS9FdmVudEhhbmRs
ZXIuY3BwIGIvU291cmNlL1dlYkNvcmUvcGFnZS9FdmVudEhhbmRsZXIuY3BwCmluZGV4IDJkY2Q0
MmMxZDU3YzFhNTU1MTNlZjI3ZGUxYTc5MWMxNDQzYTQ1ODMuLjdmNDAyZjdmMDQ2OGUzNTYxNjgz
MDA4ZjA0YmFjNWI2ZTRmMTZlOTAgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL3BhZ2UvRXZl
bnRIYW5kbGVyLmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9wYWdlL0V2ZW50SGFuZGxlci5jcHAK
QEAgLTI1MjEsOCArMjUyMSwxMCBAQCBib29sIEV2ZW50SGFuZGxlcjo6aGFuZGxlR2VzdHVyZUV2
ZW50KGNvbnN0IFBsYXRmb3JtR2VzdHVyZUV2ZW50JiBnZXN0dXJlRXZlbnQpCiAgICAgICAgIHJl
dHVybiBoYW5kbGVHZXN0dXJlVGFwKGdlc3R1cmVFdmVudCk7CiAgICAgY2FzZSBQbGF0Zm9ybUV2
ZW50OjpHZXN0dXJlVGFwRG93bjoKICAgICAgICAgcmV0dXJuIGhhbmRsZUdlc3R1cmVUYXBEb3du
KCk7CisjaWYgRU5BQkxFKENPTlRFWFRfTUVOVVMpCiAgICAgY2FzZSBQbGF0Zm9ybUV2ZW50OjpH
ZXN0dXJlTG9uZ1ByZXNzOgogICAgICAgICByZXR1cm4gaGFuZGxlR2VzdHVyZUxvbmdQcmVzcyhn
ZXN0dXJlRXZlbnQpOworI2VuZGlmCiAgICAgY2FzZSBQbGF0Zm9ybUV2ZW50OjpHZXN0dXJlRG91
YmxlVGFwOgogICAgIGNhc2UgUGxhdGZvcm1FdmVudDo6R2VzdHVyZVBpbmNoQmVnaW46CiAgICAg
Y2FzZSBQbGF0Zm9ybUV2ZW50OjpHZXN0dXJlUGluY2hFbmQ6CkBAIC0yNTcwLDYgKzI1NzIsNyBA
QCBib29sIEV2ZW50SGFuZGxlcjo6aGFuZGxlR2VzdHVyZVRhcChjb25zdCBQbGF0Zm9ybUdlc3R1
cmVFdmVudCYgZ2VzdHVyZUV2ZW50KQogICAgIHJldHVybiBkZWZhdWx0UHJldmVudGVkOwogfQog
CisjaWYgRU5BQkxFKENPTlRFWFRfTUVOVVMpCiBib29sIEV2ZW50SGFuZGxlcjo6aGFuZGxlR2Vz
dHVyZUxvbmdQcmVzcyhjb25zdCBQbGF0Zm9ybUdlc3R1cmVFdmVudCYgZ2VzdHVyZUV2ZW50KQog
ewogI2lmIE9TKEFORFJPSUQpCkBAIC0yNTg0LDYgKzI1ODcsNyBAQCBib29sIEV2ZW50SGFuZGxl
cjo6aGFuZGxlR2VzdHVyZUxvbmdQcmVzcyhjb25zdCBQbGF0Zm9ybUdlc3R1cmVFdmVudCYgZ2Vz
dHVyZUV2ZQogI2VuZGlmCiAgICAgcmV0dXJuIHNlbmRDb250ZXh0TWVudUV2ZW50Rm9yR2VzdHVy
ZShnZXN0dXJlRXZlbnQpOwogfQorI2VuZGlmIC8vIEVOQUJMRShDT05URVhUX01FTlVTKQogCiBi
b29sIEV2ZW50SGFuZGxlcjo6aGFuZGxlR2VzdHVyZVNjcm9sbFVwZGF0ZShjb25zdCBQbGF0Zm9y
bUdlc3R1cmVFdmVudCYgZ2VzdHVyZUV2ZW50KQogewpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNv
cmUvcGFnZS9FdmVudEhhbmRsZXIuaCBiL1NvdXJjZS9XZWJDb3JlL3BhZ2UvRXZlbnRIYW5kbGVy
LmgKaW5kZXggYTYwNGRhMmM4NDBlMzIxYjZkN2QxYzg1MTAyMjU5NzJkYThiMTg4YS4uN2M5YzA5
MTUwYWJlZTU3OTg5YzQ3YmM5ZDNmNTA5MzgyNTUyNjg1MSAxMDA2NDQKLS0tIGEvU291cmNlL1dl
YkNvcmUvcGFnZS9FdmVudEhhbmRsZXIuaAorKysgYi9Tb3VyY2UvV2ViQ29yZS9wYWdlL0V2ZW50
SGFuZGxlci5oCkBAIC0xNjcsNyArMTY3LDkgQEAgcHVibGljOgogI2lmIEVOQUJMRShHRVNUVVJF
X0VWRU5UUykKICAgICBib29sIGhhbmRsZUdlc3R1cmVFdmVudChjb25zdCBQbGF0Zm9ybUdlc3R1
cmVFdmVudCYpOwogICAgIGJvb2wgaGFuZGxlR2VzdHVyZVRhcChjb25zdCBQbGF0Zm9ybUdlc3R1
cmVFdmVudCYpOworI2lmIEVOQUJMRShDT05URVhUX01FTlVTKQogICAgIGJvb2wgaGFuZGxlR2Vz
dHVyZUxvbmdQcmVzcyhjb25zdCBQbGF0Zm9ybUdlc3R1cmVFdmVudCYpOworI2VuZGlmCiAgICAg
Ym9vbCBoYW5kbGVHZXN0dXJlU2Nyb2xsVXBkYXRlKGNvbnN0IFBsYXRmb3JtR2VzdHVyZUV2ZW50
Jik7CiAjZW5kaWYKIAo=
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>168010</attachid>
            <date>2012-10-10 09:11:50 -0700</date>
            <delta_ts>2012-10-10 09:54:17 -0700</delta_ts>
            <desc>Another proposed patch</desc>
            <filename>bug-98890-20121010131049.patch</filename>
            <type>text/plain</type>
            <size>1348</size>
            <attacher name="Luciano Wolf">luciano.wolf</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTMwOTA4CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggOWYzY2M4NTQ0YzMyMzNi
NTIwZmZmYzc1NmQxNmI2YjMyYzIyMzEyOC4uODkzOTJhNzhlYzVlMGI1ZWQ3NzQwNGQ5NjRhZWJi
MjcyMGM1NWZkOSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDEzIEBACisyMDEyLTEwLTEwICBMdWNp
YW5vIFdvbGYgIDxsdWNpYW5vLndvbGZAb3BlbmJvc3NhLm9yZz4KKworICAgICAgICBFdmVudEhh
bmRsZXI6OmhhbmRsZUdlc3R1cmVMb25nUHJlc3MgYnJlYWtzIGNvbXBpbGF0aW9uIGlmIENPTlRF
WFRfTUVOVSBpc24ndCBzZXQKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i
dWcuY2dpP2lkPTk4ODkwCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisK
KyAgICAgICAgKiBwYWdlL0V2ZW50SGFuZGxlci5jcHA6CisgICAgICAgIChXZWJDb3JlOjpFdmVu
dEhhbmRsZXI6OmhhbmRsZUdlc3R1cmVMb25nUHJlc3MpOgorCiAyMDEyLTEwLTEwICBNaWtlIFdl
c3QgIDxta3dzdEBnb29nbGUuY29tPgogCiAgICAgICAgIEVkaXRvckNvbW1hbmQgY2FsbHMgc2V0
UHJvcGVydHkgd2l0aCB0aGUgd3JvbmcgcGFyYW1ldGVycy4KZGlmZiAtLWdpdCBhL1NvdXJjZS9X
ZWJDb3JlL3BhZ2UvRXZlbnRIYW5kbGVyLmNwcCBiL1NvdXJjZS9XZWJDb3JlL3BhZ2UvRXZlbnRI
YW5kbGVyLmNwcAppbmRleCAyZGNkNDJjMWQ1N2MxYTU1NTEzZWYyN2RlMWE3OTFjMTQ0M2E0NTgz
Li45OWQ5OWRmNzY0NGIwOWE5NTNiZTAwYWZiZjBjZDJiZWU3NmZkODgzIDEwMDY0NAotLS0gYS9T
b3VyY2UvV2ViQ29yZS9wYWdlL0V2ZW50SGFuZGxlci5jcHAKKysrIGIvU291cmNlL1dlYkNvcmUv
cGFnZS9FdmVudEhhbmRsZXIuY3BwCkBAIC0yNTgyLDcgKzI1ODIsMTEgQEAgYm9vbCBFdmVudEhh
bmRsZXI6OmhhbmRsZUdlc3R1cmVMb25nUHJlc3MoY29uc3QgUGxhdGZvcm1HZXN0dXJlRXZlbnQm
IGdlc3R1cmVFdmUKICAgICAgICAgICAgIHJldHVybiB0cnVlOwogICAgIH0KICNlbmRpZgorI2lm
IEVOQUJMRShDT05URVhUX01FTlVTKQogICAgIHJldHVybiBzZW5kQ29udGV4dE1lbnVFdmVudEZv
ckdlc3R1cmUoZ2VzdHVyZUV2ZW50KTsKKyNlbHNlCisgICAgcmV0dXJuIGZhbHNlOworI2VuZGlm
CiB9CiAKIGJvb2wgRXZlbnRIYW5kbGVyOjpoYW5kbGVHZXN0dXJlU2Nyb2xsVXBkYXRlKGNvbnN0
IFBsYXRmb3JtR2VzdHVyZUV2ZW50JiBnZXN0dXJlRXZlbnQpCg==
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>168020</attachid>
            <date>2012-10-10 09:54:22 -0700</date>
            <delta_ts>2012-10-10 10:03:43 -0700</delta_ts>
            <desc>Patch for landing</desc>
            <filename>bug-98890-20121010135321.patch</filename>
            <type>text/plain</type>
            <size>1364</size>
            <attacher name="Luciano Wolf">luciano.wolf</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTMwOTA4CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggOWYzY2M4NTQ0YzMyMzNi
NTIwZmZmYzc1NmQxNmI2YjMyYzIyMzEyOC4uNDIwMmUyNGMzNTEyN2E3OWM3MWJkMTNkZWI0YTJj
NGNlNzIwZGQzMiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDEzIEBACisyMDEyLTEwLTEwICBMdWNp
YW5vIFdvbGYgIDxsdWNpYW5vLndvbGZAb3BlbmJvc3NhLm9yZz4KKworICAgICAgICBFdmVudEhh
bmRsZXI6OmhhbmRsZUdlc3R1cmVMb25nUHJlc3MgYnJlYWtzIGNvbXBpbGF0aW9uIGlmIENPTlRF
WFRfTUVOVVMgaXNuJ3Qgc2V0CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3df
YnVnLmNnaT9pZD05ODg5MAorCisgICAgICAgIFJldmlld2VkIGJ5IENhaW8gTWFyY2VsbyBkZSBP
bGl2ZWlyYSBGaWxobworCisgICAgICAgICogcGFnZS9FdmVudEhhbmRsZXIuY3BwOgorICAgICAg
ICAoV2ViQ29yZTo6RXZlbnRIYW5kbGVyOjpoYW5kbGVHZXN0dXJlTG9uZ1ByZXNzKToKKwogMjAx
Mi0xMC0xMCAgTWlrZSBXZXN0ICA8bWt3c3RAZ29vZ2xlLmNvbT4KIAogICAgICAgICBFZGl0b3JD
b21tYW5kIGNhbGxzIHNldFByb3BlcnR5IHdpdGggdGhlIHdyb25nIHBhcmFtZXRlcnMuCmRpZmYg
LS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9wYWdlL0V2ZW50SGFuZGxlci5jcHAgYi9Tb3VyY2UvV2Vi
Q29yZS9wYWdlL0V2ZW50SGFuZGxlci5jcHAKaW5kZXggMmRjZDQyYzFkNTdjMWE1NTUxM2VmMjdk
ZTFhNzkxYzE0NDNhNDU4My4uOTlkOTlkZjc2NDRiMDlhOTUzYmUwMGFmYmYwY2QyYmVlNzZmZDg4
MyAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvcGFnZS9FdmVudEhhbmRsZXIuY3BwCisrKyBi
L1NvdXJjZS9XZWJDb3JlL3BhZ2UvRXZlbnRIYW5kbGVyLmNwcApAQCAtMjU4Miw3ICsyNTgyLDEx
IEBAIGJvb2wgRXZlbnRIYW5kbGVyOjpoYW5kbGVHZXN0dXJlTG9uZ1ByZXNzKGNvbnN0IFBsYXRm
b3JtR2VzdHVyZUV2ZW50JiBnZXN0dXJlRXZlCiAgICAgICAgICAgICByZXR1cm4gdHJ1ZTsKICAg
ICB9CiAjZW5kaWYKKyNpZiBFTkFCTEUoQ09OVEVYVF9NRU5VUykKICAgICByZXR1cm4gc2VuZENv
bnRleHRNZW51RXZlbnRGb3JHZXN0dXJlKGdlc3R1cmVFdmVudCk7CisjZWxzZQorICAgIHJldHVy
biBmYWxzZTsKKyNlbmRpZgogfQogCiBib29sIEV2ZW50SGFuZGxlcjo6aGFuZGxlR2VzdHVyZVNj
cm9sbFVwZGF0ZShjb25zdCBQbGF0Zm9ybUdlc3R1cmVFdmVudCYgZ2VzdHVyZUV2ZW50KQo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>168022</attachid>
            <date>2012-10-10 10:03:48 -0700</date>
            <delta_ts>2012-10-10 10:50:01 -0700</delta_ts>
            <desc>Patch for landing</desc>
            <filename>bug-98890-20121010140247.patch</filename>
            <type>text/plain</type>
            <size>1365</size>
            <attacher name="Luciano Wolf">luciano.wolf</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTMwOTA4CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggOWYzY2M4NTQ0YzMyMzNi
NTIwZmZmYzc1NmQxNmI2YjMyYzIyMzEyOC4uMzIxYWRiYjgzNjA3YWMyNWFmY2U1MWY0MGVmZDAy
NWMwNjNjYmEyMCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDEzIEBACisyMDEyLTEwLTEwICBMdWNp
YW5vIFdvbGYgIDxsdWNpYW5vLndvbGZAb3BlbmJvc3NhLm9yZz4KKworICAgICAgICBFdmVudEhh
bmRsZXI6OmhhbmRsZUdlc3R1cmVMb25nUHJlc3MgYnJlYWtzIGNvbXBpbGF0aW9uIGlmIENPTlRF
WFRfTUVOVVMgaXNuJ3Qgc2V0CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3df
YnVnLmNnaT9pZD05ODg5MAorCisgICAgICAgIFJldmlld2VkIGJ5IENhaW8gTWFyY2VsbyBkZSBP
bGl2ZWlyYSBGaWxoby4KKworICAgICAgICAqIHBhZ2UvRXZlbnRIYW5kbGVyLmNwcDoKKyAgICAg
ICAgKFdlYkNvcmU6OkV2ZW50SGFuZGxlcjo6aGFuZGxlR2VzdHVyZUxvbmdQcmVzcyk6CisKIDIw
MTItMTAtMTAgIE1pa2UgV2VzdCAgPG1rd3N0QGdvb2dsZS5jb20+CiAKICAgICAgICAgRWRpdG9y
Q29tbWFuZCBjYWxscyBzZXRQcm9wZXJ0eSB3aXRoIHRoZSB3cm9uZyBwYXJhbWV0ZXJzLgpkaWZm
IC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvcGFnZS9FdmVudEhhbmRsZXIuY3BwIGIvU291cmNlL1dl
YkNvcmUvcGFnZS9FdmVudEhhbmRsZXIuY3BwCmluZGV4IDJkY2Q0MmMxZDU3YzFhNTU1MTNlZjI3
ZGUxYTc5MWMxNDQzYTQ1ODMuLjk5ZDk5ZGY3NjQ0YjA5YTk1M2JlMDBhZmJmMGNkMmJlZTc2ZmQ4
ODMgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL3BhZ2UvRXZlbnRIYW5kbGVyLmNwcAorKysg
Yi9Tb3VyY2UvV2ViQ29yZS9wYWdlL0V2ZW50SGFuZGxlci5jcHAKQEAgLTI1ODIsNyArMjU4Miwx
MSBAQCBib29sIEV2ZW50SGFuZGxlcjo6aGFuZGxlR2VzdHVyZUxvbmdQcmVzcyhjb25zdCBQbGF0
Zm9ybUdlc3R1cmVFdmVudCYgZ2VzdHVyZUV2ZQogICAgICAgICAgICAgcmV0dXJuIHRydWU7CiAg
ICAgfQogI2VuZGlmCisjaWYgRU5BQkxFKENPTlRFWFRfTUVOVVMpCiAgICAgcmV0dXJuIHNlbmRD
b250ZXh0TWVudUV2ZW50Rm9yR2VzdHVyZShnZXN0dXJlRXZlbnQpOworI2Vsc2UKKyAgICByZXR1
cm4gZmFsc2U7CisjZW5kaWYKIH0KIAogYm9vbCBFdmVudEhhbmRsZXI6OmhhbmRsZUdlc3R1cmVT
Y3JvbGxVcGRhdGUoY29uc3QgUGxhdGZvcm1HZXN0dXJlRXZlbnQmIGdlc3R1cmVFdmVudCkK
</data>
<flag name="commit-queue"
          id="180886"
          type_id="3"
          status="-"
          setter="webkit.review.bot"
    />
          </attachment>
      

    </bug>

</bugzilla>