Commit 9a9bf1d7 authored by groby@chromium.org's avatar groby@chromium.org

[Coverity] pass-by-ref instead of pass-by-val

CID=103838
R=estade@chromium.org

BUG=
TEST=


Review URL: http://codereview.chromium.org/10166012

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@133492 0039d316-1c4b-4281-b951-d872f2087c98
parent 5a322ae6
......@@ -418,7 +418,8 @@ void FindBarGtk::AudibleAlert() {
// gtk_widget_error_bell(widget());
}
gfx::Rect FindBarGtk::GetDialogPosition(gfx::Rect avoid_overlapping_rect) {
gfx::Rect FindBarGtk::GetDialogPosition(
const gfx::Rect& avoid_overlapping_rect) {
bool ltr = !base::i18n::IsRTL();
// 15 is the size of the scrollbar, copied from ScrollbarThemeChromium.
// The height is not used.
......
......@@ -104,7 +104,7 @@ class FindBarGtk : public FindBar,
// the top of the page area, (it will be converted to coordinates relative to
// the top of the browser window, when comparing against the dialog
// coordinates). The returned value is relative to the browser window.
gfx::Rect GetDialogPosition(gfx::Rect avoid_overlapping_rect);
gfx::Rect GetDialogPosition(const gfx::Rect& avoid_overlapping_rect);
// Adjust the text alignment according to the text direction of the widget
// and |text_entry_|'s content, to make sure the real text alignment is
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment