RESOLVED FIXED 84512
Repurpose ActiveDOMObject::WillShowDialog to WillDeferLoading 
https://bugs.webkit.org/show_bug.cgi?id=84512
Summary Repurpose ActiveDOMObject::WillShowDialog to WillDeferLoading
Brady Eidson
Reported 2012-04-20 16:46:42 PDT
Repurpose ActiveDOMObject::WillShowDialog to WillDeferLoading All of the ActiveDOMObject ReasonForSuspensions are great, but only a *single* subclass does anything special for a particular Reason. And it's not WillShowDialog. WillShowDialog is used whenever we defer page loading and has no special meaning beyond that. We should rename it, then use it by default in all page load deferral cases without having to pass the argument around. Noticed while exploring https://bugs.webkit.org/show_bug.cgi?id=84488
Attachments
Patch v1 - Do the rename and get rid of the unnecessary argument (11.89 KB, patch)
2012-04-20 16:51 PDT, Brady Eidson 		eric.carlson: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Brady Eidson
Comment 1 2012-04-20 16:51:48 PDT
Created attachment 138199 [details] Patch v1 - Do the rename and get rid of the unnecessary argument
Brady Eidson
Comment 2 2012-04-20 17:07:33 PDT
http://trac.webkit.org/changeset/114802
Note You need to log in before you can comment on or make changes to this bug.