<?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>80989</bug_id>
          
          <creation_ts>2012-03-13 08:22:19 -0700</creation_ts>
          <short_desc>[BlackBerry] Remove sublayers before clamping just in case the layer already exists</short_desc>
          <delta_ts>2012-03-13 10:30:23 -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>WebKit BlackBerry</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>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Konrad Piascik">kpiascik</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>anilsson</cc>
    
    <cc>robin.webkit</cc>
    
    <cc>tonikitoo</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>577335</commentid>
    <comment_count>0</comment_count>
    <who name="Konrad Piascik">kpiascik</who>
    <bug_when>2012-03-13 08:22:19 -0700</bug_when>
    <thetext>[BlackBerry] Remove sublayers before clamping just in case the layer already exists</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>577345</commentid>
    <comment_count>1</comment_count>
      <attachid>131622</attachid>
    <who name="Konrad Piascik">kpiascik</who>
    <bug_when>2012-03-13 08:32:20 -0700</bug_when>
    <thetext>Created attachment 131622
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>577348</commentid>
    <comment_count>2</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-03-13 08:35:55 -0700</bug_when>
    <thetext>Attachment 131622 did not pass style-queue:

Failed to run &quot;[&apos;Tools/Scripts/check-webkit-style&apos;, &apos;--diff-files&apos;, u&apos;Source/WebCore/ChangeLog&apos;, u&apos;Source/WebCor...&quot; exit_code: 1
Source/WebCore/ChangeLog:8:  You should remove the &apos;No new tests&apos; and either add and list tests, or explain why no new tests were possible.  [changelog/nonewtests] [5]
Total errors found: 1 in 2 files


If any of these errors are false positives, please file a bug against check-webkit-style.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>577388</commentid>
    <comment_count>3</comment_count>
      <attachid>131629</attachid>
    <who name="Konrad Piascik">kpiascik</who>
    <bug_when>2012-03-13 09:01:12 -0700</bug_when>
    <thetext>Created attachment 131629
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>577505</commentid>
    <comment_count>4</comment_count>
      <attachid>131629</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-03-13 10:30:19 -0700</bug_when>
    <thetext>Comment on attachment 131629
Patch

Clearing flags on attachment: 131629

Committed r110578: &lt;http://trac.webkit.org/changeset/110578&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>577506</commentid>
    <comment_count>5</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-03-13 10:30:23 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>131622</attachid>
            <date>2012-03-13 08:32:20 -0700</date>
            <delta_ts>2012-03-13 09:01:07 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-80989-20120313113314.patch</filename>
            <type>text/plain</type>
            <size>1599</size>
            <attacher name="Konrad Piascik">kpiascik</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTEwMjk2CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggZDA2NGUwMDIwYmUxMmYw
ZTFkODM5NDI3NmUzNzJlNTI2OTc0ZGNhNi4uYmYxMDUwNzQ3YWIxOTlmOTkzY2M5ODc1OTVlOTIw
ODZmYjI3MmZiMiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1IEBACisyMDEyLTAzLTEzICBLb25y
YWQgUGlhc2NpayAgPGtwaWFzY2lrQHJpbS5jb20+CisKKyAgICAgICAgW0JsYWNrQmVycnldIFJl
bW92ZSBzdWJsYXllcnMgYmVmb3JlIGNsYW1waW5nIGp1c3QgaW4gY2FzZSB0aGUgbGF5ZXIgYWxy
ZWFkeSBleGlzdHMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dp
P2lkPTgwOTg5CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAg
ICAgTm8gbmV3IHRlc3RzLiAoT09QUyEpCisKKyAgICAgICAgKiBwbGF0Zm9ybS9ncmFwaGljcy9i
bGFja2JlcnJ5L0xheWVyV2ViS2l0VGhyZWFkLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OkxheWVy
V2ViS2l0VGhyZWFkOjppbnNlcnRTdWJsYXllcik6CisKIDIwMTItMDMtMDkgIFpvbHRhbiBIZXJj
emVnICA8emhlcmN6ZWdAd2Via2l0Lm9yZz4KIAogICAgICAgICBBZGQgU1ZHUmVuZGVyaW5nQ29u
dGV4dCBhbmQgbW92ZSB0aGVyZSB0aGUgY29udGV4dCByZWxhdGVkIGNvZGUgZnJvbSBTVkdSZW5k
ZXJTdXBwb3J0CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9i
bGFja2JlcnJ5L0xheWVyV2ViS2l0VGhyZWFkLmNwcCBiL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3Jt
L2dyYXBoaWNzL2JsYWNrYmVycnkvTGF5ZXJXZWJLaXRUaHJlYWQuY3BwCmluZGV4IDZjYWRmMzU5
ZDBjY2IyYTYxMDI1ZjFkOTA3NWQ4NTc2ZTVhMzBhMGIuLjU1MTM2YmJhNWI0ZTQ5OGVjYTE3ZjFj
MDc5YTJiNDk3NGQ5MDY2NmIgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dy
YXBoaWNzL2JsYWNrYmVycnkvTGF5ZXJXZWJLaXRUaHJlYWQuY3BwCisrKyBiL1NvdXJjZS9XZWJD
b3JlL3BsYXRmb3JtL2dyYXBoaWNzL2JsYWNrYmVycnkvTGF5ZXJXZWJLaXRUaHJlYWQuY3BwCkBA
IC0zMDYsOCArMzA2LDggQEAgdm9pZCBMYXllcldlYktpdFRocmVhZDo6YWRkU3VibGF5ZXIoUGFz
c1JlZlB0cjxMYXllcldlYktpdFRocmVhZD4gc3VibGF5ZXIpCiAKIHZvaWQgTGF5ZXJXZWJLaXRU
aHJlYWQ6Omluc2VydFN1YmxheWVyKFBhc3NSZWZQdHI8TGF5ZXJXZWJLaXRUaHJlYWQ+IHN1Ymxh
eWVyLCBzaXplX3QgaW5kZXgpCiB7Ci0gICAgaW5kZXggPSBtaW4oaW5kZXgsIG1fc3VibGF5ZXJz
LnNpemUoKSk7CiAgICAgc3VibGF5ZXItPnJlbW92ZUZyb21TdXBlcmxheWVyKCk7CisgICAgaW5k
ZXggPSBtaW4oaW5kZXgsIG1fc3VibGF5ZXJzLnNpemUoKSk7CiAgICAgc3VibGF5ZXItPnNldFN1
cGVybGF5ZXIodGhpcyk7CiAgICAgbV9zdWJsYXllcnMuaW5zZXJ0KGluZGV4LCBzdWJsYXllcik7
CiAK
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>131629</attachid>
            <date>2012-03-13 09:01:12 -0700</date>
            <delta_ts>2012-03-13 10:30:19 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-80989-20120313120206.patch</filename>
            <type>text/plain</type>
            <size>1918</size>
            <attacher name="Konrad Piascik">kpiascik</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTEwMjk2CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggZDA2NGUwMDIwYmUxMmYw
ZTFkODM5NDI3NmUzNzJlNTI2OTc0ZGNhNi4uNDZjNzFjNDMzZTBjMzI0OTljZDM1Yzc3N2IzOTM3
OGZiODMyMTRhMCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDIwIEBACisyMDEyLTAzLTEzICBLb25y
YWQgUGlhc2NpayAgPGtwaWFzY2lrQHJpbS5jb20+CisKKyAgICAgICAgW0JsYWNrQmVycnldIFJl
bW92ZSBzdWJsYXllcnMgYmVmb3JlIGNsYW1waW5nIGp1c3QgaW4gY2FzZSB0aGUgbGF5ZXIgYWxy
ZWFkeSBleGlzdHMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dp
P2lkPTgwOTg5CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAg
ICAgRGlzY292ZXJlZCB3aGVuIHRyeWluZyB0byBhZGQgYSBoaWdobGlnaHQgc3VibGF5ZXIgZm9y
IHdlYiBpbnNwZWN0b3IuICBTaW5jZSB0aGF0IGxheWVyCisgICAgICAgIG1heSBhbHJlYWR5IGV4
aXN0IGluIHRoZSB0cmVlIHdlIG5lZWQgdG8gc2VlIGlmIGl0IHdpbGwgYmUgcHJ1bmVkIGJlZm9y
ZSB3ZSBjbGFtcCBvbiB0aGUKKyAgICAgICAgc2l6ZS4KKworICAgICAgICBUaGlzIHdhcyBtYW51
YWxseSB0ZXN0ZWQgZHVyaW5nIHRoZSB3cml0aW5nIG9mIGFub3RoZXIgcGF0Y2ggdG8gYWRkIHdl
YiBpbnNwZWN0b3IgaGlnaGxpZ2h0cworICAgICAgICB0byBhY2NlbHJhdGVkIGNvbXBvc2l0aW5n
LgorCisgICAgICAgICogcGxhdGZvcm0vZ3JhcGhpY3MvYmxhY2tiZXJyeS9MYXllcldlYktpdFRo
cmVhZC5jcHA6CisgICAgICAgIChXZWJDb3JlOjpMYXllcldlYktpdFRocmVhZDo6aW5zZXJ0U3Vi
bGF5ZXIpOgorCiAyMDEyLTAzLTA5ICBab2x0YW4gSGVyY3plZyAgPHpoZXJjemVnQHdlYmtpdC5v
cmc+CiAKICAgICAgICAgQWRkIFNWR1JlbmRlcmluZ0NvbnRleHQgYW5kIG1vdmUgdGhlcmUgdGhl
IGNvbnRleHQgcmVsYXRlZCBjb2RlIGZyb20gU1ZHUmVuZGVyU3VwcG9ydApkaWZmIC0tZ2l0IGEv
U291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvYmxhY2tiZXJyeS9MYXllcldlYktpdFRo
cmVhZC5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9ibGFja2JlcnJ5L0xh
eWVyV2ViS2l0VGhyZWFkLmNwcAppbmRleCA2Y2FkZjM1OWQwY2NiMmE2MTAyNWYxZDkwNzVkODU3
NmU1YTMwYTBiLi41NTEzNmJiYTViNGU0OThlY2ExN2YxYzA3OWEyYjQ5NzRkOTA2NjZiIDEwMDY0
NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9ibGFja2JlcnJ5L0xheWVy
V2ViS2l0VGhyZWFkLmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9i
bGFja2JlcnJ5L0xheWVyV2ViS2l0VGhyZWFkLmNwcApAQCAtMzA2LDggKzMwNiw4IEBAIHZvaWQg
TGF5ZXJXZWJLaXRUaHJlYWQ6OmFkZFN1YmxheWVyKFBhc3NSZWZQdHI8TGF5ZXJXZWJLaXRUaHJl
YWQ+IHN1YmxheWVyKQogCiB2b2lkIExheWVyV2ViS2l0VGhyZWFkOjppbnNlcnRTdWJsYXllcihQ
YXNzUmVmUHRyPExheWVyV2ViS2l0VGhyZWFkPiBzdWJsYXllciwgc2l6ZV90IGluZGV4KQogewot
ICAgIGluZGV4ID0gbWluKGluZGV4LCBtX3N1YmxheWVycy5zaXplKCkpOwogICAgIHN1YmxheWVy
LT5yZW1vdmVGcm9tU3VwZXJsYXllcigpOworICAgIGluZGV4ID0gbWluKGluZGV4LCBtX3N1Ymxh
eWVycy5zaXplKCkpOwogICAgIHN1YmxheWVyLT5zZXRTdXBlcmxheWVyKHRoaXMpOwogICAgIG1f
c3VibGF5ZXJzLmluc2VydChpbmRleCwgc3VibGF5ZXIpOwogCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>