<?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>33410</bug_id>
          
          <creation_ts>2010-01-08 16:25:40 -0800</creation_ts>
          <short_desc>Fix &quot;View Plain Diff&quot;/&quot;View Formatted Diff&quot; button</short_desc>
          <delta_ts>2010-01-08 18:17:34 -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>Tools / Tests</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Other</rep_platform>
          <op_sys>OS X 10.5</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="David Kilzer (:ddkilzer)">ddkilzer</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>abarth</cc>
    
    <cc>aroben</cc>
    
    <cc>eric</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>178868</commentid>
    <comment_count>0</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2010-01-08 16:25:40 -0800</bug_when>
    <thetext>Fix &quot;View Plain Diff&quot;/&quot;View Formatted Diff&quot; button</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>178869</commentid>
    <comment_count>1</comment_count>
      <attachid>46175</attachid>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2010-01-08 16:26:09 -0800</bug_when>
    <thetext>Created attachment 46175
Patch v1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>178878</commentid>
    <comment_count>2</comment_count>
      <attachid>46175</attachid>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-01-08 16:47:45 -0800</bug_when>
    <thetext>Comment on attachment 46175
Patch v1

Looks good.  Kind of fun to know how bugzilla works inside.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>178914</commentid>
    <comment_count>3</comment_count>
    <who name="David Kilzer (:ddkilzer)">ddkilzer</who>
    <bug_when>2010-01-08 18:17:34 -0800</bug_when>
    <thetext>Committed r53033: &lt;http://trac.webkit.org/changeset/53033&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>46175</attachid>
            <date>2010-01-08 16:26:09 -0800</date>
            <delta_ts>2010-01-08 16:47:45 -0800</delta_ts>
            <desc>Patch v1</desc>
            <filename>bug-33410-20100108162608.patch</filename>
            <type>text/plain</type>
            <size>2536</size>
            <attacher name="David Kilzer (:ddkilzer)">ddkilzer</attacher>
            
              <data encoding="base64">SW5kZXg6IEJ1Z3NTaXRlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBCdWdzU2l0ZS9DaGFuZ2VM
