Oilpan: update WebNotification and WebDOMMediaStreamTrack type conversions

Source/web/NotificationPresenterImpl.cpp

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 61 matching lines @@
     m_presenter = presenter;
 }
 
 bool NotificationPresenterImpl::isInitialized()
 {
     return !!m_presenter;
 }
 
 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)
 {
     int result = m_presenter->checkPermission(WebSecurityOrigin(context->securityOrigin()));
     return static_cast<NotificationClient::Permission>(result);
 }
 
 void NotificationPresenterImpl::requestPermission(ExecutionContext* context, WTF::PassOwnPtr<NotificationPermissionCallback> callback)
 {
     m_presenter->requestPermission(WebSecurityOrigin(context->securityOrigin()), new NotificationPermissionCallbackClient(m_presenter, context->securityOrigin(), callback));
 }
 
 } // namespace blink