Misc. cleanup: simplify some code, add a few comments

BUG=none
TEST=none
Review URL: http://codereview.chromium.org/3143016

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56106 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 3 insertions, 9 deletions

diff --git a/views/view.cc b/views/view.cc
index 02d2766..9acbd47 100644
--- a/views/view.cc
+++ b/views/view.cc
@@ -400,16 +400,12 @@ void View::ProcessPaint(gfx::Canvas* canvas) {
     // where this view is located (related to its parent).
     canvas->TranslateInt(MirroredX(), y());
 
-    // Save the state again, so that any changes don't effect PaintChildren.
-    canvas->Save();
-
     // If the View we are about to paint requested the canvas to be flipped, we
     // should change the transform appropriately.
-    bool flip_canvas = FlipCanvasOnPaintForRTLUI();
-    if (flip_canvas) {
+    canvas->Save();
+    if (FlipCanvasOnPaintForRTLUI()) {
       canvas->TranslateInt(width(), 0);
       canvas->ScaleInt(-1, 1);
-      canvas->Save();
     }
 
     Paint(canvas);
@@ -417,10 +413,8 @@ void View::ProcessPaint(gfx::Canvas* canvas) {
     // We must undo the canvas mirroring once the View is done painting so that
     // we don't pass the canvas with the mirrored transform to Views that
     // didn't request the canvas to be flipped.
-    if (flip_canvas)
-      canvas->Restore();
-
     canvas->Restore();
+
     PaintChildren(canvas);
   }