[SPDY] Remove a SPDY session from its pool only when not in an IO loop

This is in preparation for making pool removal destroy the SPDY session.

Add an in_io_loop_ variable to SpdySession to help with
the above. Assert its state in various functions.

Move the guts of CloseSessionOnError into DoCloseSession(). Make
DoCloseSession() close the session only if it's not already closed
and remove the session only if it's not in an IO loop. Make it return
a value describing what it did and add assertions for that value
whenever it's called.

Add Pump{Read,Write}Loop() functions that check for
session closure after calling Do{Read,Write}Loop()
and remove from the pool if so.

Some minor test changes to reflect the above.

BUG=255701

Review URL: https://chromiumcodereview.appspot.com/18408008

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212833 0039d316-1c4b-4281-b951-d872f2087c98
Files.app: Replace the border images for buttons with the latest assets.

BUG=none
TEST=none
TBR=mtomasz@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212834 0039d316-1c4b-4281-b951-d872f2087c98
Updating trunk VERSION from 1572.0 to 1573.0

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212836 0039d316-1c4b-4281-b951-d872f2087c98
Sanitize png files in Files.app

Ran "./build/sanitize-png-files.sh -o2 chrome/browser/resources/file_manager/images/"

Result:
- 35 files changed
- 52398 => 52246 bytes (152 bytes : 0 %)

BUG=none
TEST=none
TBR=mtomasz@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212838 0039d316-1c4b-4281-b951-d872f2087c98
Blink roll r154591 -> r154619

TBR=haraken

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

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212842 0039d316-1c4b-4281-b951-d872f2087c98
Make AppListItemView's progress bar not reset to 0 while installing app.

BUG=257961

Review URL: https://chromiumcodereview.appspot.com/19705012

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212843 0039d316-1c4b-4281-b951-d872f2087c98
Automated Commit: Committing new LKGM version 4436.0.0 for chromeos.

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212844 0039d316-1c4b-4281-b951-d872f2087c98
When dismissing OSX app launcher, do not activate Chrome if Chrome was inactive.

When an active window is closed or hidden in an active application, OSX
will focus and raise the next window in that application. For the app
launcher, this means a buried browser window can pop into focus simply
by showing and dismissing the app launcher via the dock icon.

This change checks whether OSX is about to do this, by checking whether
the app launcher is being dismissed while it is the 'main' window of
Chrome (as opposed to being dismissed simply by losing focus). Instead
of closing the app list, instead just activate the application that was
active when the app launcher was shown. The launcher will lose 'main'
status and dismiss itself again, without activating another Chrome
window, since it is no longer active.

BUG=261963
TEST=Have OSX app launcher enabled. Open Chrome browser. Focus a
non-Chrome OSX app on top of the browser. Open App launcher via dock
icon and dismiss it via dock icon. Focus should stay with the non-Chrome
app, and browser window should not be raised.

Review URL: https://chromiumcodereview.appspot.com/19762009

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212847 0039d316-1c4b-4281-b951-d872f2087c98
Adding android.webview policy to deep memory profiler.

This change basically introduce a new DMP profile which is merging
android.browser and android.renderer rules, in order to be used
for single-process WebView instances.

NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/19541008

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212848 0039d316-1c4b-4281-b951-d872f2087c98
DevTools: ListInterfaces can fail while discovering usb devices.

BUG=262441
R=vsevik@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212849 0039d316-1c4b-4281-b951-d872f2087c98
Files.app: JsDoc clean up.

This patch fixes misc errors of JsDoc and adds some additional annotation to Files.app JavaScript code. All changes are in JsDoc. No functionality is changed.

BUG=none
TEST=none
R=mtomasz@chromium.org

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212850 0039d316-1c4b-4281-b951-d872f2087c98
Disable flaky ExtensionApiTest.WallpaperPicker.

BUG=262854
NOTRY=true
TBR=bshe@chromium.org

Review URL: https://chromiumcodereview.appspot.com/19773019

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212851 0039d316-1c4b-4281-b951-d872f2087c98
Add devtools-reviews@chromium.org to the watchers of devtools commits.

TBR=paulirish

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212852 0039d316-1c4b-4281-b951-d872f2087c98
Revert the Blink roll back to r154591

TBR=haraken

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

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212854 0039d316-1c4b-4281-b951-d872f2087c98
Retry Blink roll r154591 -> r154619

This will break Android bots, so I need to land https://codereview.chromium.org/19832003
immediately after the Blink roll. I will do it.

TBR=haraken

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

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212856 0039d316-1c4b-4281-b951-d872f2087c98
Update the new path to WebKit tests data.

data directory was moved from WebKit/chromium/tests/ to web/tests/ in r154618
on Blink repository (see CL https://codereview.chromium.org/19906002/).

R=frankf@chromium.org,abarth@chromium.org
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/19832003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@212857 0039d316-1c4b-4281-b951-d872f2087c98
[SPDY] Use WeakPtr<SpdySession> everywhere but SpdySessionPool

Make SpdyHttpStream cache it's SPDY stream's LoadTimingInfo
on close. Also, make SpdyHttpStream query SpdySession::IsReused()
when it's constructed and cache the value, as that's closer to the
intent of its use.

Avoid uses of SpdySession::IsClosed() in non-test code and add
TODO to remove uses from test code. This is more correct since
testing the WeakPtr is a stronger condition than testing openness
and SpdySession functions already do the right thing if the
SpdySession is already closed.

Tweak some tests that implicitly depended on
having refs to SpdySession.

BUG=255701
R=rch@chromium.org

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

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