Disallow UI/IO thread blocking on any other thread.

content/browser/renderer_host/text_input_client_mac.mm

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #import "content/browser/renderer_host/text_input_client_mac.h"
 
 #include "base/memory/singleton.h"
 #include "base/metrics/histogram.h"
+#include "base/threading/thread_restrictions.h"
 #include "base/time.h"
 #include "content/browser/renderer_host/render_widget_host_impl.h"
 #include "content/common/text_input_client_messages.h"
 
 using content::RenderWidgetHost;
 using content::RenderWidgetHostImpl;
 
 // The amount of time in milliseconds that the browser process will wait for a
 // response from the renderer.
 // TODO(rsesek): Using the histogram data, find the best upper-bound for this
@@ skipping 15 matching lines @@
 }
 
 NSUInteger TextInputClientMac::GetCharacterIndexAtPoint(RenderWidgetHost* rwh,
     gfx::Point point) {
   base::TimeTicks start = base::TimeTicks::Now();
 
   BeforeRequest();
   RenderWidgetHostImpl* rwhi = RenderWidgetHostImpl::From(rwh);
   rwhi->Send(new TextInputClientMsg_CharacterIndexForPoint(rwhi->GetRoutingID(),
                                                           point));
+  // http://crbug.com/121917
+  base::ThreadRestrictions::ScopedAllowWait allow_wait;
   condition_.TimedWait(base::TimeDelta::FromMilliseconds(kWaitTimeout));
   AfterRequest();
 
   base::TimeDelta delta(base::TimeTicks::Now() - start);
   UMA_HISTOGRAM_TIMES("TextInputClient.CharacterIndex",
                       delta * base::Time::kMicrosecondsPerMillisecond);
 
   return character_index_;
 }
 
 NSRect TextInputClientMac::GetFirstRectForRange(RenderWidgetHost* rwh,
     NSRange range) {
   base::TimeTicks start = base::TimeTicks::Now();
 
   BeforeRequest();
   RenderWidgetHostImpl* rwhi = RenderWidgetHostImpl::From(rwh);
   rwhi->Send(
       new TextInputClientMsg_FirstRectForCharacterRange(rwhi->GetRoutingID(),
                                                         ui::Range(range)));
+  // http://crbug.com/121917
+  base::ThreadRestrictions::ScopedAllowWait allow_wait;
   condition_.TimedWait(base::TimeDelta::FromMilliseconds(kWaitTimeout));
   AfterRequest();
 
   base::TimeDelta delta(base::TimeTicks::Now() - start);
   UMA_HISTOGRAM_TIMES("TextInputClient.FirstRect",
                       delta * base::Time::kMicrosecondsPerMillisecond);
 
   return first_rect_;
 }
 
 NSAttributedString* TextInputClientMac::GetAttributedSubstringFromRange(
     RenderWidgetHost* rwh,
     NSRange range) {
   base::TimeTicks start = base::TimeTicks::Now();
 
   BeforeRequest();
   RenderWidgetHostImpl* rwhi = RenderWidgetHostImpl::From(rwh);
   rwhi->Send(new TextInputClientMsg_StringForRange(rwhi->GetRoutingID(),
                                                    ui::Range(range)));
+  // http://crbug.com/121917
+  base::ThreadRestrictions::ScopedAllowWait allow_wait;
   condition_.TimedWait(base::TimeDelta::FromMilliseconds(kWaitTimeout));
   AfterRequest();
 
   base::TimeDelta delta(base::TimeTicks::Now() - start);
   UMA_HISTOGRAM_TIMES("TextInputClient.Substring",
                       delta * base::Time::kMicrosecondsPerMillisecond);
 
   // Lookup.framework calls this method repeatedly and expects that repeated
   // calls don't deallocate previous results immediately. Returning an
   // autoreleased string is better convention anyway.
@@ skipping 31 matching lines @@
                       delta * base::Time::kMicrosecondsPerMillisecond);
 
   character_index_ = NSNotFound;
   first_rect_ = NSZeroRect;
   substring_.reset();
 }
 
 void TextInputClientMac::AfterRequest() {
   lock_.Release();
 }