<?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>34853</bug_id>
          
          <creation_ts>2010-02-11 10:52:36 -0800</creation_ts>
          <short_desc>Web Inspector: restore highlighting animation, bring back keyboard navigation, beautify search results.</short_desc>
          <delta_ts>2010-02-12 05: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>1</everconfirmed>
          <reporter name="Pavel Feldman">pfeldman</reporter>
          <assigned_to name="Pavel Feldman">pfeldman</assigned_to>
          <cc>bweinstein</cc>
    
    <cc>joepeck</cc>
    
    <cc>rik</cc>
    
    <cc>timothy</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>189743</commentid>
    <comment_count>0</comment_count>
    <who name="Pavel Feldman">pfeldman</who>
    <bug_when>2010-02-11 10:52:36 -0800</bug_when>
    <thetext>Fixing nits.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>189744</commentid>
    <comment_count>1</comment_count>
      <attachid>48583</attachid>
    <who name="Pavel Feldman">pfeldman</who>
    <bug_when>2010-02-11 10:53:57 -0800</bug_when>
    <thetext>Created attachment 48583
[PATCH] Proposed change.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>189790</commentid>
    <comment_count>2</comment_count>
      <attachid>48583</attachid>
    <who name="Oliver Hunt">oliver</who>
    <bug_when>2010-02-11 14:02:37 -0800</bug_when>
    <thetext>Comment on attachment 48583
[PATCH] Proposed change.

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>189796</commentid>
    <comment_count>3</comment_count>
      <attachid>48583</attachid>
    <who name="Timothy Hatcher">timothy</who>
    <bug_when>2010-02-11 14:15:54 -0800</bug_when>
    <thetext>Comment on attachment 48583
[PATCH] Proposed change.

Why not fade to transparent?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>189797</commentid>
    <comment_count>4</comment_count>
      <attachid>48583</attachid>
    <who name="Timothy Hatcher">timothy</who>
    <bug_when>2010-02-11 14:16:36 -0800</bug_when>
    <thetext>Comment on attachment 48583
[PATCH] Proposed change.

&gt; +    -webkit-animation: &apos;fadeout&apos; 2s 0s;

Can you use double quotes here, to match the other quotes in the file.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>190022</commentid>
    <comment_count>5</comment_count>
    <who name="Pavel Feldman">pfeldman</who>
    <bug_when>2010-02-12 05:17:12 -0800</bug_when>
    <thetext>Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	WebCore/ChangeLog
	M	WebCore/inspector/front-end/TextViewer.js
	M	WebCore/inspector/front-end/textViewer.css
Committed r54720