b2cJKHJldmlzaW9uIDUzMDIwKQorKysgQnVnc1NpdGUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkp
CkBAIC0xLDMgKzEsMTYgQEAKKzIwMTAtMDEtMDggIERhdmlkIEtpbHplciAgPGRka2lsemVyQGFw
cGxlLmNvbT4KKworICAgICAgICBGaXggIlZpZXcgUGxhaW4gRGlmZiIvIlZpZXcgRm9ybWF0dGVk
IERpZmYiIGJ1dHRvbgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisg
ICAgICAgIFRoaXMgYnV0dG9uIHdhcyBicm9rZW4gZHVyaW5nIGEgcHJldmlvdXMgQnVnemlsbGEg
dXBncmFkZSBhZnRlcgorICAgICAgICBjaGFuZ2VzIHdlcmUgbWFkZSB0byB0aGUgbmFtZXMgb2Yg
b2JqZWN0cyBleHBvc2VkIG9uIHRlbXBsYXRlcy4KKworICAgICAgICAqIHRlbXBsYXRlL2VuL2N1
c3RvbS9hdHRhY2htZW50L3Jldmlld2Zvcm0uaHRtbC50bXBsOiBDaGFuZ2VkCisgICAgICAgIGF0
dGFjaGlkIHRvIGF0dGFjaG1lbnQuaWQgYW5kIGlzcGF0Y2ggdG8gYXR0YWNobWVudC5pc3BhdGNo
LgorICAgICAgICBBbHNvIHR3ZWFrZWQgdGhlIGZvcm1hdCB0byB1c2UgNCB0YWJsZSBjb2x1bW5z
IGluc3RlYWQgb2YgNS4KKwogMjAxMC0wMS0wOCAgRGF2aWQgS2lsemVyICA8ZGRraWx6ZXJAYXBw
bGUuY29tPgogCiAgICAgICAgIEluY2x1ZGVkIFVOQ09ORklSTUVEIGJ1Z3MgYnkgZGVmYXVsdCBp
biBhZHZhbmNlZCBzZWFyY2ggcXVlcnkKSW5kZXg6IEJ1Z3NTaXRlL3RlbXBsYXRlL2VuL2N1c3Rv
bS9hdHRhY2htZW50L3Jldmlld2Zvcm0uaHRtbC50bXBsCj09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIEJ1Z3NTaXRl
L3RlbXBsYXRlL2VuL2N1c3RvbS9hdHRhY2htZW50L3Jldmlld2Zvcm0uaHRtbC50bXBsCShyZXZp
c2lvbiA1MzAxNikKKysrIEJ1Z3NTaXRlL3RlbXBsYXRlL2VuL2N1c3RvbS9hdHRhY2htZW50L3Jl
dmlld2Zvcm0uaHRtbC50bXBsCSh3b3JraW5nIGNvcHkpCkBAIC0zNSw3ICszNSw3IEBACiAgIGZ1
bmN0aW9uIHZpZXdQcmV0dHlQYXRjaCgpCiAgICAgewogICAgICAgdmlld2luZ19mb3JtYXR0ZWRf
ZGlmZiA9ICF2aWV3aW5nX2Zvcm1hdHRlZF9kaWZmOwotICAgICAgdmFyIHNyYyA9ICJhdHRhY2ht
ZW50LmNnaT9pZD1bJSBhdHRhY2hpZCAlXSI7CisgICAgICB2YXIgc3JjID0gImF0dGFjaG1lbnQu
Y2dpP2lkPVslIGF0dGFjaG1lbnQuaWQgJV0iOwogICAgICAgdmFyIGJ1dHRvblRleHQgPSAiVmll
dyBGb3JtYXR0ZWQgRGlmZiI7CiAgICAgICBpZiAodmlld2luZ19mb3JtYXR0ZWRfZGlmZikgewog
ICAgICAgICBzcmMgKz0gIiZhY3Rpb249cHJldHR5cGF0Y2giCkBAIC0xMDAsMjIgKzEwMCwyMCBA
QAogWyUgRU5EICVdCiA8L3RkPgogPHRkPgotQmFjayB0byA8YSBocmVmPSJzaG93X2J1Zy5jZ2k/
aWQ9WyUgYXR0YWNobWVudC5idWdfaWQgJV0iIHRhcmdldD0iX3RvcCI+WyUgdGVybXMuQnVnICVd
ICNbJSBhdHRhY2htZW50LmJ1Z19pZCAlXTwvYT4KLTwvdGQ+Ci08dGQ+Ci0gICAgWyUgSUYgaXNw
YXRjaCAlXQotICAgICAgPHNjcmlwdCB0eXBlPSJ0ZXh0L2phdmFzY3JpcHQiPgotICAgICAgICA8
IS0tCisgICAgPGRpdj5CYWNrIHRvIDxhIGhyZWY9InNob3dfYnVnLmNnaT9pZD1bJSBhdHRhY2ht
ZW50LmJ1Z19pZCAlXSIgdGFyZ2V0PSJfdG9wIj5bJSB0ZXJtcy5CdWcgJV0gI1slIGF0dGFjaG1l
bnQuYnVnX2lkICVdPC9hPjwvZGl2PgorWyUgSUYgYXR0YWNobWVudC5pc3BhdGNoICVdCisgICAg
PHNjcmlwdCB0eXBlPSJ0ZXh0L2phdmFzY3JpcHQiPgorICAgIDwhLS0KICAgICAgICAgaWYgKHR5
cGVvZiBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCA9PSAiZnVuY3Rpb24iKSB7Ci0gICAgICAgICAg
ZG9jdW1lbnQud3JpdGUoJzxidXR0b24gdHlwZT0iYnV0dG9uIiBpZD0idmlld1ByZXR0eVBhdGNo
QnV0dG9uIiBvbmNsaWNrPSJ2aWV3UHJldHR5UGF0Y2goKTsiPlZpZXcgUGxhaW4gRGlmZjwvYnV0
dG9uPicpOworICAgICAgICAgIGRvY3VtZW50LndyaXRlKCc8cD48YnV0dG9uIHR5cGU9ImJ1dHRv
biIgaWQ9InZpZXdQcmV0dHlQYXRjaEJ1dHRvbiIgb25jbGljaz0idmlld1ByZXR0eVBhdGNoKCk7
Ij5WaWV3IFBsYWluIERpZmY8L2J1dHRvbj48L3A+Jyk7CiAgICAgICAgIH0KLSAgICAgICAgLy8t
LT4KLSAgICAgIDwvc2NyaXB0PgotICAgIFslIEVORCAlXQorICAgIC8vLS0+CisgICAgPC9zY3Jp
cHQ+CitbJSBFTkQgJV0KIDwvdGQ+CiA8L3RyPgogPHRyPgotPHRkIGNvbHNwYW49IjUiPgorPHRk
IGNvbHNwYW49IjQiPgogRW50ZXIgY29tbWVudHMgYmVsb3c6PGJyPgogPHRleHRhcmVhIGlkPWNv
bW1lbnQgbmFtZT0iY29tbWVudCIgd3JhcD0ic29mdCI+WyUgcXVvdGVkcGF0Y2ggRklMVEVSIGh0
bWwgJV08L3RleHRhcmVhPjxicj4KIDwvdHI+Cg==
</data>
<flag name="review"
          id="28398"
          type_id="1"
          status="+"
          setter="abarth"
    />
          </attachment>
      

    </bug>

</bugzilla>