diff options
| author | groby@chromium.org <groby@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-08-20 23:48:51 +0000 |
|---|---|---|
| committer | groby@chromium.org <groby@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-08-20 23:48:51 +0000 |
| commit | db26138ce3cd2ac975f01a93dcd278049707abbd (patch) | |
| tree | 59784f5f382c4b8e1b7533e8f371e869412d44fc | |
| parent | f778d1f29be6a8f9702cceb9f9804cefbe833fba (diff) | |
| download | chromium_src-db26138ce3cd2ac975f01a93dcd278049707abbd.zip chromium_src-db26138ce3cd2ac975f01a93dcd278049707abbd.tar.gz chromium_src-db26138ce3cd2ac975f01a93dcd278049707abbd.tar.bz2 | |
[Coverity] Pass-by-ref instead of pass-by-val
CID=104687,104688
CID_COUNT=2
R=sky@chromium.org
BUG=none
Review URL: https://chromiumcodereview.appspot.com/10823385
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@152430 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | chrome/browser/favicon/select_favicon_frames.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/chrome/browser/favicon/select_favicon_frames.cc b/chrome/browser/favicon/select_favicon_frames.cc index 605ba69..f06c2c9 100644 --- a/chrome/browser/favicon/select_favicon_frames.cc +++ b/chrome/browser/favicon/select_favicon_frames.cc @@ -24,7 +24,9 @@ size_t BiggestCandidate(const std::vector<SkBitmap>& bitmaps) { return max_index; } -SkBitmap PadWithBorder(SkBitmap contents, int desired_size, int source_size) { +SkBitmap PadWithBorder(const SkBitmap& contents, + int desired_size, + int source_size) { SkBitmap bitmap; bitmap.setConfig( SkBitmap::kARGB_8888_Config, desired_size, desired_size); @@ -41,7 +43,7 @@ SkBitmap PadWithBorder(SkBitmap contents, int desired_size, int source_size) { return bitmap; } -SkBitmap SampleNearestNeighbor(SkBitmap contents, int desired_size) { +SkBitmap SampleNearestNeighbor(const SkBitmap& contents, int desired_size) { SkBitmap bitmap; bitmap.setConfig( SkBitmap::kARGB_8888_Config, desired_size, desired_size); |