(In reply to comment #4)
&gt; (From update of attachment 48583 [details])
&gt; &gt; +    -webkit-animation: &apos;fadeout&apos; 2s 0s;
&gt; 
&gt; Can you use double quotes here, to match the other quotes in the file.

Done.

(In reply to comment #3)
&gt; (From update of attachment 48583 [details])
&gt; Why not fade to transparent?

I just took the old SourceFrame code...</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>48583</attachid>
            <date>2010-02-11 10:53:57 -0800</date>
            <delta_ts>2010-02-11 14:16:36 -0800</delta_ts>
            <desc>[PATCH] Proposed change.</desc>
            <filename>nits</filename>
            <type>text/plain</type>
            <size>2384</size>
            <attacher name="Pavel Feldman">pfeldman</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg
ZjNlNDFkMy4uZTRiZTdmNSAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvV2Vi
Q29yZS9DaGFuZ2VMb2cKQEAgLTEsNSArMSwxOCBAQAogMjAxMC0wMi0xMSAgUGF2ZWwgRmVsZG1h
biAgPHBmZWxkbWFuQGNocm9taXVtLm9yZz4KIAorICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkg
KE9PUFMhKS4KKworICAgICAgICBXZWIgSW5zcGVjdG9yOiByZXN0b3JlIGhpZ2hsaWdodGluZyBh
bmltYXRpb24sIGJyaW5nIGJhY2sga2V5Ym9hcmQgbmF2aWdhdGlvbiwgYmVhdXRpZnkgc2VhcmNo
IHJlc3VsdHMuCisKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dp
P2lkPTM0ODUzCisKKyAgICAgICAgKiBpbnNwZWN0b3IvZnJvbnQtZW5kL1RleHRWaWV3ZXIuanM6
CisgICAgICAgIChXZWJJbnNwZWN0b3IuVGV4dFZpZXdlcik6CisgICAgICAgIChXZWJJbnNwZWN0
b3IuVGV4dFZpZXdlci5wcm90b3R5cGUuaGlnaGxpZ2h0TGluZSk6CisgICAgICAgICogaW5zcGVj
dG9yL2Zyb250LWVuZC90ZXh0Vmlld2VyLmNzczoKKworMjAxMC0wMi0xMSAgUGF2ZWwgRmVsZG1h
biAgPHBmZWxkbWFuQGNocm9taXVtLm9yZz4KKwogICAgICAgICBSZXZpZXdlZCBieSBUaW1vdGh5
IEhhdGNoZXIuCiAKICAgICAgICAgV2ViIEluc3BlY3Rvcjogc3BsaXQgc291cmNlIGNvZGUgaW50
byBjaHVua3MgaW4gb3JkZXIgdG8gaW1wcm92ZSB0ZXh0CmRpZmYgLS1naXQgYS9XZWJDb3JlL2lu
c3BlY3Rvci9mcm9udC1lbmQvVGV4dFZpZXdlci5qcyBiL1dlYkNvcmUvaW5zcGVjdG9yL2Zyb250
LWVuZC9UZXh0Vmlld2VyLmpzCmluZGV4IGNlZDcyMzIuLjA5NjQ2NGYgMTAwNjQ0Ci0tLSBhL1dl
YkNvcmUvaW5zcGVjdG9yL2Zyb250LWVuZC9UZXh0Vmlld2VyLmpzCisrKyBiL1dlYkNvcmUvaW5z
cGVjdG9yL2Zyb250LWVuZC9UZXh0Vmlld2VyLmpzCkBAIC0zNiw2ICszNiw3IEBAIFdlYkluc3Bl
Y3Rvci5UZXh0Vmlld2VyID0gZnVuY3Rpb24odGV4dE1vZGVsLCBwbGF0Zm9ybSwgdXJsKQogCiAg
ICAgdGhpcy5lbGVtZW50ID0gZG9jdW1lbnQuY3JlYXRlRWxlbWVudCgiZGl2Iik7CiAgICAgdGhp
cy5lbGVtZW50LmNsYXNzTmFtZSA9ICJ0ZXh0LWVkaXRvciBtb25vc3BhY2UiOworICAgIHRoaXMu
ZWxlbWVudC50YWJJbmRleCA9IDA7CiAKICAgICB0aGlzLmVsZW1lbnQuYWRkRXZlbnRMaXN0ZW5l
cigic2Nyb2xsIiwgdGhpcy5fc2Nyb2xsLmJpbmQodGhpcyksIGZhbHNlKTsKIApAQCAtMTAwLDcg
KzEwMSw2IEBAIFdlYkluc3BlY3Rvci5UZXh0Vmlld2VyLnByb3RvdHlwZSA9IHsKIAogICAgIGhp
Z2hsaWdodExpbmU6IGZ1bmN0aW9uKGxpbmVOdW1iZXIpCiAgICAgewotICAgICAgICAvLyBGSVhN
RTogcmVzdG9yZSBhbmltYXRpb24uCiAgICAgICAgIGlmICh0eXBlb2YgdGhpcy5faGlnaGxpZ2h0
ZWRMaW5lID09PSAibnVtYmVyIikgewogICAgICAgICAgICAgdmFyIGNodW5rID0gdGhpcy5fbWFr
ZUxpbmVBQ2h1bmsodGhpcy5faGlnaGxpZ2h0ZWRMaW5lKTsKICAgICAgICAgICAgIGNodW5rLnJl
bW92ZURlY29yYXRpb24oIndlYmtpdC1oaWdobGlnaHRlZC1saW5lIik7CmRpZmYgLS1naXQgYS9X
ZWJDb3JlL2luc3BlY3Rvci9mcm9udC1lbmQvdGV4dFZpZXdlci5jc3MgYi9XZWJDb3JlL2luc3Bl
Y3Rvci9mcm9udC1lbmQvdGV4dFZpZXdlci5jc3MKaW5kZXggNTYzYzYxYi4uYjAzYWY0NCAxMDA2
NDQKLS0tIGEvV2ViQ29yZS9pbnNwZWN0b3IvZnJvbnQtZW5kL3RleHRWaWV3ZXIuY3NzCisrKyBi
L1dlYkNvcmUvaW5zcGVjdG9yL2Zyb250LWVuZC90ZXh0Vmlld2VyLmNzcwpAQCAtMTMxLDkgKzEz
MSwxOSBAQAogfQogCiAud2Via2l0LW1hcmt1cCB7CisgICAgLXdlYmtpdC1ib3JkZXItcmFkaXVz
OiA0cHg7CisgICAgcGFkZGluZzogMnB4IDFweCAycHggM3B4OworICAgIG1hcmdpbi1sZWZ0OiAt
NHB4OworICAgIG1hcmdpbi10b3A6IC0ycHg7CisgICAgLXdlYmtpdC1ib3gtc2hhZG93OiByZ2Jh
KDAsIDAsIDAsIC41KSAzcHggM3B4IDRweDsKICAgICBiYWNrZ3JvdW5kLWNvbG9yOiByZ2IoMjQx
LCAyMzQsIDApOwogfQogCiAud2Via2l0LWhpZ2hsaWdodGVkLWxpbmUgLndlYmtpdC1saW5lLWNv
bnRlbnQgewotICAgIGJhY2tncm91bmQtY29sb3I6IHJnYigyNTUsIDI1NSwgMTIwKTsKKyAgICAt
d2Via2l0LWFuaW1hdGlvbjogJ2ZhZGVvdXQnIDJzIDBzOworfQorCitALXdlYmtpdC1rZXlmcmFt
ZXMgZmFkZW91dCB7CisgICAgZnJvbSB7YmFja2dyb3VuZC1jb2xvcjogcmdiKDI1NSwgMjU1LCAx
MjApOyB9CisgICAgdG8geyBiYWNrZ3JvdW5kLWNvbG9yOiB3aGl0ZTsgfQogfQo=
</data>
<flag name="review"
          id="31293"
          type_id="1"
          status="+"
          setter="oliver"
    />
          </attachment>
      

    </bug>

</bugzilla>