[webkit-reviews] review requested: [Bug 15212] Useless checking for Item in FrameLoader::goBackOrForward(int distance) : [Attachment 16397] modified patch

[bugzilla-daemon at webkit.org]

Maxime BRITTO <mbritto at pleyo.com> has asked  for review:
Bug 15212: Useless checking for Item in FrameLoader::goBackOrForward(int
distance)
http://bugs.webkit.org/show_bug.cgi?id=15212

Attachment 16397: modified patch
http://bugs.webkit.org/attachment.cgi?id=16397&action=edit

------- Additional Comments from Maxime BRITTO <mbritto at pleyo.com>
After the talk mitzpettel and I had, we decided to let the check for the
reachable item in the function because it's more conveniant for the future
updates.
Though I added an assert to help future debug.
I let that last check to avoid an eventually crash on release versions :
    if (item)
	page->goToItem(item, FrameLoadTypeIndexedBackForward);