<?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>129706</bug_id>
          
          <creation_ts>2014-03-04 14:29:53 -0800</creation_ts>
          <short_desc>[CSS Shapes] Remove CSSBoxType member from BasicShape and CSSBasicShape</short_desc>
          <delta_ts>2014-04-05 01:23:06 -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>CSS</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>124173</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Bear Travis">betravis</reporter>
          <assigned_to name="Zoltan Horvath">zoltan</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>esprehn+autocc</cc>
    
    <cc>glenn</cc>
    
    <cc>gyuyoung.kim</cc>
    
    <cc>kondapallykalyan</cc>
    
    <cc>macpherson</cc>
    
    <cc>menard</cc>
    
    <cc>zoltan</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>986981</commentid>
    <comment_count>0</comment_count>
    <who name="Bear Travis">betravis</who>
    <bug_when>2014-03-04 14:29:53 -0800</bug_when>
    <thetext>The LayoutBox has been moved up to ShapeValue and is no longer needed in the BasicShape classes.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>997775</commentid>
    <comment_count>1</comment_count>
      <attachid>228640</attachid>
    <who name="Zoltan Horvath">zoltan</who>
    <bug_when>2014-04-04 18:06:16 -0700</bug_when>
    <thetext>Created attachment 228640
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>997855</commentid>
    <comment_count>2</comment_count>
      <attachid>228640</attachid>
    <who name="Andreas Kling">kling</who>
    <bug_when>2014-04-05 00:52:24 -0700</bug_when>
    <thetext>Comment on attachment 228640
Patch

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>997860</commentid>
    <comment_count>3</comment_count>
      <attachid>228640</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2014-04-05 01:23:01 -0700</bug_when>
    <thetext>Comment on attachment 228640
Patch

Clearing flags on attachment: 228640

