Remove usage of deprecated DOM API

Use defaultPrevented instead of returnValue

BUG=None

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@228073 0039d316-1c4b-4281-b951-d872f2087c98
SVN changes up to revision 228073
cc: Reset resource update controller on closing of layer tree host

This fixes a potential use-after-free bug.

The ResourceUpdateController might have posted a task using a weak
pointer to itself. That task can result in a commit, which assumes
a number of resources are still available on the LayerTreeHostImpl.

However, ThreadProxy::LayerTreeHostClosedOnImplThread might have
freed those resources before the ThreadProxy itself is destroyed,
allowing the ResourceUpdateController's posted task to run without
the resources available.

This patch resets the ResourceUpdateController in
ThreadProxy::LayerTreeHostClosedOnImplThread now to avoid the race
between the posted task and the destruction of the ThreadProxy.

BUG=303828

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@228074 0039d316-1c4b-4281-b951-d872f2087c98
Make app list profile selector reload on non-active user signin status change.

The AppListViewDelegate only listens to signin events from the current app list
profile. This CL makes it listen to all profiles which is necessary to update
the signin details in the menu when a user that isn't the active app list user
signs into chrome.

BUG=305475

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@228075 0039d316-1c4b-4281-b951-d872f2087c98
SVN changes up to revision 228075