Require ACK for editor-related changes not originating from browser.

content/browser/renderer_host/render_widget_host_view_android.h

 // 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.
 
 #ifndef CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_ANDROID_H_
 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_ANDROID_H_
 
 #include <map>
 #include <queue>
 
@@ skipping 207 matching lines @@
   void RequestContentClipping(const gfx::Rect& clipping,
                               const gfx::Size& content_size);
 
   // Returns true when animation ticks are still needed. This avoids a separate
   // round-trip for requesting follow-up animation.
   bool Animate(base::TimeTicks frame_time);
 
   void SynchronousFrameMetadata(
       const cc::CompositorFrameMetadata& frame_metadata);
 
- private:
+private:

[kochi, 2013/07/08 04:28:43]

nit: not necessary to remove a space.

[nyquist, 2013/07/09 07:47:36]

Done.

   void BuffersSwapped(const gpu::Mailbox& mailbox,
                       const base::Closure& ack_callback);
 
   void RunAckCallbacks();
 
   void SwapDelegatedFrame(scoped_ptr<cc::DelegatedFrameData> frame_data);
   void SendDelegatedFrameAck();
 
   void UpdateContentViewCoreFrameMetadata(
       const cc::CompositorFrameMetadata& frame_metadata);
   void ComputeContentsSize(const cc::CompositorFrameMetadata& frame_metadata);
   void ResetClipping();
   void ClipContents(const gfx::Rect& clipping, const gfx::Size& content_size);
 
   void AttachLayers();
   void RemoveLayers();
 
   void CreateOverscrollEffectIfNecessary();
   void UpdateAnimationSize(const cc::CompositorFrameMetadata& frame_metadata);
   void ScheduleAnimationIfNecessary();
 
+  // Returns whether strict IME processing has been enabled.
+  bool has_strict_ime_processing() { return strict_ime_processing_; }

[kochi, 2013/07/08 04:28:43]

Usually we use the same name for a simple accessor - probably rename the member
to bool has_strict_ime_processing_?

[nyquist, 2013/07/09 07:47:36]

Done.

+
   // The model object.
   RenderWidgetHostImpl* host_;
 
   // Used to track whether this render widget needs a BeginFrame.
   bool needs_begin_frame_;
 
   // Whether or not this widget is potentially attached to the view hierarchy.
   // This view may not actually be attached if this is true, but it should be
   // treated as such, because as soon as a ContentViewCore is set the layer
   // will be attached automatically.
   bool are_layers_attached_;
 
   // ContentViewCoreImpl is our interface to the view system.
   ContentViewCoreImpl* content_view_core_;
 
   ImeAdapterAndroid ime_adapter_android_;
 
+  // Specifies if the strict IME processing has been enabled.
+  bool strict_ime_processing_;
+
   // Body background color of the underlying document.
   SkColor cached_background_color_;
 
   // The texture layer for this view when using browser-side compositing.
   scoped_refptr<cc::TextureLayer> texture_layer_;
 
   scoped_refptr<cc::DelegatedRendererLayer> delegated_renderer_layer_;
 
   // The layer used for rendering the contents of this view.
   // It is either owned by texture_layer_ or surface_texture_transport_
@@ skipping 22 matching lines @@
   // Used to render overscroll overlays.
   bool overscroll_effect_enabled_;
   scoped_ptr<OverscrollGlow> overscroll_effect_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderWidgetHostViewAndroid);
 };
 
 } // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_RENDER_WIDGET_HOST_VIEW_ANDROID_H_