Committed r166830: &lt;http://trac.webkit.org/changeset/166830&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>997861</commentid>
    <comment_count>4</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2014-04-05 01:23:06 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>228640</attachid>
            <date>2014-04-04 18:06:16 -0700</date>
            <delta_ts>2014-04-05 01:23:01 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>removeCSSBoxTypeFromBasicShapes.patch</filename>
            <type>text/plain</type>
            <size>3427</size>
            <attacher name="Zoltan Horvath">zoltan</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZwppbmRleCA5MjAwYTA0Li5hY2ViNjY1IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y
ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMjQg
QEAKKzIwMTQtMDQtMDQgIFpvbHRhbiBIb3J2YXRoICA8em9sdGFuQHdlYmtpdC5vcmc+CisKKyAg
ICAgICAgW0NTUyBTaGFwZXNdIFJlbW92ZSBDU1NCb3hUeXBlIG1lbWJlciBmcm9tIEJhc2ljU2hh
cGUgYW5kIENTU0Jhc2ljU2hhcGUKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hv
d19idWcuY2dpP2lkPTEyOTcwNgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEp
LgorCisgICAgICAgIFRoZSBDU1NCb3hUeXBlIGhhcyBiZWVuIG1vdmVkIHVwIHRvIFNoYXBlVmFs
dWUgYW5kIGlzIG5vIGxvbmdlciBuZWVkZWQgaW4gdGhlIEJhc2ljU2hhcGUgY2xhc3Nlcy4KKwor
ICAgICAgICBObyBuZXcgdGVzdHMsIG5vIGJlaGF2aW9yIGNoYW5nZS4KKworICAgICAgICAqIGNz
cy9CYXNpY1NoYXBlRnVuY3Rpb25zLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OnZhbHVlRm9yQmFz
aWNTaGFwZSk6CisgICAgICAgIChXZWJDb3JlOjpiYXNpY1NoYXBlRm9yVmFsdWUpOgorICAgICAg
ICAqIHJlbmRlcmluZy9zdHlsZS9CYXNpY1NoYXBlcy5jcHA6CisgICAgICAgIChXZWJDb3JlOjpC
YXNpY1NoYXBlOjpjYW5CbGVuZCk6CisgICAgICAgICogcmVuZGVyaW5nL3N0eWxlL0Jhc2ljU2hh
cGVzLmg6CisgICAgICAgIChXZWJDb3JlOjpCYXNpY1NoYXBlOjpyZWZlcmVuY2VCb3gpOiBEZWxl
dGVkLgorICAgICAgICAoV2ViQ29yZTo6QmFzaWNTaGFwZTo6c2V0UmVmZXJlbmNlQm94KTogRGVs
ZXRlZC4KKyAgICAgICAgKFdlYkNvcmU6OkJhc2ljU2hhcGU6OkJhc2ljU2hhcGUpOiBEZWxldGVk
LgorCiAyMDE0LTA0LTA0ICBZdXN1a2UgU3V6dWtpICA8dXRhdGFuZS50ZWFAZ21haWwuY29tPgog
CiAgICAgICAgIFVwZ3JhZGUgdG8gU2VsZWN0b3JGYWlsc0FsbFNpYmxpbmdzIHdoZW4gQ2hpbGQg
c2VsZWN0b3IgaXMgZmFpbGVkLgpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvY3NzL0Jhc2lj
U2hhcGVGdW5jdGlvbnMuY3BwIGIvU291cmNlL1dlYkNvcmUvY3NzL0Jhc2ljU2hhcGVGdW5jdGlv
bnMuY3BwCmluZGV4IDYzMzkzNjAuLjg5MDUyODYgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3Jl
L2Nzcy9CYXNpY1NoYXBlRnVuY3Rpb25zLmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9jc3MvQmFz
aWNTaGFwZUZ1bmN0aW9ucy5jcHAKQEAgLTEyNCw5ICsxMjQsNiBAQCBQYXNzUmVmUHRyPENTU1Zh
bHVlPiB2YWx1ZUZvckJhc2ljU2hhcGUoY29uc3QgUmVuZGVyU3R5bGUqIHN0eWxlLCBjb25zdCBC
YXNpY1NoYQogICAgICAgICBicmVhazsKICAgICB9CiAKLSAgICBpZiAoYmFzaWNTaGFwZS0+cmVm
ZXJlbmNlQm94KCkgIT0gQm94TWlzc2luZykKLSAgICAgICAgYmFzaWNTaGFwZVZhbHVlLT5zZXRS
ZWZlcmVuY2VCb3gocG9vbC5jcmVhdGVWYWx1ZShiYXNpY1NoYXBlLT5yZWZlcmVuY2VCb3goKSkp
OwotCiAgICAgcmV0dXJuIHBvb2wuY3JlYXRlVmFsdWUoYmFzaWNTaGFwZVZhbHVlLnJlbGVhc2Uo
KSk7CiB9CiAKQEAgLTI2NCw5ICsyNjEsNiBAQCBQYXNzUmVmUHRyPEJhc2ljU2hhcGU+IGJhc2lj
U2hhcGVGb3JWYWx1ZShjb25zdCBSZW5kZXJTdHlsZSogc3R5bGUsIGNvbnN0IFJlbmRlcgogICAg
ICAgICBicmVhazsKICAgICB9CiAKLSAgICBpZiAoYmFzaWNTaGFwZVZhbHVlLT5yZWZlcmVuY2VC
b3goKSkKLSAgICAgICAgYmFzaWNTaGFwZS0+c2V0UmVmZXJlbmNlQm94KENTU0JveFR5cGUoKmJh
c2ljU2hhcGVWYWx1ZS0+cmVmZXJlbmNlQm94KCkpKTsKLQogICAgIHJldHVybiBiYXNpY1NoYXBl
LnJlbGVhc2UoKTsKIH0KIApkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvcmVuZGVyaW5nL3N0
eWxlL0Jhc2ljU2hhcGVzLmNwcCBiL1NvdXJjZS9XZWJDb3JlL3JlbmRlcmluZy9zdHlsZS9CYXNp
Y1NoYXBlcy5jcHAKaW5kZXggZDJhMGM0ZC4uMWVjYjJlNiAxMDA2NDQKLS0tIGEvU291cmNlL1dl
YkNvcmUvcmVuZGVyaW5nL3N0eWxlL0Jhc2ljU2hhcGVzLmNwcAorKysgYi9Tb3VyY2UvV2ViQ29y
ZS9yZW5kZXJpbmcvc3R5bGUvQmFzaWNTaGFwZXMuY3BwCkBAIC02NCwxMCArNjQsNiBAQCBib29s
IEJhc2ljU2hhcGU6OmNhbkJsZW5kKGNvbnN0IEJhc2ljU2hhcGUqIG90aGVyKSBjb25zdAogICAg
IGlmICh0eXBlKCkgIT0gb3RoZXItPnR5cGUoKSkKICAgICAgICAgcmV0dXJuIGZhbHNlOwogCi0g
ICAgLy8gQm90aCBzaGFwZXMgbXVzdCB1c2UgdGhlIHNhbWUgcmVmZXJlbmNlIGJveC4KLSAgICBp
ZiAocmVmZXJlbmNlQm94KCkgIT0gb3RoZXItPnJlZmVyZW5jZUJveCgpKQotICAgICAgICByZXR1
cm4gZmFsc2U7Ci0KICAgICAvLyBKdXN0IHBvbHlnb25zIHdpdGggc2FtZSBudW1iZXIgb2YgdmVy
dGljZXMgY2FuIGJlIGFuaW1hdGVkLgogICAgIGlmICh0eXBlKCkgPT0gQmFzaWNTaGFwZTo6QmFz
aWNTaGFwZVBvbHlnb25UeXBlCiAgICAgICAgICYmIChzdGF0aWNfY2FzdDxjb25zdCBCYXNpY1No
YXBlUG9seWdvbio+KHRoaXMpLT52YWx1ZXMoKS5zaXplKCkgIT0gc3RhdGljX2Nhc3Q8Y29uc3Qg
QmFzaWNTaGFwZVBvbHlnb24qPihvdGhlciktPnZhbHVlcygpLnNpemUoKQpkaWZmIC0tZ2l0IGEv
U291cmNlL1dlYkNvcmUvcmVuZGVyaW5nL3N0eWxlL0Jhc2ljU2hhcGVzLmggYi9Tb3VyY2UvV2Vi
Q29yZS9yZW5kZXJpbmcvc3R5bGUvQmFzaWNTaGFwZXMuaAppbmRleCBmMmFiNGJkLi44ZTE4ZjQ2
IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9yZW5kZXJpbmcvc3R5bGUvQmFzaWNTaGFwZXMu
aAorKysgYi9Tb3VyY2UvV2ViQ29yZS9yZW5kZXJpbmcvc3R5bGUvQmFzaWNTaGFwZXMuaApAQCAt
NjIsMTggKzYyLDYgQEAgcHVibGljOgogICAgIHZpcnR1YWwgUGFzc1JlZlB0cjxCYXNpY1NoYXBl
PiBibGVuZChjb25zdCBCYXNpY1NoYXBlKiwgZG91YmxlKSBjb25zdCA9IDA7CiAKICAgICB2aXJ0
dWFsIFR5cGUgdHlwZSgpIGNvbnN0ID0gMDsKLQotICAgIENTU0JveFR5cGUgcmVmZXJlbmNlQm94
KCkgY29uc3QgeyByZXR1cm4gbV9yZWZlcmVuY2VCb3g7IH0KLSAgICB2b2lkIHNldFJlZmVyZW5j
ZUJveChDU1NCb3hUeXBlIHJlZmVyZW5jZUJveCkgeyBtX3JlZmVyZW5jZUJveCA9IHJlZmVyZW5j
ZUJveDsgfQotCi1wcm90ZWN0ZWQ6Ci0gICAgQmFzaWNTaGFwZSgpCi0gICAgICAgIDogbV9yZWZl
cmVuY2VCb3goQm94TWlzc2luZykKLSAgICB7Ci0gICAgfQotCi1wcml2YXRlOgotICAgIENTU0Jv
eFR5cGUgbV9yZWZlcmVuY2VCb3g7CiB9OwogCiBjbGFzcyBCYXNpY1NoYXBlQ2VudGVyQ29vcmRp
bmF0ZSB7Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>