<?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>62632</bug_id>
          
          <creation_ts>2011-06-14 06:35:29 -0700</creation_ts>
          <short_desc>[Qt] Don&apos;t include convenience &quot;QWebFoo&quot; headers in WebKit code.</short_desc>
          <delta_ts>2011-06-14 09:25:20 -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>Tools / Tests</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>Qt, QtTriaged</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Andreas Kling">kling</reporter>
          <assigned_to name="Andreas Kling">kling</assigned_to>
          <cc>ademar</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>420368</commentid>
    <comment_count>0</comment_count>
    <who name="Andreas Kling">kling</who>
    <bug_when>2011-06-14 06:35:29 -0700</bug_when>
    <thetext>We shouldn&apos;t make assumptions about these headers being available and pointing to the right qwebfoo.h</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>420370</commentid>
    <comment_count>1</comment_count>
      <attachid>97107</attachid>
    <who name="Andreas Kling">kling</who>
    <bug_when>2011-06-14 06:37:21 -0700</bug_when>
    <thetext>Created attachment 97107
Proposed patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>420371</commentid>
    <comment_count>2</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-06-14 06:38:38 -0700</bug_when>
    <thetext>Attachment 97107 did not pass style-queue:

Failed to run &quot;[&apos;Tools/Scripts/check-webkit-style&apos;, &apos;--diff-files&apos;, u&apos;Source/WebKit/qt/ChangeLog&apos;, u&apos;Source/WebK...&quot; exit_code: 1

Source/WebKit/qt/WebCoreSupport/FrameNetworkingContextQt.cpp:26:  Alphabetical sorting problem.  [build/include_order] [4]
Total errors found: 1 in 2 files


If any of these errors are false positives, please file a bug against check-webkit-style.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>420382</commentid>
    <comment_count>3</comment_count>
    <who name="Andreas Kling">kling</who>
    <bug_when>2011-06-14 06:54:21 -0700</bug_when>
    <thetext>Committed r88795: &lt;http://trac.webkit.org/changeset/88795&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>420484</commentid>
    <comment_count>4</comment_count>
    <who name="Ademar Reis">ademar</who>
    <bug_when>2011-06-14 09:25:20 -0700</bug_when>
    <thetext>Revision r88795 cherry-picked into qtwebkit-2.2 with commit ceea7a3 &lt;http://gitorious.org/webkit/qtwebkit/commit/ceea7a3&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>97107</attachid>
            <date>2011-06-14 06:37:21 -0700</date>
            <delta_ts>2011-06-14 06:38:25 -0700</delta_ts>
            <desc>Proposed patch</desc>
            <filename>bug-62632.diff</filename>
            <type>text/plain</type>
            <size>1038</size>
            <attacher name="Andreas Kling">kling</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvcXQvQ2hhbmdlTG9nIGIvU291cmNlL1dlYktpdC9x
dC9DaGFuZ2VMb2cKaW5kZXggYmE2NTEwOS4uOTIwNmI3MCAxMDA2NDQKLS0tIGEvU291cmNlL1dl
YktpdC9xdC9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYktpdC9xdC9DaGFuZ2VMb2cKQEAgLTEs
MyArMSwxMiBAQAorMjAxMS0wNi0xNCAgQW5kcmVhcyBLbGluZyAgPGtsaW5nQHdlYmtpdC5vcmc+
CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgW1F0XSBE
b24ndCBpbmNsdWRlIGNvbnZlbmllbmNlICJRV2ViRm9vIiBoZWFkZXJzIGluIFdlYktpdCBjb2Rl
LgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NjI2MzIK
KworICAgICAgICAqIFdlYkNvcmVTdXBwb3J0L0ZyYW1lTmV0d29ya2luZ0NvbnRleHRRdC5jcHA6
CisKIDIwMTEtMDYtMTIgIEFkYW0gQmFydGggIDxhYmFydGhAd2Via2l0Lm9yZz4KIAogICAgICAg
ICBSZXZpZXdlZCBieSBBbGV4ZXkgUHJvc2t1cnlha292LgpkaWZmIC0tZ2l0IGEvU291cmNlL1dl
YktpdC9xdC9XZWJDb3JlU3VwcG9ydC9GcmFtZU5ldHdvcmtpbmdDb250ZXh0UXQuY3BwIGIvU291
cmNlL1dlYktpdC9xdC9XZWJDb3JlU3VwcG9ydC9GcmFtZU5ldHdvcmtpbmdDb250ZXh0UXQuY3Bw
CmluZGV4IDUwMmE0MTguLjNkZDhkOGIgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvcXQvV2Vi
Q29yZVN1cHBvcnQvRnJhbWVOZXR3b3JraW5nQ29udGV4dFF0LmNwcAorKysgYi9Tb3VyY2UvV2Vi
S2l0L3F0L1dlYkNvcmVTdXBwb3J0L0ZyYW1lTmV0d29ya2luZ0NvbnRleHRRdC5jcHAKQEAgLTIz
LDggKzIzLDggQEAKIAogI2luY2x1ZGUgPFFOZXR3b3JrQWNjZXNzTWFuYWdlcj4KICNpbmNsdWRl
IDxRT2JqZWN0PgotI2luY2x1ZGUgPFFXZWJGcmFtZT4KLSNpbmNsdWRlIDxRV2ViUGFnZT4KKyNp
bmNsdWRlICJxd2ViZnJhbWUuaCIKKyNpbmNsdWRlICJxd2VicGFnZS5oIgogCiBuYW1lc3BhY2Ug
V2ViQ29yZSB7CiAK
</data>
<flag name="review"
          id="90961"
          type_id="1"
          status="+"
          setter="benjamin"
    />
          </attachment>
      

    </bug>

</bugzilla>