<?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>24092</bug_id>
          
          <creation_ts>2009-02-23 06:29:39 -0800</creation_ts>
          <short_desc>Autofocus readonly inputs</short_desc>
          <delta_ts>2011-09-27 11:16:45 -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>Forms</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc>http://lachy.id.au/dev/markup/tests/html5/autofocus/004.html</bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>HasReduction</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>19264</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Anthony Ricaud">rik</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>adele</cc>
    
    <cc>ap</cc>
    
    <cc>arv</cc>
    
    <cc>donggwan.kim</cc>
    
    <cc>ian</cc>
    
    <cc>kaustubh.ra</cc>
    
    <cc>michelangelo</cc>
    
    <cc>ojan</cc>
    
    <cc>rniwa</cc>
    
    <cc>tkent</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>110571</commentid>
    <comment_count>0</comment_count>
    <who name="Anthony Ricaud">rik</who>
    <bug_when>2009-02-23 06:29:39 -0800</bug_when>
    <thetext>This test by Lachlan Hunt fails. readonly inputs can&apos;t be autofocused.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>469337</commentid>
    <comment_count>1</comment_count>
      <attachid>107836</attachid>
    <who name="Kaustubh Atrawalkar">kaustubh.ra</who>
    <bug_when>2011-09-19 05:24:00 -0700</bug_when>
    <thetext>Created attachment 107836
Patch

Removed the check for readonly in shouldAutoFocus(). As per specs autofocus should be ignored only for hidden &amp; output controls.
http://www.whatwg.org/specs/web-forms/current-work/#the-autofocus</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>469462</commentid>
    <comment_count>2</comment_count>
    <who name="Kaustubh Atrawalkar">kaustubh.ra</who>
    <bug_when>2011-09-19 09:55:22 -0700</bug_when>
    <thetext>Ryoduke can you review patch once? Thanks.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>469464</commentid>
    <comment_count>3</comment_count>
      <attachid>107836</attachid>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2011-09-19 09:58:06 -0700</bug_when>
    <thetext>Comment on attachment 107836
Patch

What do other browsers do?  What does spec say?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>469465</commentid>
    <comment_count>4</comment_count>
      <attachid>107836</attachid>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2011-09-19 10:00:21 -0700</bug_when>
    <thetext>Comment on attachment 107836
Patch

r-, since this change doesn&apos;t have a rationale. Please feel free to request review on the same patch once you add a rationale (do both IE and Firefox agree? what does HTML5 say?)

This code looks deliberate - did you check when and why it was added to WebKit?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>469468</commentid>
    <comment_count>5</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2011-09-19 10:03:49 -0700</bug_when>
    <thetext>The check was introduced http://trac.webkit.org/changeset/34626, the original implementation of autofocus attribute.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>469470</commentid>
    <comment_count>6</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2011-09-19 10:06:15 -0700</bug_when>
    <thetext>&quot;what does HTML5 say?&quot;

Apparently, many reviewers have a difficulty reading the whole bug (one of my patches was rejected for no rationale last week even though it was right there in bug description). Slightly more of it should be in bug title and/or in ChangeLog to make the process more smooth.

We strive for spec compliance, which can be achieved by changing WebKit, or by changing specs. It&apos;s important to investigate the reasons for existing behavior, and compatibility story.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>469929</commentid>
    <comment_count>7</comment_count>
    <who name="Kaustubh Atrawalkar">kaustubh.ra</who>
    <bug_when>2011-09-19 23:37:26 -0700</bug_when>
    <thetext>I completely agree with both of you. We should be adding rationale in the bug description and make it somehow mandatory. This will ease for the committers and reviewers both to analyze the behavior and may be the cause of existence of the bug easily. 

For the current bug here is the rationale -

MSIE 9 - Fail 
Firefox 6 - Fail
Opera 11 - Fail
Google Chrome - Fail

But as per specs, it does not mention to ignore autofocus attribute for readonly element. One use case may be user want to always focus on the text in the readonly input to allow user to copy it.

