Changeset 24490 in webkit
- Timestamp:
- Jul 20, 2007, 3:35:50 PM (18 years ago)
- Location:
- trunk
- Files:
-
- 3 added
- 5 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/LayoutTests/ChangeLog
r24487 r24490 1 2007-07-20 Maciej Stachowiak <mjs@apple.com> 2 3 Reviewed by Oliver. 4 5 - test case for <rdar://problem/5326303> REGRESSION: blur event fires upon navigating away from a page with focused element 6 7 (Test mostly by Alice.) 8 9 * fast/events/no-blur-on-page-leave-expected.txt: Added. 10 * fast/events/no-blur-on-page-leave.html: Added. 11 * fast/events/resources/no-blur-result.html: Added. 12 1 13 2007-07-20 Sam Weinig <sam@webkit.org> 2 14 -
trunk/WebCore/ChangeLog
r24486 r24490 1 2007-07-20 Maciej Stachowiak <mjs@apple.com> 2 3 Reviewed by Oliver. 4 5 <rdar://problem/5326303> REGRESSION: blur event fires upon navigating away from a page with focused element 6 7 8 9 The proximate cause was the recent change to keep focus on the 10 WebHTMLView when switching views. But the reason this happend was 11 that didCommitLoadForFrame got called at a time when the new view 12 was in place, but the old document was still there. This was a 13 longstanding problem with the delegate, and fixing it cleans it 14 up. 15 16 * loader/FrameLoader.cpp: 17 (WebCore::FrameLoader::begin): Don't send dispatchWindowObjectAvailable if 18 caller asked not to (this is to allow order of delegates to be the same). 19 (WebCore::FrameLoader::transitionToCommitted): Don't emit didCommitLoadForFrame here. 20 (WebCore::FrameLoader::receivedFirstData): Instead do it here, after calling begin(), 21 so the new document is set up. 22 * loader/FrameLoader.h: Add new optional parameter to begin(). 23 1 24 2007-07-20 Sam Weinig <sam@webkit.org> 2 25 -
trunk/WebCore/WebCore.xcodeproj/project.pbxproj
r24485 r24490 11735 11735 isa = PBXProject; 11736 11736 buildConfigurationList = 149C284308902B11008A9EFC /* Build configuration list for PBXProject "WebCore" */; 11737 compatibilityVersion = "Xcode 2.4"; 11737 11738 hasScannedForEncodings = 1; 11738 11739 knownRegions = ( … … 11748 11749 productRefGroup = 034768DFFF38A50411DB9C8B /* Products */; 11749 11750 projectDirPath = ""; 11751 projectRoot = ""; 11750 11752 targets = ( 11751 11753 93F198A508245E59001E9ABC /* WebCore */, -
trunk/WebCore/loader/FrameLoader.cpp
r24434 r24490 818 818 void FrameLoader::receivedFirstData() 819 819 { 820 begin(m_workingURL); 820 begin(m_workingURL, false); 821 822 dispatchDidCommitLoad(); 823 dispatchWindowObjectAvailable(); 824 825 String ptitle = m_documentLoader->title(); 826 // If we have a title let the WebView know about it. 827 if (!ptitle.isNull()) 828 m_client->dispatchDidReceiveTitle(ptitle); 821 829 822 830 m_frame->document()->docLoader()->setCachePolicy(m_cachePolicy); … … 853 861 } 854 862 855 void FrameLoader::begin(const KURL& url )863 void FrameLoader::begin(const KURL& url, bool dispatch) 856 864 { 857 865 clear(); 858 dispatchWindowObjectAvailable(); 866 if (dispatch) 867 dispatchWindowObjectAvailable(); 859 868 860 869 m_needsClear = true; … … 2431 2440 // Handle adding the URL to the back/forward list. 2432 2441 DocumentLoader* dl = m_documentLoader.get(); 2433 String ptitle = dl->title();2434 2442 2435 2443 switch (m_loadType) { … … 2486 2494 2487 2495 m_committedFirstRealDocumentLoad = true; 2488 2489 dispatchDidCommitLoad();2490 2491 // If we have a title let the WebView know about it.2492 if (!ptitle.isNull())2493 m_client->dispatchDidReceiveTitle(ptitle);2494 2496 } 2495 2497 -
trunk/WebCore/loader/FrameLoader.h
r24367 r24490 310 310 311 311 void begin(); 312 void begin(const KURL&); 312 void begin(const KURL&, bool dispatchWindowObjectAvailable = true); 313 313 314 void write(const char* str, int len = -1, bool flush = false); 314 315 void write(const String&);
Note:
See TracChangeset
for help on using the changeset viewer.