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

Unified Diff: chrome/renderer/chrome_render_view_observer.cc

Issue 10939010: Cleanup: avoid foo ? true : false, part 1. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: rebase Created 8 years, 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/ui/webui/quota_internals_types.cc ('k') | content/common/gpu/media/h264_parser.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/renderer/chrome_render_view_observer.cc
===================================================================
--- chrome/renderer/chrome_render_view_observer.cc (revision 159076)
+++ chrome/renderer/chrome_render_view_observer.cc (working copy)
@@ -1012,8 +1012,7 @@
S16CPU new_width = static_cast<S16CPU>(src_bmp.height() * dest_aspect);
S16CPU x_offset = (src_bmp_width - new_width) / 2;
src_rect.set(x_offset, 0, new_width + x_offset, src_bmp.height());
- score->good_clipping =
- (src_aspect >= ThumbnailScore::kTooWideAspectRatio) ? false : true;
+ score->good_clipping = (src_aspect < ThumbnailScore::kTooWideAspectRatio);
} else {
src_rect.set(0, 0, src_bmp_width,
static_cast<S16CPU>(src_bmp_width / dest_aspect));
« no previous file with comments | « chrome/browser/ui/webui/quota_internals_types.cc ('k') | content/common/gpu/media/h264_parser.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698