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

Side by Side Diff: content/browser/renderer_host/render_view_host_impl.cc

Issue 10855036: Adding selection text direction to SelectionBoundsChanged. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Synced Created 8 years, 4 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 | Annotate | Revision Log
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 #include "content/browser/renderer_host/render_view_host_impl.h" 5 #include "content/browser/renderer_host/render_view_host_impl.h"
6 6
7 #include <set> 7 #include <set>
8 #include <string> 8 #include <string>
9 #include <utility> 9 #include <utility>
10 #include <vector> 10 #include <vector>
(...skipping 1281 matching lines...) Expand 10 before | Expand all | Expand 10 after
1292 1292
1293 void RenderViewHostImpl::OnMsgSelectionChanged(const string16& text, 1293 void RenderViewHostImpl::OnMsgSelectionChanged(const string16& text,
1294 size_t offset, 1294 size_t offset,
1295 const ui::Range& range) { 1295 const ui::Range& range) {
1296 if (view_) 1296 if (view_)
1297 view_->SelectionChanged(text, offset, range); 1297 view_->SelectionChanged(text, offset, range);
1298 } 1298 }
1299 1299
1300 void RenderViewHostImpl::OnMsgSelectionBoundsChanged( 1300 void RenderViewHostImpl::OnMsgSelectionBoundsChanged(
1301 const gfx::Rect& start_rect, 1301 const gfx::Rect& start_rect,
1302 const gfx::Rect& end_rect) { 1302 WebKit::WebTextDirection start_direction,
1303 if (view_) 1303 const gfx::Rect& end_rect,
1304 view_->SelectionBoundsChanged(start_rect, end_rect); 1304 WebKit::WebTextDirection end_direction) {
1305 if (view_) {
1306 view_->SelectionBoundsChanged(start_rect, start_direction,
1307 end_rect, end_direction);
1308 }
1305 } 1309 }
1306 1310
1307 void RenderViewHostImpl::OnMsgRouteCloseEvent() { 1311 void RenderViewHostImpl::OnMsgRouteCloseEvent() {
1308 // Have the delegate route this to the active RenderViewHost. 1312 // Have the delegate route this to the active RenderViewHost.
1309 delegate_->RouteCloseEvent(this); 1313 delegate_->RouteCloseEvent(this);
1310 } 1314 }
1311 1315
1312 void RenderViewHostImpl::OnMsgRouteMessageEvent( 1316 void RenderViewHostImpl::OnMsgRouteMessageEvent(
1313 const ViewMsg_PostMessage_Params& params) { 1317 const ViewMsg_PostMessage_Params& params) {
1314 // Give to the delegate to route to the active RenderViewHost. 1318 // Give to the delegate to route to the active RenderViewHost.
(...skipping 570 matching lines...) Expand 10 before | Expand all | Expand 10 after
1885 // can cause navigations to be ignored in OnMsgNavigate. 1889 // can cause navigations to be ignored in OnMsgNavigate.
1886 is_waiting_for_beforeunload_ack_ = false; 1890 is_waiting_for_beforeunload_ack_ = false;
1887 is_waiting_for_unload_ack_ = false; 1891 is_waiting_for_unload_ack_ = false;
1888 } 1892 }
1889 1893
1890 void RenderViewHostImpl::ClearPowerSaveBlockers() { 1894 void RenderViewHostImpl::ClearPowerSaveBlockers() {
1891 STLDeleteValues(&power_save_blockers_); 1895 STLDeleteValues(&power_save_blockers_);
1892 } 1896 }
1893 1897
1894 } // namespace content 1898 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/renderer_host/render_view_host_impl.h ('k') | content/browser/renderer_host/render_widget_host_view_aura.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698