Deploy adoptPtr to WebCore/loader
Created attachment 65382 [details] Patch
Attachment 65382 [details] did not build on chromium: Build output: http://queues.webkit.org/results/3805075
Created attachment 65429 [details] Patch for landing
Comment on attachment 65429 [details] Patch for landing I don't think that using adoptPtr on classes derived from CachedResource is helpful until we figure out the ownership approach. Adding adoptPtr/leakPtr to call sites doesn't do a lot of harm, but I also don't think it does much good. Since no icon database is ever deleted, it is too bad we are compiling code that calls the icon database destructor. It might be better for us to come up with a different way to handle immortal objects, rather than using OwnPtr which will compile in never-called code paths that destroy the object. I suggest omitting the IconDatabase changes.
Created attachment 65451 [details] Patch for landing
Comment on attachment 65451 [details] Patch for landing Rejecting patch 65451 from commit-queue. Failed to run "['WebKitTools/Scripts/run-webkit-tests', '--no-launch-safari', '--exit-after-n-failures=1', '--wait-for-httpd', '--ignore-tests', 'compositing,media', '--quiet']" exit_code: 1 Running build-dumprendertree Compiling Java tests make: Nothing to be done for `default'. Running tests from /Users/eseidel/Projects/CommitQueue/LayoutTests Testing 20823 test cases. svg/dom/SVGScriptElement/script-set-href.svg -> failed Exiting early after 1 failures. 18347 tests run. 542.96s total testing time 18346 test cases (99%) succeeded 1 test case (<1%) had incorrect layout 26 test cases (<1%) had stderr output Full output: http://queues.webkit.org/results/3740638
Comment on attachment 65451 [details] Patch for landing Another flaky test.
Comment on attachment 65451 [details] Patch for landing Clearing flags on attachment: 65451 Committed r66083: <http://trac.webkit.org/changeset/66083>
All reviewed patches have been landed. Closing bug.