Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(24)

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 2355723004: [Presentation API] 1-UA: send message between controller and receiver page (Closed)

Created:
4 years, 3 months ago by zhaobin
Modified:
4 years, 2 months ago
Reviewers:
mark a. foltz, imcheng
CC:
chromium-reviews, mlamouri+watch-content_chromium.org, feature-media-reviews_chromium.org, jam, haraken, media-router+watch_chromium.org, dglazkov+blink, darin-cc_chromium.org, blink-reviews, blink-reviews-api_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

[Presentation API] 1-UA: send message between controller and receiver page - Add didReceiveSessionTextMessage() and didReceiveSessionBinaryMessage() functions to PresentationReceiver class - In controller/receiver frame's PSImpl::SendSessionMessage(), invoke receiver/controller frame's (message observer) PSImpl::OnSessionMessages() BUG=525660

Patch Set 1 #

Patch Set 2 : Merge with changes in Issue 2343013002 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+179 lines, -69 lines) Patch
M chrome/browser/media/router/offscreen_presentation_manager.h View 1 3 chunks +6 lines, -4 lines 0 comments Download
M chrome/browser/media/router/offscreen_presentation_manager.cc View 1 3 chunks +10 lines, -5 lines 0 comments Download
M chrome/browser/media/router/offscreen_presentation_manager_unittest.cc View 1 15 chunks +45 lines, -38 lines 0 comments Download
M chrome/browser/media/router/presentation_service_delegate_impl.cc View 1 1 chunk +2 lines, -2 lines 0 comments Download
M content/browser/presentation/presentation_service_impl.h View 1 1 chunk +5 lines, -5 lines 0 comments Download
M content/browser/presentation/presentation_service_impl.cc View 1 3 chunks +16 lines, -3 lines 0 comments Download
M content/public/browser/presentation_service_delegate.h View 1 1 chunk +5 lines, -0 lines 0 comments Download
M content/renderer/presentation/presentation_dispatcher.cc View 1 4 chunks +11 lines, -7 lines 0 comments Download
M third_party/WebKit/Source/modules/presentation/PresentationConnectionList.h View 2 chunks +3 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/modules/presentation/PresentationConnectionList.cpp View 1 chunk +10 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/modules/presentation/PresentationReceiver.h View 1 chunk +2 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/modules/presentation/PresentationReceiver.cpp View 1 chunk +20 lines, -0 lines 0 comments Download
M third_party/WebKit/Source/modules/presentation/PresentationReceiverTest.cpp View 1 4 chunks +31 lines, -0 lines 0 comments Download
M third_party/WebKit/public/platform/modules/presentation/WebPresentationClient.h View 1 1 chunk +9 lines, -0 lines 0 comments Download
M third_party/WebKit/public/platform/modules/presentation/WebPresentationController.h View 1 chunk +2 lines, -2 lines 0 comments Download
M third_party/WebKit/public/platform/modules/presentation/WebPresentationReceiver.h View 1 chunk +2 lines, -3 lines 0 comments Download

Depends on Patchset:

Issue 2343013002 Patch 20001

Messages

Total messages: 2 (1 generated)
zhaobin
4 years, 3 months ago (2016-09-23 20:05:23 UTC) #2 

          

          

        


      

    

    

  











      		
    

  





  
    
  






  Powered by Google App Engine





  
This is Rietveld 408576698