<?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>12506</bug_id>
          
          <creation_ts>2007-01-31 09:48:47 -0800</creation_ts>
          <short_desc>REGRESSION: Safari doesn&apos;t display hebrew text on a web page, displayed correctly on Tiger</short_desc>
          <delta_ts>2007-02-01 22:03:23 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Text</component>
          <version>420+</version>
          <rep_platform>Mac</rep_platform>
          <op_sys>OS X 10.4</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar, Regression</keywords>
          <priority>P1</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Darin Adler">darin</reporter>
          <assigned_to name="Alexey Proskuryakov">ap</assigned_to>
          <cc>ap</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>29732</commentid>
    <comment_count>0</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2007-01-31 09:48:47 -0800</bug_when>
    <thetext>&lt;GMT14-Nov-2006 21:30:26GMT&gt; Dary Mihova:
* SUMMARY:
The Hebrew text on the web page 
http://www.icellcom.co.il/javagames/
is displayed unreadable from Leopard 9a302 Safari.
The same web page is displayed without any problems on Tiger Safari 2.0.4 (419.3).

* STEPS TO REPRODUCE:
Install Tiger 10.48.
Open Safari and go to the above URL =&gt; the Hebrew text is displayed correctly (the left side of the attached screenshot)

Install Leopard 9a302
Open Safari and go to the above URL =&gt; the Hebrew text is broken (the right side of the attached screenshot)

&lt;GMT16-Nov-2006 17:34:00GMT&gt; Dave Wiley:
FYI

&lt;GMT16-Nov-2006 20:14:27GMT&gt; Ping Huey:
Active on 9A305.

&lt;GMT27-Nov-2006 21:49:19GMT&gt; Stephanie Lewis:
Safari BRB Reviewed

&lt;GMT29-Nov-2006 22:44:18GMT&gt; Oliver Hunt:
This looks like it could be us incorrectly assuming ASCII over UTF-8, which seems odd

&lt;GMT01-Dec-2006 21:24:35GMT&gt; Jill Surdzial:
Intl BRB:  Setting Intl Blocker/P1.

&lt;GMT15-Jan-2007 22:18:27GMT&gt; Alice Liu:
Safari beta blocker reviewed

&lt;GMT26-Jan-2007 01:56:51GMT&gt; Darin Adler:
I don&apos;t think it&apos;s right to say that this is blocked by bug 4892428.

This specific bug is happening because we&apos;re confused by text in the a &lt;meta&gt; tag that looks like a tag to us. We could easily fix this specific problem and I&apos;d like to see why this is a regression from the old code. Bug 4892428 is a far broader one and not a regression.

&lt;GMT26-Jan-2007 01:57:18GMT&gt; Darin Adler:
The key question here is: why did the Tiger version of the charset sniffer work better on this page?

