Don't repaint button when its click() method is called

When click() method is called, previously we change the active status
of the element to true and then back to false. This doesn't change
anything for a button (or other elements that status doesn't change
after click()) but causes a repaint.

Don't change active status when click() method is called. (Elements
that status changes after click() call (e.g. checkboxes) will still
repaint.)

This doesn't change how we handle real mouse down/up/click events.

BUG=369358
TEST=fast/repaint/button-checkbox-click-method-repaint.html
TEST=fast/events/button-mouse-active.html # ensures :active is correct on real mouse down events.

Review URL: https://codereview.chromium.org/291563009

git-svn-id: svn://svn.chromium.org/blink/trunk@174988 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Fix under-repaint in non-RAL mode

The under-repaint occurs in non-RAL mode when an object needs both
repaint and non-full layout. https://codereview.chromium.org/265703012/
caused this because it skips the direct repaint() call in the above
case.

BUG=376306
TEST=fast/repaint/change-text-content-and-background-color.html

Review URL: https://codereview.chromium.org/298973011

git-svn-id: svn://svn.chromium.org/blink/trunk@174989 bbb929c8-8fbe-4397-9dbb-9b2b20218538