[SCM] WebKit Debian packaging branch, debian/experimental, updated. upstream/1.3.3-9427-gc2be6fc
commit-queue at webkit.org
commit-queue at webkit.org
Wed Dec 22 13:53:55 UTC 2010
The following commit has been merged in the debian/experimental branch:
commit 57662911d2823531ac02e63f777e7dba2da7f6ec
Author: commit-queue at webkit.org <commit-queue at webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Wed Sep 29 00:14:17 2010 +0000
2010-09-28 Jia Pu <jpu at apple.com>
Reviewed by Dan Bernstein.
Delete and forward delete shouldn't start autocorrection panel timer.
https://bugs.webkit.org/show_bug.cgi?id=46679
<rdar://problem/8463917>
* editing/TypingCommand.cpp:
(WebCore::TypingCommand::markMisspellingsAfterTyping): Only start correction panel timer
when the editing command is InsertText.
(WebCore::TypingCommand::typingAddedToOpenCommand): Add an argument to pass in command type.
* editing/TypingCommand.h: Add an argument to typingAddedToOpenCommand().
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@68613 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 65eb364..56708ea 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -1,3 +1,18 @@
+2010-09-28 Jia Pu <jpu at apple.com>
+
+ Reviewed by Dan Bernstein.
+
+ Delete and forward delete shouldn't start autocorrection panel timer.
+ https://bugs.webkit.org/show_bug.cgi?id=46679
+ <rdar://problem/8463917>
+
+ * editing/TypingCommand.cpp:
+ (WebCore::TypingCommand::markMisspellingsAfterTyping): Only start correction panel timer
+ when the editing command is InsertText.
+ (WebCore::TypingCommand::typingAddedToOpenCommand): Add an argument to pass in command type.
+
+ * editing/TypingCommand.h: Add an argument to typingAddedToOpenCommand().
+
2010-09-28 Mihai Parparita <mihaip at chromium.org>
Reviewed by Dimitri Glazkov.
diff --git a/WebCore/editing/TypingCommand.cpp b/WebCore/editing/TypingCommand.cpp
index 1d1183a..60b3f7b 100644
--- a/WebCore/editing/TypingCommand.cpp
+++ b/WebCore/editing/TypingCommand.cpp
@@ -283,7 +283,7 @@ EditAction TypingCommand::editingAction() const
return EditActionTyping;
}
-void TypingCommand::markMisspellingsAfterTyping()
+void TypingCommand::markMisspellingsAfterTyping(ETypingCommand commandType)
{
#if PLATFORM(MAC) && !defined(BUILDING_ON_TIGER) && !defined(BUILDING_ON_LEOPARD)
if (!document()->frame()->editor()->isContinuousSpellCheckingEnabled()
@@ -308,8 +308,10 @@ void TypingCommand::markMisspellingsAfterTyping()
if (p1 != p2)
document()->frame()->editor()->markMisspellingsAfterTypingToPosition(p1);
#if PLATFORM(MAC) && !defined(BUILDING_ON_TIGER) && !defined(BUILDING_ON_LEOPARD) && !defined(BUILDING_ON_SNOW_LEOPARD)
- else
+ else if (commandType == TypingCommand::InsertText)
document()->frame()->editor()->startCorrectionPanelTimer();
+#else
+ UNUSED_PARAM(commandType);
#endif
}
}
@@ -317,14 +319,14 @@ void TypingCommand::markMisspellingsAfterTyping()
void TypingCommand::typingAddedToOpenCommand(ETypingCommand commandTypeForAddedTyping)
{
updatePreservesTypingStyle(commandTypeForAddedTyping);
-
+
#if PLATFORM(MAC) && !defined(BUILDING_ON_TIGER) && !defined(BUILDING_ON_LEOPARD)
document()->frame()->editor()->appliedEditing(this);
// Since the spellchecking code may also perform corrections and other replacements, it should happen after the typing changes.
- markMisspellingsAfterTyping();
+ markMisspellingsAfterTyping(commandTypeForAddedTyping);
#else
// The old spellchecking code requires that checking be done first, to prevent issues like that in 6864072, where <doesn't> is marked as misspelled.
- markMisspellingsAfterTyping();
+ markMisspellingsAfterTyping(commandTypeForAddedTyping);
document()->frame()->editor()->appliedEditing(this);
#endif
}
diff --git a/WebCore/editing/TypingCommand.h b/WebCore/editing/TypingCommand.h
index 7c89a7c..60b5f2a 100644
--- a/WebCore/editing/TypingCommand.h
+++ b/WebCore/editing/TypingCommand.h
@@ -82,10 +82,10 @@ private:
virtual bool preservesTypingStyle() const { return m_preservesTypingStyle; }
void updatePreservesTypingStyle(ETypingCommand);
- void markMisspellingsAfterTyping();
+ void markMisspellingsAfterTyping(ETypingCommand);
void typingAddedToOpenCommand(ETypingCommand);
bool makeEditableRootEmpty();
-
+
ETypingCommand m_commandType;
String m_textToInsert;
bool m_openForMoreTyping;
--
WebKit Debian packaging
More information about the Pkg-webkit-commits
mailing list