RESOLVED FIXED 58080
Make IconSnapshot and PageURLRecord member variables private. 
https://bugs.webkit.org/show_bug.cgi?id=58080
Summary Make IconSnapshot and PageURLRecord member variables private.
David Levin
Reported 2011-04-07 14:27:11 PDT
This helps to ensure that they are only modified in given ways and in general is a good C++ practice.
Attachments
Patch (10.04 KB, patch)
2011-04-07 14:31 PDT, David Levin 		darin: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
David Levin
Comment 1 2011-04-07 14:31:00 PDT
Created attachment 88699 [details] Patch
Darin Adler
Comment 2 2011-04-07 16:21:24 PDT
Comment on attachment 88699 [details] Patch Since we never want to change these after creating them, seems like a reasonable way to go. The other direction would be to make them structs.
David Levin
Comment 3 2011-04-07 17:56:30 PDT
Committed as http://trac.webkit.org/changeset/83234
WebKit Review Bot
Comment 4 2011-04-07 18:44:17 PDT
http://trac.webkit.org/changeset/83234 might have broken Qt Linux Release The following tests are not passing: http/tests/security/contentSecurityPolicy/xsl-allowed.php
Note You need to log in before you can comment on or make changes to this bug.