<?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>50713</bug_id>
          
          <creation_ts>2010-12-08 13:43:41 -0800</creation_ts>
          <short_desc>[Qt] Move to BUILDING_WEBKIT flag as used by other ports (and away from BUILD_WEBKIT)</short_desc>
          <delta_ts>2010-12-09 04:37:33 -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>WebKit Qt</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>Qt</keywords>
          <priority>P3</priority>
          <bug_severity>Minor</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Siddharth Mathur">s.mathur</reporter>
          <assigned_to name="Siddharth Mathur">s.mathur</assigned_to>
          <cc>abarth</cc>
    
    <cc>ademar</cc>
    
    <cc>commit-queue</cc>
    
    <cc>eric</cc>
    
    <cc>hausmann</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>318997</commentid>
    <comment_count>0</comment_count>
    <who name="Siddharth Mathur">s.mathur</who>
    <bug_when>2010-12-08 13:43:41 -0800</bug_when>
    <thetext>All Webkit ports except Qt use BUILDING_WEBKIT to signal that Webkit&apos;s shared library/DLL is being compiled. We should preferably stick to this convention and not use our own BUILD_WEBKIT</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>319215</commentid>
    <comment_count>1</comment_count>
      <attachid>76003</attachid>
    <who name="Siddharth Mathur">s.mathur</who>
    <bug_when>2010-12-08 19:21:42 -0800</bug_when>
    <thetext>Created attachment 76003
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>319297</commentid>
    <comment_count>2</comment_count>
      <attachid>76003</attachid>
    <who name="Laszlo Gombos">laszlo.gombos</who>
    <bug_when>2010-12-08 23:54:16 -0800</bug_when>
    <thetext>Comment on attachment 76003
Patch

LGTM, r+. 

Next time please add &quot;[Qt] &quot;  as a prefix of the title of the ChangeLog/bug if the change is specific to the Qt port.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>319371</commentid>
    <comment_count>3</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-12-09 03:11:23 -0800</bug_when>
    <thetext>The commit-queue encountered the following flaky tests while processing attachment 76003:

http/tests/appcache/update-cache.html

Please file bugs against the tests.  These tests were authored by ap@webkit.org.  The commit-queue is continuing to process your patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>319373</commentid>
    <comment_count>4</comment_count>
      <attachid>76003</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-12-09 03:13:09 -0800</bug_when>
    <thetext>Comment on attachment 76003
Patch

Clearing flags on attachment: 76003

Committed r73602: &lt;http://trac.webkit.org/changeset/73602&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>319374</commentid>
    <comment_count>5</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-12-09 03:13:14 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>319402</commentid>
    <comment_count>6</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2010-12-09 04:37:33 -0800</bug_when>
    <thetext>http://trac.webkit.org/changeset/73602 might have broken GTK Linux 64-bit Debug
The following tests are not passing:
editing/selection/extend-by-character-002.html</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>76003</attachid>
            <date>2010-12-08 19:21:42 -0800</date>
            <delta_ts>2010-12-09 03:13:09 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>50713.patch</filename>
            <type>text/plain</type>
            <size>1962</size>
            <attacher name="Siddharth Mathur">s.mathur</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg
