Simplify the String management in CSSSelector::RareData

CSSSelector::setValue was manually managing the lifetime of a StringImpl*
from inside an AtomicString which was very confusing. Lets just use a real
String instead.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@164187 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Do not use v8::Integer::New{,Unsigned} with weird argument order.

R=dcarney@chromium.org
BUG=324225

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

git-svn-id: svn://svn.chromium.org/blink/trunk@164188 bbb929c8-8fbe-4397-9dbb-9b2b20218538
HTMLOptionElement code style: consistently access select owner.

For methods that access the select owner, do this consistently
when otherwise possible.

R=
BUG=

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

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