Revert of [Presentation API] Implement ondefaultsessionstart in PSImpl. (patchset...
commit89a5c0d4bc4e43ac69ee4db6a01916f41ccf2c0f
authorvmpstr <vmpstr@chromium.org>
Wed, 22 Apr 2015 22:16:01 +0000 (22 15:16 -0700)
committerCommit bot <commit-bot@chromium.org>
Wed, 22 Apr 2015 22:16:09 +0000 (22 22:16 +0000)
tree58a1d2726b4d9c048ed3ddae95d977ed8b122a76
parent2b6d0da267480a7222551b982bf12a83e0ea4df1
Revert of [Presentation API] Implement ondefaultsessionstart in PSImpl. (patchset #7 id:180001 of https://codereview.chromium.org/1073893003/)

Reason for revert:
Speculative revert to fix content_unittests failures: http://build.chromium.org/p/chromium.linux/builders/Android%20Tests%20%28dbg%29/builds/27420

Original issue's description:
> [Presentation API] Implement ondefaultsessionstart in PSImpl.
>
> Added DefaultSessionStartContext for coordinating sending default
> session back to PresentationDispatcher.
>
> When ListenForDefaultSessionStart is called, DefaultSessionStartContext
> will be installed on PresentationServiceImpl. When both the default
> session and PresentationDispatcher's callback are available, the
> callback will be invoked with the session.
>
> On Reset(), if a callback is available, it will be invoked with null.
> Changed PresentationDispatcher to not update Blink in that case.
>
> Also, PSImpl now keeps track of the corresponding RFH's ID instead of
> RFH* since most of the time we only need to use the ID.
>
> Changed PresentationServiceDelegate's Add/RemoveObserver interface,
> since the PresentationServiceDelegate need to be able to correlate
> an Observer with a RFH. (at most 1 per RFH, as it stands today).
>
> Added OnDefaultPresentationStarted to
> PresentationServiceDelegate::Observer interface and implemented it
> in PresentationServiceImpl.
>
> Added tests in PresentationServiceImpl.
>
> BUG=459001
>
> Committed: https://crrev.com/79ddfb2383dca537e481eb9e0a477267dd48bc60
> Cr-Commit-Position: refs/heads/master@{#326353}

TBR=avayvod@google.com,avi@chromium.org,imcheng@chromium.org,mfoltz@chromium.org,avayvod@chromium.org,imcheng@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=459001

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

Cr-Commit-Position: refs/heads/master@{#326378}
content/browser/presentation/presentation_service_impl.cc
content/browser/presentation/presentation_service_impl.h
content/browser/presentation/presentation_service_impl_unittest.cc
content/common/presentation/presentation_service.mojom
content/public/browser/presentation_service_delegate.h
content/renderer/presentation/presentation_dispatcher.cc