<?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>38632</bug_id>
          
          <creation_ts>2010-05-06 01:39:53 -0700</creation_ts>
          <short_desc>[DRT/Chromium] ImageDiff should depend on WebKit</short_desc>
          <delta_ts>2010-05-06 10:26: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>New Bugs</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>0</everconfirmed>
          <reporter>jochen</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>abarth</cc>
    
    <cc>commit-queue</cc>
    
    <cc>dglazkov</cc>
    
    <cc>eric</cc>
    
    <cc>tkent</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>221568</commentid>
    <comment_count>0</comment_count>
    <who name="">jochen</who>
    <bug_when>2010-05-06 01:39:53 -0700</bug_when>
    <thetext>[DRT/Chromium] ImageDiff should depend on WebKit</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>221571</commentid>
    <comment_count>1</comment_count>
      <attachid>55216</attachid>
    <who name="">jochen</who>
    <bug_when>2010-05-06 01:46:45 -0700</bug_when>
    <thetext>Created attachment 55216
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>221572</commentid>
    <comment_count>2</comment_count>
      <attachid>55217</attachid>
    <who name="">jochen</who>
    <bug_when>2010-05-06 01:48:06 -0700</bug_when>
    <thetext>Created attachment 55217
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>221581</commentid>
    <comment_count>3</comment_count>
    <who name="">jochen</who>
    <bug_when>2010-05-06 01:55:42 -0700</bug_when>
    <thetext>If you compile ImageDiff from within chromium with shared libraries, you run into undefined symbols: CurrentTime and ChromiumBridge::callOnMainThread. That&apos;s because these symbols are declared in WTF (and used) but defined in WebKit/chromium/src/ChromiumBridge.cpp</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>221586</commentid>
    <comment_count>4</comment_count>
      <attachid>55217</attachid>
    <who name="Kent Tamura">tkent</who>
    <bug_when>2010-05-06 02:11:43 -0700</bug_when>
    <thetext>Comment on attachment 55217
Patch

This change looks ok.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>221695</commentid>
    <comment_count>5</comment_count>
      <attachid>55217</attachid>
    <who name="Dimitri Glazkov (Google)">dglazkov</who>
    <bug_when>2010-05-06 08:31:48 -0700</bug_when>
    <thetext>Comment on attachment 55217
Patch

ok.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>221749</commentid>
    <comment_count>6</comment_count>
      <attachid>55217</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-05-06 09:46:50 -0700</bug_when>
    <thetext>Comment on attachment 55217
Patch

Clearing flags on attachment: 55217

Committed r58889: &lt;http://trac.webkit.org/changeset/58889&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>221750</commentid>
    <comment_count>7</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-05-06 09:46:56 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>221784</commentid>
    <comment_count>8</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2010-05-06 10:26:23 -0700</bug_when>
    <thetext>http://trac.webkit.org/changeset/58889 might have broken GTK Linux 64-bit Release</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>55216</attachid>
            <date>2010-05-06 01:46:45 -0700</date>
            <delta_ts>2010-05-06 01:48:01 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-38632-20100506104642.patch</filename>
            <type>text/plain</type>
            <size>1397</size>
            <attacher>jochen</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL0NoYW5nZUxvZyBiL1dlYktpdFRvb2xzL0NoYW5nZUxv