&lt;rdar://problem/4836738&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>29721</commentid>
    <comment_count>1</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2007-01-31 10:15:48 -0800</bug_when>
    <thetext>(In reply to comment #0)
&gt; The key question here is: why did the Tiger version of the charset sniffer work
&gt; better on this page?

In Tiger, all unknown tags were allowed in HEAD:

  int id = khtml::getTagID(tmp, len);
  if(end) id += ID_CLOSE_TAG;
...
  case (ID_META+ID_CLOSE_TAG):
  case ID_SCRIPT:
  case (ID_SCRIPT+ID_CLOSE_TAG):
...other tags explicitly allowed in HEAD...
  case ID_HTML:
  case ID_HEAD:
  case 0:
  case (0 + ID_CLOSE_TAG ):
    break;
  default:
    body = true;

I don&apos;t think we want to re-add this behavior.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>29715</commentid>
    <comment_count>2</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2007-01-31 10:24:46 -0800</bug_when>
    <thetext>(In reply to comment #1)
&gt; I don&apos;t think we want to re-add this behavior.

At the same time, I suspect the change was non-intentional (likely made at the time when we switched from tag IDs to AtomicStrings).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>29710</commentid>
    <comment_count>3</comment_count>
      <attachid>12830</attachid>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2007-01-31 11:32:33 -0800</bug_when>
    <thetext>Created attachment 12830
proposed fix

This patch fixes more than the regression - known tags hidden in attributes don&apos;t fool the sniffer anymore, too.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>29698</commentid>
    <comment_count>4</comment_count>
      <attachid>12830</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2007-01-31 12:12:31 -0800</bug_when>
    <thetext>Comment on attachment 12830
proposed fix

I believe that the HTML parser is tolerant of spurious quote marks in tags attribute lists -- see line 927 of HTMLTokenizer.cpp, the part that says &quot;ignore any quotes we encounter and treat them like spaces&quot;. This code is strict about that. I&apos;d like to see some more test cases that cover these strange quoting cases to see if there are any repercussions of that.

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>29530</commentid>
    <comment_count>5</comment_count>
    <who name="Sam Weinig">sam</who>
    <bug_when>2007-01-31 20:36:06 -0800</bug_when>
    <thetext>Landed in r19324.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>29512</commentid>
    <comment_count>6</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2007-01-31 21:21:43 -0800</bug_when>
    <thetext>I&apos;m going to look into how the real parser handles quote marks; will re-open if the fix needs to be corrected.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>29095</commentid>
    <comment_count>7</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2007-02-01 09:37:14 -0800</bug_when>
    <thetext>Turns out the only reason the test case was succeeding was a bug in the code! The patch did not fix the bug, as demonstrated by the layout test.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>29096</commentid>
    <comment_count>8</comment_count>
      <attachid>12830</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2007-02-01 09:38:44 -0800</bug_when>
    <thetext>Comment on attachment 12830
proposed fix

Clearing review flag on this, since it was already landed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>29100</commentid>
    <comment_count>9</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2007-02-01 10:03:16 -0800</bug_when>
    <thetext>(In reply to comment #7)
&gt; Turns out the only reason the test case was succeeding was a bug in the code!
&gt; The patch did not fix the bug, as demonstrated by the layout test.

I have a fix for this; looking into whether the issue with spurious quote marks can be fixed without copying most of HTMLTokenizer.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>29107</commentid>
    <comment_count>10</comment_count>
      <attachid>12857</attachid>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2007-02-01 11:16:12 -0800</bug_when>
    <thetext>Created attachment 12857
fix the fix

The spurious quotes quirk is very context-dependent, I don&apos;t see a way to support it without stateful code similar to that found in HTMLTokenizer. Maybe we should find a way to re-use that.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>29146</commentid>
    <comment_count>11</comment_count>
      <attachid>12857</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2007-02-01 18:14:16 -0800</bug_when>
    <thetext>Comment on attachment 12857
fix the fix

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>29161</commentid>
    <comment_count>12</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2007-02-01 22:03:23 -0800</bug_when>
    <thetext>Committed revision 19348.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>12830</attachid>
            <date>2007-01-31 11:32:33 -0800</date>
            <delta_ts>2007-02-01 09:38:44 -0800</delta_ts>
            <desc>proposed fix</desc>
            <filename>12506r1_patch.txt</filename>
            <type>text/plain</type>
            <size>4678</size>
            <attacher name="Alexey Proskuryakov">ap</attacher>
            
              <data encoding="base64">SW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9D
aGFuZ2VMb2cJKHJldmlzaW9uIDE5MzA2KQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3Jr
aW5nIGNvcHkpCkBAIC0xLDMgKzEsMTQgQEAKKzIwMDctMDEtMzEgIEFsZXhleSBQcm9za3VyeWFr
b3YgIDxhcEB3ZWJraXQub3JnPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEp
LgorCisgICAgICAgIGh0dHA6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTEyNTA2
CisgICAgICAgIFJFR1JFU1NJT046IFNhZmFyaSBkb2Vzbid0IGRpc3BsYXkgaGVicmV3IHRleHQg
b24gYSB3ZWIgcGFnZSwKKyAgICAgICAgZGlzcGxheWVkIGNvcnJlY3RseSBvbiBUaWdlcgorCisg
ICAgICAgICogZmFzdC9lbmNvZGluZy9wc2V1ZG8tdGFncy1pbi1hdHRyaWJ1dGVzLWV4cGVjdGVk
LnR4dDogQWRkZWQuCisgICAgICAgICogZmFzdC9lbmNvZGluZy9wc2V1ZG8tdGFncy1pbi1hdHRy
aWJ1dGVzLmh0bWw6IEFkZGVkLgorCiAyMDA3LTAxLTMwICBEYXZpZCBLaWx6ZXIgIDxkZGtpbHpl
ckB3ZWJraXQub3JnPgogCiAgICAgICAgIFJldmlld2VkIGJ5IERhcmluLgpJbmRleDogTGF5b3V0
VGVzdHMvZmFzdC9lbmNvZGluZy9wc2V1ZG8tdGFncy1pbi1hdHRyaWJ1dGVzLWV4cGVjdGVkLnR4
dAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9mYXN0L2VuY29kaW5nL3BzZXVkby10YWdzLWlu
LWF0dHJpYnV0ZXMtZXhwZWN0ZWQudHh0CShyZXZpc2lvbiAwKQorKysgTGF5b3V0VGVzdHMvZmFz
dC9lbmNvZGluZy9wc2V1ZG8tdGFncy1pbi1hdHRyaWJ1dGVzLWV4cGVjdGVkLnR4dAkocmV2aXNp
b24gMCkKQEAgLTAsMCArMSw3IEBACitUZXN0IGZvciBidWcgMTI1MDY6IFJFR1JFU1NJT046IFNh
ZmFyaSBkb2Vzbid0IGRpc3BsYXkgaGVicmV3IHRleHQgb24gYSB3ZWIgcGFnZSwgZGlzcGxheWVk
IGNvcnJlY3RseSBvbiBUaWdlci4KKworU1XQodCh0JVTUworCitDaGFyc2V0OiBVVEYtOAorCisK
ClByb3BlcnR5IGNoYW5nZXMgb246IExheW91dFRlc3RzL2Zhc3QvZW5jb2RpbmcvcHNldWRvLXRh
Z3MtaW4tYXR0cmlidXRlcy1leHBlY3RlZC50eHQKX19fX19fX19fX19fX19fX19fX19fX19fX19f
X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpOYW1lOiBzdm46bWltZS10
eXBlCiAgICsgdGV4dC9wbGFpbgpOYW1lOiBzdm46ZW9sLXN0eWxlCiAgICsgbmF0aXZlCgpJbmRl
eDogTGF5b3V0VGVzdHMvZmFzdC9lbmNvZGluZy9wc2V1ZG8tdGFncy1pbi1hdHRyaWJ1dGVzLmh0
bWwKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvZmFzdC9lbmNvZGluZy9wc2V1ZG8tdGFncy1p
bi1hdHRyaWJ1dGVzLmh0bWwJKHJldmlzaW9uIDApCisrKyBMYXlvdXRUZXN0cy9mYXN0L2VuY29k
aW5nL3BzZXVkby10YWdzLWluLWF0dHJpYnV0ZXMuaHRtbAkocmV2aXNpb24gMCkKQEAgLTAsMCAr
MSwyMCBAQAorPEhUTUw+CisgIDxIRUFEPgorICAgIDxUSVRMRT48L1RJVExFPgorICAgIDxMSU5L
IHJlbD0ic3R5bGVzaGVldCIgdHlwZT0iPGZvb2Jhcj48PiIgaHJlZj0iIj4KKyAgICA8bWV0YSBu
YW1lPSJrZXl3b3JkcyIgY29udGVudD0iPGJvZHk+LCA8Zm9vYmFyPiwgPD4iPgorICAgIDxNRVRB
IEhUVFAtRVFVSVY9IkNvbnRlbnQtVHlwZSIgQ09OVEVOVD0idGV4dC9odG1sIiBjaGFyc2V0PSJ1
dGYtOCI+CisKKyAgPC9IRUFEPgorICA8Qk9EWT4KKyAgPFA+VGVzdCBmb3IgPGEgaHJlZj0iaHR0
cDovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTI1MDYiPmJ1ZyAxMjUwNjwvYT46
CisgIFJFR1JFU1NJT046IFNhZmFyaSBkb2Vzbid0IGRpc3BsYXkgaGVicmV3IHRleHQgb24gYSB3
ZWIgcGFnZSwgZGlzcGxheWVkIGNvcnJlY3RseSBvbiBUaWdlci48L1A+CisgIDxQPlNV0KHQodCV
U1M8L1A+CisgIAorICA8U0NSSVBUPgorICBkb2N1bWVudC53cml0ZSgiPHA+Q2hhcnNldDogIiAr
IGRvY3VtZW50LmlucHV0RW5jb2RpbmcgKyAiPC9wPiIpOworICBpZiAod2luZG93LmxheW91dFRl
c3RDb250cm9sbGVyKQorICAgIGxheW91dFRlc3RDb250cm9sbGVyLmR1bXBBc1RleHQoKTsKKyAg
PC9TQ1JJUFQ+CisgIDwvQk9EWT4KKzwvSFRNTD4KClByb3BlcnR5IGNoYW5nZXMgb246IExheW91
dFRlc3RzL2Zhc3QvZW5jb2RpbmcvcHNldWRvLXRhZ3MtaW4tYXR0cmlidXRlcy5odG1sCl9fX19f
X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f
X19fX18KTmFtZTogc3ZuOm1pbWUtdHlwZQogICArIHRleHQvaHRtbAoKSW5kZXg6IFdlYkNvcmUv
Q2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9nCShyZXZpc2lvbiAxOTMw
NikKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTUgQEAK
KzIwMDctMDEtMzEgIEFsZXhleSBQcm9za3VyeWFrb3YgIDxhcEB3ZWJraXQub3JnPgorCisgICAg
ICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIGh0dHA6Ly9idWdzLndl
YmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTEyNTA2CisgICAgICAgIFJFR1JFU1NJT046IFNhZmFy
aSBkb2Vzbid0IGRpc3BsYXkgaGVicmV3IHRleHQgb24gYSB3ZWIgcGFnZSwKKyAgICAgICAgZGlz
cGxheWVkIGNvcnJlY3RseSBvbiBUaWdlcgorCisgICAgICAgICogbG9hZGVyL1RleHRSZXNvdXJj
ZURlY29kZXIuY3BwOgorICAgICAgICAoV2ViQ29yZTo6VGV4dFJlc291cmNlRGVjb2Rlcjo6Y2hl
Y2tGb3JIZWFkQ2hhcnNldCk6IFNraXAgb3ZlciAKKyAgICAgICAgcXVvdGVkIGF0dHJpYnV0ZXMu
CisKIDIwMDctMDEtMzEgIERhcmluIEFkbGVyICA8ZGFyaW5AYXBwbGUuY29tPgogCiAgICAgICAg
IC0gZml4IGJ1aWxkCkluZGV4OiBXZWJDb3JlL2xvYWRlci9UZXh0UmVzb3VyY2VEZWNvZGVyLmNw
cAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09Ci0tLSBXZWJDb3JlL2xvYWRlci9UZXh0UmVzb3VyY2VEZWNvZGVyLmNwcAko
cmV2aXNpb24gMTkzMDYpCisrKyBXZWJDb3JlL2xvYWRlci9UZXh0UmVzb3VyY2VEZWNvZGVyLmNw
cAkod29ya2luZyBjb3B5KQpAQCAtNTU5LDEzICs1NTksMjYgQEAgYm9vbCBUZXh0UmVzb3VyY2VE
ZWNvZGVyOjpjaGVja0ZvckhlYWRDaAogICAgICAgICAgICAgICAgICAgICBlbmNsb3NpbmdUYWdO
YW1lID0gbm9zY3JpcHRUYWcubG9jYWxOYW1lKCkuaW1wbCgpOwogICAgICAgICAgICAgfQogICAg
ICAgICAgICAgCisgICAgICAgICAgICAvLyBGaW5kIHdoZXJlIHRoZSBvcGVuaW5nIHRhZyBlbmRz
LgorICAgICAgICAgICAgY29uc3QgY2hhciogdGFnQ29udGVudFN0YXJ0ID0gcHRyOworICAgICAg
ICAgICAgaWYgKCFlbmQpIHsKKyAgICAgICAgICAgICAgICB3aGlsZSAoKnB0ciAhPSAnPicpIHsK
KyAgICAgICAgICAgICAgICAgICAgaWYgKCpwdHIgPT0gJ1wwJykKKyAgICAgICAgICAgICAgICAg
ICAgICAgIHJldHVybiBmYWxzZTsKKyAgICAgICAgICAgICAgICAgICAgaWYgKCpwdHIgPT0gJ1wn
JyB8fCAqcHRyID09ICciJykgeworICAgICAgICAgICAgICAgICAgICAgICAgY2hhciBxdW90ZU1h
cmsgPSAqcHRyOworICAgICAgICAgICAgICAgICAgICAgICAgd2hpbGUgKCpwdHIgIT0gcXVvdGVN
YXJrKSB7CisgICAgICAgICAgICAgICAgICAgICAgICAgICAgaWYgKCpwdHIgPT0gJ1wwJykKKyAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgcmV0dXJuIGZhbHNlOworICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICsrcHRyOworICAgICAgICAgICAgICAgICAgICAgICAgfQorICAgICAg
ICAgICAgICAgICAgICB9CisgICAgICAgICAgICAgICAgICAgICsrcHRyOworICAgICAgICAgICAg
ICAgIH0KKyAgICAgICAgICAgIH0KKyAgICAgICAgICAgIAogICAgICAgICAgICAgaWYgKCFlbmQg
JiYgdGFnID09IG1ldGFUYWcpIHsKLSAgICAgICAgICAgICAgICBjb25zdCBjaGFyKiBlbmQgPSBw
dHI7Ci0gICAgICAgICAgICAgICAgd2hpbGUgKCplbmQgIT0gJz4nICYmICplbmQgIT0gJ1wwJykK
LSAgICAgICAgICAgICAgICAgICAgZW5kKys7Ci0gICAgICAgICAgICAgICAgaWYgKCplbmQgPT0g
J1wwJykKLSAgICAgICAgICAgICAgICAgICAgYnJlYWs7Ci0gICAgICAgICAgICAgICAgRGVwcmVj
YXRlZENTdHJpbmcgc3RyKHB0ciwgKGVuZC1wdHIpKzEpOworICAgICAgICAgICAgICAgIERlcHJl
Y2F0ZWRDU3RyaW5nIHN0cih0YWdDb250ZW50U3RhcnQsIHRhZ0NvbnRlbnRTdGFydCAtIHB0cik7
CiAgICAgICAgICAgICAgICAgc3RyID0gc3RyLmxvd2VyKCk7CiAgICAgICAgICAgICAgICAgaW50
IHBvcyA9IDA7CiAgICAgICAgICAgICAgICAgd2hpbGUgKHBvcyA8IChpbnQpc3RyLmxlbmd0aCgp
KSB7Cg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>12857</attachid>
            <date>2007-02-01 11:16:12 -0800</date>
            <delta_ts>2007-02-01 18:14:16 -0800</delta_ts>
            <desc>fix the fix</desc>
            <filename>12506_3.txt</filename>
            <type>text/plain</type>
            <size>2487</size>
            <attacher name="Alexey Proskuryakov">ap</attacher>
            
              <data encoding="base64">SW5kZXg6IExheW91dFRlc3RzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBMYXlvdXRUZXN0cy9D
aGFuZ2VMb2cJKHJldmlzaW9uIDE5MzM5KQorKysgTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCSh3b3Jr
aW5nIGNvcHkpCkBAIC0xLDMgKzEsMTMgQEAKKzIwMDctMDItMDEgIEFsZXhleSBQcm9za3VyeWFr
b3YgIDxhcEB3ZWJraXQub3JnPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEp
LgorCisgICAgICAgIGh0dHA6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTEyNTA2
CisgICAgICAgIFJFR1JFU1NJT046IFNhZmFyaSBkb2Vzbid0IGRpc3BsYXkgaGVicmV3IHRleHQg
b24gYSB3ZWIgcGFnZSwKKyAgICAgICAgZGlzcGxheWVkIGNvcnJlY3RseSBvbiBUaWdlcgorCisg
ICAgICAgICogZmFzdC9lbmNvZGluZy9wc2V1ZG8tdGFncy1pbi1hdHRyaWJ1dGVzLWV4cGVjdGVk
LnR4dDogQ2hhbmdpbmcgYmFjayB0byBzdWNjZXNzLgorCiAyMDA3LTAyLTAxICBEYXJpbiBBZGxl
ciAgPGRhcmluQGFwcGxlLmNvbT4KIAogICAgICAgICAqIGZhc3QvZW5jb2RpbmcvcHNldWRvLXRh
Z3MtaW4tYXR0cmlidXRlcy1leHBlY3RlZC50eHQ6IENoYW5nZSB0ZXN0IHJlc3VsdCB0byByZWZs
ZWN0IGZhaWx1cmUKSW5kZXg6IExheW91dFRlc3RzL2Zhc3QvZW5jb2RpbmcvcHNldWRvLXRhZ3Mt
aW4tYXR0cmlidXRlcy1leHBlY3RlZC50eHQKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvZmFz
dC9lbmNvZGluZy9wc2V1ZG8tdGFncy1pbi1hdHRyaWJ1dGVzLWV4cGVjdGVkLnR4dAkocmV2aXNp
b24gMTkzMzkpCisrKyBMYXlvdXRUZXN0cy9mYXN0L2VuY29kaW5nL3BzZXVkby10YWdzLWluLWF0
dHJpYnV0ZXMtZXhwZWN0ZWQudHh0CSh3b3JraW5nIGNvcHkpCkBAIC0xLDcgKzEsNyBAQAogVGVz
dCBmb3IgYnVnIDEyNTA2OiBSRUdSRVNTSU9OOiBTYWZhcmkgZG9lc24ndCBkaXNwbGF5IGhlYnJl
dyB0ZXh0IG9uIGEgd2ViIHBhZ2UsIGRpc3BsYXllZCBjb3JyZWN0bHkgb24gVGlnZXIuCiAKLVNV
w5DCocOQwqHDkOKAolNTCitTVdCh0KHQlVNTCiAKLUNoYXJzZXQ6IElTTy04ODU5LTEKK0NoYXJz
ZXQ6IFVURi04CiAKIApJbmRleDogV2ViQ29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDE5MzM5KQorKysgV2ViQ29yZS9DaGFuZ2VMb2cJKHdv
cmtpbmcgY29weSkKQEAgLTEsMyArMSwxNCBAQAorMjAwNy0wMi0wMSAgQWxleGV5IFByb3NrdXJ5
YWtvdiAgPGFwQHdlYmtpdC5vcmc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BT
ISkuCisKKyAgICAgICAgaHR0cDovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTI1
MDYKKyAgICAgICAgUkVHUkVTU0lPTjogU2FmYXJpIGRvZXNuJ3QgZGlzcGxheSBoZWJyZXcgdGV4
dCBvbiBhIHdlYiBwYWdlLAorICAgICAgICBkaXNwbGF5ZWQgY29ycmVjdGx5IG9uIFRpZ2VyCisK
KyAgICAgICAgKiBsb2FkZXIvVGV4dFJlc291cmNlRGVjb2Rlci5jcHA6CisgICAgICAgIChXZWJD
b3JlOjpUZXh0UmVzb3VyY2VEZWNvZGVyOjpjaGVja0ZvckhlYWRDaGFyc2V0KTogQWR2YW5jZSB0
aGUgcG9pbnRlciB3aGVuIG5lY2Vzc2FyeS4KKwogMjAwNy0wMi0wMSAgWmFjayBSdXNpbiAgPHpy
dXNpbkB0cm9sbHRlY2guY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5IExhcnMKSW5kZXg6IFdl
YkNvcmUvbG9hZGVyL1RleHRSZXNvdXJjZURlY29kZXIuY3BwCj09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNv
cmUvbG9hZGVyL1RleHRSZXNvdXJjZURlY29kZXIuY3BwCShyZXZpc2lvbiAxOTMzOSkKKysrIFdl
YkNvcmUvbG9hZGVyL1RleHRSZXNvdXJjZURlY29kZXIuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC01
NjcsNiArNTY3LDcgQEAgYm9vbCBUZXh0UmVzb3VyY2VEZWNvZGVyOjpjaGVja0ZvckhlYWRDaAog
ICAgICAgICAgICAgICAgICAgICAgICAgcmV0dXJuIGZhbHNlOwogICAgICAgICAgICAgICAgICAg
ICBpZiAoKnB0ciA9PSAnXCcnIHx8ICpwdHIgPT0gJyInKSB7CiAgICAgICAgICAgICAgICAgICAg
ICAgICBjaGFyIHF1b3RlTWFyayA9ICpwdHI7CisgICAgICAgICAgICAgICAgICAgICAgICArK3B0
cjsKICAgICAgICAgICAgICAgICAgICAgICAgIHdoaWxlICgqcHRyICE9IHF1b3RlTWFyaykgewog
ICAgICAgICAgICAgICAgICAgICAgICAgICAgIGlmICgqcHRyID09ICdcMCcpCiAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgIHJldHVybiBmYWxzZTsK
</data>
<flag name="review"
          id="4926"
          type_id="1"
          status="+"
          setter="darin"
    />
          </attachment>
      

    </bug>

</bugzilla>