<?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>47901</bug_id>
          
          <creation_ts>2010-10-19 06:40:27 -0700</creation_ts>
          <short_desc>fast/canvas/canvas-getImageData-negative-source.html fails on Mac</short_desc>
          <delta_ts>2011-04-19 13:51:05 -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>Canvas</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Mac</rep_platform>
          <op_sys>OS X 10.5</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="Andreas Kling">kling</reporter>
          <assigned_to name="Mihai Parparita">mihaip</assigned_to>
          <cc>agl</cc>
    
    <cc>commit-queue</cc>
    
    <cc>jamesr</cc>
    
    <cc>jschuh</cc>
    
    <cc>mdelaney7</cc>
    
    <cc>mihaip</cc>
    
    <cc>senorblanco</cc>
    
    <cc>wjmaclean</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>296146</commentid>
    <comment_count>0</comment_count>
    <who name="Andreas Kling">kling</who>
    <bug_when>2010-10-19 06:40:27 -0700</bug_when>
    <thetext>fast/canvas/canvas-getImageData-negative-source.html fails on Mac.
It was added in &lt;http://trac.webkit.org/changeset/70050&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>296376</commentid>
    <comment_count>1</comment_count>
    <who name="James Robinson">jamesr</who>
    <bug_when>2010-10-19 13:47:42 -0700</bug_when>
    <thetext>Fails on chromium mac as well (but passes on other chromium platforms).  Looks like a CoreGraphics issue.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>319180</commentid>
    <comment_count>2</comment_count>
      <attachid>75998</attachid>
    <who name="Mihai Parparita">mihaip</who>
    <bug_when>2010-12-08 18:23:06 -0800</bug_when>
    <thetext>Created attachment 75998
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>319181</commentid>
    <comment_count>3</comment_count>
      <attachid>75998</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2010-12-08 18:25:49 -0800</bug_when>
    <thetext>Comment on attachment 75998
Patch

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

Looks fine.

&gt; WebCore/html/HTMLCanvasElement.cpp:349
&gt;  #if PLATFORM(SKIA)
&gt; -    if (wf &gt; MaxSkiaDim || hf &gt; MaxSkiaDim)
&gt; +    if (width &gt; MaxSkiaDim || height &gt; MaxSkiaDim)
&gt;          return IntSize();
&gt;  #endif