ZwppbmRleCBjMDZlNzg4NmZiYmQ4ODcxMDI1OTI1MjcxODUzZDM5NjA2NDMwMmYyLi5kYmZiOWMx
YjBlYTVkNGZmMDU5YmU0NWFmZGNkMjFlNTI4MmNhMjFkIDEwMDY0NAotLS0gYS9XZWJLaXRUb29s
cy9DaGFuZ2VMb2cKKysrIGIvV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTIgQEAK
KzIwMTAtMDUtMDYgIEpvY2hlbiBFaXNpbmdlciAgPGpvY2hlbkBjaHJvbWl1bS5vcmc+CisKKyAg
ICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgTWFrZSBJbWFnZURp
ZmYgZGVwZW5kIG9uIFdlYktpdC4gV2hlbiBjb21waWxlZCBmcm9tIHdpdGhpbiBDaHJvbWl1bSwg
V1RGIGlzIG5vdCBhIHN0YW5kYWxvbmUgZHluYW1pYyBsaWJyYXJ5IGJ1dCBkZXBlbmRzIG9uIFdl
YktpdC4KKwlodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9Mzg2MzIKKwor
ICAgICAgICAqIER1bXBSZW5kZXJUcmVlL0R1bXBSZW5kZXJUcmVlLmd5cC9EdW1wUmVuZGVyVHJl
ZS5neXA6CisKIDIwMTAtMDUtMDUgIEpvc2VwaCBQZWNvcmFybyAgPGpvZXBlY2tAd2Via2l0Lm9y
Zz4KIAogICAgICAgICBSZXZpZXdlZCBieSBQYXZlbCBGZWxkbWFuLgpkaWZmIC0tZ2l0IGEvV2Vi
S2l0VG9vbHMvRHVtcFJlbmRlclRyZWUvRHVtcFJlbmRlclRyZWUuZ3lwL0R1bXBSZW5kZXJUcmVl
Lmd5cCBiL1dlYktpdFRvb2xzL0R1bXBSZW5kZXJUcmVlL0R1bXBSZW5kZXJUcmVlLmd5cC9EdW1w
UmVuZGVyVHJlZS5neXAKaW5kZXggYWNjZjhlYTljMmYyMWQ0MjFhZDdlZDkyYzliNjhjNWNiOTFj
MTYzNi4uMGM2YjhiZGE2YmM5MzlkZjVlYjYzMWEwNGNjZjU3MDg2YTM4NmI0ZSAxMDA2NDQKLS0t
IGEvV2ViS2l0VG9vbHMvRHVtcFJlbmRlclRyZWUvRHVtcFJlbmRlclRyZWUuZ3lwL0R1bXBSZW5k
ZXJUcmVlLmd5cAorKysgYi9XZWJLaXRUb29scy9EdW1wUmVuZGVyVHJlZS9EdW1wUmVuZGVyVHJl
ZS5neXAvRHVtcFJlbmRlclRyZWUuZ3lwCkBAIC0xNDcsNiArMTQ3LDcgQEAKICAgICAgICAgICAg
ICd0YXJnZXRfbmFtZSc6ICdJbWFnZURpZmYnLAogICAgICAgICAgICAgJ3R5cGUnOiAnZXhlY3V0
YWJsZScsCiAgICAgICAgICAgICAnZGVwZW5kZW5jaWVzJzogWworICAgICAgICAgICAgICAgICc8
KHdlYmtpdF9hcGlfZGlyKS9XZWJLaXQuZ3lwOndlYmtpdCcsCiAgICAgICAgICAgICAgICAgJzwo
d2Via2l0X3RvcCkvSmF2YVNjcmlwdENvcmUvSmF2YVNjcmlwdENvcmUuZ3lwL0phdmFTY3JpcHRD
b3JlLmd5cDp3dGYnLAogICAgICAgICAgICAgICAgICc8KGNocm9taXVtX3NyY19kaXIpL2dmeC9n
ZnguZ3lwOmdmeCcsCiAgICAgICAgICAgICBdLAo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>55217</attachid>
            <date>2010-05-06 01:48:06 -0700</date>
            <delta_ts>2010-05-06 09:46:50 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-38632-20100506104804.patch</filename>
            <type>text/plain</type>
            <size>1404</size>
            <attacher>jochen</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL0NoYW5nZUxvZyBiL1dlYktpdFRvb2xzL0NoYW5nZUxv
