Update GeolocationController to call startUpdating() / stopUpdating() only if needed

Update GeolocationController to call startUpdating() / stopUpdating() only if needed
as some clients (e.g. the geolocation mock client) do not accept consecutive calls
to either of these methods.

This was done by introducing startUpdatingIfNeeded() / stopUpdatingIfNeeded() private
methods which update a boolean flag to avoid useless calls.

This fixes a possible assertion hit in Debug.

R=jochen
BUG=310326

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160480 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove virtual destructor for Pair class

This destructor was empty and marked as virtual. Removing it avoids creating
an unnecessary vtable since this class is never subclassed nor does it have
any virtual function.

BUG=

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160481 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove virtual destructors for CSSMatrix and CSSVariablesMap

Remove destructors for CSSMatrix and CSSVariablesMap as these were empty
and marked as virtual. This avoids creating an unnecessary vtable
as these classes are never subclasses nor do they have any virtual
function.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160482 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Move stale TestExpectations lines to StaleTestExpectations.

This moves all the platform-specific and/or virtual stale lines.

BUG=308779, 247466, 239465, 31488, 30536, 108833, 14009, 250935, 23476, 7482, 27569, 51571, 58257, 58735, 66087, 83290, 237003, 227473, 232061, 234758, 251129, 143475, 241576, 260660, 178745, 243248, 243649, 243732, 243738, 243894, 244123, 247889, 247979, 248065, 248067, 257325, 246077, 259547, 240374
TBR=adamk@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160483 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Adding myself to the rendering watchlist

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160484 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Don't error if NeedsRebaseline is in a comment in TestExpectations.

auto-rebaseline was failing because it was trying to parse a comment
line as a standard format expectations line and it failed. Instead,
strip comments first.

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160485 bbb929c8-8fbe-4397-9dbb-9b2b20218538
Remove unused Perl in-file support

All the clients of this code have been converted to Python.

R=eseidel@chromium.org

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

git-svn-id: svn://svn.chromium.org/blink/trunk@160486 bbb929c8-8fbe-4397-9dbb-9b2b20218538
DevTools: Extract RemoteObject.getProperty method.

So that it could be reused soon.

BUG=
R=yurys@chromium.org

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

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