[Webkit-unassigned] [Bug 48812] FrameLoader::checkLoadCompleteForThisFrame uses wrong history item

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jan 13 08:36:23 PST 2011


Brady Eidson <beidson at apple.com> changed:

           What    |Removed                     |Added
  Attachment #75044|review+                     |review-
               Flag|                            |

--- Comment #43 from Brady Eidson <beidson at apple.com>  2011-01-13 08:36:23 PST ---
(From update of attachment 75044)
Sorry this patch has been sitting here with an r+ for awhile and that I'm just doing this, but reading my bugzilla CC email I noticed something I really dislike here.

I understand the motivation for reordering the arguments was the "null vs bool" ambiguity.  A much better way to fix that besides reordering the arguments would be to take this opportunity to replace the bool with an enum.  

Since it's been so long I'm going to change to r- in hopes this can be done before landing.

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

More information about the webkit-unassigned mailing list