Bug 52956

Summary: [Qt] Support layoutTestController.addURLToRedirect()
Product: WebKit Reporter: Robert Hogan <robert>
Component: New BugsAssignee: Robert Hogan <robert>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, kling, ossy, tonikitoo
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: OS X 10.5   
Attachments:
Description Flags
Patch
none
Patch none

Robert Hogan
Reported 2011-01-22 04:25:42 PST
[Qt] Support layoutTestController.addURLToRedirect()
Attachments
Patch (7.71 KB, patch)
2011-01-22 04:27 PST, Robert Hogan 		no flags
DetailsFormatted DiffDiff
Patch (7.76 KB, patch)
2011-01-22 06:36 PST, Robert Hogan 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Robert Hogan
Comment 1 2011-01-22 04:27:04 PST
Created attachment 79842 [details] Patch
Andreas Kling
Comment 2 2011-01-22 05:01:23 PST
Comment on attachment 79842 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=79842&action=review r=me, one comment. > Source/WebKit/qt/WebCoreSupport/FrameLoaderClientQt.cpp:1027 > + QString url = newRequest.url().string(); > + if (URLsToRedirect.contains(url)) > + newRequest.setURL(QUrl(URLsToRedirect[url])); Let's only do this if QWebPagePrivate::drtRun is true.
Robert Hogan
Comment 3 2011-01-22 06:36:36 PST
Created attachment 79845 [details] Patch
Robert Hogan
Comment 4 2011-01-22 09:28:12 PST
Committed r76451: <http://trac.webkit.org/changeset/76451>
Note You need to log in before you can comment on or make changes to this bug.