<?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>79915</bug_id>
          
          <creation_ts>2012-02-29 07:54:22 -0800</creation_ts>
          <short_desc>Web Inspector: hide color picker on Esc and Enter.</short_desc>
          <delta_ts>2012-03-01 08:17:12 -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>Web Inspector (Deprecated)</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</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="Pavel Feldman">pfeldman</reporter>
          <assigned_to name="Pavel Feldman">pfeldman</assigned_to>
          <cc>apavlov</cc>
    
    <cc>bweinstein</cc>
    
    <cc>joepeck</cc>
    
    <cc>keishi</cc>
    
    <cc>loislo</cc>
    
    <cc>pfeldman</cc>
    
    <cc>pmuellr</cc>
    
    <cc>rik</cc>
    
    <cc>timothy</cc>
    
    <cc>yurys</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>567626</commentid>
    <comment_count>0</comment_count>
    <who name="Pavel Feldman">pfeldman</who>
    <bug_when>2012-02-29 07:54:22 -0800</bug_when>
    <thetext>Patch to follow.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>567627</commentid>
    <comment_count>1</comment_count>
      <attachid>129452</attachid>
    <who name="Pavel Feldman">pfeldman</who>
    <bug_when>2012-02-29 07:55:48 -0800</bug_when>
    <thetext>Created attachment 129452
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>568644</commentid>
    <comment_count>2</comment_count>
      <attachid>129452</attachid>
    <who name="Vsevolod Vlasov">vsevik</who>
    <bug_when>2012-03-01 07:14:14 -0800</bug_when>
    <thetext>Comment on attachment 129452
Patch

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

&gt; Source/WebCore/inspector/front-end/Spectrum.js:410
&gt; +        this._previousFocusElement = WebInspector.currentFocusElement();

You are going to lose previous focus element once reposition() is called more than once.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>568674</commentid>
    <comment_count>3</comment_count>
    <who name="Pavel Feldman">pfeldman</who>
    <bug_when>2012-03-01 08:17:12 -0800</bug_when>
    <thetext>Committed r109348: &lt;http://trac.webkit.org/changeset/109348&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>129452</attachid>
            <date>2012-02-29 07:55:48 -0800</date>
            <delta_ts>2012-03-01 07:14:14 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-79915-20120229195546.patch</filename>
            <type>text/plain</type>
            <size>2832</size>
            <attacher name="Pavel Feldman">pfeldman</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTA5MjExCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggNWE1YWM2YmFiMWZhZWFi
