Changeset 90344 in webkit


Ignore:
Timestamp:
Jul 3, 2011 11:29:08 AM (13 years ago)
Author:
commit-queue@webkit.org
Message:

2011-07-03 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r90342.
​http://trac.webkit.org/changeset/90342
​https://bugs.webkit.org/show_bug.cgi?id=63875

Newly added tests fail on Mac, GTK, and Win7 (Requested by
abarth on #webkit).

  • http/tests/navigation/scrollstate-after-http-equiv-refresh-expected.txt: Removed.
  • http/tests/navigation/scrollstate-after-http-equiv-refresh-fragment-identifier-2-expected.txt: Removed.
  • http/tests/navigation/scrollstate-after-http-equiv-refresh-fragment-identifier-2.html: Removed.
  • http/tests/navigation/scrollstate-after-http-equiv-refresh-fragment-identifier-expected.txt: Removed.
  • http/tests/navigation/scrollstate-after-http-equiv-refresh-fragment-identifier.html: Removed.
  • http/tests/navigation/scrollstate-after-http-equiv-refresh.html: Removed.
  • http/tests/navigation/scrollstate-after-location-reload-expected.txt: Removed.
  • http/tests/navigation/scrollstate-after-location-reload.html: Removed.

2011-07-03 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r90342.
​http://trac.webkit.org/changeset/90342
​https://bugs.webkit.org/show_bug.cgi?id=63875

Newly added tests fail on Mac, GTK, and Win7 (Requested by
abarth on #webkit).

  • loader/FrameLoader.cpp: (WebCore::FrameLoader::loadURL):
  • loader/NavigationScheduler.cpp:
Location:
trunk
Files:
8 deleted
4 edited

Legend:

Unmodified
Added
Removed

  • trunk/LayoutTests/ChangeLog

    r90343 r90344  
     12011-07-03  Sheriff Bot  <webkit.review.bot@gmail.com>
     2
     3        Unreviewed, rolling out r90342.
     4        http://trac.webkit.org/changeset/90342
     5        https://bugs.webkit.org/show_bug.cgi?id=63875
     6
     7        Newly added tests fail on Mac, GTK, and Win7 (Requested by
     8        abarth on #webkit).
     9
     10        * http/tests/navigation/scrollstate-after-http-equiv-refresh-expected.txt: Removed.
     11        * http/tests/navigation/scrollstate-after-http-equiv-refresh-fragment-identifier-2-expected.txt: Removed.
     12        * http/tests/navigation/scrollstate-after-http-equiv-refresh-fragment-identifier-2.html: Removed.
     13        * http/tests/navigation/scrollstate-after-http-equiv-refresh-fragment-identifier-expected.txt: Removed.
     14        * http/tests/navigation/scrollstate-after-http-equiv-refresh-fragment-identifier.html: Removed.
     15        * http/tests/navigation/scrollstate-after-http-equiv-refresh.html: Removed.
     16        * http/tests/navigation/scrollstate-after-location-reload-expected.txt: Removed.
     17        * http/tests/navigation/scrollstate-after-location-reload.html: Removed.
     18
    1192011-07-03  Robert Hogan  <robert@webkit.org>
    220

  • trunk/Source/WebCore/ChangeLog

    r90342 r90344  
     12011-07-03  Sheriff Bot  <webkit.review.bot@gmail.com>
     2
     3        Unreviewed, rolling out r90342.
     4        http://trac.webkit.org/changeset/90342
     5        https://bugs.webkit.org/show_bug.cgi?id=63875
     6
     7        Newly added tests fail on Mac, GTK, and Win7 (Requested by
     8        abarth on #webkit).
     9
     10        * loader/FrameLoader.cpp:
     11        (WebCore::FrameLoader::loadURL):
     12        * loader/NavigationScheduler.cpp:
     13
    1142011-07-03  Robert Hogan  <robert@webkit.org>
    215

  • trunk/Source/WebCore/loader/FrameLoader.cpp

    r90342 r90344  
    12181218            if (m_provisionalDocumentLoader)
    12191219                m_provisionalDocumentLoader->setIsClientRedirect(true);
    1220         } else if (sameURL && newLoadType != FrameLoadTypeReload && newLoadType != FrameLoadTypeReloadFromOrigin)
     1220        } else if (sameURL)
    12211221            // Example of this case are sites that reload the same URL with a different cookie
    12221222            // driving the generated content, or a master frame with links that drive a target

  • trunk/Source/WebCore/loader/NavigationScheduler.cpp

    r90342 r90344  
    154154
    155155    virtual bool shouldStartTimer(Frame* frame) { return frame->loader()->allAncestorsAreComplete(); }
    156 
    157     virtual void fire(Frame* frame)
    158     {
    159         UserGestureIndicator gestureIndicator(wasUserGesture() ? DefinitelyProcessingUserGesture : DefinitelyNotProcessingUserGesture);
    160         bool refresh = equalIgnoringFragmentIdentifier(frame->document()->url(), KURL(ParsedURLString, url()));
    161         frame->loader()->changeLocation(securityOrigin(), KURL(ParsedURLString, url()), referrer(), lockHistory(), lockBackForwardList(), refresh);
    162     }
    163156};
    164157
Note: See TracChangeset for help on using the changeset viewer.