From 55d2cb286895e9183609e318b5c0b7801364bd2c Mon Sep 17 00:00:00 2001
From: Lukhnos Liu <lukhnos@lukhnos.org>
Date: Thu, 24 Jan 2013 23:28:42 -0800
Subject: [PATCH] Defer updating scroller.

---
 .../VTVerticalCandidateController.m           | 20 ++++++++++++-------
 1 file changed, 13 insertions(+), 7 deletions(-)

diff --git a/Source/CandidateUI/VTVerticalCandidateController.m b/Source/CandidateUI/VTVerticalCandidateController.m
index 7b78bd38..b5d63826 100644
--- a/Source/CandidateUI/VTVerticalCandidateController.m
+++ b/Source/CandidateUI/VTVerticalCandidateController.m
@@ -41,6 +41,7 @@ static const CGFloat kCandidateTextLeftMargin = 8.0;
 - (BOOL)scrollPageByOne:(BOOL)forward;
 - (BOOL)moveSelectionByOne:(BOOL)forward;
 - (void)layoutCandidateView;
+- (void)forceUpdateVerticalScroller;
 @end
 
 @implementation VTVerticalCandidateController
@@ -244,7 +245,7 @@ static const CGFloat kCandidateTextLeftMargin = 8.0;
 
     // fix a subtle on 10.7 that, since we force the scroller to appear, scrolling sometimes shows a temporarily "broken" scroll bar (but quickly disappears)
     if ([_scrollView hasVerticalScroller]) {
-        [[_scrollView verticalScroller] setNeedsDisplay];
+        [self forceUpdateVerticalScroller];
     }
     
     return attrString;
@@ -338,12 +339,6 @@ static const CGFloat kCandidateTextLeftMargin = 8.0;
 }
                            
 - (void)layoutCandidateView
-{
-    [NSObject cancelPreviousPerformRequestsWithTarget:self selector:@selector(deferredLayoutCandidateView) object:nil];
-    [self performSelector:@selector(deferredLayoutCandidateView) withObject:nil afterDelay:0.0];
-}
-
-- (void)deferredLayoutCandidateView
 {
     NSUInteger count = [_delegate candidateCountForController:self];
     if (!count) {
@@ -402,4 +397,15 @@ static const CGFloat kCandidateTextLeftMargin = 8.0;
     [_scrollView setFrame:NSMakeRect(stripWidth + 1.0, 0.0, tableViewStartWidth, windowHeight)];
     [[self window] setFrame:frameRect display:YES];
 }
+
+- (void)forceUpdateVerticalScroller
+{
+    [NSObject cancelPreviousPerformRequestsWithTarget:self selector:@selector(deferredForceUpdateVerticalScroller) object:nil];
+    [self performSelector:@selector(deferredForceUpdateVerticalScroller) withObject:nil afterDelay:0.0];
+}
+
+- (void)deferredForceUpdateVerticalScroller
+{
+    [[_scrollView verticalScroller] setNeedsDisplay];
+}
 @end