Apply OwnPtr|PassOwnPtr to handle member variables and arguments in MIDIClient

Source/web/MIDIClientProxy.cpp

 /*
  * Copyright (C) 2013 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 26 matching lines @@
 
 using WebCore::MIDIAccess;
 
 namespace blink {
 
 MIDIClientProxy::MIDIClientProxy(WebMIDIClient* client)
     : m_client(client)
 {
 }
 
+PassOwnPtr<MIDIClientProxy> MIDIClientProxy::create(WebMIDIClient* client)
+{
+    return adoptPtr(new MIDIClientProxy(client));

[jochen (gone - plz use gerrit), 2014/04/02 09:28:59]

nit should go into the header

+}
+
 void MIDIClientProxy::requestSysexPermission(PassRefPtrWillBeRawPtr<MIDIAccess> access)
 {
     if (m_client)

[haraken, 2014/04/02 01:08:54]

Is it possible that m_client is 0?

[gyuyoung-inactive, 2014/04/02 01:17:35]

Yes, client parameter of WebViewImpl's ctor can be 0. So, if the client is null,
WebViewImpl has passed 0 to MIDIClientProxy. That's why MIDIClientProxy's
m_client can be 0.

- m_midiClientProxy(adoptPtr(new MIDIClientProxy(client ?
client->webMIDIClient() : 0)))
+ provideMIDITo(*m_page, MIDIClientProxy::create(client ?
client->webMIDIClient() : 0));

         m_client->requestSysexPermission(WebMIDIPermissionRequest(access));
     else
         access->setSysexEnabled(false);
 }
 
 void MIDIClientProxy::cancelSysexPermissionRequest(MIDIAccess* access)
 {
     if (m_client)

[haraken, 2014/04/02 01:08:54]

Ditto.

         m_client->cancelSysexPermissionRequest(WebMIDIPermissionRequest(access));
 }
 
 } // namespace blink