NotificationController handles an ownership of NotificaitonClient

Source/web/WebLocalFrameImpl.h

 /*
  * 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 317 matching lines @@
     void setWebCoreFrame(PassRefPtr<WebCore::LocalFrame>);
 
     void loadJavaScriptURL(const WebCore::KURL&);
 
     // Returns a hit-tested VisiblePosition for the given point
     WebCore::VisiblePosition visiblePositionForWindowPoint(const WebPoint&);
 
     WebPlugin* focusedPluginIfInputMethodSupported();
 
     // Returns the provider of desktop notifications.
-    NotificationPresenterImpl* notificationPresenterImpl();
+    PassOwnPtr<NotificationPresenterImpl> notificationPresenterImpl();
 
     FrameLoaderClientImpl m_frameLoaderClientImpl;
 
     // The embedder retains a reference to the WebCore LocalFrame while it is active in the DOM. This
     // reference is released when the frame is removed from the DOM or the entire page is closed.
     // FIXME: These will need to change to WebFrame when we introduce WebFrameProxy.
     RefPtr<WebCore::LocalFrame> m_frame;
     WebLocalFrameImpl* m_parent;
     WebLocalFrameImpl* m_previousSibling;
     WebLocalFrameImpl* m_nextSibling;
@@ skipping 16 matching lines @@
 
     // Valid between calls to BeginPrint() and EndPrint(). Containts the print
     // information. Is used by PrintPage().
     OwnPtr<ChromePrintContext> m_printContext;
 
     // Stores the additional input events offset and scale when device metrics emulation is enabled.
     WebCore::IntSize m_inputEventsOffsetForEmulation;
     float m_inputEventsScaleFactorForEmulation;
 
     // The provider of desktop notifications;
-    NotificationPresenterImpl m_notificationPresenter;
+    OwnPtr<NotificationPresenterImpl> m_notificationPresenter;

[jochen (gone - plz use gerrit), 2014/05/05 07:35:14]

why do you need this member now?

[gyuyoung-inactive, 2014/05/06 15:26:09]

Yes, we don't need to keep this member and notificationPresenterImpl() indeed.
Moved those inside setWebCoreFrame() function and removed from header file.

 
     UserMediaClientImpl m_userMediaClientImpl;
 };
 
 DEFINE_TYPE_CASTS(WebLocalFrameImpl, WebFrame, frame, frame->isWebLocalFrame(), frame.isWebLocalFrame());
 
 } // namespace blink
 
 #endif