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

Unified Diff: views/window/custom_frame_view.cc

Issue 145017: Fix dialog edges - error was because we made the top edges the same height bu... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 11 years, 6 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: views/window/custom_frame_view.cc
===================================================================
--- views/window/custom_frame_view.cc (revision 18861)
+++ views/window/custom_frame_view.cc (working copy)
@@ -422,11 +422,7 @@
SkBitmap* left = rb.GetBitmapNamed(IDR_CONTENT_LEFT_SIDE);
// Top.
- // This next calculation is necessary because the top center bitmap is shorter
- // than the top left and right bitmaps. We need their top edges to line up,
- // and we need the left and right edges to start below the corners' bottoms.
int top_edge_y = client_area_top - top->height();
- client_area_top = top_edge_y + top_left->height();
canvas->DrawBitmapInt(*top_left, client_area_bounds.x() - top_left->width(),
top_edge_y);
canvas->TileImageInt(*top, client_area_bounds.x(), top_edge_y,
@@ -454,8 +450,8 @@
// Draw the toolbar color to fill in the edges.
canvas->DrawRectInt(ResourceBundle::toolbar_color,
- client_area_bounds.x() - 1, client_area_top - 2,
- client_area_bounds.width() + 1, client_area_bottom - client_area_top + 2);
+ client_area_bounds.x() - 1, client_area_top - 1,
+ client_area_bounds.width() + 1, client_area_bottom - client_area_top + 1);
}
void CustomFrameView::LayoutWindowControls() {
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698