Bug 1016184 part 3a - Fix typos and minor style changes. r=ehsan

Also group the private section together, and remove the protect section
since TouchCaret is a MOZ_FINAL class.
This commit is contained in:
Ting-Yu Lin 2014-07-27 00:13:00 +02:00
parent 27b0e27ad5
commit 570744db3c
2 changed files with 5 additions and 7 deletions

View File

@ -125,7 +125,7 @@ TouchCaret::SetVisibility(bool aVisible)
// Set touch caret expiration time.
mVisible ? LaunchExpirationTimer() : CancelExpirationTimer();
// We must call SetHasTouchCaret() in order to get APZC to wait until the
// We must call SetMayHaveTouchCaret() in order to get APZC to wait until the
// event has been round-tripped and check whether it has been handled,
// otherwise B2G will end up panning the document when the user tries to drag
// touch caret.
@ -329,7 +329,7 @@ TouchCaret::IsOnTouchCaret(const nsPoint& aPoint)
nsresult
TouchCaret::NotifySelectionChanged(nsIDOMDocument* aDoc, nsISelection* aSel,
int16_t aReason)
int16_t aReason)
{
// Hide touch caret while no caret exists.
nsCOMPtr<nsIPresShell> presShell = do_QueryReferent(mPresShell);

View File

@ -19,16 +19,13 @@
namespace mozilla {
/**
* The TouchCaret places a touch caret according to caret postion when the
* The TouchCaret places a touch caret according to caret position when the
* caret is shown.
* TouchCaret is also responsible for touch caret visibility. Touch caret
* won't be shown when timer expires or while key event causes selection change.
*/
class TouchCaret MOZ_FINAL : public nsISelectionListener
{
private:
~TouchCaret();
public:
explicit TouchCaret(nsIPresShell* aPresShell);
@ -73,6 +70,8 @@ private:
// Hide default constructor.
TouchCaret() MOZ_DELETE;
~TouchCaret();
/**
* Find the nsCanvasFrame which holds the touch caret.
*/
@ -226,7 +225,6 @@ private:
return sTouchCaretExpirationTime;
}
protected:
nsWeakPtr mPresShell;
// Touch caret visibility