Chromium Code Reviews| Index: content/renderer/render_view_impl.cc |
| diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc |
| index 2374e0c989bd6950b190e540e530b2e4f2bcb0f1..fd0134b2c342aadccecf336cd0e33e0a4b869a81 100644 |
| --- a/content/renderer/render_view_impl.cc |
| +++ b/content/renderer/render_view_impl.cc |
| @@ -764,6 +764,9 @@ RenderViewImpl::RenderViewImpl(RenderViewImplParams* params) |
| target_url_status_(TARGET_NONE), |
| selection_text_offset_(0), |
| selection_range_(ui::Range::InvalidRange()), |
| +#if defined(OS_ANDROID) |
| + top_controls_constraints_(cc::INVALID), |
|
aelias_OOO_until_Jul13
2013/06/13 03:29:25
You don't ever set this value? Did you forget to
Jinsuk Kim
2013/06/13 04:18:13
Thanks for catching it. Now it is done in RenderVi
|
| +#endif |
| cached_is_main_frame_pinned_to_left_(false), |
| cached_is_main_frame_pinned_to_right_(false), |
| cached_has_main_frame_horizontal_scrollbar_(false), |
| @@ -2342,6 +2345,16 @@ bool RenderViewImpl::handleCurrentKeyboardEvent() { |
| return did_execute_command; |
| } |
| +void RenderViewImpl::didScrollWithKeyboard(const WebKit::WebSize& delta) { |
| +#if defined(OS_ANDROID) |
| + if (top_controls_constraints_ == cc::INVALID || delta.height == 0) |
| + return; |
| + compositor_->UpdateTopControlsState(top_controls_constraints_, |
|
aelias_OOO_until_Jul13
2013/06/13 03:29:25
If the field is moved to RenderViewImplAndroid, th
Jinsuk Kim
2013/06/13 04:18:13
Done.
|
| + delta.height < 0 ? cc::SHOWN : cc::HIDDEN, |
| + true); |
| +#endif |
| +} |
| + |
| WebKit::WebColorChooser* RenderViewImpl::createColorChooser( |
| WebKit::WebColorChooserClient* client, |
| const WebKit::WebColor& initial_color) { |