RIM PR: 210858 For network stream classes, they get the playerId() from class FrameLoaderClientBlackBerry::playerId(), which gets the id from WebCore::WebPageClient::getInstanceId(). There's also a special stream class: MediaSSLHandlerStream which is responsible for ssl verification of mmr engine, and this stream class's object is created inside a MMRPlayer and can only access to the PageClientBlackBerry. So I'm thinking of adding a new member function playerId() for PageClientBlackBerry, by doing this we can provide playerId() to MediaSSLHandlerStream directly and to other network streams via FrameLoaderClientBlackBerry as usual. Then we can upstream our media ssl handling codes for RIM PR 116205.
Created attachment 164722 [details] Patch
Comment on attachment 164722 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=164722&action=review otherwise LGTM > Source/WebKit/blackberry/Api/WebPage_p.h:182 > + virtual int playerId() const; according to WebKit style or existing code, we should name it playerID() rather than playerId.
Created attachment 164741 [details] Patch
Comment on attachment 164741 [details] Patch Clearing flags on attachment: 164741 Committed r129010: <http://trac.webkit.org/changeset/129010>
All reviewed patches have been landed. Closing bug.