Really? Why does Skia need this at this level? Why doesn’t any other graphics system need this?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>319751</commentid>
    <comment_count>4</comment_count>
    <who name="Mihai Parparita">mihaip</who>
    <bug_when>2010-12-09 15:46:31 -0800</bug_when>
    <thetext>(In reply to comment #3)
&gt; Really? Why does Skia need this at this level? Why doesn’t any other graphics system need this?

This was done in http://trac.webkit.org/changeset/63219 because of a security bug (bug 41962) that I don&apos;t have permission to see, perhaps one of the people involved with that patch can say for sure.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>319779</commentid>
    <comment_count>5</comment_count>
    <who name="James Robinson">jamesr</who>
    <bug_when>2010-12-09 16:26:59 -0800</bug_when>
    <thetext>(In reply to comment #3)
&gt; (From update of attachment 75998 [details])
&gt; View in context: https://bugs.webkit.org/attachment.cgi?id=75998&amp;action=review
&gt; 
&gt; Looks fine.
&gt; 
&gt; &gt; WebCore/html/HTMLCanvasElement.cpp:349
&gt; &gt;  #if PLATFORM(SKIA)
&gt; &gt; -    if (wf &gt; MaxSkiaDim || hf &gt; MaxSkiaDim)
&gt; &gt; +    if (width &gt; MaxSkiaDim || height &gt; MaxSkiaDim)
&gt; &gt;          return IntSize();
&gt; &gt;  #endif
&gt; 
&gt; Really? Why does Skia need this at this level? Why doesn’t any other graphics system need this?

The short story here is that Skia&apos;s limitations are more strict than that of other graphics systems - every graphics system has a max area cap, but Skia also has a maximum value it can handle in any one dimension.  I think the decision was made to enforce the Skia limits in the same places that the limitations of other graphics systems are enforced rather than reducing everyone&apos;s limits to that of skia.  I agree it&apos;s kind of weird.

At this point, I&apos;m not sure if the limits for Skia are even necessary any more (we&apos;ve switched to using floating point math instead of fixed point math in some of Skia&apos;s internals) - so it&apos;s possible that we can just drop all of the Skia guards and stick the cross-platform limits.  Stephen or Adam would probably know if I&apos;m just imagining things here.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>319796</commentid>
    <comment_count>6</comment_count>
      <attachid>75998</attachid>
    <who name="Stephen White">senorblanco</who>
    <bug_when>2010-12-09 17:00:51 -0800</bug_when>
    <thetext>Comment on attachment 75998
Patch

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

&gt;&gt;&gt; WebCore/html/HTMLCanvasElement.cpp:349
&gt;&gt;&gt;  #endif
&gt;&gt; 
&gt;&gt; Really? Why does Skia need this at this level? Why doesn’t any other graphics system need this?
&gt; 
&gt; The short story here is that Skia&apos;s limitations are more strict than that of other graphics systems - every graphics system has a max area cap, but Skia also has a maximum value it can handle in any one dimension.  I think the decision was made to enforce the Skia limits in the same places that the limitations of other graphics systems are enforced rather than reducing everyone&apos;s limits to that of skia.  I agree it&apos;s kind of weird.
&gt; 
&gt; At this point, I&apos;m not sure if the limits for Skia are even necessary any more (we&apos;ve switched to using floating point math instead of fixed point math in some of Skia&apos;s internals) - so it&apos;s possible that we can just drop all of the Skia guards and stick the cross-platform limits.  Stephen or Adam would probably know if I&apos;m just imagining things here.

No, unfortunately we still need this.  See r63219 where this was added.

One way to make it less ugly would be to have a function or enum on GraphicsContext.h to retrieve the (platform-specific) maximum dimension, rather than having an #if PLATFORM here.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>319869</commentid>
    <comment_count>7</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-12-09 19:08:18 -0800</bug_when>
    <thetext>The commit-queue encountered the following flaky tests while processing attachment 75998:

http/tests/appcache/foreign-fallback.html
fast/css/font-face-download-error.html

Please file bugs against the tests.  These tests were authored by ap@webkit.org and yuzo@google.com.  The commit-queue is continuing to process your patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>319870</commentid>
    <comment_count>8</comment_count>
      <attachid>75998</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-12-09 19:09:31 -0800</bug_when>
    <thetext>Comment on attachment 75998
Patch

Rejecting patch 75998 from commit-queue.

Failed to run &quot;[&apos;./WebKitTools/Scripts/webkit-patch&apos;, &apos;--status-host=queues.webkit.org&apos;, &apos;--bot-id=eseidel-cq-sl&apos;, &apos;apply-attachment&apos;, &apos;--force-clean&apos;, &apos;--non-interactive&apos;, 75998]&quot; exit_code: 2
Last 500 characters of output:
s.txt
Hunk #1 FAILED at 2412.
Hunk #2 succeeded at 2919 (offset -15 lines).
Hunk #3 FAILED at 2991.
2 out of 3 hunks FAILED -- saving rejects to file LayoutTests/platform/chromium/test_expectations.txt.rej
patching file WebCore/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file WebCore/html/HTMLCanvasElement.cpp
patching file WebCore/html/HTMLCanvasElement.h

Failed to run &quot;[u&apos;/Projects/CommitQueue/WebKitTools/Scripts/svn-apply&apos;, u&apos;--reviewer&apos;, u&apos;Darin Adler&apos;, u&apos;--force&apos;]&quot; exit_code: 1

Full output: http://queues.webkit.org/results/6953031</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>320243</commentid>
    <comment_count>9</comment_count>
    <who name="Mihai Parparita">mihaip</who>
    <bug_when>2010-12-10 10:48:05 -0800</bug_when>
    <thetext>Committed r73744: &lt;http://trac.webkit.org/changeset/73744&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>388762</commentid>
    <comment_count>10</comment_count>
    <who name="Matthew Delaney">mdelaney7</who>
    <bug_when>2011-04-19 13:51:05 -0700</bug_when>
    <thetext>This test appears to be still on the platform/mac/Skipped list. If it&apos;s fixed for all/mac, it should be taken off all/mac skipped lists.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>75998</attachid>
            <date>2010-12-08 18:23:06 -0800</date>
            <delta_ts>2010-12-09 19:09:31 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-47901-20101208182304.patch</filename>
            <type>text/plain</type>
            <size>9940</size>
            <attacher name="Mihai Parparita">mihaip</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv
ZwppbmRleCBmNzlhOTY2MmU4NmFhOWJkN2UzNGE2MWU2ZTBkODkwNDY1YjlkZGNhLi4yNTlmMzFh
ZGZjZjJkNTEwNzNjZGE3Y2M5MzU5NmFjNDIwMzQzZTg5IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0
cy9DaGFuZ2VMb2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTcgQEAK
KzIwMTAtMTItMDggIE1paGFpIFBhcnBhcml0YSAgPG1paGFpcEBjaHJvbWl1bS5vcmc+CisKKyAg
ICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgZmFzdC9jYW52YXMv
Y2FudmFzLWdldEltYWdlRGF0YS1uZWdhdGl2ZS1zb3VyY2UuaHRtbCBmYWlscyBvbiBNYWMKKyAg
ICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTQ3OTAxCisgICAg
ICAgIAorICAgICAgICBBZGQgdGVzdCBmb3Igcm91bmRpbmcgYmVoYXZpb3IuIFJlbW92ZSBub3ct
cGFzc2luZyB0ZXN0cyBmcm9tCisgICAgICAgIHRlc3RfZXhwZWN0YXRpb25zLnR4dC4KKworICAg
ICAgICAqIGZhc3QvY2FudmFzL2NhbnZhcy1nZXRJbWFnZURhdGEtcm91bmRpbmctZXhwZWN0ZWQu
dHh0OiBBZGRlZC4KKyAgICAgICAgKiBmYXN0L2NhbnZhcy9jYW52YXMtZ2V0SW1hZ2VEYXRhLXJv
dW5kaW5nLmh0bWw6IEFkZGVkLgorICAgICAgICAqIHBsYXRmb3JtL2Nocm9taXVtL3Rlc3RfZXhw
ZWN0YXRpb25zLnR4dDoKKwogMjAxMC0xMi0wOCAgUnlvc3VrZSBOaXdhICA8cm5pd2FAd2Via2l0
Lm9yZz4KIAogICAgICAgICBVbnJldmlld2VkIENocm9taXVtIHRlc3QgZXhwZWN0YXRpb25zIHVw
ZGF0ZS4KZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL2Zhc3QvY2FudmFzL2NhbnZhcy1nZXRJbWFn
ZURhdGEtcm91bmRpbmctZXhwZWN0ZWQudHh0IGIvTGF5b3V0VGVzdHMvZmFzdC9jYW52YXMvY2Fu
dmFzLWdldEltYWdlRGF0YS1yb3VuZGluZy1leHBlY3RlZC50eHQKbmV3IGZpbGUgbW9kZSAxMDA2
NDQKaW5kZXggMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMC4uZTdlYTI4
YmI3NDdkNzNmZjVkMDVkMGQ2NDJjOGY3ZGFkOTIzNDYzYQotLS0gL2Rldi9udWxsCisrKyBiL0xh
eW91dFRlc3RzL2Zhc3QvY2FudmFzL2NhbnZhcy1nZXRJbWFnZURhdGEtcm91bmRpbmctZXhwZWN0
ZWQudHh0CkBAIC0wLDAgKzEsMzYgQEAKK1Rlc3QgdGhlIGhhbmRsaW5nIG9mIG5vbi1pbnRlZ2Vy
IHNvdXJjZSBjb29yZGluYXRlcyBpbiBnZXRJbWFnZURhdGEoKS4KKworT24gc3VjY2VzcywgeW91
IHdpbGwgc2VlIGEgc2VyaWVzIG9mICJQQVNTIiBtZXNzYWdlcywgZm9sbG93ZWQgYnkgIlRFU1Qg
Q09NUExFVEUiLgorCisKK2dldEltYWdlRGF0YSgwLCAwLCAyMCwgMTApCitQQVNTIGltYWdlRGF0
YS53aWR0aCBpcyAyMAorUEFTUyBpbWFnZURhdGEuaGVpZ2h0IGlzIDEwCitnZXRJbWFnZURhdGEo
MC4xLCAwLjIsIDIwLCAxMCkKK1BBU1MgaW1hZ2VEYXRhLndpZHRoIGlzIDIxCitQQVNTIGltYWdl
RGF0YS5oZWlnaHQgaXMgMTEKK2dldEltYWdlRGF0YSgwLjksIDAuOCwgMjAsIDEwKQorUEFTUyBp
bWFnZURhdGEud2lkdGggaXMgMjEKK1BBU1MgaW1hZ2VEYXRhLmhlaWdodCBpcyAxMQorZ2V0SW1h
Z2VEYXRhKDAsIDAsIDE5LjksIDkuOSkKK1BBU1MgaW1hZ2VEYXRhLndpZHRoIGlzIDIwCitQQVNT
IGltYWdlRGF0YS5oZWlnaHQgaXMgMTAKK2dldEltYWdlRGF0YSgwLCAwLCAxOS4xLCA5LjEpCitQ
QVNTIGltYWdlRGF0YS53aWR0aCBpcyAyMAorUEFTUyBpbWFnZURhdGEuaGVpZ2h0IGlzIDEwCitn
ZXRJbWFnZURhdGEoMC45LCAwLCAwLjIsIDEwKQorUEFTUyBpbWFnZURhdGEud2lkdGggaXMgMgor
UEFTUyBpbWFnZURhdGEuaGVpZ2h0IGlzIDEwCitnZXRJbWFnZURhdGEoLTEsIC0xLCAyMCwgMTAp
CitQQVNTIGltYWdlRGF0YS53aWR0aCBpcyAyMAorUEFTUyBpbWFnZURhdGEuaGVpZ2h0IGlzIDEw
CitnZXRJbWFnZURhdGEoLTEuMSwgMCwgMjAsIDEwKQorUEFTUyBpbWFnZURhdGEud2lkdGggaXMg
MjEKK1BBU1MgaW1hZ2VEYXRhLmhlaWdodCBpcyAxMAorZ2V0SW1hZ2VEYXRhKC0xLjksIDAsIDIw
LCAxMCkKK1BBU1MgaW1hZ2VEYXRhLndpZHRoIGlzIDIxCitQQVNTIGltYWdlRGF0YS5oZWlnaHQg
aXMgMTAKK1BBU1Mgc3VjY2Vzc2Z1bGx5UGFyc2VkIGlzIHRydWUKKworVEVTVCBDT01QTEVURQor
CmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9mYXN0L2NhbnZhcy9jYW52YXMtZ2V0SW1hZ2VEYXRh
LXJvdW5kaW5nLmh0bWwgYi9MYXlvdXRUZXN0cy9mYXN0L2NhbnZhcy9jYW52YXMtZ2V0SW1hZ2VE
YXRhLXJvdW5kaW5nLmh0bWwKbmV3IGZpbGUgbW9kZSAxMDA2NDQKaW5kZXggMDAwMDAwMDAwMDAw
MDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMC4uZmU0ZjA3ZjNhMDAxZDQ0NTVmYWZkMDkxZGFi
NGE1OWNiMDNiYzI5MwotLS0gL2Rldi9udWxsCisrKyBiL0xheW91dFRlc3RzL2Zhc3QvY2FudmFz
L2NhbnZhcy1nZXRJbWFnZURhdGEtcm91bmRpbmcuaHRtbApAQCAtMCwwICsxLDQ4IEBACis8IURP
Q1RZUEUgaHRtbD4KKzxodG1sPgorPGhlYWQ+Cis8bGluayByZWw9InN0eWxlc2hlZXQiIGhyZWY9
Ii4uL2pzL3Jlc291cmNlcy9qcy10ZXN0LXN0eWxlLmNzcyI+Cis8c2NyaXB0IHNyYz0iLi4vanMv
cmVzb3VyY2VzL2pzLXRlc3QtcHJlLmpzIj48L3NjcmlwdD4KKzwvaGVhZD4KKzxib2R5PgorPHAg
aWQ9ImRlc2NyaXB0aW9uIj48L3A+Cis8ZGl2IGlkPSJjb25zb2xlIj48L2Rpdj4KKzxzY3JpcHQ+
CitkZXNjcmlwdGlvbigiVGVzdCB0aGUgaGFuZGxpbmcgb2Ygbm9uLWludGVnZXIgc291cmNlIGNv
b3JkaW5hdGVzIGluIGdldEltYWdlRGF0YSgpLiIpOworCitjdHggPSBkb2N1bWVudC5jcmVhdGVF
bGVtZW50KCdjYW52YXMnKS5nZXRDb250ZXh0KCcyZCcpOworCitmdW5jdGlvbiBkaW1lbnNpb25z
U2hvdWxkQmUoc3gsIHN5LCBzdywgc2gsIHdpZHRoLCBoZWlnaHQpCit7CisgICAgaW1hZ2VEYXRh
ID0gY3R4LmdldEltYWdlRGF0YShzeCwgc3ksIHN3LCBzaCk7CisgICAgZGVidWcoJ2dldEltYWdl
RGF0YSgnICsgc3ggKyAnLCAnICsgc3kgKyAnLCAnICsgc3cgKyAnLCAnICsgc2ggKyAnKScpOwor
ICAgIHNob3VsZEJlKCdpbWFnZURhdGEud2lkdGgnLCB3aWR0aCArICcnKTsKKyAgICBzaG91bGRC
ZSgnaW1hZ2VEYXRhLmhlaWdodCcsIGhlaWdodCArICcnKTsKK30KKworLy8gQmFzaWMgaW50ZWdl
ciB2YWx1ZXMKK2RpbWVuc2lvbnNTaG91bGRCZSggICAwLCAgMCwgICAyMCwgIDEwLCAgICAgMjAs
IDEwKTsKKworLy8gU291cmNlIHBvaW50IGlzIG5vdCBhbiBpbnRlZ2VyCitkaW1lbnNpb25zU2hv
dWxkQmUoICAuMSwgLjIsICAgMjAsICAxMCwgICAgIDIxLCAxMSk7CitkaW1lbnNpb25zU2hvdWxk
QmUoICAuOSwgLjgsICAgMjAsICAxMCwgICAgIDIxLCAxMSk7CisKKy8vIFNpemUgaXMgbm90IGFu
IGludGVnZXIKK2RpbWVuc2lvbnNTaG91bGRCZSggICAwLCAgMCwgMTkuOSwgOS45LCAgICAgMjAs
IDEwKTsKK2RpbWVuc2lvbnNTaG91bGRCZSggICAwLCAgMCwgMTkuMSwgOS4xLCAgICAgMjAsIDEw
KTsKKworLy8gV2lkdGggc3RyYWRkbGVzIGEgcGl4ZWwgYm91bmRhcnkKK2RpbWVuc2lvbnNTaG91
bGRCZSggIC45LCAgMCwgICAuMiwgIDEwLCAgICAgIDIsIDEwKTsKKworLy8gQmFzaWMgaW50ZWdl
ciBuZWdhdGl2ZSB2YWx1ZXMKK2RpbWVuc2lvbnNTaG91bGRCZSggIC0xLCAtMSwgICAyMCwgIDEw
LCAgICAgMjAsIDEwKTsKKworLy8gTm9uLWludGVnZXIgbmVnYXRpdmUgdmFsdWVzCitkaW1lbnNp
b25zU2hvdWxkQmUoLTEuMSwgIDAsICAgMjAsICAxMCwgICAgIDIxLCAxMCk7CitkaW1lbnNpb25z
U2hvdWxkQmUoLTEuOSwgIDAsICAgMjAsICAxMCwgICAgIDIxLCAxMCk7CisKK3N1Y2Nlc3NmdWxs
eVBhcnNlZCA9IHRydWU7Cis8L3NjcmlwdD4KKzxzY3JpcHQgc3JjPSIuLi9qcy9yZXNvdXJjZXMv
anMtdGVzdC1wb3N0LmpzIj48L3NjcmlwdD4KKzwvYm9keT4KKzwvaHRtbD4KZGlmZiAtLWdpdCBh
L0xheW91dFRlc3RzL3BsYXRmb3JtL2Nocm9taXVtL3Rlc3RfZXhwZWN0YXRpb25zLnR4dCBiL0xh
eW91dFRlc3RzL3BsYXRmb3JtL2Nocm9taXVtL3Rlc3RfZXhwZWN0YXRpb25zLnR4dAppbmRleCA4
YTIzNGMxMTliZDBkYzE2YzA0MjYwN2Y0M2RiMTU5M2Y1OTg0MDJhLi5mY2FlNWRmNjhjOTI4MzJm
MDFjYTYwN2VjMmRiMzZiZWU2ZWZjMWEzIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9wbGF0Zm9y
bS9jaHJvbWl1bS90ZXN0X2V4cGVjdGF0aW9ucy50eHQKKysrIGIvTGF5b3V0VGVzdHMvcGxhdGZv
cm0vY2hyb21pdW0vdGVzdF9leHBlY3RhdGlvbnMudHh0CkBAIC0yNDEyLDggKzI0MTIsNiBAQCBC
VUdXSzQ1OTkxIE1BQyA6IGNhbnZhcy9waGlsaXAvdGVzdHMvMmQuc3Ryb2tlUmVjdC56ZXJvLjQu
aHRtbCA9IFRFWFQgUEFTUwogQlVHV0s0NTk5MSBNQUMgOiBjYW52YXMvcGhpbGlwL3Rlc3RzLzJk
LnRleHQuZHJhdy5iYXNlbGluZS5pZGVvZ3JhcGhpYy5odG1sID0gVEVYVCBQQVNTCiBCVUdXSzQ1
OTkxIE1BQyA6IGNhbnZhcy9waGlsaXAvdGVzdHMvdG9EYXRhVVJMLmpwZWcuYWxwaGEuaHRtbCA9
IFRFWFQgUEFTUwogCi1CVUdXSzQ1OTkxIE1BQyA6IGNhbnZhcy9waGlsaXAvdGVzdHMvMmQuaW1h
Z2VEYXRhLmdldC5zb3VyY2Uub3V0c2lkZS5odG1sID0gVEVYVAotCiAvLyBUaGlzIHNlZW1zIHRv
IGJlIGZhaWxpbmcgb24gd2luZG93cyBvbmx5LgogQlVHV0s0NTk5MSBXSU4gOiBjYW52YXMvcGhp
bGlwL3Rlc3RzLzJkLmdyYWRpZW50LnJhZGlhbC50b3VjaDIuaHRtbCA9IFRFWFQKIApAQCAtMjkz
Niw4ICsyOTM0LDYgQEAgQlVHNDc1NzggOiBwcmludGluZy9zaW11bHRhbmVvdXMtcG9zaXRpb24t
ZmxvYXQtY2hhbmdlLmh0bWwgPSBJTUFHRStURVhUCiAKIEJVRzU5NjQ3IFdJTiA6IHN2Zy9kb20v
bGVuZ3RoLWxpc3QtcGFyc2VyLmh0bWwgPSBQQVNTIENSQVNICiAKLUJVR1dLNDc5MDEgTUFDIDog
ZmFzdC9jYW52YXMvY2FudmFzLWdldEltYWdlRGF0YS1uZWdhdGl2ZS1zb3VyY2UuaHRtbCA9IFRF
WFQKLQogLy8gV2UgZG9uJ3Qgc3VwcG9ydCBsYXlvdXRUZXN0Q29udHJvbGxlci5ub2Rlc0Zyb21S
ZWN0IHNvIHRoaXMgdGVzdCBqdXN0IHRpbWVzIG91dAogQlVHV0s0NjYwMCBTS0lQIDogZmFzdC9k
b20vbm9kZXNGcm9tUmVjdC1saW5rcy1hbmQtdGV4dC5odG1sID0gVEVYVAogQlVHV0s0NjYwMCBT
S0lQIDogZmFzdC9kb20vbm9kZXNGcm9tUmVjdC1pbm5lci1kb2N1bWVudHMuaHRtbCA9IFRFWFQK
QEAgLTMwMTAsNyArMzAwNiw2IEBAIEJVR1dLNjEyNDYgOiBmYXN0L2pzL2ludGVycHJldGVyLW5v
LWFjdGl2YXRpb24uaHRtbCA9IFRFWFQKIEJVR1dLNjEyNjAgOiBtZWRpYS9tZWRpYS1jYW4tcGxh
eS13YXYtYXVkaW8uaHRtbCA9IFRFWFQKIAogLy8gRmFpbHVyZXMgYWZ0ZXIgV2ViS2l0IHJvbGwg
NzA5NDk6NzEwMDAKLUJVR1dLNzEwMDAgTUFDIDogY2FudmFzL3BoaWxpcC90ZXN0cy8yZC5pbWFn
ZURhdGEuZ2V0LnNvdXJjZS5uZWdhdGl2ZS5odG1sID0gVEVYVAogQlVHNjE0MDYgOiBzdmcvZHlu
YW1pYy11cGRhdGVzL1NWR0ZFQ29sb3JNYXRyaXhFbGVtZW50LXN2Z2RvbS12YWx1ZXMtcHJvcC5o
dG1sID0gSU1BR0UrVEVYVAogQlVHNjE0MDYgOiBzdmcvZHluYW1pYy11cGRhdGVzL1NWR0ZFQ29s
b3JNYXRyaXhFbGVtZW50LXN2Z2RvbS10eXBlLXByb3AuaHRtbCA9IElNQUdFCiBCVUc2MTQwNiBX
SU4gTElOVVggOiBzdmcvZHluYW1pYy11cGRhdGVzL1NWR0ZFVHVyYnVsZW5jZUVsZW1lbnQtc3Zn
ZG9tLWJhc2VGcmVxdWVuY3ktcHJvcC5odG1sID0gSU1BR0UKZGlmZiAtLWdpdCBhL1dlYkNvcmUv
Q2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggMmE4Y2Y0ZTljZTc2YjM0ZDU2YmM4
YjJjZTcxOTY0NzgzZDJjOGJkMS4uNGI2ZjUyYmNjNzBlMzlkZjEyYzMzYzg1YTcyNDNhMWZlODNm
YTVkNiAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvV2ViQ29yZS9DaGFuZ2VM
b2cKQEAgLTEsMyArMSwyNCBAQAorMjAxMC0xMi0wOCAgTWloYWkgUGFycGFyaXRhICA8bWloYWlw
QGNocm9taXVtLm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKwor
ICAgICAgICBmYXN0L2NhbnZhcy9jYW52YXMtZ2V0SW1hZ2VEYXRhLW5lZ2F0aXZlLXNvdXJjZS5o
dG1sIGZhaWxzIG9uIE1hYworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1
Zy5jZ2k/aWQ9NDc5MDEKKworICAgICAgICBUZXN0OiBmYXN0L2NhbnZhcy9jYW52YXMtZ2V0SW1h
Z2VEYXRhLXJvdW5kaW5nLmh0bWwKKyAgICAgICAgCisgICAgICAgIHN0YXRpY19jYXN0PHVuc2ln
bmVkPiBpcyBnZW5lcmFsbHkgbm90IHdoYXQgd2Ugd2FudCBpbgorICAgICAgICBjb252ZXJ0TG9n
aWNhbFRvRGV2aWNlLiBJdCBwcm9kdWNlcyBpbmNvbnNpc3RlbnQgcmVzdWx0cyB3aGVuIGNvbXBp
bGluZworICAgICAgICBmb3IgMzItYml0IHZzLiA2NC1iaXQsIGFuZCBpbiBhbnkgY2FzZSB3ZSB3
ZXJlbid0IGdldHRpbmcgY29ycmVjdAorICAgICAgICByb3VuZGluZyBiZWhhdmlvciBmb3Igc291
cmNlIHJlY3RhbmdsZXMgKGUuZy4gd2Ugc2hvdWxkIGdldCBhIHNvdXJjZSByZWN0CisgICAgICAg
IG9mIHdpZHRoIDIgaWYgdGhlIHNvdXJjZSBYIGlzIDAuOSBhbmQgdGhlIHNvdXJjZSB3aWR0aCBp
cyAwLjIsIGJ1dCB3ZQorICAgICAgICB3ZXJlIGdldHRpbmcgb25seSBvbmUgb2Ygd2lkdGggMSku
CisKKyAgICAgICAgKiBodG1sL0hUTUxDYW52YXNFbGVtZW50LmNwcDoKKyAgICAgICAgKFdlYkNv
cmU6OkhUTUxDYW52YXNFbGVtZW50Ojpjb252ZXJ0TG9naWNhbFRvRGV2aWNlKToKKyAgICAgICAg
KFdlYkNvcmU6OkhUTUxDYW52YXNFbGVtZW50Ojpjb252ZXJ0VG9WYWxpZERldmljZVNpemUpOgor
ICAgICAgICAqIGh0bWwvSFRNTENhbnZhc0VsZW1lbnQuaDoKKwogMjAxMC0xMi0wOCAgQW5kZXJz
IENhcmxzc29uICA8YW5kZXJzY2FAYXBwbGUuY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5IERh
cmluIEFkbGVyLgpkaWZmIC0tZ2l0IGEvV2ViQ29yZS9odG1sL0hUTUxDYW52YXNFbGVtZW50LmNw
cCBiL1dlYkNvcmUvaHRtbC9IVE1MQ2FudmFzRWxlbWVudC5jcHAKaW5kZXggZmVhNzBkMTYzNWUw
NTFmMTU1MjRhNThmYmE0YTYyOWFjYzYzOGI2OC4uNDEzMGUzODJmZGY0MjBmZGYwN2RjMzg3YmMx
ZWExNjhmZjIxN2Q3NCAxMDA2NDQKLS0tIGEvV2ViQ29yZS9odG1sL0hUTUxDYW52YXNFbGVtZW50
LmNwcAorKysgYi9XZWJDb3JlL2h0bWwvSFRNTENhbnZhc0VsZW1lbnQuY3BwCkBAIC0zMjIsMzEg
KzMyMiwzMyBAQCBTdHJpbmcgSFRNTENhbnZhc0VsZW1lbnQ6OnRvRGF0YVVSTChjb25zdCBTdHJp
bmcmIG1pbWVUeXBlLCBjb25zdCBkb3VibGUqIHF1YWxpdAogCiBJbnRSZWN0IEhUTUxDYW52YXNF
bGVtZW50Ojpjb252ZXJ0TG9naWNhbFRvRGV2aWNlKGNvbnN0IEZsb2F0UmVjdCYgbG9naWNhbFJl
Y3QpIGNvbnN0CiB7Ci0gICAgcmV0dXJuIEludFJlY3QoY29udmVydExvZ2ljYWxUb0RldmljZShs
b2dpY2FsUmVjdC5sb2NhdGlvbigpKSwgY29udmVydExvZ2ljYWxUb0RldmljZShsb2dpY2FsUmVj
dC5zaXplKCkpKTsKKyAgICBmbG9hdCBsZWZ0ID0gZmxvb3JmKGxvZ2ljYWxSZWN0LmxlZnQoKSAq
IG1fcGFnZVNjYWxlRmFjdG9yKTsKKyAgICBmbG9hdCB0b3AgPSBmbG9vcmYobG9naWNhbFJlY3Qu
dG9wKCkgKiBtX3BhZ2VTY2FsZUZhY3Rvcik7CisgICAgZmxvYXQgcmlnaHQgPSBjZWlsZihsb2dp
Y2FsUmVjdC5yaWdodCgpICogbV9wYWdlU2NhbGVGYWN0b3IpOworICAgIGZsb2F0IGJvdHRvbSA9
IGNlaWxmKGxvZ2ljYWxSZWN0LmJvdHRvbSgpICogbV9wYWdlU2NhbGVGYWN0b3IpOworICAgIAor
ICAgIHJldHVybiBJbnRSZWN0KEludFBvaW50KGxlZnQsIHRvcCksIGNvbnZlcnRUb1ZhbGlkRGV2
aWNlU2l6ZShyaWdodCAtIGxlZnQsIGJvdHRvbSAtIHRvcCkpOwogfQogCiBJbnRTaXplIEhUTUxD
YW52YXNFbGVtZW50Ojpjb252ZXJ0TG9naWNhbFRvRGV2aWNlKGNvbnN0IEZsb2F0U2l6ZSYgbG9n
aWNhbFNpemUpIGNvbnN0CiB7Ci0gICAgZmxvYXQgd2YgPSBjZWlsZihsb2dpY2FsU2l6ZS53aWR0
aCgpICogbV9wYWdlU2NhbGVGYWN0b3IpOwotICAgIGZsb2F0IGhmID0gY2VpbGYobG9naWNhbFNp
emUuaGVpZ2h0KCkgKiBtX3BhZ2VTY2FsZUZhY3Rvcik7CisgICAgcmV0dXJuIGNvbnZlcnRUb1Zh
bGlkRGV2aWNlU2l6ZShsb2dpY2FsU2l6ZS53aWR0aCgpICogbV9wYWdlU2NhbGVGYWN0b3IsIGxv
Z2ljYWxTaXplLmhlaWdodCgpICogbV9wYWdlU2NhbGVGYWN0b3IpOworfQogCi0gICAgaWYgKCEo
d2YgPj0gMSAmJiBoZiA+PSAxICYmIHdmICogaGYgPD0gTWF4Q2FudmFzQXJlYSkpCitJbnRTaXpl
IEhUTUxDYW52YXNFbGVtZW50Ojpjb252ZXJ0VG9WYWxpZERldmljZVNpemUoZmxvYXQgd2lkdGgs
IGZsb2F0IGhlaWdodCkgY29uc3QKK3sKKyAgICB3aWR0aCA9IGNlaWxmKHdpZHRoKTsKKyAgICBo
ZWlnaHQgPSBjZWlsZihoZWlnaHQpOworICAgIAorICAgIGlmICh3aWR0aCA8IDEgfHwgaGVpZ2h0
IDwgMSB8fCB3aWR0aCAqIGhlaWdodCA+IE1heENhbnZhc0FyZWEpCiAgICAgICAgIHJldHVybiBJ
bnRTaXplKCk7CiAKICNpZiBQTEFURk9STShTS0lBKQotICAgIGlmICh3ZiA+IE1heFNraWFEaW0g
fHwgaGYgPiBNYXhTa2lhRGltKQorICAgIGlmICh3aWR0aCA+IE1heFNraWFEaW0gfHwgaGVpZ2h0
ID4gTWF4U2tpYURpbSkKICAgICAgICAgcmV0dXJuIEludFNpemUoKTsKICNlbmRpZgogCi0gICAg
cmV0dXJuIEludFNpemUoc3RhdGljX2Nhc3Q8dW5zaWduZWQ+KHdmKSwgc3RhdGljX2Nhc3Q8dW5z
aWduZWQ+KGhmKSk7Ci19Ci0KLUludFBvaW50IEhUTUxDYW52YXNFbGVtZW50Ojpjb252ZXJ0TG9n
aWNhbFRvRGV2aWNlKGNvbnN0IEZsb2F0UG9pbnQmIGxvZ2ljYWxQb3MpIGNvbnN0Ci17Ci0gICAg
ZmxvYXQgeGYgPSBsb2dpY2FsUG9zLngoKSAqIG1fcGFnZVNjYWxlRmFjdG9yOwotICAgIGZsb2F0
IHlmID0gbG9naWNhbFBvcy55KCkgKiBtX3BhZ2VTY2FsZUZhY3RvcjsKLQotICAgIHJldHVybiBJ
bnRQb2ludChzdGF0aWNfY2FzdDx1bnNpZ25lZD4oeGYpLCBzdGF0aWNfY2FzdDx1bnNpZ25lZD4o
eWYpKTsKKyAgICByZXR1cm4gSW50U2l6ZSh3aWR0aCwgaGVpZ2h0KTsKIH0KIAogY29uc3QgU2Vj
dXJpdHlPcmlnaW4mIEhUTUxDYW52YXNFbGVtZW50OjpzZWN1cml0eU9yaWdpbigpIGNvbnN0CmRp
ZmYgLS1naXQgYS9XZWJDb3JlL2h0bWwvSFRNTENhbnZhc0VsZW1lbnQuaCBiL1dlYkNvcmUvaHRt
bC9IVE1MQ2FudmFzRWxlbWVudC5oCmluZGV4IGQ5Yjc3MzY4MTI2MDFmNTlmYjBlMTdjMTI5MTZk
ODdlZGVhZWEwYzkuLjEwODFjMzNhZmViZjFmYTNmNTU2ODRjNDA3NTI0YmJkMWFlYWYxMzEgMTAw
NjQ0Ci0tLSBhL1dlYkNvcmUvaHRtbC9IVE1MQ2FudmFzRWxlbWVudC5oCisrKyBiL1dlYkNvcmUv
aHRtbC9IVE1MQ2FudmFzRWxlbWVudC5oCkBAIC0xMDcsNyArMTA3LDcgQEAgcHVibGljOgogCiAg
ICAgSW50UmVjdCBjb252ZXJ0TG9naWNhbFRvRGV2aWNlKGNvbnN0IEZsb2F0UmVjdCYpIGNvbnN0
OwogICAgIEludFNpemUgY29udmVydExvZ2ljYWxUb0RldmljZShjb25zdCBGbG9hdFNpemUmKSBj
b25zdDsKLSAgICBJbnRQb2ludCBjb252ZXJ0TG9naWNhbFRvRGV2aWNlKGNvbnN0IEZsb2F0UG9p
bnQmKSBjb25zdDsKKyAgICBJbnRTaXplIGNvbnZlcnRUb1ZhbGlkRGV2aWNlU2l6ZShmbG9hdCB3
aWR0aCwgZmxvYXQgaGVpZ2h0KSBjb25zdDsKIAogICAgIGNvbnN0IFNlY3VyaXR5T3JpZ2luJiBz
ZWN1cml0eU9yaWdpbigpIGNvbnN0OwogICAgIHZvaWQgc2V0T3JpZ2luVGFpbnRlZCgpIHsgbV9v
cmlnaW5DbGVhbiA9IGZhbHNlOyB9Cg==
</data>
<flag name="review"
          id="66861"
          type_id="1"
          status="+"
          setter="darin"
    />
    <flag name="commit-queue"
          id="67012"
          type_id="3"
          status="-"
          setter="commit-queue"
    />
          </attachment>
      

    </bug>

</bugzilla>