YjZlMjMwMC4uNjdmOTQ3YyAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvV2Vi
Q29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxMiBAQAorMjAxMC0xMi0wOCAgU2lkZGhhcnRoIE1h
dGh1ciAgPHNpZGRoYXJ0aC5tYXRodXJAbm9raWEuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5
IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFVzZSBCVUlMRElOR19XRUJLSVQgbGlrZSBvdGhl
ciBwb3J0cworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9
NTA3MTMKKworICAgICAgICAqIFdlYkNvcmUucHJvOiBDaGFuZ2UgQlVJTERfV0VCS0lUIHRvIEJV
SUxESU5HX1dFQktJVAorCiAyMDEwLTEyLTA4ICBTYW0gV2VpbmlnICA8c2FtQHdlYmtpdC5vcmc+
CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgRGFyaW4gQWRsZXIuCmRpZmYgLS1naXQgYS9XZWJDb3Jl
L1dlYkNvcmUucHJvIGIvV2ViQ29yZS9XZWJDb3JlLnBybwppbmRleCAwZjg4NTM4Li4wZWFmZjVh
IDEwMDY0NAotLS0gYS9XZWJDb3JlL1dlYkNvcmUucHJvCisrKyBiL1dlYkNvcmUvV2ViQ29yZS5w
cm8KQEAgLTEyNSw3ICsxMjUsNyBAQCBDT05GSUcocmVsZWFzZSk6IUNPTkZJRyhzdGFuZGFsb25l
X3BhY2thZ2UpIHsKICAgICB1bml4OmNvbnRhaW5zKFFUX0NPTkZJRywgcmVkdWNlX3JlbG9jYXRp
b25zKTpDT05GSUcgKz0gYnN5bWJvbGljX2Z1bmN0aW9ucwogfQogCi1ERUZJTkVTICs9IEJVSUxE
X1dFQktJVAorREVGSU5FUyArPSBCVUlMRElOR19XRUJLSVQKIAogIyBSZW1vdmUgd2hvbGUgcHJv
Z3JhbSBvcHRpbWl6YXRpb25zIGR1ZSB0byBtaXNjb21waWxhdGlvbnMKIHdpbjMyLW1zdmMyMDA1
fHdpbjMyLW1zdmMyMDA4fHdpbmNlKjp7CmRpZmYgLS1naXQgYS9XZWJLaXQvcXQvQXBpL3F3ZWJr
aXRnbG9iYWwuaCBiL1dlYktpdC9xdC9BcGkvcXdlYmtpdGdsb2JhbC5oCmluZGV4IGE2YmU1NzEu
LjdlZTU2ZGUgMTAwNjQ0Ci0tLSBhL1dlYktpdC9xdC9BcGkvcXdlYmtpdGdsb2JhbC5oCisrKyBi
L1dlYktpdC9xdC9BcGkvcXdlYmtpdGdsb2JhbC5oCkBAIC0yOSw3ICsyOSw3IEBACiAjZGVmaW5l
IFFUV0VCS0lUX1ZFUlNJT05fQ0hFQ0sobWFqb3IsIG1pbm9yLCBwYXRjaCkgKChtYWpvcjw8MTYp
fChtaW5vcjw8OCl8KHBhdGNoKSkKIAogI2lmIGRlZmluZWQoUVRfTUFLRURMTCkgICAgICAgIC8q
IGNyZWF0ZSBhIFF0IERMTCBsaWJyYXJ5ICovCi0jICBpZiBkZWZpbmVkKEJVSUxEX1dFQktJVCkK
KyMgIGlmIGRlZmluZWQoQlVJTERJTkdfV0VCS0lUKQogIyAgICAgIGRlZmluZSBRV0VCS0lUX0VY
UE9SVCBRX0RFQ0xfRVhQT1JUCiAjICBlbHNlCiAjICAgICAgZGVmaW5lIFFXRUJLSVRfRVhQT1JU
IFFfREVDTF9JTVBPUlQKZGlmZiAtLWdpdCBhL1dlYktpdC9xdC9DaGFuZ2VMb2cgYi9XZWJLaXQv
cXQvQ2hhbmdlTG9nCmluZGV4IDQ1ZDQxNmQuLjM5MGYzMjEgMTAwNjQ0Ci0tLSBhL1dlYktpdC9x
dC9DaGFuZ2VMb2cKKysrIGIvV2ViS2l0L3F0L0NoYW5nZUxvZwpAQCAtMSwzICsxLDEyIEBACisy
MDEwLTEyLTA4ICBTaWRkaGFydGggTWF0aHVyICA8c2lkZGhhcnRoLm1hdGh1ckBub2tpYS5jb20+
CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgVXNlIEJV
SUxESU5HX1dFQktJVCBsaWtlIG90aGVyIHBvcnRzCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJr
aXQub3JnL3Nob3dfYnVnLmNnaT9pZD01MDcxMworCisgICAgICAgICogQXBpL3F3ZWJraXRnbG9i
YWwuaDogY2hhbmdlIEJVSUxEX1dFQktJVCB0byBCVUlMRElOR19XRUJLSVQKKwogMjAxMC0xMi0w
NyAgQnJpYW4gV2VpbnN0ZWluICA8YndlaW5zdGVpbkBhcHBsZS5jb20+CiAKICAgICAgICAgUmV2
aWV3ZWQgYnkgSm9obiBTdWxsaXZhbi4K
</data>

          </attachment>
      

    </bug>

</bugzilla>