<?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>159029</bug_id>
          
          <creation_ts>2016-06-22 11:34:45 -0700</creation_ts>
          <short_desc>Exception is not thrown when shipping method is an invalid amount</short_desc>
          <delta_ts>2016-06-22 11:43:30 -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>New Bugs</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Anders Carlsson">andersca</reporter>
          <assigned_to name="Anders Carlsson">andersca</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>mkwst</cc>
    
    <cc>thorton</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1204597</commentid>
    <comment_count>0</comment_count>
    <who name="Anders Carlsson">andersca</who>
    <bug_when>2016-06-22 11:34:45 -0700</bug_when>
    <thetext>Exception is not thrown when shipping method is an invalid amount</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1204598</commentid>
    <comment_count>1</comment_count>
      <attachid>281852</attachid>
    <who name="Anders Carlsson">andersca</who>
    <bug_when>2016-06-22 11:36:53 -0700</bug_when>
    <thetext>Created attachment 281852
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1204599</commentid>
    <comment_count>2</comment_count>
    <who name="Tim Horton">thorton</who>
    <bug_when>2016-06-22 11:38:44 -0700</bug_when>
    <thetext>Tests!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1204603</commentid>
    <comment_count>3</comment_count>
    <who name="Anders Carlsson">andersca</who>
    <bug_when>2016-06-22 11:43:30 -0700</bug_when>
    <thetext>Committed r202341: &lt;http://trac.webkit.org/changeset/202341&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>281852</attachid>
            <date>2016-06-22 11:36:53 -0700</date>
            <delta_ts>2016-06-22 11:38:30 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-159029-20160622113645.patch</filename>
            <type>text/plain</type>
            <size>4413</size>
            <attacher name="Anders Carlsson">andersca</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjAyMzI4CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggZDJjZTM0ZDlmYjcwMjI0
