Oilpan: update WebNotification and WebDOMMediaStreamTrack type conversions

Source/web/NotificationPresenterImpl.cpp

Index: Source/web/NotificationPresenterImpl.cpp
diff --git a/Source/web/NotificationPresenterImpl.cpp b/Source/web/NotificationPresenterImpl.cpp
index 0d926ea7eee001cc75ce02c030cc954db889bae5..ae8361577162e587efc8978ac17a861f44536848 100644
--- a/Source/web/NotificationPresenterImpl.cpp
+++ b/Source/web/NotificationPresenterImpl.cpp
@@ -79,20 +79,20 @@ bool NotificationPresenterImpl::isInitialized()
 
 bool NotificationPresenterImpl::show(Notification* notification)
 {
-    return m_presenter->show(PassRefPtr<Notification>(notification));
+    return m_presenter->show(RawPtr<Notification>(notification));

[Mads Ager (chromium), 2014/06/06 06:45:17]

I think these methods already take actual raw pointers, so can we just remove
the conversion here and pass the pointer as is?

[sof, 2014/06/06 06:50:28]

No, that runs into issues with converting a Notification* to a const
WebNotification&. We only have conversions over (const RawPtr<Notification>).

This is what I referred to in https://codereview.chromium.org/310393004/#msg2

[Mads Ager (chromium), 2014/06/06 07:13:20]

OK, thanks. :(

[sof, 2014/06/06 07:23:06]

There's stuff to be looked at wrt WebPrivatePtr<>, but I'd prefer to address
that separately.

(It is painfully slow (at this end) to make changes to WebPrivatePtr & test them
out.)

 }
 
 void NotificationPresenterImpl::close(Notification* notification)
 {
-    m_presenter->close(PassRefPtr<Notification>(notification));
+    m_presenter->close(RawPtr<Notification>(notification));
 
     // FIXME: Remove the duplicated call to cancel() when Chromium updated to override close() instead.
-    m_presenter->cancel(PassRefPtr<Notification>(notification));
+    m_presenter->cancel(RawPtr<Notification>(notification));
 }
 
 void NotificationPresenterImpl::notificationObjectDestroyed(Notification* notification)
 {
-    m_presenter->objectDestroyed(PassRefPtr<Notification>(notification));
+    m_presenter->objectDestroyed(RawPtr<Notification>(notification));
 }
 
 NotificationClient::Permission NotificationPresenterImpl::checkPermission(ExecutionContext* context)