ZwppbmRleCBjMDZlNzg4NmZiYmQ4ODcxMDI1OTI1MjcxODUzZDM5NjA2NDMwMmYyLi41NTJmOTBh
NTJiNDkzOTcyYzdiMDMzMTllNzAzNWM3YWRlMzNmOWM5IDEwMDY0NAotLS0gYS9XZWJLaXRUb29s
cy9DaGFuZ2VMb2cKKysrIGIvV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTIgQEAK
KzIwMTAtMDUtMDYgIEpvY2hlbiBFaXNpbmdlciAgPGpvY2hlbkBjaHJvbWl1bS5vcmc+CisKKyAg
ICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgTWFrZSBJbWFnZURp
ZmYgZGVwZW5kIG9uIFdlYktpdC4gV2hlbiBjb21waWxlZCBmcm9tIHdpdGhpbiBDaHJvbWl1bSwg
V1RGIGlzIG5vdCBhIHN0YW5kYWxvbmUgZHluYW1pYyBsaWJyYXJ5IGJ1dCBkZXBlbmRzIG9uIFdl
YktpdC4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTM4
NjMyCisKKyAgICAgICAgKiBEdW1wUmVuZGVyVHJlZS9EdW1wUmVuZGVyVHJlZS5neXAvRHVtcFJl
bmRlclRyZWUuZ3lwOgorCiAyMDEwLTA1LTA1ICBKb3NlcGggUGVjb3Jhcm8gIDxqb2VwZWNrQHdl
YmtpdC5vcmc+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgUGF2ZWwgRmVsZG1hbi4KZGlmZiAtLWdp
dCBhL1dlYktpdFRvb2xzL0R1bXBSZW5kZXJUcmVlL0R1bXBSZW5kZXJUcmVlLmd5cC9EdW1wUmVu
ZGVyVHJlZS5neXAgYi9XZWJLaXRUb29scy9EdW1wUmVuZGVyVHJlZS9EdW1wUmVuZGVyVHJlZS5n
eXAvRHVtcFJlbmRlclRyZWUuZ3lwCmluZGV4IGFjY2Y4ZWE5YzJmMjFkNDIxYWQ3ZWQ5MmM5YjY4
YzVjYjkxYzE2MzYuLjBjNmI4YmRhNmJjOTM5ZGY1ZWI2MzFhMDRjY2Y1NzA4NmEzODZiNGUgMTAw
NjQ0Ci0tLSBhL1dlYktpdFRvb2xzL0R1bXBSZW5kZXJUcmVlL0R1bXBSZW5kZXJUcmVlLmd5cC9E
dW1wUmVuZGVyVHJlZS5neXAKKysrIGIvV2ViS2l0VG9vbHMvRHVtcFJlbmRlclRyZWUvRHVtcFJl
bmRlclRyZWUuZ3lwL0R1bXBSZW5kZXJUcmVlLmd5cApAQCAtMTQ3LDYgKzE0Nyw3IEBACiAgICAg
ICAgICAgICAndGFyZ2V0X25hbWUnOiAnSW1hZ2VEaWZmJywKICAgICAgICAgICAgICd0eXBlJzog
J2V4ZWN1dGFibGUnLAogICAgICAgICAgICAgJ2RlcGVuZGVuY2llcyc6IFsKKyAgICAgICAgICAg
ICAgICAnPCh3ZWJraXRfYXBpX2RpcikvV2ViS2l0Lmd5cDp3ZWJraXQnLAogICAgICAgICAgICAg
ICAgICc8KHdlYmtpdF90b3ApL0phdmFTY3JpcHRDb3JlL0phdmFTY3JpcHRDb3JlLmd5cC9KYXZh
U2NyaXB0Q29yZS5neXA6d3RmJywKICAgICAgICAgICAgICAgICAnPChjaHJvbWl1bV9zcmNfZGly
KS9nZngvZ2Z4Lmd5cDpnZngnLAogICAgICAgICAgICAgXSwK
</data>

          </attachment>
      

    </bug>

</bugzilla>