Revert of [Presentation API] (5th) (1-UA) integrate controller and receiver side for 1-UA messaging

Revert of [Presentation API] (5th) (1-UA) integrate controller and receiver side for 1-UA messaging (patchset #16 id:400001 of https://codereview.chromium.org/2471573005/ )

Reason for revert:
Reverting as this is a suspect CL causing a presentation webkit test to fail:
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Linux%20Trusty/builds/22948

Debug output:
10:33:07.330 27811 renderer crash, pid = None, error_line = #CRASHED - renderer
10:33:07.331 27811 killed pid 18285
10:33:07.331 27811 worker/2 presentation/presentation-reconnect.html crashed, (stderr lines):
10:33:07.332 27811   [18285:18329:0206/103306.890280:1569541808:WARNING:url_request_job_manager.cc(90)] Failed to map: layout-test-mojom://content/shell/renderer/layout_test/frame_interface_registry
10:33:07.332 27811   [18285:18329:0206/103306.890393:1569541918:WARNING:url_request_job_manager.cc(90)] Failed to map: layout-test-mojom://content/shell/renderer/layout_test/interface_registry
10:33:07.332 27811   [18285:18329:0206/103306.891027:1569542552:WARNING:url_request_job_manager.cc(90)] Failed to map: layout-test-mojom://Mojo Helpers
10:33:07.332 27811   [1:1:0206/103306.896454:1569552070:ERROR:mojo_context_state.cc(194)] Failed to fetch source for module "content/shell/renderer/layout_test/frame_interface_registry"
10:33:07.332 27811   [1:1:0206/103306.900643:1569552511:ERROR:mojo_context_state.cc(194)] Failed to fetch source for module "content/shell/renderer/layout_test/interface_registry"
10:33:07.332 27811   [1:1:0206/103306.901223:1569553276:ERROR:mojo_context_state.cc(194)] Failed to fetch source for module "Mojo Helpers"
10:33:07.332 27811   [18285:18329:0206/103306.932287:1569583816:WARNING:crash_handler_host_linux.cc(324)] Could not translate tid, attempt = 1 retry ...
10:33:07.333 27725 [42777/48739] presentation/presentation-reconnect.html failed unexpectedly (renderer crashed)

Original issue's description:
> [Presentation API] (5th)(1-UA) integrate controller and receiver side for 1-UA messaging
> 
> This CL actually turns on 1-UA messaging:
> 
> - When presentation starts, a new route is created and returned to MR. In PresentationDispatcher, we send controller_connection_proxy and receiver_connection_request back to browser via PresentationService::SetPresentationConnection()
> 
> For 1-UA:
>   - In PresentationServiceDelegateImpl::ConnectToPresentation(), we register proxy and request as a controller connection entry to OPM. If there has been a receiver_call_back, invoke it;
>   - When offscreen tab is rendered, we register a receiverPSDImpl, which registers a receiver_call_back to OPM. If there is any pending controller connection entry, invoke receiver_call_back for each controller connection entry;
>   - In receiver_call_back (PresentationDispatcher::OnReceiverConnectionAvailable() function), we create a receiver_connection_proxy object, bind it to receiver_connection_request (input parameter) and connect it with controller_connection_proxy (input parameter). Message sending between controller Blink connection and receiver Blink connection may happen afterwards.
> 
> For 2-UA:
>   - In PresentationServiceDelegateImpl::ConnectToPresentation(), we create a browser_connection_proxy object, bind it to receiver_connection_request (input parameter) and connect it with controller_connection_proxy (input parameter). Message sending from controller Blink connection to media router may happen afterwards.
> 
> More details in code comments and design doc (use chromium.org account): https://docs.google.com/document/d/1XM3jhMJTQyhEC5PDAAJFNIaKh6UUEihqZDz_ztEe4Co/edit#heading=h.hadpx5oi0gml
> 
> Patch set 4 updates:
> - added BrowserPresentationConnectionProxy class
> - Added PresentationFrame::IsOffscreen() and PresentationFrame::OnBrowserConnectionAvailable()
> - PresentationConnection::OnMessage() takes a callback
> - PresentationDispatcher::SendXXX() takes PresentationConnectionProxy as parameter
> 
> BUG=525660, 513859
> 
> Review-Url: https://codereview.chromium.org/2471573005
> Cr-Commit-Position: refs/heads/master@{#448299}
> Committed: https://chromium.googlesource.com/chromium/src/+/4b94a1fe3f58aacad2c46bbfd84aa780469e15be

TBR=mfoltz@chromium.org,imcheng@chromium.org,jam@chromium.org,miu@chromium.org,zhaobin@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=525660, 513859
Review-Url: https://codereview.chromium.org/2674273003
Cr-Commit-Position: refs/heads/master@{#448341}
Committed: https://chromium.googlesource.com/chromium/src/+/85671a55bef6e30d53982b94647b5ba85a7eee77

[joedow, 2017-02-06 19:13:28]

The CQ bit was checked by joedow@chromium.org

[joedow, 2017-02-06 19:13:29]

Created Revert of [Presentation API] (5th) (1-UA) integrate controller and
receiver side for 1-UA messaging

[commit-bot: I haz the power, 2017-02-06 19:14:10]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-06 19:17:08]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486408408910220, "parent_rev":
"8aced40d3b566e3f0a04e4ae3e2453ab9db43b74", "commit_rev":
"85671a55bef6e30d53982b94647b5ba85a7eee77"}

[commit-bot: I haz the power, 2017-02-06 19:17:44]

Description was changed from

==========
Revert of [Presentation API] (5th) (1-UA) integrate controller and receiver side
for 1-UA messaging (patchset #16 id:400001 of
https://codereview.chromium.org/2471573005/ )

Reason for revert:
Reverting as this is a suspect CL causing a presentation webkit test to fail:
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Linu...

Debug output:
10:33:07.330 27811 renderer crash, pid = None, error_line = #CRASHED - renderer
10:33:07.331 27811 killed pid 18285
10:33:07.331 27811 worker/2 presentation/presentation-reconnect.html crashed,
(stderr lines):
10:33:07.332 27811  
[18285:18329:0206/103306.890280:1569541808:WARNING:url_request_job_manager.cc(90)]
Failed to map:
layout-test-mojom://content/shell/renderer/layout_test/frame_interface_registry
10:33:07.332 27811  
[18285:18329:0206/103306.890393:1569541918:WARNING:url_request_job_manager.cc(90)]
Failed to map:
layout-test-mojom://content/shell/renderer/layout_test/interface_registry
10:33:07.332 27811  
[18285:18329:0206/103306.891027:1569542552:WARNING:url_request_job_manager.cc(90)]
Failed to map: layout-test-mojom://Mojo Helpers
10:33:07.332 27811  
[1:1:0206/103306.896454:1569552070:ERROR:mojo_context_state.cc(194)] Failed to
fetch source for module
"content/shell/renderer/layout_test/frame_interface_registry"
10:33:07.332 27811  
[1:1:0206/103306.900643:1569552511:ERROR:mojo_context_state.cc(194)] Failed to
fetch source for module "content/shell/renderer/layout_test/interface_registry"
10:33:07.332 27811  
[1:1:0206/103306.901223:1569553276:ERROR:mojo_context_state.cc(194)] Failed to
fetch source for module "Mojo Helpers"
10:33:07.332 27811  
[18285:18329:0206/103306.932287:1569583816:WARNING:crash_handler_host_linux.cc(324)]
Could not translate tid, attempt = 1 retry ...
10:33:07.333 27725 [42777/48739] presentation/presentation-reconnect.html failed
unexpectedly (renderer crashed)

Original issue's description:
> [Presentation API] (5th)(1-UA) integrate controller and receiver side for 1-UA
messaging
> 
> This CL actually turns on 1-UA messaging:
> 
> - When presentation starts, a new route is created and returned to MR. In
PresentationDispatcher, we send controller_connection_proxy and
receiver_connection_request back to browser via
PresentationService::SetPresentationConnection()
> 
> For 1-UA:
>   - In PresentationServiceDelegateImpl::ConnectToPresentation(), we register
proxy and request as a controller connection entry to OPM. If there has been a
receiver_call_back, invoke it;
>   - When offscreen tab is rendered, we register a receiverPSDImpl, which
registers a receiver_call_back to OPM. If there is any pending controller
connection entry, invoke receiver_call_back for each controller connection
entry;
>   - In receiver_call_back
(PresentationDispatcher::OnReceiverConnectionAvailable() function), we create a
receiver_connection_proxy object, bind it to receiver_connection_request (input
parameter) and connect it with controller_connection_proxy (input parameter).
Message sending between controller Blink connection and receiver Blink
connection may happen afterwards.
> 
> For 2-UA:
>   - In PresentationServiceDelegateImpl::ConnectToPresentation(), we create a
browser_connection_proxy object, bind it to receiver_connection_request (input
parameter) and connect it with controller_connection_proxy (input parameter).
Message sending from controller Blink connection to media router may happen
afterwards.
> 
> More details in code comments and design doc (use chromium.org account):
https://docs.google.com/document/d/1XM3jhMJTQyhEC5PDAAJFNIaKh6UUEihqZDz_ztEe4...
> 
> Patch set 4 updates:
> - added BrowserPresentationConnectionProxy class
> - Added PresentationFrame::IsOffscreen() and
PresentationFrame::OnBrowserConnectionAvailable()
> - PresentationConnection::OnMessage() takes a callback
> - PresentationDispatcher::SendXXX() takes PresentationConnectionProxy as
parameter
> 
> BUG=525660,513859
> 
> Review-Url: https://codereview.chromium.org/2471573005
> Cr-Commit-Position: refs/heads/master@{#448299}
> Committed:
https://chromium.googlesource.com/chromium/src/+/4b94a1fe3f58aacad2c46bbfd84a...

TBR=mfoltz@chromium.org,imcheng@chromium.org,jam@chromium.org,miu@chromium.or...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=525660,513859
==========

to

==========
Revert of [Presentation API] (5th) (1-UA) integrate controller and receiver side
for 1-UA messaging (patchset #16 id:400001 of
https://codereview.chromium.org/2471573005/ )

Reason for revert:
Reverting as this is a suspect CL causing a presentation webkit test to fail:
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Linu...

Debug output:
10:33:07.330 27811 renderer crash, pid = None, error_line = #CRASHED - renderer
10:33:07.331 27811 killed pid 18285
10:33:07.331 27811 worker/2 presentation/presentation-reconnect.html crashed,
(stderr lines):
10:33:07.332 27811  
[18285:18329:0206/103306.890280:1569541808:WARNING:url_request_job_manager.cc(90)]
Failed to map:
layout-test-mojom://content/shell/renderer/layout_test/frame_interface_registry
10:33:07.332 27811  
[18285:18329:0206/103306.890393:1569541918:WARNING:url_request_job_manager.cc(90)]
Failed to map:
layout-test-mojom://content/shell/renderer/layout_test/interface_registry
10:33:07.332 27811  
[18285:18329:0206/103306.891027:1569542552:WARNING:url_request_job_manager.cc(90)]
Failed to map: layout-test-mojom://Mojo Helpers
10:33:07.332 27811  
[1:1:0206/103306.896454:1569552070:ERROR:mojo_context_state.cc(194)] Failed to
fetch source for module
"content/shell/renderer/layout_test/frame_interface_registry"
10:33:07.332 27811  
[1:1:0206/103306.900643:1569552511:ERROR:mojo_context_state.cc(194)] Failed to
fetch source for module "content/shell/renderer/layout_test/interface_registry"
10:33:07.332 27811  
[1:1:0206/103306.901223:1569553276:ERROR:mojo_context_state.cc(194)] Failed to
fetch source for module "Mojo Helpers"
10:33:07.332 27811  
[18285:18329:0206/103306.932287:1569583816:WARNING:crash_handler_host_linux.cc(324)]
Could not translate tid, attempt = 1 retry ...
10:33:07.333 27725 [42777/48739] presentation/presentation-reconnect.html failed
unexpectedly (renderer crashed)

Original issue's description:
> [Presentation API] (5th)(1-UA) integrate controller and receiver side for 1-UA
messaging
> 
> This CL actually turns on 1-UA messaging:
> 
> - When presentation starts, a new route is created and returned to MR. In
PresentationDispatcher, we send controller_connection_proxy and
receiver_connection_request back to browser via
PresentationService::SetPresentationConnection()
> 
> For 1-UA:
>   - In PresentationServiceDelegateImpl::ConnectToPresentation(), we register
proxy and request as a controller connection entry to OPM. If there has been a
receiver_call_back, invoke it;
>   - When offscreen tab is rendered, we register a receiverPSDImpl, which
registers a receiver_call_back to OPM. If there is any pending controller
connection entry, invoke receiver_call_back for each controller connection
entry;
>   - In receiver_call_back
(PresentationDispatcher::OnReceiverConnectionAvailable() function), we create a
receiver_connection_proxy object, bind it to receiver_connection_request (input
parameter) and connect it with controller_connection_proxy (input parameter).
Message sending between controller Blink connection and receiver Blink
connection may happen afterwards.
> 
> For 2-UA:
>   - In PresentationServiceDelegateImpl::ConnectToPresentation(), we create a
browser_connection_proxy object, bind it to receiver_connection_request (input
parameter) and connect it with controller_connection_proxy (input parameter).
Message sending from controller Blink connection to media router may happen
afterwards.
> 
> More details in code comments and design doc (use chromium.org account):
https://docs.google.com/document/d/1XM3jhMJTQyhEC5PDAAJFNIaKh6UUEihqZDz_ztEe4...
> 
> Patch set 4 updates:
> - added BrowserPresentationConnectionProxy class
> - Added PresentationFrame::IsOffscreen() and
PresentationFrame::OnBrowserConnectionAvailable()
> - PresentationConnection::OnMessage() takes a callback
> - PresentationDispatcher::SendXXX() takes PresentationConnectionProxy as
parameter
> 
> BUG=525660,513859
> 
> Review-Url: https://codereview.chromium.org/2471573005
> Cr-Commit-Position: refs/heads/master@{#448299}
> Committed:
https://chromium.googlesource.com/chromium/src/+/4b94a1fe3f58aacad2c46bbfd84a...

TBR=mfoltz@chromium.org,imcheng@chromium.org,jam@chromium.org,miu@chromium.or...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=525660,513859

Review-Url: https://codereview.chromium.org/2674273003
Cr-Commit-Position: refs/heads/master@{#448341}
Committed:
https://chromium.googlesource.com/chromium/src/+/85671a55bef6e30d53982b94647b...
==========

[commit-bot: I haz the power, 2017-02-06 19:17:46]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/85671a55bef6e30d53982b94647b...