Changeset 24934 in webkit


Ignore:
Timestamp:
Aug 8, 2007 9:37:48 AM (17 years ago)
Author:
beidson
Message:

LayoutTests:

Reviewed and landed by Brady

Updated test case because of fix for <http://bugs.webkit.org/show_bug.cgi?id=13422>

Bug 13422: REGRESSION: Page reload loses page position

  • http/tests/navigation/success200-reload-expected.txt:

WebCore:

Reviewed and landed by Brady

Fixes <http://bugs.webkit.org/show_bug.cgi?id=13422>

Bug 13422: REGRESSION: Page reload loses page position

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::updateHistoryForReload):
Location:
trunk
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • trunk/LayoutTests/ChangeLog

    r24924 r24934  
     12007-08-08  Cameron Zwarich  <cwzwarich@uwaterloo.ca>
     2
     3        Reviewed and landed by Brady
     4
     5        Updated test case because of fix for <http://bugs.webkit.org/show_bug.cgi?id=13422>
     6
     7        Bug 13422: REGRESSION: Page reload loses page position
     8
     9        * http/tests/navigation/success200-reload-expected.txt:
     10
    1112007-08-07  Sam Weinig  <sam@webkit.org>
    212
  • trunk/LayoutTests/http/tests/navigation/success200-reload-expected.txt

    r24604 r24934  
    105105      text run at (188,0) width 0: " "
    106106    RenderBR {BR} at (3,13) size 0x13
     107scrolled to 0,50
    107108
    108109============== Back Forward List ==============
  • trunk/WebCore/ChangeLog

    r24930 r24934  
     12007-08-08  Cameron Zwarich  <cwzwarich@uwaterloo.ca>
     2
     3        Reviewed and landed by Brady
     4
     5        Fixes <http://bugs.webkit.org/show_bug.cgi?id=13422>
     6
     7        Bug 13422: REGRESSION: Page reload loses page position
     8
     9        * loader/FrameLoader.cpp:
     10        (WebCore::FrameLoader::updateHistoryForReload):
     11
    1122007-08-08  Holger Hans Peter Freyther  <zecke@selfish.org>
    213
  • trunk/WebCore/loader/FrameLoader.cpp

    r24900 r24934  
    41594159#endif
    41604160
    4161     if (m_previousHistoryItem) {
    4162         pageCache()->remove(m_previousHistoryItem.get());
     4161    if (m_currentHistoryItem) {
     4162        pageCache()->remove(m_currentHistoryItem.get());
    41634163   
    41644164        if (loadType() == FrameLoadTypeReload)
    4165             saveScrollPositionAndViewStateToItem(m_previousHistoryItem.get());
     4165            saveScrollPositionAndViewStateToItem(m_currentHistoryItem.get());
    41664166   
    41674167        // Sometimes loading a page again leads to a different result because of cookies. Bugzilla 4072
    41684168        if (documentLoader()->unreachableURL().isEmpty())
    4169             m_previousHistoryItem->setURL(documentLoader()->requestURL());
     4169            m_currentHistoryItem->setURL(documentLoader()->requestURL());
    41704170    }
    41714171   
Note: See TracChangeset for help on using the changeset viewer.