Yjg1Y2ExYTQxYzE4YjViOWU0NmY0MTQyMy4uZmU1ZmQwYzY1YjA2ZTU5MWFmMWI2MmJiNTg0Y2Y1
YzU1YzgxNjJjMCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDI3IEBACisyMDE2LTA2LTIyICBBbmRl
cnMgQ2FybHNzb24gIDxhbmRlcnNjYUBhcHBsZS5jb20+CisKKyAgICAgICAgRXhjZXB0aW9uIGlz
IG5vdCB0aHJvd24gd2hlbiBzaGlwcGluZyBtZXRob2QgaXMgYW4gaW52YWxpZCBhbW91bnQKKyAg
ICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE1OTAyOQorICAg
ICAgICByZGFyOi8vcHJvYmxlbS8yNjcwMDQxMworCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9E
WSAoT09QUyEpLgorCisgICAgICAgICogTW9kdWxlcy9hcHBsZXBheS9QYXltZW50UmVxdWVzdC5o
OgorICAgICAgICBDaGFuZ2UgU2hpcHBpbmdNZXRob2Q6OmFtb3VudCB0byBiZSBhIHNpZ25lZCA2
NC1iaXQgaW50ZWdlci4KKworICAgICAgICAqIE1vZHVsZXMvYXBwbGVwYXkvUGF5bWVudFJlcXVl
c3RWYWxpZGF0b3IuY3BwOgorICAgICAgICAoV2ViQ29yZTo6UGF5bWVudFJlcXVlc3RWYWxpZGF0
b3I6OnZhbGlkYXRlKToKKyAgICAgICAgQ2FsbCB2YWxpZGF0ZVNoaXBwaW5nTWV0aG9kcy4KKwor
ICAgICAgICAoV2ViQ29yZTo6UGF5bWVudFJlcXVlc3RWYWxpZGF0b3I6OnZhbGlkYXRlU2hpcHBp
bmdNZXRob2RzKToKKyAgICAgICAgVmFsaWRhdGUgYWxsIHRoZSBzaGlwcGluZyBtZXRob2RzLgor
CisgICAgICAgIChXZWJDb3JlOjpQYXltZW50UmVxdWVzdFZhbGlkYXRvcjo6dmFsaWRhdGVTaGlw
cGluZ01ldGhvZCk6CisgICAgICAgIENoZWNrIHRoYXQgdGhlIGFtb3VudCBpcyA+PSAwLgorCisg
ICAgICAgICogTW9kdWxlcy9hcHBsZXBheS9QYXltZW50UmVxdWVzdFZhbGlkYXRvci5oOgorICAg
ICAgICBBZGQgbmV3IG1lbWJlcnMuCisKIDIwMTYtMDYtMjIgIENhcmxvcyBHYXJjaWEgQ2FtcG9z
ICA8Y2dhcmNpYUBpZ2FsaWEuY29tPgogCiAgICAgICAgIFtHVEtdIEFkZCBzdXBwb3J0IGZvciB2
YXJpYWRpYyBwYXJhbWV0ZXJzIHRvIEdPYmplY3QgRE9NIGJpbmRpbmdzCmRpZmYgLS1naXQgYS9T
b3VyY2UvV2ViQ29yZS9Nb2R1bGVzL2FwcGxlcGF5L1BheW1lbnRSZXF1ZXN0LmggYi9Tb3VyY2Uv
V2ViQ29yZS9Nb2R1bGVzL2FwcGxlcGF5L1BheW1lbnRSZXF1ZXN0LmgKaW5kZXggNmRmYmM3ZTg4
YmUzMmYyZjRkMTZjYmYwOGIxMTk1ZDNhZmQ0MTlhYi4uODdkZTMyMDVmOTMxNTA3ODYzMjc3NDNh
NTQ0YTA4YWZjMjhlNjZjOCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvTW9kdWxlcy9hcHBs
ZXBheS9QYXltZW50UmVxdWVzdC5oCisrKyBiL1NvdXJjZS9XZWJDb3JlL01vZHVsZXMvYXBwbGVw
YXkvUGF5bWVudFJlcXVlc3QuaApAQCAtMTEyLDcgKzExMiw3IEBAIHB1YmxpYzoKICAgICBzdHJ1
Y3QgU2hpcHBpbmdNZXRob2QgewogICAgICAgICBTdHJpbmcgbGFiZWw7CiAgICAgICAgIFN0cmlu
ZyBkZXRhaWw7Ci0gICAgICAgIHVpbnQ2NF90IGFtb3VudDsKKyAgICAgICAgaW50NjRfdCBhbW91
bnQ7CiAKICAgICAgICAgU3RyaW5nIGlkZW50aWZpZXI7CiAgICAgfTsKZGlmZiAtLWdpdCBhL1Nv
dXJjZS9XZWJDb3JlL01vZHVsZXMvYXBwbGVwYXkvUGF5bWVudFJlcXVlc3RWYWxpZGF0b3IuY3Bw
IGIvU291cmNlL1dlYkNvcmUvTW9kdWxlcy9hcHBsZXBheS9QYXltZW50UmVxdWVzdFZhbGlkYXRv
ci5jcHAKaW5kZXggNTVhNjA2ODJlOTA3ZDBkZDU4NDVmODM0OTUyYWY4ZDIyNzc4YWZjMC4uOTZi
YmFlZjI0MzRiOGFlYmVjNTJmZGI1NzFkN2RlYTc3NjEzYmFiNSAxMDA2NDQKLS0tIGEvU291cmNl
L1dlYkNvcmUvTW9kdWxlcy9hcHBsZXBheS9QYXltZW50UmVxdWVzdFZhbGlkYXRvci5jcHAKKysr
IGIvU291cmNlL1dlYkNvcmUvTW9kdWxlcy9hcHBsZXBheS9QYXltZW50UmVxdWVzdFZhbGlkYXRv
ci5jcHAKQEAgLTU3LDcgKzU3LDggQEAgYm9vbCBQYXltZW50UmVxdWVzdFZhbGlkYXRvcjo6dmFs
aWRhdGUoY29uc3QgUGF5bWVudFJlcXVlc3QmIHBheW1lbnRSZXF1ZXN0KSBjb24KICAgICAgICAg
cmV0dXJuIGZhbHNlOwogICAgIGlmICghdmFsaWRhdGVUb3RhbChwYXltZW50UmVxdWVzdC50b3Rh
bCgpKSkKICAgICAgICAgcmV0dXJuIGZhbHNlOwotCisgICAgaWYgKCF2YWxpZGF0ZVNoaXBwaW5n
TWV0aG9kcyhwYXltZW50UmVxdWVzdC5zaGlwcGluZ01ldGhvZHMoKSkpCisgICAgICAgIHJldHVy
biBmYWxzZTsKICAgICByZXR1cm4gdHJ1ZTsKIH0KIApAQCAtMTM2LDYgKzEzNywxNiBAQCBib29s
IFBheW1lbnRSZXF1ZXN0VmFsaWRhdG9yOjp2YWxpZGF0ZU1lcmNoYW50Q2FwYWJpbGl0aWVzKGNv
bnN0IFBheW1lbnRSZXF1ZXN0OgogICAgIHJldHVybiB0cnVlOwogfQogCitib29sIFBheW1lbnRS
ZXF1ZXN0VmFsaWRhdG9yOjp2YWxpZGF0ZVNoaXBwaW5nTWV0aG9kKGNvbnN0IFBheW1lbnRSZXF1
ZXN0OjpTaGlwcGluZ01ldGhvZCYgc2hpcHBpbmdNZXRob2QpIGNvbnN0Cit7CisgICAgaWYgKHNo
aXBwaW5nTWV0aG9kLmFtb3VudCA8IDApIHsKKyAgICAgICAgbV93aW5kb3cucHJpbnRFcnJvck1l
c3NhZ2UoIlNoaXBwaW5nIG1ldGhvZCBhbW91bnQgbXVzdCBiZSBncmVhdGVyIHRoYW4gb3IgZXF1
YWwgdG8gemVyby4iKTsKKyAgICAgICAgcmV0dXJuIGZhbHNlOworICAgIH0KKworICAgIHJldHVy
biB0cnVlOworfQorCiBib29sIFBheW1lbnRSZXF1ZXN0VmFsaWRhdG9yOjp2YWxpZGF0ZVN1cHBv
cnRlZE5ldHdvcmtzKGNvbnN0IFBheW1lbnRSZXF1ZXN0OjpTdXBwb3J0ZWROZXR3b3JrcyYgc3Vw
cG9ydGVkTmV0d29ya3MpIGNvbnN0CiB7CiAgICAgaWYgKCFzdXBwb3J0ZWROZXR3b3Jrcy5hbWV4
ICYmICFzdXBwb3J0ZWROZXR3b3Jrcy5jaGluYVVuaW9uUGF5ICYmICFzdXBwb3J0ZWROZXR3b3Jr
cy5kaXNjb3ZlciAmJiAhc3VwcG9ydGVkTmV0d29ya3MuaW50ZXJhYyAmJiAhc3VwcG9ydGVkTmV0
d29ya3MubWFzdGVyQ2FyZCAmJiAhc3VwcG9ydGVkTmV0d29ya3MucHJpdmF0ZUxhYmVsICYmIXN1
cHBvcnRlZE5ldHdvcmtzLnZpc2EpIHsKQEAgLTE0Niw2ICsxNTcsMTYgQEAgYm9vbCBQYXltZW50
UmVxdWVzdFZhbGlkYXRvcjo6dmFsaWRhdGVTdXBwb3J0ZWROZXR3b3Jrcyhjb25zdCBQYXltZW50
UmVxdWVzdDo6U3UKICAgICByZXR1cm4gdHJ1ZTsKIH0KIAorYm9vbCBQYXltZW50UmVxdWVzdFZh
bGlkYXRvcjo6dmFsaWRhdGVTaGlwcGluZ01ldGhvZHMoY29uc3QgVmVjdG9yPFBheW1lbnRSZXF1
ZXN0OjpTaGlwcGluZ01ldGhvZD4mIHNoaXBwaW5nTWV0aG9kcykgY29uc3QKK3sKKyAgICBmb3Ig
KGNvbnN0IGF1dG8mIHNoaXBwaW5nTWV0aG9kIDogc2hpcHBpbmdNZXRob2RzKSB7CisgICAgICAg
IGlmICghdmFsaWRhdGVTaGlwcGluZ01ldGhvZChzaGlwcGluZ01ldGhvZCkpCisgICAgICAgICAg
ICByZXR1cm4gZmFsc2U7CisgICAgfQorCisgICAgcmV0dXJuIHRydWU7Cit9CisKIH0KIAogI2Vu
ZGlmCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9Nb2R1bGVzL2FwcGxlcGF5L1BheW1lbnRS
ZXF1ZXN0VmFsaWRhdG9yLmggYi9Tb3VyY2UvV2ViQ29yZS9Nb2R1bGVzL2FwcGxlcGF5L1BheW1l
bnRSZXF1ZXN0VmFsaWRhdG9yLmgKaW5kZXggMjJkMTUwY2JiZGNkNzBjZTIwM2E1ZmU5NjMwMzJi
Yzk4ZWQwZjM4My4uYTVkNGUxZTM5ZGExYTNjZTY4ZWMxZjAzOTMzOThlYTAwNDQ5MWIyMCAxMDA2
NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvTW9kdWxlcy9hcHBsZXBheS9QYXltZW50UmVxdWVzdFZh
bGlkYXRvci5oCisrKyBiL1NvdXJjZS9XZWJDb3JlL01vZHVsZXMvYXBwbGVwYXkvUGF5bWVudFJl
cXVlc3RWYWxpZGF0b3IuaApAQCAtNDcsNiArNDcsOSBAQCBwcml2YXRlOgogICAgIGJvb2wgdmFs
aWRhdGVNZXJjaGFudENhcGFiaWxpdGllcyhjb25zdCBQYXltZW50UmVxdWVzdDo6TWVyY2hhbnRD
YXBhYmlsaXRpZXMmKSBjb25zdDsKICAgICBib29sIHZhbGlkYXRlU3VwcG9ydGVkTmV0d29ya3Mo
Y29uc3QgUGF5bWVudFJlcXVlc3Q6OlN1cHBvcnRlZE5ldHdvcmtzJikgY29uc3Q7CiAKKyAgICBi
b29sIHZhbGlkYXRlU2hpcHBpbmdNZXRob2RzKGNvbnN0IFZlY3RvcjxQYXltZW50UmVxdWVzdDo6
U2hpcHBpbmdNZXRob2Q+JikgY29uc3Q7CisgICAgYm9vbCB2YWxpZGF0ZVNoaXBwaW5nTWV0aG9k
KGNvbnN0IFBheW1lbnRSZXF1ZXN0OjpTaGlwcGluZ01ldGhvZCYpIGNvbnN0OworCiAgICAgRE9N
V2luZG93JiBtX3dpbmRvdzsKIH07CiAK
</data>
<flag name="review"
          id="305677"
          type_id="1"
          status="+"
          setter="thorton"
    />
          </attachment>
      

    </bug>

</bugzilla>