NmVhZWIwMjA3MzgzNzMwYjIyMzQ3OWU4NC4uMDM5OWY3YTg0MTczM2U5MDYzYzE4ODA3ZmYyYTMz
NmM0ZTY4ZjI2OSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSw1ICsxLDE3IEBACiAyMDEyLTAyLTI5ICBQYXZl
bCBGZWxkbWFuICA8cGZlbGRtYW5AZ29vZ2xlLmNvbT4KIAorICAgICAgICBXZWIgSW5zcGVjdG9y
OiBoaWRlIGNvbG9yIHBpY2tlciBvbiBFc2MgYW5kIEVudGVyLgorICAgICAgICBodHRwczovL2J1
Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9Nzk5MTUKKworICAgICAgICBSZXZpZXdlZCBi
eSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAqIGluc3BlY3Rvci9mcm9udC1lbmQvU3BlY3Ry
dW0uanM6CisgICAgICAgIChXZWJJbnNwZWN0b3IuU3BlY3RydW0ucHJvdG90eXBlLnJlcG9zaXRp
b24pOgorICAgICAgICAoV2ViSW5zcGVjdG9yLlNwZWN0cnVtLnByb3RvdHlwZS5oaWRlKToKKyAg
ICAgICAgKFdlYkluc3BlY3Rvci5TcGVjdHJ1bS5wcm90b3R5cGUuX29uS2V5RG93bik6CisKKzIw
MTItMDItMjkgIFBhdmVsIEZlbGRtYW4gIDxwZmVsZG1hbkBnb29nbGUuY29tPgorCiAgICAgICAg
IFdlYiBJbnNwZWN0b3I6IEN0cmwgUiBzaG91bGQgcmVsb2FkIHBhZ2UgZnJvbSB0aGUgY29uc29s
ZSBwYW5lbCBhcyB3ZWxsLgogICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1
Zy5jZ2k/aWQ9Nzk4ODMKIApkaWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvaW5zcGVjdG9yL2Zy
b250LWVuZC9TcGVjdHJ1bS5qcyBiL1NvdXJjZS9XZWJDb3JlL2luc3BlY3Rvci9mcm9udC1lbmQv
U3BlY3RydW0uanMKaW5kZXggM2Y5YTU4ZTQ0MmU5Yjg0NzI4NGY5OTQ1ZTllYzFhYjYxMTEyZWI5
MC4uZjU4ZTk3YjQzMGNhNDVmNTFmNTdiMjFhMTcxMTkxNjI2NWZiYmU2OSAxMDA2NDQKLS0tIGEv
U291cmNlL1dlYkNvcmUvaW5zcGVjdG9yL2Zyb250LWVuZC9TcGVjdHJ1bS5qcworKysgYi9Tb3Vy
Y2UvV2ViQ29yZS9pbnNwZWN0b3IvZnJvbnQtZW5kL1NwZWN0cnVtLmpzCkBAIC0zOCw2ICszOCw4
IEBAIFdlYkluc3BlY3Rvci5TcGVjdHJ1bSA9IGZ1bmN0aW9uKCkKIAogICAgIHRoaXMuX2NvbnRh
aW5lckVsZW1lbnQgPSBkb2N1bWVudC5jcmVhdGVFbGVtZW50KCdkaXYnKTsKICAgICB0aGlzLl9j
b250YWluZXJFbGVtZW50LmNsYXNzTmFtZSA9ICJzcGVjdHJ1bS1jb250YWluZXIiOworICAgIHRo
aXMuX2NvbnRhaW5lckVsZW1lbnQudGFiSW5kZXggPSAwOworICAgIHRoaXMuX2NvbnRhaW5lckVs
ZW1lbnQuYWRkRXZlbnRMaXN0ZW5lcigia2V5ZG93biIsIHRoaXMuX29uS2V5RG93bi5iaW5kKHRo
aXMpLCBmYWxzZSk7CiAKICAgICB2YXIgdG9wRWxlbWVudCA9IHRoaXMuX2NvbnRhaW5lckVsZW1l
bnQuY3JlYXRlQ2hpbGQoImRpdiIsICJzcGVjdHJ1bS10b3AiKTsKICAgICB0b3BFbGVtZW50LmNy
ZWF0ZUNoaWxkKCJkaXYiLCAic3BlY3RydW0tZmlsbCIpOwpAQCAtNDA1LDExICs0MDcsMTUgQEAg
V2ViSW5zcGVjdG9yLlNwZWN0cnVtLnByb3RvdHlwZSA9IHsKIAogICAgIHJlcG9zaXRpb246IGZ1
bmN0aW9uKGVsZW1lbnQpCiAgICAgeworICAgICAgICB0aGlzLl9wcmV2aW91c0ZvY3VzRWxlbWVu
dCA9IFdlYkluc3BlY3Rvci5jdXJyZW50Rm9jdXNFbGVtZW50KCk7CiAgICAgICAgIHRoaXMuX3Bv
cG92ZXIuc2hvdyh0aGlzLl9jb250YWluZXJFbGVtZW50LCBlbGVtZW50KTsKKyAgICAgICAgV2Vi
SW5zcGVjdG9yLm1hcmtCZWluZ0VkaXRlZCh0aGlzLl9jb250YWluZXJFbGVtZW50LCB0cnVlKTsg
ICAgCisgICAgICAgIFdlYkluc3BlY3Rvci5zZXRDdXJyZW50Rm9jdXNFbGVtZW50KHRoaXMuX2Nv
bnRhaW5lckVsZW1lbnQpOwogICAgIH0sCiAKICAgICBoaWRlOiBmdW5jdGlvbigpCiAgICAgewor
ICAgICAgICBXZWJJbnNwZWN0b3IubWFya0JlaW5nRWRpdGVkKHRoaXMuX2NvbnRhaW5lckVsZW1l
bnQsIGZhbHNlKTsKICAgICAgICAgdGhpcy5fcG9wb3Zlci5oaWRlKCk7CiAKICAgICAgICAgZG9j
dW1lbnQucmVtb3ZlRXZlbnRMaXN0ZW5lcigibW91c2Vkb3duIiwgdGhpcy5faGlkZVByb3h5LCBm
YWxzZSk7CkBAIC00MTcsOCArNDIzLDE5IEBAIFdlYkluc3BlY3Rvci5TcGVjdHJ1bS5wcm90b3R5
cGUgPSB7CiAKICAgICAgICAgdGhpcy5kaXNwYXRjaEV2ZW50VG9MaXN0ZW5lcnMoV2ViSW5zcGVj
dG9yLlNwZWN0cnVtLkV2ZW50cy5IaWRkZW4pOwogCisgICAgICAgIFdlYkluc3BlY3Rvci5zZXRD
dXJyZW50Rm9jdXNFbGVtZW50KHRoaXMuX3ByZXZpb3VzRm9jdXNFbGVtZW50KTsKKwogICAgICAg
ICBkZWxldGUgdGhpcy5hbmNob3JFbGVtZW50OworICAgIH0sCisKKyAgICBfb25LZXlEb3duOiBm
dW5jdGlvbihldmVudCkKKyAgICB7CisgICAgICAgIGlmIChldmVudC5rZXlJZGVudGlmaWVyID09
PSAiRW50ZXIiIHx8IGV2ZW50LmtleUlkZW50aWZpZXIgPT09ICJVKzAwMUIiKSB7IC8vIEVzY2Fw
ZSBrZXkKKyAgICAgICAgICAgIHRoaXMuaGlkZSgpOworICAgICAgICAgICAgZXZlbnQuc3RvcFBy
b3BhZ2F0aW9uKCk7CisgICAgICAgICAgICBldmVudC5wcmV2ZW50RGVmYXVsdCgpOworICAgICAg
ICB9CiAgICAgfQotfTsKK30KIAogV2ViSW5zcGVjdG9yLlNwZWN0cnVtLnByb3RvdHlwZS5fX3By
b3RvX18gPSBXZWJJbnNwZWN0b3IuT2JqZWN0LnByb3RvdHlwZTsK
</data>
<flag name="review"
          id="132166"
          type_id="1"
          status="+"
          setter="vsevik"
    />
          </attachment>
      

    </bug>

</bugzilla>