(In reply to comment #6)
&gt; &quot;what does HTML5 say?&quot;
&gt; 
&gt; Apparently, many reviewers have a difficulty reading the whole bug (one of my patches was rejected for no rationale last week even though it was right there in bug description). Slightly more of it should be in bug title and/or in ChangeLog to make the process more smooth.
&gt; 
&gt; We strive for spec compliance, which can be achieved by changing WebKit, or by changing specs. It&apos;s important to investigate the reasons for existing behavior, and compatibility story.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>470080</commentid>
    <comment_count>8</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2011-09-20 08:33:20 -0700</bug_when>
    <thetext>(In reply to comment #7)
&gt; I completely agree with both of you. We should be adding rationale in the bug description and make it somehow mandatory. This will ease for the committers and reviewers both to analyze the behavior and may be the cause of existence of the bug easily. 
&gt; 
&gt; For the current bug here is the rationale -
&gt; 
&gt; MSIE 9 - Fail 
&gt; Firefox 6 - Fail
&gt; Opera 11 - Fail
&gt; Google Chrome - Fail

So all major browsers do not auto-focus the element?  Mn... that isn&apos;t what I found.  The test case attached on this bug doesn&apos;t work property (has a bug) so I created my own and Firefox appeared to have set focus.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>470618</commentid>
    <comment_count>9</comment_count>
    <who name="Kaustubh Atrawalkar">kaustubh.ra</who>
    <bug_when>2011-09-20 23:40:10 -0700</bug_when>
    <thetext>&gt; So all major browsers do not auto-focus the element?  Mn... that isn&apos;t what I found.  The test case attached on this bug doesn&apos;t work property (has a bug) so I created my own and Firefox appeared to have set focus.

Ohh.., I modified the same test case to create Layout test case. I check that, and it fails. Can you upload your own test case?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>470976</commentid>
    <comment_count>10</comment_count>
      <attachid>108191</attachid>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2011-09-21 11:23:43 -0700</bug_when>
    <thetext>Created attachment 108191
test</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>470989</commentid>
    <comment_count>11</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2011-09-21 11:36:45 -0700</bug_when>
    <thetext>With my test, IE9 doesn&apos;t autofocus while FF6 does.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>471369</commentid>
    <comment_count>12</comment_count>
    <who name="Kaustubh Atrawalkar">kaustubh.ra</who>
    <bug_when>2011-09-21 23:18:51 -0700</bug_when>
    <thetext>(In reply to comment #11)
&gt; With my test, IE9 doesn&apos;t autofocus while FF6 does.

Ryosuke, your test case does autofocus on Opera 11.51 as well. So readonly inputs does receive autofocus in FF6, Opera. As per specs we should set focus to readonly. What you suggest? Should we fix?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>471378</commentid>
    <comment_count>13</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2011-09-21 23:38:36 -0700</bug_when>
    <thetext>It seems reasonable to fix this bug. Please update your patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>471381</commentid>
    <comment_count>14</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2011-09-21 23:43:18 -0700</bug_when>
    <thetext>One more thing. Are there websites affected by this bug?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>471386</commentid>
    <comment_count>15</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2011-09-22 00:01:02 -0700</bug_when>
    <thetext>What will it mean in practice to have a disabled input focused (I see that the test uses :focused, but that doesn&apos;t seem very practical)? Will the page be scrolled to make it visible?

I&apos;m also curious about disabled input autofocus behavior.

But it probably doesn&apos;t matter much either way.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>471390</commentid>
    <comment_count>16</comment_count>
      <attachid>108285</attachid>
    <who name="Kaustubh Atrawalkar">kaustubh.ra</who>
    <bug_when>2011-09-22 00:09:59 -0700</bug_when>
    <thetext>Created attachment 108285
Updated Patch

I have updated patch and the Layout test case as well to use only one input readonly box as per your test case. That should be enough I guess.
Currently I m not aware on any websites which will get affected. But still will keep on looking.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>471392</commentid>
    <comment_count>17</comment_count>
    <who name="Kaustubh Atrawalkar">kaustubh.ra</who>
    <bug_when>2011-09-22 00:13:06 -0700</bug_when>
    <thetext>(In reply to comment #15)
&gt; What will it mean in practice to have a disabled input focused (I see that the test uses :focused, but that doesn&apos;t seem very practical)? Will the page be scrolled to make it visible?
&gt; 
I agree with you that there are not much practical use case for this issue. But, yes the page gets scrolled and shows the auto focused element. This might be to some extent useful to web devs.  
&gt; I&apos;m also curious about disabled input autofocus behavior.
&gt; 
&gt; But it probably doesn&apos;t matter much either way.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>471395</commentid>
    <comment_count>18</comment_count>
      <attachid>108287</attachid>
    <who name="Kaustubh Atrawalkar">kaustubh.ra</who>
    <bug_when>2011-09-22 00:17:22 -0700</bug_when>
    <thetext>Created attachment 108287
Updated Patch

Sorry for the wrong patch. Something messed up in svn up.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>472015</commentid>
    <comment_count>19</comment_count>
    <who name="Ojan Vafai">ojan</who>
    <bug_when>2011-09-22 19:00:50 -0700</bug_when>
    <thetext>(In reply to comment #15)
&gt; What will it mean in practice to have a disabled input focused (I see that the test uses :focused, but that doesn&apos;t seem very practical)? Will the page be scrolled to make it visible?
&gt; 
&gt; I&apos;m also curious about disabled input autofocus behavior.
&gt; 
&gt; But it probably doesn&apos;t matter much either way.

I agree that focusing a readonly element is a bit weird, but they are currently focusable, we just don&apos;t autofocus them.

The HTML spec seems to imply that if the element is focusable is should be autofocused, which makes sense to me: http://www.whatwg.org/specs/web-apps/current-work/#attr-fe-autofocus

&quot;Queue a task that checks to see if the element is focusable, and if so, runs the focusing steps for that element. User agents may also change the scrolling position of the document, or perform some other action that brings the element to the user&apos;s attention.&quot;

kaustubh, since browsers currently disagree, it&apos;s probably worth getting the spec to explicitly say one way or the other. Would you mind sending an email to whatwg@whatwg.org asking for clarification on whether readonly/disabled/hidden inputs should be focusable?

I&apos;m inclined to say that they should only be focusable if you explicitly set a tabIndex, but I don&apos;t really feel strongly about it.

Also, I don&apos;t see why form controls are the only things that are autofocusable (at least, that&apos;s how shouldAutofocus is currently written). It seems that anything that is focusable should accept autofocus.

As the spec is currently written, I&apos;d expect the code to look like this:
static bool shouldAutofocus(HTMLFormControlElement* element)
{
    if (!element-&gt;autofocus())
        return false;
    if (!element-&gt;renderer())
        return false;
    if (element-&gt;document()-&gt;ignoreAutofocus())
        return false;
    return element-&gt;isFocusable();
}</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>472029</commentid>
    <comment_count>20</comment_count>
    <who name="Ian &apos;Hixie&apos; Hickson">ian</who>
    <bug_when>2011-09-22 19:50:39 -0700</bug_when>
    <thetext>Ojan&apos;s interpretation of the spec in comment 19 is correct — there&apos;s nothing in the spec that special cases readonlyness as far as focusing goes, whether for regular focusing or for autofocusing.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>472058</commentid>
    <comment_count>21</comment_count>
    <who name="Kaustubh Atrawalkar">kaustubh.ra</who>
    <bug_when>2011-09-22 21:19:23 -0700</bug_when>
    <thetext>Ojan, I will surely mail them today itself about the clarification. And as of now Opera and Firefox supports the autofocusing of readonly elements. My initial test case was little buggy and Ryosuke has notified me with his proper test case. According to specs your view about how the shouldHaveFocus functions would be is exactly right. Current patch is to achieve the same. But still we should check with whatwg for clarity.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>472694</commentid>
    <comment_count>22</comment_count>
    <who name="Ojan Vafai">ojan</who>
    <bug_when>2011-09-23 19:48:08 -0700</bug_when>
    <thetext>whatwg discussion: http://lists.whatwg.org/htdig.cgi/whatwg-whatwg.org/2011-September/033279.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>472983</commentid>
    <comment_count>23</comment_count>
    <who name="Kaustubh Atrawalkar">kaustubh.ra</who>
    <bug_when>2011-09-25 23:56:07 -0700</bug_when>
    <thetext>Should we land this under regression may be?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>472988</commentid>
    <comment_count>24</comment_count>
      <attachid>108629</attachid>
    <who name="Kaustubh Atrawalkar">kaustubh.ra</who>
    <bug_when>2011-09-26 00:05:35 -0700</bug_when>
    <thetext>Created attachment 108629
Updated patch

Adding updated patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>473285</commentid>
    <comment_count>25</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2011-09-26 12:19:22 -0700</bug_when>
    <thetext>Given Ian&apos;s response, it seems reasonable to fix this bug. Also, I highly doubt this can cause serious compat. issues since users can always move focus around.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>473286</commentid>
    <comment_count>26</comment_count>
      <attachid>108629</attachid>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2011-09-26 12:20:05 -0700</bug_when>
    <thetext>Comment on attachment 108629
Updated patch

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

&gt; LayoutTests/fast/forms/autofocus-readonly-attribute.html:11
&gt; +(document.activeElement == document.getElementById(&quot;test&quot;)) ? log(&quot;PASS&quot;):log(&quot;FAIL&quot;);

Should we also check document.getElementById(&quot;test&quot;).focused?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>473380</commentid>
    <comment_count>27</comment_count>
    <who name="Ojan Vafai">ojan</who>
    <bug_when>2011-09-26 14:22:26 -0700</bug_when>
    <thetext>As per my comments at http://lists.whatwg.org/htdig.cgi/whatwg-whatwg.org/2011-September/033327.html, it&apos;s clear to me that the code I wrote in comment 19 is the most spec-compliant code.

I&apos;d rather see that patch done, but this patch is fine as is since it&apos;s an incremental step in that direction.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>473384</commentid>
    <comment_count>28</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2011-09-26 14:25:17 -0700</bug_when>
    <thetext>(In reply to comment #27)
&gt; As per my comments at http://lists.whatwg.org/htdig.cgi/whatwg-whatwg.org/2011-September/033327.html, it&apos;s clear to me that the code I wrote in comment 19 is the most spec-compliant code.

Yes. But I think that change can be made in a separate patch since this bug explicitly mentions readonly input element.

&gt; I&apos;d rather see that patch done, but this patch is fine as is since it&apos;s an incremental step in that direction.

Agreed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>473734</commentid>
    <comment_count>29</comment_count>
      <attachid>108796</attachid>
    <who name="Kaustubh Atrawalkar">kaustubh.ra</who>
    <bug_when>2011-09-27 00:24:45 -0700</bug_when>
    <thetext>Created attachment 108796
Updated Patch

Updated patch using onfocus() property to check for the focus instead of activeElement. As focused element is always activeElement but activeElement need not be always focused.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>473744</commentid>
    <comment_count>30</comment_count>
      <attachid>108796</attachid>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2011-09-27 00:34:20 -0700</bug_when>
    <thetext>Comment on attachment 108796
Updated Patch

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

&gt; LayoutTests/fast/forms/autofocus-readonly-attribute.html:9
&gt; +function runTest()
&gt; +{
&gt; +    if (window.layoutTestController)
&gt; +        layoutTestController.dumpAsText();
&gt; +}

It seems like there&apos;s no need for us to wrap this in a function. Please get rid of the function and just run if &amp; dumpAsText().

&gt; LayoutTests/fast/forms/autofocus-readonly-attribute.html:12
&gt; +    var ele = document.getElementById(&quot;console&quot;);

Please don&apos;t use the abbreviation &quot;ele&quot;.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>473761</commentid>
    <comment_count>31</comment_count>
      <attachid>108804</attachid>
    <who name="Kaustubh Atrawalkar">kaustubh.ra</who>
    <bug_when>2011-09-27 00:51:08 -0700</bug_when>
    <thetext>Created attachment 108804
Patch

Reformatted layout test case as per comments.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>473769</commentid>
    <comment_count>32</comment_count>
      <attachid>108804</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-09-27 00:58:01 -0700</bug_when>
    <thetext>Comment on attachment 108804
Patch

Rejecting attachment 108804 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:
LFormControlElement.cpp
Hunk #1 FAILED at 127.
1 out of 1 hunk FAILED -- saving rejects to file Source/WebCore/html/HTMLFormControlElement.cpp.rej
patching file LayoutTests/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file LayoutTests/fast/forms/autofocus-readonly-attribute-expected.txt
patching file LayoutTests/fast/forms/autofocus-readonly-attribute.html

Failed to run &quot;[u&apos;/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply&apos;, u&apos;--reviewer&apos;, u&apos;Ryosuke Niwa&apos;, u&apos;--for...&quot; exit_code: 1

Full output: http://queues.webkit.org/results/9880193</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>473780</commentid>
    <comment_count>33</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2011-09-27 01:42:49 -0700</bug_when>
    <thetext>It appears that something is wrong with your patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>473783</commentid>
    <comment_count>34</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2011-09-27 01:45:51 -0700</bug_when>
    <thetext>It appears that your patch is conflicting with http://trac.webkit.org/changeset/96078. I&apos;ll probably land it manually tomorrow morning.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>473786</commentid>
    <comment_count>35</comment_count>
    <who name="Kaustubh Atrawalkar">kaustubh.ra</who>
    <bug_when>2011-09-27 01:51:24 -0700</bug_when>
    <thetext>(In reply to comment #34)
&gt; It appears that your patch is conflicting with http://trac.webkit.org/changeset/96078. I&apos;ll probably land it manually tomorrow morning.
I got that error. Think i needed to updated before sending new patch. Just did that. If possible u can do it now.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>473787</commentid>
    <comment_count>36</comment_count>
      <attachid>108812</attachid>
    <who name="Kaustubh Atrawalkar">kaustubh.ra</who>
    <bug_when>2011-09-27 01:51:52 -0700</bug_when>
    <thetext>Created attachment 108812
Updated Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>474060</commentid>
    <comment_count>37</comment_count>
    <who name="Kaustubh Atrawalkar">kaustubh.ra</who>
    <bug_when>2011-09-27 10:41:42 -0700</bug_when>
    <thetext>(In reply to comment #36)
&gt; Created an attachment (id=108812) [details]
&gt; Updated Patch

Thanks Darin, seems i forgot to add it as patch :(</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>474098</commentid>
    <comment_count>38</comment_count>
      <attachid>108812</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-09-27 11:16:38 -0700</bug_when>
    <thetext>Comment on attachment 108812
Updated Patch

Clearing flags on attachment: 108812

Committed r96134: &lt;http://trac.webkit.org/changeset/96134&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>474099</commentid>
    <comment_count>39</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-09-27 11:16:45 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>107836</attachid>
            <date>2011-09-19 05:24:00 -0700</date>
            <delta_ts>2011-09-22 00:09:59 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>patch.txt</filename>
            <type>text/plain</type>
            <size>3366</size>
            <attacher name="Kaustubh Atrawalkar">kaustubh.ra</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDk1NDA2KQorKysgU291cmNlL1dlYkNvcmUvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTcgQEAKKzIwMTEtMDktMTkgIEthdXN0dWJo
IEF0cmF3YWxrYXIgIDxrYXVzdHViaEBtb3Rvcm9sYS5jb20+CisKKyAgICAgICAgQXV0b2ZvY3Vz
IG9uIHJlYWRvbmx5IGlucHV0cyBkb2VzIG5vdCBmb2N1cyB0aGUgZWxlbWVudC4KKyAgICAgICAg
aHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTI0MDkyCisKKyAgICAgICAg
UmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgUmVhZG9ubHkgaW5wdXQgZWxl
bWVudHMgc2hvdWxkIGJlIGF1dG9mb2N1c2FibGUuIFJlbW92ZWQgdGhlIGNoZWNrLgorCisgICAg
ICAgIFRlc3RzOiBmYXN0L2Zvcm1zL2F1dG9mb2N1cy1yZWFkb25seS1hdHRyaWJ1dGUuaHRtbAor
CisgICAgICAgICogaHRtbC9IVE1MRm9ybUNvbnRyb2xFbGVtZW50LmNwcDoKKyAgICAgICAgKFdl
YkNvcmU6OnNob3VsZEF1dG9mb2N1cyk6CisKIDIwMTEtMDktMTkgIFNoYXduIFNpbmdoICA8c2hh
d25zaW5naEBjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgUmUtbmFtZSBMYXllckNocm9taXVtIGJv
cmRlciBmdW5jdGlvbnMgdG8gcmVmbGVjdCB0aGF0CkluZGV4OiBTb3VyY2UvV2ViQ29yZS9odG1s
L0hUTUxGb3JtQ29udHJvbEVsZW1lbnQuY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJDb3Jl
L2h0bWwvSFRNTEZvcm1Db250cm9sRWxlbWVudC5jcHAJKHJldmlzaW9uIDk1NDA2KQorKysgU291
cmNlL1dlYkNvcmUvaHRtbC9IVE1MRm9ybUNvbnRyb2xFbGVtZW50LmNwcAkod29ya2luZyBjb3B5
KQpAQCAtMTI3LDggKzEyNyw2IEBAIHN0YXRpYyBib29sIHNob3VsZEF1dG9mb2N1cyhIVE1MRm9y
bUNvbnQKICAgICAgICAgcmV0dXJuIGZhbHNlOwogICAgIGlmIChlbGVtZW50LT5kb2N1bWVudCgp
LT5pZ25vcmVBdXRvZm9jdXMoKSkKICAgICAgICAgcmV0dXJuIGZhbHNlOwotICAgIGlmIChlbGVt
ZW50LT5pc1JlYWRPbmx5Rm9ybUNvbnRyb2woKSkKLSAgICAgICAgcmV0dXJuIGZhbHNlOwogCiAg
ICAgLy8gRklYTUU6IFNob3VsZCB0aGlzIHNldCBvZiBoYXNUYWdOYW1lIGNoZWNrcyBiZSByZXBs
YWNlZCBieSBhCiAgICAgLy8gdmlydHVhbCBtZW1iZXIgZnVuY3Rpb24/CkluZGV4OiBMYXlvdXRU
ZXN0cy9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCShyZXZp
c2lvbiA5NTQwNikKKysrIExheW91dFRlc3RzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAt
MSwzICsxLDEzIEBACisyMDExLTA5LTE5ICBLYXVzdHViaCBBdHJhd2Fsa2FyICA8a2F1c3R1YmhA
bW90b3JvbGEuY29tPgorCisgICAgICAgIEF1dG9mb2N1cyBvbiByZWFkb25seSBpbnB1dHMgZG9l
cyBub3QgZm9jdXMgdGhlIGVsZW1lbnQuCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3Jn
L3Nob3dfYnVnLmNnaT9pZD0yNDA5MgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09Q
UyEpLgorCisgICAgICAgICogZmFzdC9mb3Jtcy9hdXRvZm9jdXMtcmVhZG9ubHktYXR0cmlidXRl
LWV4cGVjdGVkLnR4dDogQWRkZWQuCisgICAgICAgICogZmFzdC9mb3Jtcy9hdXRvZm9jdXMtcmVh
ZG9ubHktYXR0cmlidXRlLmh0bWw6IEFkZGVkLgorCiAyMDExLTA5LTE4ICBJbHlhIFRpa2hvbm92
c2t5ICA8bG9pc2xvQGNocm9taXVtLm9yZz4KIAogICAgICAgICBXZWIgSW5zcGVjdG9yOiByZXF1
ZXN0QW5pbWF0aW9uRnJhbWUgY2FsbGJhY2tzIGRvbid0IHNob3cgdXAgaW4gdGhlIHRpbWVsaW5l
IHBhbmVsLgpJbmRleDogTGF5b3V0VGVzdHMvZmFzdC9mb3Jtcy9hdXRvZm9jdXMtcmVhZG9ubHkt
YXR0cmlidXRlLmh0bWwKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvZmFzdC9mb3Jtcy9hdXRv
Zm9jdXMtcmVhZG9ubHktYXR0cmlidXRlLmh0bWwJKHJldmlzaW9uIDApCisrKyBMYXlvdXRUZXN0
cy9mYXN0L2Zvcm1zL2F1dG9mb2N1cy1yZWFkb25seS1hdHRyaWJ1dGUuaHRtbAkocmV2aXNpb24g
MCkKQEAgLTAsMCArMSwyMCBAQAorPCFET0NZVFBFIGh0bWw+Cis8aHRtbD4KKzxib2R5PgorPHA+
VGhlIHNlY29uZCB0ZXh0IGJveCBzaG91bGQgaGF2ZSBmb2N1cyBjb25zb2xlIHNob3VsZCBwcmlu
dCBQQVNTPC9wPgorPHA+PGlucHV0IHR5cGU9InRleHQiIG5hbWU9InRlc3QiIGF1dG9mb2N1cyBp
ZD0idGVzdDEiPjwvcD4KKzxwPjxpbnB1dCB0eXBlPSJ0ZXh0IiBuYW1lPSJ0ZXN0IiBhdXRvZm9j
dXMgcmVhZG9ubHkgaWQ9InRlc3QyIj48L3A+Cis8ZGl2IGlkPSJjb25zb2xlIj48L2Rpdj4KKzxz
Y3JpcHQ+CitpZiAod2luZG93LmxheW91dFRlc3RDb250cm9sbGVyKQorICAgIGxheW91dFRlc3RD
b250cm9sbGVyLmR1bXBBc1RleHQoKTsKKworKGRvY3VtZW50LmFjdGl2ZUVsZW1lbnQgPT0gZG9j
dW1lbnQuZ2V0RWxlbWVudEJ5SWQoInRlc3QyIikpID8gbG9nKCJQQVNTIik6bG9nKCJGQUlMIik7
CisKK2Z1bmN0aW9uIGxvZyhtc2cpIHsKKyAgICB2YXIgY29uc29sZSA9IGRvY3VtZW50LmdldEVs
ZW1lbnRCeUlkKCJjb25zb2xlIik7CisgICAgY29uc29sZS5pbm5lckhUTUwgPSBjb25zb2xlLmlu
bmVySFRNTCArIG1zZyArICI8YnI+IjsKK30KKzwvc2NyaXB0PgorPC9ib2R5PgorPC9odG1sPgpJ
bmRleDogTGF5b3V0VGVzdHMvZmFzdC9mb3Jtcy9hdXRvZm9jdXMtcmVhZG9ubHktYXR0cmlidXRl
LWV4cGVjdGVkLnR4dAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9mYXN0L2Zvcm1zL2F1dG9m
b2N1cy1yZWFkb25seS1hdHRyaWJ1dGUtZXhwZWN0ZWQudHh0CShyZXZpc2lvbiAwKQorKysgTGF5
b3V0VGVzdHMvZmFzdC9mb3Jtcy9hdXRvZm9jdXMtcmVhZG9ubHktYXR0cmlidXRlLWV4cGVjdGVk
LnR4dAkocmV2aXNpb24gMCkKQEAgLTAsMCArMSw4IEBACitUaGUgc2Vjb25kIHRleHQgYm94IHNo
b3VsZCBoYXZlIGZvY3VzIGNvbnNvbGUgc2hvdWxkIHByaW50IFBBU1MKKworCisKKworCitQQVNT
CisK
</data>
<flag name="review"
          id="104620"
          type_id="1"
          status="-"
          setter="ap"
    />
    <flag name="commit-queue"
          id="104621"
          type_id="3"
          status="-"
          setter="ap"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>108191</attachid>
            <date>2011-09-21 11:23:43 -0700</date>
            <delta_ts>2011-09-21 11:23:43 -0700</delta_ts>
            <desc>test</desc>
            <filename>autofocus.html</filename>
            <type>text/html</type>
            <size>189</size>
            <attacher name="Ryosuke Niwa">rniwa</attacher>
            
              <data encoding="base64">PCFET0NUWVBFIGh0bWw+CjxodG1sPgo8aGVhZD4KPHN0eWxlPiBpbnB1dDpmb2N1cyB7YmFja2dy
b3VuZC1jb2xvcjogZ3JlZW47fSA8L3N0eWxlPgo8L2hlYWQ+Cjxib2R5Pgo8cD48aW5wdXQgdHlw
ZT0idGV4dCIgbmFtZT0idGVzdCIgdmFsdWU9ImhlbGxvIiBhdXRvZm9jdXMgcmVhZG9ubHk+PC9w
Pgo8L2JvZHk+CjwvaHRtbD4K
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>108285</attachid>
            <date>2011-09-22 00:09:59 -0700</date>
            <delta_ts>2011-09-22 00:17:22 -0700</delta_ts>
            <desc>Updated Patch</desc>
            <filename>24092_patch.txt</filename>
            <type>text/plain</type>
            <size>3239</size>
            <attacher name="Kaustubh Atrawalkar">kaustubh.ra</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDk1Njk4KQorKysgU291cmNlL1dlYkNvcmUvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTcgQEAKKzIwMTEtMDktMTkgIEthdXN0dWJo
IEF0cmF3YWxrYXIgIDxrYXVzdHViaEBtb3Rvcm9sYS5jb20+CisKKyAgICAgICAgQXV0b2ZvY3Vz
IG9uIHJlYWRvbmx5IGlucHV0cyBkb2VzIG5vdCBmb2N1cyB0aGUgZWxlbWVudC4KKyAgICAgICAg
aHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTI0MDkyCisKKyAgICAgICAg
UmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgUmVhZG9ubHkgaW5wdXQgZWxl
bWVudHMgc2hvdWxkIGJlIGF1dG9mb2N1c2FibGUuIFJlbW92ZWQgdGhlIGNoZWNrLgorCisgICAg
ICAgIFRlc3RzOiBmYXN0L2Zvcm1zL2F1dG9mb2N1cy1yZWFkb25seS1hdHRyaWJ1dGUuaHRtbAor
CisgICAgICAgICogaHRtbC9IVE1MRm9ybUNvbnRyb2xFbGVtZW50LmNwcDoKKyAgICAgICAgKFdl
YkNvcmU6OnNob3VsZEF1dG9mb2N1cyk6CisKIDIwMTEtMDktMjEgIEJldGggRGFraW4gIDxiZGFr
aW5AYXBwbGUuY29tPgogCiAgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVn
LmNnaT9pZD02NzQxNQpJbmRleDogU291cmNlL1dlYkNvcmUvaHRtbC9IVE1MRm9ybUNvbnRyb2xF
bGVtZW50LmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViQ29yZS9odG1sL0hUTUxGb3JtQ29u
dHJvbEVsZW1lbnQuY3BwCShyZXZpc2lvbiA5NTQwNikKKysrIFNvdXJjZS9XZWJDb3JlL2h0bWwv
SFRNTEZvcm1Db250cm9sRWxlbWVudC5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTEyNyw4ICsxMjcs
NiBAQCBzdGF0aWMgYm9vbCBzaG91bGRBdXRvZm9jdXMoSFRNTEZvcm1Db250CiAgICAgICAgIHJl
dHVybiBmYWxzZTsKICAgICBpZiAoZWxlbWVudC0+ZG9jdW1lbnQoKS0+aWdub3JlQXV0b2ZvY3Vz
KCkpCiAgICAgICAgIHJldHVybiBmYWxzZTsKLSAgICBpZiAoZWxlbWVudC0+aXNSZWFkT25seUZv
cm1Db250cm9sKCkpCi0gICAgICAgIHJldHVybiBmYWxzZTsKIAogICAgIC8vIEZJWE1FOiBTaG91
bGQgdGhpcyBzZXQgb2YgaGFzVGFnTmFtZSBjaGVja3MgYmUgcmVwbGFjZWQgYnkgYQogICAgIC8v
IHZpcnR1YWwgbWVtYmVyIGZ1bmN0aW9uPwpJbmRleDogTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCj09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT0KLS0tIExheW91dFRlc3RzL0NoYW5nZUxvZwkocmV2aXNpb24gOTU2OTgpCisrKyBM
YXlvdXRUZXN0cy9DaGFuZ2VMb2cJKHdvcmtpbmcgY29weSkKQEAgLTEsMyArMSwxMyBAQAorMjAx
MS0wOS0xOSAgS2F1c3R1YmggQXRyYXdhbGthciAgPGthdXN0dWJoQG1vdG9yb2xhLmNvbT4KKwor
ICAgICAgICBBdXRvZm9jdXMgb24gcmVhZG9ubHkgaW5wdXRzIGRvZXMgbm90IGZvY3VzIHRoZSBl
bGVtZW50LgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9
MjQwOTIKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAq
IGZhc3QvZm9ybXMvYXV0b2ZvY3VzLXJlYWRvbmx5LWF0dHJpYnV0ZS1leHBlY3RlZC50eHQ6IEFk
ZGVkLgorICAgICAgICAqIGZhc3QvZm9ybXMvYXV0b2ZvY3VzLXJlYWRvbmx5LWF0dHJpYnV0ZS5o
dG1sOiBBZGRlZC4KKwogMjAxMS0wOS0yMSAgQWJoaXNoZWsgQXJ5YSAgPGluZmVybm9AY2hyb21p
dW0ub3JnPgogCiAgICAgICAgIFVucmV2aWV3ZWQuIENocm9taXVtIHJlYmFzZWxpbmVzIGZvciBy
OTU2NzEuICAgICAgICAKSW5kZXg6IExheW91dFRlc3RzL2Zhc3QvZm9ybXMvYXV0b2ZvY3VzLXJl
YWRvbmx5LWF0dHJpYnV0ZS5odG1sCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3RzL2Zhc3QvZm9y
bXMvYXV0b2ZvY3VzLXJlYWRvbmx5LWF0dHJpYnV0ZS5odG1sCShyZXZpc2lvbiAwKQorKysgTGF5
b3V0VGVzdHMvZmFzdC9mb3Jtcy9hdXRvZm9jdXMtcmVhZG9ubHktYXR0cmlidXRlLmh0bWwJKHJl
dmlzaW9uIDApCkBAIC0wLDAgKzEsMjAgQEAKKzwhRE9DWVRQRSBodG1sPgorPGh0bWw+Cis8Ym9k
eT4KKzxwPlRoZSB0ZXh0IGJveCBzaG91bGQgaGF2ZSBmb2N1cyBjb25zb2xlIHNob3VsZCBwcmlu
dCBQQVNTPC9wPgorPHA+PGlucHV0IHR5cGU9InRleHQiIG5hbWU9InRlc3QiIGF1dG9mb2N1cyBy
ZWFkb25seSBpZD0idGVzdCI+PC9wPgorPGRpdiBpZD0iY29uc29sZSI+PC9kaXY+Cis8c2NyaXB0
PgoraWYgKHdpbmRvdy5sYXlvdXRUZXN0Q29udHJvbGxlcikKKyAgICBsYXlvdXRUZXN0Q29udHJv
bGxlci5kdW1wQXNUZXh0KCk7CisKKyhkb2N1bWVudC5hY3RpdmVFbGVtZW50ID09IGRvY3VtZW50
LmdldEVsZW1lbnRCeUlkKCJ0ZXN0IikpID8gbG9nKCJQQVNTIik6bG9nKCJGQUlMIik7CisKK2Z1
bmN0aW9uIGxvZyhtc2cpIHsKKyAgICB2YXIgY29uc29sZSA9IGRvY3VtZW50LmdldEVsZW1lbnRC
eUlkKCJjb25zb2xlIik7CisgICAgY29uc29sZS5pbm5lckhUTUwgPSBjb25zb2xlLmlubmVySFRN
TCArIG1zZyArICI8YnI+IjsKK30KKzwvc2NyaXB0PgorPC9ib2R5PgorPC9odG1sPgpJbmRleDog
TGF5b3V0VGVzdHMvZmFzdC9mb3Jtcy9hdXRvZm9jdXMtcmVhZG9ubHktYXR0cmlidXRlLWV4cGVj
dGVkLnR4dAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9mYXN0L2Zvcm1zL2F1dG9mb2N1cy1y
ZWFkb25seS1hdHRyaWJ1dGUtZXhwZWN0ZWQudHh0CShyZXZpc2lvbiAwKQorKysgTGF5b3V0VGVz
dHMvZmFzdC9mb3Jtcy9hdXRvZm9jdXMtcmVhZG9ubHktYXR0cmlidXRlLWV4cGVjdGVkLnR4dAko
cmV2aXNpb24gMCkKQEAgLTAsMCArMSw4IEBACitUaGUgdGV4dCBib3ggc2hvdWxkIGhhdmUgZm9j
dXMgY29uc29sZSBzaG91bGQgcHJpbnQgUEFTUworCisKKworCisKK1BBU1MKKwo=
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>108287</attachid>
            <date>2011-09-22 00:17:22 -0700</date>
            <delta_ts>2011-09-26 00:05:35 -0700</delta_ts>
            <desc>Updated Patch</desc>
            <filename>patch.txt</filename>
            <type>text/plain</type>
            <size>3194</size>
            <attacher name="Kaustubh Atrawalkar">kaustubh.ra</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDk1Njk4KQorKysgU291cmNlL1dlYkNvcmUvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTcgQEAKKzIwMTEtMDktMTkgIEthdXN0dWJo
IEF0cmF3YWxrYXIgIDxrYXVzdHViaEBtb3Rvcm9sYS5jb20+CisKKyAgICAgICAgQXV0b2ZvY3Vz
IG9uIHJlYWRvbmx5IGlucHV0cyBkb2VzIG5vdCBmb2N1cyB0aGUgZWxlbWVudC4KKyAgICAgICAg
aHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTI0MDkyCisKKyAgICAgICAg
UmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgUmVhZG9ubHkgaW5wdXQgZWxl
bWVudHMgc2hvdWxkIGJlIGF1dG9mb2N1c2FibGUuIFJlbW92ZWQgdGhlIGNoZWNrLgorCisgICAg
ICAgIFRlc3RzOiBmYXN0L2Zvcm1zL2F1dG9mb2N1cy1yZWFkb25seS1hdHRyaWJ1dGUuaHRtbAor
CisgICAgICAgICogaHRtbC9IVE1MRm9ybUNvbnRyb2xFbGVtZW50LmNwcDoKKyAgICAgICAgKFdl
YkNvcmU6OnNob3VsZEF1dG9mb2N1cyk6CisKIDIwMTEtMDktMjEgIEJldGggRGFraW4gIDxiZGFr
aW5AYXBwbGUuY29tPgogCiAgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVn
LmNnaT9pZD02NzQxNQpJbmRleDogU291cmNlL1dlYkNvcmUvaHRtbC9IVE1MRm9ybUNvbnRyb2xF
bGVtZW50LmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViQ29yZS9odG1sL0hUTUxGb3JtQ29u
dHJvbEVsZW1lbnQuY3BwCShyZXZpc2lvbiA5NTY5OCkKKysrIFNvdXJjZS9XZWJDb3JlL2h0bWwv
SFRNTEZvcm1Db250cm9sRWxlbWVudC5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTEyNyw4ICsxMjcs
NiBAQAogICAgICAgICByZXR1cm4gZmFsc2U7CiAgICAgaWYgKGVsZW1lbnQtPmRvY3VtZW50KCkt
Pmlnbm9yZUF1dG9mb2N1cygpKQogICAgICAgICByZXR1cm4gZmFsc2U7Ci0gICAgaWYgKGVsZW1l
bnQtPmlzUmVhZE9ubHlGb3JtQ29udHJvbCgpKQotICAgICAgICByZXR1cm4gZmFsc2U7CiAKICAg
ICAvLyBGSVhNRTogU2hvdWxkIHRoaXMgc2V0IG9mIGhhc1RhZ05hbWUgY2hlY2tzIGJlIHJlcGxh
Y2VkIGJ5IGEKICAgICAvLyB2aXJ0dWFsIG1lbWJlciBmdW5jdGlvbj8KSW5kZXg6IExheW91dFRl
c3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cJKHJldmlz
aW9uIDk1Njk4KQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0x
LDMgKzEsMTMgQEAKKzIwMTEtMDktMTkgIEthdXN0dWJoIEF0cmF3YWxrYXIgIDxrYXVzdHViaEBt
b3Rvcm9sYS5jb20+CisKKyAgICAgICAgQXV0b2ZvY3VzIG9uIHJlYWRvbmx5IGlucHV0cyBkb2Vz
IG5vdCBmb2N1cyB0aGUgZWxlbWVudC4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcv
c2hvd19idWcuY2dpP2lkPTI0MDkyCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BT
ISkuCisKKyAgICAgICAgKiBmYXN0L2Zvcm1zL2F1dG9mb2N1cy1yZWFkb25seS1hdHRyaWJ1dGUt
ZXhwZWN0ZWQudHh0OiBBZGRlZC4KKyAgICAgICAgKiBmYXN0L2Zvcm1zL2F1dG9mb2N1cy1yZWFk
b25seS1hdHRyaWJ1dGUuaHRtbDogQWRkZWQuCisKIDIwMTEtMDktMjEgIEFiaGlzaGVrIEFyeWEg
IDxpbmZlcm5vQGNocm9taXVtLm9yZz4KIAogICAgICAgICBVbnJldmlld2VkLiBDaHJvbWl1bSBy
ZWJhc2VsaW5lcyBmb3Igcjk1NjcxLiAgICAgICAgCkluZGV4OiBMYXlvdXRUZXN0cy9mYXN0L2Zv
cm1zL2F1dG9mb2N1cy1yZWFkb25seS1hdHRyaWJ1dGUtZXhwZWN0ZWQudHh0Cj09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0K
LS0tIExheW91dFRlc3RzL2Zhc3QvZm9ybXMvYXV0b2ZvY3VzLXJlYWRvbmx5LWF0dHJpYnV0ZS1l
eHBlY3RlZC50eHQJKHJldmlzaW9uIDApCisrKyBMYXlvdXRUZXN0cy9mYXN0L2Zvcm1zL2F1dG9m
b2N1cy1yZWFkb25seS1hdHRyaWJ1dGUtZXhwZWN0ZWQudHh0CShyZXZpc2lvbiAwKQpAQCAtMCww
ICsxLDYgQEAKK1RoZSB0ZXh0IGJveCBzaG91bGQgaGF2ZSBmb2N1cyBjb25zb2xlIHNob3VsZCBw
cmludCBQQVNTCisKKworCitQQVNTCisKSW5kZXg6IExheW91dFRlc3RzL2Zhc3QvZm9ybXMvYXV0
b2ZvY3VzLXJlYWRvbmx5LWF0dHJpYnV0ZS5odG1sCj09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3Rz
L2Zhc3QvZm9ybXMvYXV0b2ZvY3VzLXJlYWRvbmx5LWF0dHJpYnV0ZS5odG1sCShyZXZpc2lvbiAw
KQorKysgTGF5b3V0VGVzdHMvZmFzdC9mb3Jtcy9hdXRvZm9jdXMtcmVhZG9ubHktYXR0cmlidXRl
Lmh0bWwJKHJldmlzaW9uIDApCkBAIC0wLDAgKzEsMTkgQEAKKzwhRE9DWVRQRSBodG1sPgorPGh0
bWw+Cis8Ym9keT4KKzxwPlRoZSB0ZXh0IGJveCBzaG91bGQgaGF2ZSBmb2N1cyBjb25zb2xlIHNo
b3VsZCBwcmludCBQQVNTPC9wPgorPHA+PGlucHV0IHR5cGU9InRleHQiIG5hbWU9InRlc3QiIGF1
dG9mb2N1cyByZWFkb25seSBpZD0idGVzdCI+PC9wPgorPGRpdiBpZD0iY29uc29sZSI+PC9kaXY+
Cis8c2NyaXB0PgoraWYgKHdpbmRvdy5sYXlvdXRUZXN0Q29udHJvbGxlcikKKyAgICBsYXlvdXRU
ZXN0Q29udHJvbGxlci5kdW1wQXNUZXh0KCk7CisKKyhkb2N1bWVudC5hY3RpdmVFbGVtZW50ID09
IGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJ0ZXN0IikpID8gbG9nKCJQQVNTIik6bG9nKCJGQUlM
Iik7CisKK2Z1bmN0aW9uIGxvZyhtc2cpIHsKKyAgICB2YXIgY29uc29sZSA9IGRvY3VtZW50Lmdl
dEVsZW1lbnRCeUlkKCJjb25zb2xlIik7CisgICAgY29uc29sZS5pbm5lckhUTUwgPSBjb25zb2xl
LmlubmVySFRNTCArIG1zZyArICI8YnI+IjsKK30KKzwvc2NyaXB0PgorPC9ib2R5PgorPC9odG1s
Pgo=
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>108629</attachid>
            <date>2011-09-26 00:05:35 -0700</date>
            <delta_ts>2011-09-27 00:24:45 -0700</delta_ts>
            <desc>Updated patch</desc>
            <filename>patch.txt</filename>
            <type>text/plain</type>
            <size>3199</size>
            <attacher name="Kaustubh Atrawalkar">kaustubh.ra</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDk1OTM0KQorKysgU291cmNlL1dlYkNvcmUvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTcgQEAKKzIwMTEtMDktMjYgIEthdXN0dWJo
IEF0cmF3YWxrYXIgIDxrYXVzdHViaEBtb3Rvcm9sYS5jb20+CisKKyAgICAgICAgQXV0b2ZvY3Vz
IG9uIHJlYWRvbmx5IGlucHV0cyBkb2VzIG5vdCBmb2N1cyB0aGUgZWxlbWVudC4KKyAgICAgICAg
aHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTI0MDkyCisKKyAgICAgICAg
UmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgUmVhZG9ubHkgaW5wdXQgZWxl
bWVudHMgc2hvdWxkIGJlIGF1dG9mb2N1c2FibGUuIFJlbW92ZWQgdGhlIGNoZWNrLgorCisgICAg
ICAgIFRlc3RzOiBmYXN0L2Zvcm1zL2F1dG9mb2N1cy1yZWFkb25seS1hdHRyaWJ1dGUuaHRtbAor
CisgICAgICAgICogaHRtbC9IVE1MRm9ybUNvbnRyb2xFbGVtZW50LmNwcDoKKyAgICAgICAgKFdl
YkNvcmU6OnNob3VsZEF1dG9mb2N1cyk6CisKIDIwMTEtMDktMjUgIEFkYW0gQmFydGggIDxhYmFy
dGhAd2Via2l0Lm9yZz4KIAogICAgICAgICBGaW5pc2ggcmVtb3ZpbmcgUExBVEZPUk0oQlJFV01Q
KSBieSByZW1vdmluZyBhc3NvY2lhdGVkIGNvZGUKSW5kZXg6IFNvdXJjZS9XZWJDb3JlL2h0bWwv
SFRNTEZvcm1Db250cm9sRWxlbWVudC5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL1dlYkNvcmUv
aHRtbC9IVE1MRm9ybUNvbnRyb2xFbGVtZW50LmNwcAkocmV2aXNpb24gOTU5MzQpCisrKyBTb3Vy
Y2UvV2ViQ29yZS9odG1sL0hUTUxGb3JtQ29udHJvbEVsZW1lbnQuY3BwCSh3b3JraW5nIGNvcHkp
CkBAIC0xMjcsOCArMTI3LDYgQEAKICAgICAgICAgcmV0dXJuIGZhbHNlOwogICAgIGlmIChlbGVt
ZW50LT5kb2N1bWVudCgpLT5pZ25vcmVBdXRvZm9jdXMoKSkKICAgICAgICAgcmV0dXJuIGZhbHNl
OwotICAgIGlmIChlbGVtZW50LT5pc1JlYWRPbmx5Rm9ybUNvbnRyb2woKSkKLSAgICAgICAgcmV0
dXJuIGZhbHNlOwogCiAgICAgLy8gRklYTUU6IFNob3VsZCB0aGlzIHNldCBvZiBoYXNUYWdOYW1l
IGNoZWNrcyBiZSByZXBsYWNlZCBieSBhCiAgICAgLy8gdmlydHVhbCBtZW1iZXIgZnVuY3Rpb24/
CkluZGV4OiBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMv
Q2hhbmdlTG9nCShyZXZpc2lvbiA5NTkzNCkKKysrIExheW91dFRlc3RzL0NoYW5nZUxvZwkod29y
a2luZyBjb3B5KQpAQCAtMSwzICsxLDEzIEBACisyMDExLTA5LTI2ICBLYXVzdHViaCBBdHJhd2Fs
a2FyICA8a2F1c3R1YmhAbW90b3JvbGEuY29tPgorCisgICAgICAgIEF1dG9mb2N1cyBvbiByZWFk
b25seSBpbnB1dHMgZG9lcyBub3QgZm9jdXMgdGhlIGVsZW1lbnQuCisgICAgICAgIGh0dHBzOi8v
YnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yNDA5MgorCisgICAgICAgIFJldmlld2Vk
IGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogZmFzdC9mb3Jtcy9hdXRvZm9jdXMtcmVh
ZG9ubHktYXR0cmlidXRlLWV4cGVjdGVkLnR4dDogQWRkZWQuCisgICAgICAgICogZmFzdC9mb3Jt
cy9hdXRvZm9jdXMtcmVhZG9ubHktYXR0cmlidXRlLmh0bWw6IEFkZGVkLgorCiAyMDExLTA5LTI1
ICBLZW50YXJvIEhhcmEgIDxoYXJha2VuQGNocm9taXVtLm9yZz4KIAogICAgICAgICBJbXBsZW1l
bnQgYSBDbG9zZUV2ZW50IGNvbnN0cnVjdG9yIGZvciBKU0MKSW5kZXg6IExheW91dFRlc3RzL2Zh
c3QvZm9ybXMvYXV0b2ZvY3VzLXJlYWRvbmx5LWF0dHJpYnV0ZS1leHBlY3RlZC50eHQKPT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PQotLS0gTGF5b3V0VGVzdHMvZmFzdC9mb3Jtcy9hdXRvZm9jdXMtcmVhZG9ubHktYXR0cmli
dXRlLWV4cGVjdGVkLnR4dAkocmV2aXNpb24gMCkKKysrIExheW91dFRlc3RzL2Zhc3QvZm9ybXMv
YXV0b2ZvY3VzLXJlYWRvbmx5LWF0dHJpYnV0ZS1leHBlY3RlZC50eHQJKHJldmlzaW9uIDApCkBA
IC0wLDAgKzEsNiBAQAorVGhlIHRleHQgYm94IHNob3VsZCBoYXZlIGZvY3VzIGNvbnNvbGUgc2hv
dWxkIHByaW50IFBBU1MKKworCisKK1BBU1MKKwpJbmRleDogTGF5b3V0VGVzdHMvZmFzdC9mb3Jt
cy9hdXRvZm9jdXMtcmVhZG9ubHktYXR0cmlidXRlLmh0bWwKPT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gTGF5b3V0
VGVzdHMvZmFzdC9mb3Jtcy9hdXRvZm9jdXMtcmVhZG9ubHktYXR0cmlidXRlLmh0bWwJKHJldmlz
aW9uIDApCisrKyBMYXlvdXRUZXN0cy9mYXN0L2Zvcm1zL2F1dG9mb2N1cy1yZWFkb25seS1hdHRy
aWJ1dGUuaHRtbAkocmV2aXNpb24gMCkKQEAgLTAsMCArMSwxOSBAQAorPCFET0NZVFBFIGh0bWw+
Cis8aHRtbD4KKzxib2R5PgorPHA+VGhlIHRleHQgYm94IHNob3VsZCBoYXZlIGZvY3VzIGNvbnNv
bGUgc2hvdWxkIHByaW50IFBBU1M8L3A+Cis8cD48aW5wdXQgdHlwZT0idGV4dCIgbmFtZT0idGVz
dCIgYXV0b2ZvY3VzIHJlYWRvbmx5IGlkPSJ0ZXN0Ij48L3A+Cis8ZGl2IGlkPSJjb25zb2xlIj48
L2Rpdj4KKzxzY3JpcHQ+CitpZiAod2luZG93LmxheW91dFRlc3RDb250cm9sbGVyKQorICAgIGxh
eW91dFRlc3RDb250cm9sbGVyLmR1bXBBc1RleHQoKTsKKworKGRvY3VtZW50LmFjdGl2ZUVsZW1l
bnQgPT0gZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoInRlc3QiKSkgPyBsb2coIlBBU1MiKTpsb2co
IkZBSUwiKTsKKworZnVuY3Rpb24gbG9nKG1zZykgeworICAgIHZhciBjb25zb2xlID0gZG9jdW1l
bnQuZ2V0RWxlbWVudEJ5SWQoImNvbnNvbGUiKTsKKyAgICBjb25zb2xlLmlubmVySFRNTCA9IGNv
bnNvbGUuaW5uZXJIVE1MICsgbXNnICsgIjxicj4iOworfQorPC9zY3JpcHQ+Cis8L2JvZHk+Cis8
L2h0bWw+Cg==
</data>
<flag name="review"
          id="105642"
          type_id="1"
          status="+"
          setter="rniwa"
    />
          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>108796</attachid>
            <date>2011-09-27 00:24:45 -0700</date>
            <delta_ts>2011-09-27 00:51:08 -0700</delta_ts>
            <desc>Updated Patch</desc>
            <filename>patch.txt</filename>
            <type>text/plain</type>
            <size>3210</size>
            <attacher name="Kaustubh Atrawalkar">kaustubh.ra</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDk2MDcyKQorKysgU291cmNlL1dlYkNvcmUvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTcgQEAKKzIwMTEtMDktMjYgIEthdXN0dWJo
IEF0cmF3YWxrYXIgIDxrYXVzdHViaEBtb3Rvcm9sYS5jb20+CisKKyAgICAgICAgQXV0b2ZvY3Vz
IG9uIHJlYWRvbmx5IGlucHV0cyBkb2VzIG5vdCBmb2N1cyB0aGUgZWxlbWVudC4KKyAgICAgICAg
aHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTI0MDkyCisKKyAgICAgICAg
UmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgUmVhZG9ubHkgaW5wdXQgZWxl
bWVudHMgc2hvdWxkIGJlIGF1dG9mb2N1c2FibGUuIFJlbW92ZWQgdGhlIGNoZWNrLgorCisgICAg
ICAgIFRlc3RzOiBmYXN0L2Zvcm1zL2F1dG9mb2N1cy1yZWFkb25seS1hdHRyaWJ1dGUuaHRtbAor
CisgICAgICAgICogaHRtbC9IVE1MRm9ybUNvbnRyb2xFbGVtZW50LmNwcDoKKyAgICAgICAgKFdl
YkNvcmU6OnNob3VsZEF1dG9mb2N1cyk6CisKIDIwMTEtMDktMDkgIFNpbW9uIEZyYXNlciAgPHNp
bW9uLmZyYXNlckBhcHBsZS5jb20+CiAKICAgICAgICAgUGl4ZWwgcmVzdWx0IHNob3dzIHRoYXQg
Y29tcG9zaXRpbmcvaWZyYW1lcy9yZXBhaW50LWFmdGVyLWxvc2luZy1zY3JvbGxiYXJzLmh0bWwg
aXMgZmFpbGluZwpJbmRleDogU291cmNlL1dlYkNvcmUvaHRtbC9IVE1MRm9ybUNvbnRyb2xFbGVt
ZW50LmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViQ29yZS9odG1sL0hUTUxGb3JtQ29udHJv
bEVsZW1lbnQuY3BwCShyZXZpc2lvbiA5NjA3MikKKysrIFNvdXJjZS9XZWJDb3JlL2h0bWwvSFRN
TEZvcm1Db250cm9sRWxlbWVudC5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTEyNyw4ICsxMjcsNiBA
QAogICAgICAgICByZXR1cm4gZmFsc2U7CiAgICAgaWYgKGVsZW1lbnQtPmRvY3VtZW50KCktPmln
bm9yZUF1dG9mb2N1cygpKQogICAgICAgICByZXR1cm4gZmFsc2U7Ci0gICAgaWYgKGVsZW1lbnQt
PmlzUmVhZE9ubHlGb3JtQ29udHJvbCgpKQotICAgICAgICByZXR1cm4gZmFsc2U7CiAKICAgICAv
LyBGSVhNRTogU2hvdWxkIHRoaXMgc2V0IG9mIGhhc1RhZ05hbWUgY2hlY2tzIGJlIHJlcGxhY2Vk
IGJ5IGEKICAgICAvLyB2aXJ0dWFsIG1lbWJlciBmdW5jdGlvbj8KSW5kZXg6IExheW91dFRlc3Rz
L0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cJKHJldmlzaW9u
IDk2MDcyKQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMg
KzEsMTMgQEAKKzIwMTEtMDktMjYgIEthdXN0dWJoIEF0cmF3YWxrYXIgIDxrYXVzdHViaEBtb3Rv
cm9sYS5jb20+CisKKyAgICAgICAgQXV0b2ZvY3VzIG9uIHJlYWRvbmx5IGlucHV0cyBkb2VzIG5v
dCBmb2N1cyB0aGUgZWxlbWVudC4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hv
d19idWcuY2dpP2lkPTI0MDkyCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku
CisKKyAgICAgICAgKiBmYXN0L2Zvcm1zL2F1dG9mb2N1cy1yZWFkb25seS1hdHRyaWJ1dGUtZXhw
ZWN0ZWQudHh0OiBBZGRlZC4KKyAgICAgICAgKiBmYXN0L2Zvcm1zL2F1dG9mb2N1cy1yZWFkb25s
eS1hdHRyaWJ1dGUuaHRtbDogQWRkZWQuCisKIDIwMTEtMDktMDkgIFNpbW9uIEZyYXNlciAgPHNp
bW9uLmZyYXNlckBhcHBsZS5jb20+CiAKICAgICAgICAgVHJhbnNsdWNlbnQgc2Nyb2xsYmFycyBv
biBjb21wb3NpdGVkIGxheWVycyByZW5kZXIgaW5jb3JyZWN0bHkKSW5kZXg6IExheW91dFRlc3Rz
L2Zhc3QvZm9ybXMvYXV0b2ZvY3VzLXJlYWRvbmx5LWF0dHJpYnV0ZS1leHBlY3RlZC50eHQKPT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PQotLS0gTGF5b3V0VGVzdHMvZmFzdC9mb3Jtcy9hdXRvZm9jdXMtcmVhZG9ubHktYXR0
cmlidXRlLWV4cGVjdGVkLnR4dAkocmV2aXNpb24gMCkKKysrIExheW91dFRlc3RzL2Zhc3QvZm9y
bXMvYXV0b2ZvY3VzLXJlYWRvbmx5LWF0dHJpYnV0ZS1leHBlY3RlZC50eHQJKHJldmlzaW9uIDAp
CkBAIC0wLDAgKzEsNCBAQAorVGhlIHRleHQgYm94IHNob3VsZCBoYXZlIGZvY3VzIGNvbnNvbGUg
c2hvdWxkIHByaW50IFBBU1MKKworUEFTUworCkluZGV4OiBMYXlvdXRUZXN0cy9mYXN0L2Zvcm1z
L2F1dG9mb2N1cy1yZWFkb25seS1hdHRyaWJ1dGUuaHRtbAo9PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRU
ZXN0cy9mYXN0L2Zvcm1zL2F1dG9mb2N1cy1yZWFkb25seS1hdHRyaWJ1dGUuaHRtbAkocmV2aXNp
b24gMCkKKysrIExheW91dFRlc3RzL2Zhc3QvZm9ybXMvYXV0b2ZvY3VzLXJlYWRvbmx5LWF0dHJp
YnV0ZS5odG1sCShyZXZpc2lvbiAwKQpAQCAtMCwwICsxLDIyIEBACis8IURPQ1lUUEUgaHRtbD4K
KzxodG1sPgorPGhlYWQ+Cis8c2NyaXB0PgorZnVuY3Rpb24gcnVuVGVzdCgpCit7CisgICAgaWYg
KHdpbmRvdy5sYXlvdXRUZXN0Q29udHJvbGxlcikKKyAgICAgICAgbGF5b3V0VGVzdENvbnRyb2xs
ZXIuZHVtcEFzVGV4dCgpOworfQorCitmdW5jdGlvbiBsb2cobXNnKSB7CisgICAgdmFyIGVsZSA9
IGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJjb25zb2xlIik7CisgICAgZWxlLmlubmVyVGV4dCA9
IG1zZzsKK30KKzwvc2NyaXB0PgorPC9oZWFkPgorPGJvZHkgb25sb2FkPSdydW5UZXN0KCknPgor
PHA+VGhlIHRleHQgYm94IHNob3VsZCBoYXZlIGZvY3VzIGNvbnNvbGUgc2hvdWxkIHByaW50IFBB
U1M8L3A+Cis8ZGl2IGlkPSJjb25zb2xlIj48L2Rpdj4KKzxwPjxpbnB1dCB0eXBlPSJ0ZXh0IiBu
YW1lPSJ0ZXN0IiBpZD0idGVzdCIgcmVhZG9ubHkgb25mb2N1cz0nbG9nKCJQQVNTIiknPjwvcD4K
KzwvYm9keT4KKzwvaHRtbD4K
</data>
<flag name="review"
          id="105848"
          type_id="1"
          status="-"
          setter="rniwa"
    />
          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>108804</attachid>
            <date>2011-09-27 00:51:08 -0700</date>
            <delta_ts>2011-09-27 01:51:52 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>patch.txt</filename>
            <type>text/plain</type>
            <size>3175</size>
            <attacher name="Kaustubh Atrawalkar">kaustubh.ra</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDk2MDcyKQorKysgU291cmNlL1dlYkNvcmUvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTcgQEAKKzIwMTEtMDktMjYgIEthdXN0dWJo
IEF0cmF3YWxrYXIgIDxrYXVzdHViaEBtb3Rvcm9sYS5jb20+CisKKyAgICAgICAgQXV0b2ZvY3Vz
IG9uIHJlYWRvbmx5IGlucHV0cyBkb2VzIG5vdCBmb2N1cyB0aGUgZWxlbWVudC4KKyAgICAgICAg
aHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTI0MDkyCisKKyAgICAgICAg
UmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgUmVhZG9ubHkgaW5wdXQgZWxl
bWVudHMgc2hvdWxkIGJlIGF1dG9mb2N1c2FibGUuIFJlbW92ZWQgdGhlIGNoZWNrLgorCisgICAg
ICAgIFRlc3RzOiBmYXN0L2Zvcm1zL2F1dG9mb2N1cy1yZWFkb25seS1hdHRyaWJ1dGUuaHRtbAor
CisgICAgICAgICogaHRtbC9IVE1MRm9ybUNvbnRyb2xFbGVtZW50LmNwcDoKKyAgICAgICAgKFdl
YkNvcmU6OnNob3VsZEF1dG9mb2N1cyk6CisKIDIwMTEtMDktMDkgIFNpbW9uIEZyYXNlciAgPHNp
bW9uLmZyYXNlckBhcHBsZS5jb20+CiAKICAgICAgICAgUGl4ZWwgcmVzdWx0IHNob3dzIHRoYXQg
Y29tcG9zaXRpbmcvaWZyYW1lcy9yZXBhaW50LWFmdGVyLWxvc2luZy1zY3JvbGxiYXJzLmh0bWwg
aXMgZmFpbGluZwpJbmRleDogU291cmNlL1dlYkNvcmUvaHRtbC9IVE1MRm9ybUNvbnRyb2xFbGVt
ZW50LmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViQ29yZS9odG1sL0hUTUxGb3JtQ29udHJv
bEVsZW1lbnQuY3BwCShyZXZpc2lvbiA5NjA3MikKKysrIFNvdXJjZS9XZWJDb3JlL2h0bWwvSFRN
TEZvcm1Db250cm9sRWxlbWVudC5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTEyNyw4ICsxMjcsNiBA
QAogICAgICAgICByZXR1cm4gZmFsc2U7CiAgICAgaWYgKGVsZW1lbnQtPmRvY3VtZW50KCktPmln
bm9yZUF1dG9mb2N1cygpKQogICAgICAgICByZXR1cm4gZmFsc2U7Ci0gICAgaWYgKGVsZW1lbnQt
PmlzUmVhZE9ubHlGb3JtQ29udHJvbCgpKQotICAgICAgICByZXR1cm4gZmFsc2U7CiAKICAgICAv
LyBGSVhNRTogU2hvdWxkIHRoaXMgc2V0IG9mIGhhc1RhZ05hbWUgY2hlY2tzIGJlIHJlcGxhY2Vk
IGJ5IGEKICAgICAvLyB2aXJ0dWFsIG1lbWJlciBmdW5jdGlvbj8KSW5kZXg6IExheW91dFRlc3Rz
L0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9DaGFuZ2VMb2cJKHJldmlzaW9u
IDk2MDcyKQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMg
KzEsMTMgQEAKKzIwMTEtMDktMjYgIEthdXN0dWJoIEF0cmF3YWxrYXIgIDxrYXVzdHViaEBtb3Rv
cm9sYS5jb20+CisKKyAgICAgICAgQXV0b2ZvY3VzIG9uIHJlYWRvbmx5IGlucHV0cyBkb2VzIG5v
dCBmb2N1cyB0aGUgZWxlbWVudC4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hv
d19idWcuY2dpP2lkPTI0MDkyCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku
CisKKyAgICAgICAgKiBmYXN0L2Zvcm1zL2F1dG9mb2N1cy1yZWFkb25seS1hdHRyaWJ1dGUtZXhw
ZWN0ZWQudHh0OiBBZGRlZC4KKyAgICAgICAgKiBmYXN0L2Zvcm1zL2F1dG9mb2N1cy1yZWFkb25s
eS1hdHRyaWJ1dGUuaHRtbDogQWRkZWQuCisKIDIwMTEtMDktMDkgIFNpbW9uIEZyYXNlciAgPHNp
bW9uLmZyYXNlckBhcHBsZS5jb20+CiAKICAgICAgICAgVHJhbnNsdWNlbnQgc2Nyb2xsYmFycyBv
biBjb21wb3NpdGVkIGxheWVycyByZW5kZXIgaW5jb3JyZWN0bHkKSW5kZXg6IExheW91dFRlc3Rz
L2Zhc3QvZm9ybXMvYXV0b2ZvY3VzLXJlYWRvbmx5LWF0dHJpYnV0ZS1leHBlY3RlZC50eHQKPT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PQotLS0gTGF5b3V0VGVzdHMvZmFzdC9mb3Jtcy9hdXRvZm9jdXMtcmVhZG9ubHktYXR0
cmlidXRlLWV4cGVjdGVkLnR4dAkocmV2aXNpb24gMCkKKysrIExheW91dFRlc3RzL2Zhc3QvZm9y
bXMvYXV0b2ZvY3VzLXJlYWRvbmx5LWF0dHJpYnV0ZS1leHBlY3RlZC50eHQJKHJldmlzaW9uIDAp
CkBAIC0wLDAgKzEsNCBAQAorVGhlIHRleHQgYm94IHNob3VsZCBoYXZlIGZvY3VzIGNvbnNvbGUg
c2hvdWxkIHByaW50IFBBU1MKKworUEFTUworCkluZGV4OiBMYXlvdXRUZXN0cy9mYXN0L2Zvcm1z
L2F1dG9mb2N1cy1yZWFkb25seS1hdHRyaWJ1dGUuaHRtbAo9PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRU
ZXN0cy9mYXN0L2Zvcm1zL2F1dG9mb2N1cy1yZWFkb25seS1hdHRyaWJ1dGUuaHRtbAkocmV2aXNp
b24gMCkKKysrIExheW91dFRlc3RzL2Zhc3QvZm9ybXMvYXV0b2ZvY3VzLXJlYWRvbmx5LWF0dHJp
YnV0ZS5odG1sCShyZXZpc2lvbiAwKQpAQCAtMCwwICsxLDE5IEBACis8IURPQ1lUUEUgaHRtbD4K
KzxodG1sPgorPGhlYWQ+Cis8c2NyaXB0PgoraWYgKHdpbmRvdy5sYXlvdXRUZXN0Q29udHJvbGxl
cikKKyAgICBsYXlvdXRUZXN0Q29udHJvbGxlci5kdW1wQXNUZXh0KCk7CisKK2Z1bmN0aW9uIGxv
Zyhtc2cpIHsKKyAgICB2YXIgY29uc29sZSA9IGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJjb25z
b2xlIik7CisgICAgY29uc29sZS5pbm5lclRleHQgPSBtc2c7Cit9Cis8L3NjcmlwdD4KKzwvaGVh
ZD4KKzxib2R5PgorPHA+VGhlIHRleHQgYm94IHNob3VsZCBoYXZlIGZvY3VzIGNvbnNvbGUgc2hv
dWxkIHByaW50IFBBU1M8L3A+Cis8ZGl2IGlkPSJjb25zb2xlIj48L2Rpdj4KKzxwPjxpbnB1dCB0
eXBlPSJ0ZXh0IiBuYW1lPSJ0ZXN0IiBpZD0idGVzdCIgcmVhZG9ubHkgYXV0b2ZvY3VzIG9uZm9j
dXM9J2xvZygiUEFTUyIpJz48L3A+Cis8L2JvZHk+Cis8L2h0bWw+Cg==
</data>
<flag name="review"
          id="105861"
          type_id="1"
          status="+"
          setter="rniwa"
    />
    <flag name="commit-queue"
          id="105862"
          type_id="3"
          status="-"
          setter="webkit.review.bot"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>108812</attachid>
            <date>2011-09-27 01:51:52 -0700</date>
            <delta_ts>2011-09-27 11:16:38 -0700</delta_ts>
            <desc>Updated Patch</desc>
            <filename>patch.txt</filename>
            <type>text/plain</type>
            <size>3185</size>
            <attacher name="Kaustubh Atrawalkar">kaustubh.ra</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDk2MDgyKQorKysgU291cmNlL1dlYkNvcmUvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTcgQEAKKzIwMTEtMDktMjYgIEthdXN0dWJo
IEF0cmF3YWxrYXIgIDxrYXVzdHViaEBtb3Rvcm9sYS5jb20+CisKKyAgICAgICAgQXV0b2ZvY3Vz
IG9uIHJlYWRvbmx5IGlucHV0cyBkb2VzIG5vdCBmb2N1cyB0aGUgZWxlbWVudC4KKyAgICAgICAg
aHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTI0MDkyCisKKyAgICAgICAg
UmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgUmVhZG9ubHkgaW5wdXQgZWxl
bWVudHMgc2hvdWxkIGJlIGF1dG9mb2N1c2FibGUuIFJlbW92ZWQgdGhlIGNoZWNrLgorCisgICAg
ICAgIFRlc3RzOiBmYXN0L2Zvcm1zL2F1dG9mb2N1cy1yZWFkb25seS1hdHRyaWJ1dGUuaHRtbAor
CisgICAgICAgICogaHRtbC9IVE1MRm9ybUNvbnRyb2xFbGVtZW50LmNwcDoKKyAgICAgICAgKFdl
YkNvcmU6OnNob3VsZEF1dG9mb2N1cyk6CisKIDIwMTEtMDktMjcgIEFydW4gUGF0b2xlICA8Ym1m
ODM0QG1vdG9yb2xhLmNvbT4KIAogICAgICAgICBBdWRpbyBlbGVtZW50IGRvZXNuJ3QgZW1pdCB0
aGUgJ3BsYXlpbmcnIGV2ZW50IGV2ZXJ5IHRpbWUgaXQgc3RhcnRzIHBsYXlpbmcsIGFmdGVyIGl0
IGhhcyBmaW5pc2hlZCBwbGF5aW5nLgpJbmRleDogU291cmNlL1dlYkNvcmUvaHRtbC9IVE1MRm9y
bUNvbnRyb2xFbGVtZW50LmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViQ29yZS9odG1sL0hU
TUxGb3JtQ29udHJvbEVsZW1lbnQuY3BwCShyZXZpc2lvbiA5NjA4MikKKysrIFNvdXJjZS9XZWJD
b3JlL2h0bWwvSFRNTEZvcm1Db250cm9sRWxlbWVudC5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTEy
OCw4ICsxMjgsNiBAQAogICAgICAgICByZXR1cm4gZmFsc2U7CiAgICAgaWYgKGVsZW1lbnQtPmRv
Y3VtZW50KCktPmlnbm9yZUF1dG9mb2N1cygpKQogICAgICAgICByZXR1cm4gZmFsc2U7Ci0gICAg
aWYgKGVsZW1lbnQtPmlzUmVhZE9ubHlGb3JtQ29udHJvbCgpKQotICAgICAgICByZXR1cm4gZmFs
c2U7CiAgICAgaWYgKGVsZW1lbnQtPmhhc0F1dG9mb2N1c2VkKCkpCiAgICAgICAgIHJldHVybiBm
YWxzZTsKIApJbmRleDogTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIExheW91
dFRlc3RzL0NoYW5nZUxvZwkocmV2aXNpb24gOTYwODIpCisrKyBMYXlvdXRUZXN0cy9DaGFuZ2VM
b2cJKHdvcmtpbmcgY29weSkKQEAgLTEsMyArMSwxMyBAQAorMjAxMS0wOS0yNiAgS2F1c3R1Ymgg
QXRyYXdhbGthciAgPGthdXN0dWJoQG1vdG9yb2xhLmNvbT4KKworICAgICAgICBBdXRvZm9jdXMg
b24gcmVhZG9ubHkgaW5wdXRzIGRvZXMgbm90IGZvY3VzIHRoZSBlbGVtZW50LgorICAgICAgICBo
dHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MjQwOTIKKworICAgICAgICBS
ZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAqIGZhc3QvZm9ybXMvYXV0b2Zv
Y3VzLXJlYWRvbmx5LWF0dHJpYnV0ZS1leHBlY3RlZC50eHQ6IEFkZGVkLgorICAgICAgICAqIGZh
c3QvZm9ybXMvYXV0b2ZvY3VzLXJlYWRvbmx5LWF0dHJpYnV0ZS5odG1sOiBBZGRlZC4KKwogMjAx
MS0wOS0yNyAgQXJ1biBQYXRvbGUgIDxibWY4MzRAbW90b3JvbGEuY29tPgogCiAgICAgICAgIEF1
ZGlvIGVsZW1lbnQgZG9lc24ndCBlbWl0IHRoZSAncGxheWluZycgZXZlbnQgZXZlcnkgdGltZSBp
dCBzdGFydHMgcGxheWluZywgYWZ0ZXIgaXQgaGFzIGZpbmlzaGVkIHBsYXlpbmcuCkluZGV4OiBM
YXlvdXRUZXN0cy9mYXN0L2Zvcm1zL2F1dG9mb2N1cy1yZWFkb25seS1hdHRyaWJ1dGUtZXhwZWN0
ZWQudHh0Cj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3RzL2Zhc3QvZm9ybXMvYXV0b2ZvY3VzLXJl
YWRvbmx5LWF0dHJpYnV0ZS1leHBlY3RlZC50eHQJKHJldmlzaW9uIDApCisrKyBMYXlvdXRUZXN0
cy9mYXN0L2Zvcm1zL2F1dG9mb2N1cy1yZWFkb25seS1hdHRyaWJ1dGUtZXhwZWN0ZWQudHh0CShy
ZXZpc2lvbiAwKQpAQCAtMCwwICsxLDQgQEAKK1RoZSB0ZXh0IGJveCBzaG91bGQgaGF2ZSBmb2N1
cyBjb25zb2xlIHNob3VsZCBwcmludCBQQVNTCisKK1BBU1MKKwpJbmRleDogTGF5b3V0VGVzdHMv
ZmFzdC9mb3Jtcy9hdXRvZm9jdXMtcmVhZG9ubHktYXR0cmlidXRlLmh0bWwKPT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQot
LS0gTGF5b3V0VGVzdHMvZmFzdC9mb3Jtcy9hdXRvZm9jdXMtcmVhZG9ubHktYXR0cmlidXRlLmh0
bWwJKHJldmlzaW9uIDApCisrKyBMYXlvdXRUZXN0cy9mYXN0L2Zvcm1zL2F1dG9mb2N1cy1yZWFk
b25seS1hdHRyaWJ1dGUuaHRtbAkocmV2aXNpb24gMCkKQEAgLTAsMCArMSwxOSBAQAorPCFET0NZ
VFBFIGh0bWw+Cis8aHRtbD4KKzxoZWFkPgorPHNjcmlwdD4KK2lmICh3aW5kb3cubGF5b3V0VGVz
dENvbnRyb2xsZXIpCisgICAgbGF5b3V0VGVzdENvbnRyb2xsZXIuZHVtcEFzVGV4dCgpOworCitm
dW5jdGlvbiBsb2cobXNnKSB7CisgICAgdmFyIGNvbnNvbGUgPSBkb2N1bWVudC5nZXRFbGVtZW50
QnlJZCgiY29uc29sZSIpOworICAgIGNvbnNvbGUuaW5uZXJUZXh0ID0gbXNnOworfQorPC9zY3Jp
cHQ+Cis8L2hlYWQ+Cis8Ym9keT4KKzxwPlRoZSB0ZXh0IGJveCBzaG91bGQgaGF2ZSBmb2N1cyBj
b25zb2xlIHNob3VsZCBwcmludCBQQVNTPC9wPgorPGRpdiBpZD0iY29uc29sZSI+PC9kaXY+Cis8
cD48aW5wdXQgdHlwZT0idGV4dCIgbmFtZT0idGVzdCIgaWQ9InRlc3QiIHJlYWRvbmx5IGF1dG9m
b2N1cyBvbmZvY3VzPSdsb2coIlBBU1MiKSc+PC9wPgorPC9ib2R5PgorPC9odG1sPgo=
</data>

          </attachment>
      

    </bug>

</bugzilla>