Bug 127528

Summary: Set full URLs for kLSActivePageUserVisibleOriginsKey
Product: WebKit Reporter: Gavin Barraclough <barraclough>
Component: WebKit2Assignee: Gavin Barraclough <barraclough>
Status: NEW    
Severity: Normal    
Priority: P2    
Version: 525.x (Safari 3.1)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Fix mrowe: review+

Gavin Barraclough
Reported 2014-01-23 17:54:01 PST
In many cases the domain name is not sufficiently useful.
Attachments
Fix (1.94 KB, patch)
2014-01-23 17:55 PST, Gavin Barraclough 		mrowe: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Gavin Barraclough
Comment 1 2014-01-23 17:55:22 PST
Created attachment 222052 [details] Fix
Mark Rowe (bdash)
Comment 2 2014-01-25 14:35:09 PST
Comment on attachment 222052 [details] Fix View in context: https://bugs.webkit.org/attachment.cgi?id=222052&action=review > Source/WebKit2/WebProcess/mac/WebProcessMac.mm:240 > + NSURL *originAsNSURL = [NSURL URLWithString:mainFrame->url()]; Elsewhere in WebKit2 we use +_web_URLWithWTFString: to convert from a String representation of a URL to NSURL. Should we do the same here?
Note You need to log in before you can comment on or make changes to this bug.