Bug 28993 - [Qt] Remove Qt v4.6 check from the prefetchDNS implementation
Summary: [Qt] Remove Qt v4.6 check from the prefetchDNS implementation
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Qt (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-09-04 17:08 PDT by Laszlo Gombos
Modified: 2009-09-07 00:10 PDT (History)
1 user (show)

See Also:


Attachments
proposed patch. (1.76 KB, patch)
2009-09-04 17:12 PDT, Laszlo Gombos
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Laszlo Gombos 2009-09-04 17:08:11 PDT
Functionality behind r47425 (http://trac.webkit.org/changeset/47425) is guarded with Qt v4.6 build check. Typically we do not use a build time Qt version check for newly introduced QtWebKit functionality. The usage of Qt version check is should be restricted to guard QtWebKit functionality that depends on a Qt functionality that is not available in all Qt versions. 

In this particular case the functionality introduced by r47425 builds and functions just fine using Qt v4.5. In addition removing the build flag also gets rid of a build warning (from moc).
Comment 1 Laszlo Gombos 2009-09-04 17:12:32 PDT
Created attachment 39104 [details]
proposed patch.
Comment 2 Eric Seidel (no email) 2009-09-05 16:29:07 PDT
Comment on attachment 39104 [details]
proposed patch.

OK.  rs=me.  I can't remember if you're a committer.  Assuming not.  cq+
Comment 3 Eric Seidel (no email) 2009-09-05 16:40:29 PDT
Comment on attachment 39104 [details]
proposed patch.

Rejecting patch 39104 from commit-queue.  This patch will require manual commit.

['WebKitTools/Scripts/run-webkit-tests', '--no-launch-safari', '--quiet', '--exit-after-n-failures=1'] failed with exit code 1
Comment 4 Laszlo Gombos 2009-09-05 17:13:35 PDT
(In reply to comment #3)
> (From update of attachment 39104 [details])
> Rejecting patch 39104 from commit-queue.  This patch will require manual
> commit.
> 
> ['WebKitTools/Scripts/run-webkit-tests', '--no-launch-safari', '--quiet',
> '--exit-after-n-failures=1'] failed with exit code 1

Eric, I can/will commit, thanks for the review.
Comment 5 Eric Seidel (no email) 2009-09-06 02:02:27 PDT
Comment on attachment 39104 [details]
proposed patch.

Another victim of bug 28845.
media/video-source-error.html -> timed out
Comment 6 Eric Seidel (no email) 2009-09-06 02:09:33 PDT
Comment on attachment 39104 [details]
proposed patch.

Rejecting patch 39104 from commit-queue.  This patch will require manual commit.

['WebKitTools/Scripts/run-webkit-tests', '--no-launch-safari', '--quiet', '--exit-after-n-failures=1'] failed with exit code 1
Comment 7 Laszlo Gombos 2009-09-06 05:39:37 PDT
(In reply to comment #6)
> (From update of attachment 39104 [details])
> Rejecting patch 39104 from commit-queue.  This patch will require manual
> commit.
> 
> ['WebKitTools/Scripts/run-webkit-tests', '--no-launch-safari', '--quiet',
> '--exit-after-n-failures=1'] failed with exit code 1

Committed - http://trac.webkit.org/changeset/48100
Comment 8 Eric Seidel (no email) 2009-09-07 00:08:48 PDT
Thank you.  The commit-queue is kinda paralyzed by bug 28845 at the moment.  This hit it *again* when trying to land.
Comment 9 Eric Seidel (no email) 2009-09-07 00:10:17 PDT
Actually it hit a different test the final time, same bug though:
compositing/geometry/abs-position-inside-opacity.html -> timed out