Bug 173159

Summary: Konqueror does not warn about unsubmitted changes
Product: [Applications] konqueror Reporter: Albert Astals Cid <aacid>
Component: generalAssignee: Konqueror Bugs <konqueror-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: germain
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Unspecified   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Albert Astals Cid 2008-10-19 20:09:13 UTC 
Version:            (using Devel)
Installed from:    Compiled sources

How to reproduce:
 * Go to www.google.com
 * Type something in the search line
 * Close the window

On KDE 3 you got a warning about unsubmitted changes, in KDE 4 you do not.
Comment 1 Maksim Orlovich 2008-10-19 20:48:15 UTC 
SVN commit 873587 by orlovich:

DocumentImpl::unsubmittedChanges actually relied on modified things having 'M' at the end of their state 
serialization, which broke notification of changes when the N/null flag got added. Fix this by adding 
an appropriate abstraction.

BUG:173159 


 M  +3 -0      html/html_formimpl.h  
 M  +4 -2      xml/dom_docimpl.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=873587
Comment 2 Harri Porten 2008-10-19 20:53:07 UTC 
SVN commit 873589 by porten:

Merged revision 873587:
DocumentImpl::unsubmittedChanges actually relied on modified things having 'M' at the end of their state 
serialization, which broke notification of changes when the N/null flag got added. Fix this by adding 
an appropriate abstraction.

BUG:173159 

 M  +3 -0      html/html_formimpl.h  
 M  +4 -2      xml/dom_docimpl.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=873589
Comment 3 Germain Garand 2008-10-19 21:40:19 UTC 
was my fault, sorry ;(