Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(420)

Side by Side Diff: content/browser/renderer_host/text_input_client_mac.mm

Issue 1318483007: Implement "Look Up In Dictionary" context menu item asynchronously. (OS X) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: addressed palmer's comment. Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #import "content/browser/renderer_host/text_input_client_mac.h" 5 #import "content/browser/renderer_host/text_input_client_mac.h"
6 6
7 #include "base/memory/singleton.h" 7 #include "base/memory/singleton.h"
8 #include "base/metrics/histogram.h" 8 #include "base/metrics/histogram.h"
9 #include "base/threading/thread_restrictions.h" 9 #include "base/threading/thread_restrictions.h"
10 #include "base/time/time.h" 10 #include "base/time/time.h"
(...skipping 19 matching lines...) Expand all
30 30
31 // static 31 // static
32 TextInputClientMac* TextInputClientMac::GetInstance() { 32 TextInputClientMac* TextInputClientMac::GetInstance() {
33 return Singleton<TextInputClientMac>::get(); 33 return Singleton<TextInputClientMac>::get();
34 } 34 }
35 35
36 void TextInputClientMac::GetStringAtPoint( 36 void TextInputClientMac::GetStringAtPoint(
37 RenderWidgetHost* rwh, 37 RenderWidgetHost* rwh,
38 gfx::Point point, 38 gfx::Point point,
39 void (^replyHandler)(NSAttributedString*, NSPoint)) { 39 void (^replyHandler)(NSAttributedString*, NSPoint)) {
40 DCHECK(replyHandler_.get() == nil); 40 DCHECK(replyForPointHandler_.get() == nil);
41 replyHandler_.reset(replyHandler, base::scoped_policy::RETAIN); 41 replyForPointHandler_.reset(replyHandler, base::scoped_policy::RETAIN);
42 RenderWidgetHostImpl* rwhi = RenderWidgetHostImpl::From(rwh); 42 RenderWidgetHostImpl* rwhi = RenderWidgetHostImpl::From(rwh);
43 rwhi->Send(new TextInputClientMsg_StringAtPoint(rwhi->GetRoutingID(), point)); 43 rwhi->Send(new TextInputClientMsg_StringAtPoint(rwhi->GetRoutingID(), point));
44 } 44 }
45 45
46 void TextInputClientMac::GetStringAtPointReply(NSAttributedString* string, 46 void TextInputClientMac::GetStringAtPointReply(NSAttributedString* string,
47 NSPoint point) { 47 NSPoint point) {
48 if (replyHandler_.get()) { 48 if (replyForPointHandler_.get()) {
49 replyHandler_.get()(string, point); 49 replyForPointHandler_.get()(string, point);
50 replyHandler_.reset(); 50 replyForPointHandler_.reset();
51 } 51 }
52 } 52 }
53 53
54 void TextInputClientMac::GetStringFromRange(
55 RenderWidgetHost* rwh,
56 NSRange range,
57 void (^replyHandler)(NSAttributedString*, NSPoint)) {
58 DCHECK(replyForRangeHandler_.get() == nil);
59 replyForRangeHandler_.reset(replyHandler, base::scoped_policy::RETAIN);
60 RenderWidgetHostImpl* rwhi = RenderWidgetHostImpl::From(rwh);
61 rwhi->Send(new TextInputClientMsg_StringForRange(rwhi->GetRoutingID(),
62 gfx::Range(range)));
63 }
64
65 void TextInputClientMac::GetStringFromRangeReply(NSAttributedString* string,
66 NSPoint point) {
67 if (replyForRangeHandler_.get()) {
68 replyForRangeHandler_.get()(string, point);
69 replyForRangeHandler_.reset();
70 }
71 }
72
54 NSUInteger TextInputClientMac::GetCharacterIndexAtPoint(RenderWidgetHost* rwh, 73 NSUInteger TextInputClientMac::GetCharacterIndexAtPoint(RenderWidgetHost* rwh,
55 gfx::Point point) { 74 gfx::Point point) {
56 base::TimeTicks start = base::TimeTicks::Now(); 75 base::TimeTicks start = base::TimeTicks::Now();
57 76
58 BeforeRequest(); 77 BeforeRequest();
59 RenderWidgetHostImpl* rwhi = RenderWidgetHostImpl::From(rwh); 78 RenderWidgetHostImpl* rwhi = RenderWidgetHostImpl::From(rwh);
60 rwhi->Send(new TextInputClientMsg_CharacterIndexForPoint(rwhi->GetRoutingID(), 79 rwhi->Send(new TextInputClientMsg_CharacterIndexForPoint(rwhi->GetRoutingID(),
61 point)); 80 point));
62 // http://crbug.com/121917 81 // http://crbug.com/121917
63 base::ThreadRestrictions::ScopedAllowWait allow_wait; 82 base::ThreadRestrictions::ScopedAllowWait allow_wait;
(...skipping 85 matching lines...) Expand 10 before | Expand all | Expand 10 after
149 character_index_ = NSNotFound; 168 character_index_ = NSNotFound;
150 first_rect_ = NSZeroRect; 169 first_rect_ = NSZeroRect;
151 substring_.reset(); 170 substring_.reset();
152 } 171 }
153 172
154 void TextInputClientMac::AfterRequest() { 173 void TextInputClientMac::AfterRequest() {
155 lock_.Release(); 174 lock_.Release();
156 } 175 }
157 176
158 } // namespace content 177 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698