Add PLATFORM_EXPORT to MediaStreamComponent::AudioSourceProviderImpl

Add PLATFORM_EXPORT to MediaStreamComponent::AudioSourceProviderImpl

On non-Windows, PLATFORM_EXPORT expands to a visibility attribute,
which is inherited by nested classes. On Windows, it's expanded to
a dllexport/dllimport attribute, which is not inherited.

When using Clang on Windows, MediaStreamComponent's dtor got
inlined, which destroys m_sourceProvider, whose destructor got inlined,
and tried to reference provideInput for the vtable, but failed to link
since it wasn't in the right dll. Exporting the inner class fixes this.

BUG=82385
TEST=build blink_web.dll with Clang on Windows in Release mode

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=176468

[hans, 2014-06-18 20:31:26]

Please take a look.

[Nico, 2014-06-18 20:43:24]

lgtm

[Nico, 2014-06-18 20:43:57]

The CQ bit was checked by thakis@chromium.org

[commit-bot: I haz the power, 2014-06-18 20:45:05]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/hans@chromium.org/343763002/1

[Reid Kleckner, 2014-06-18 20:45:30]

There is no inline ctor, so I think it must have been the dtor and not the ctor.

lgtm if I'm right and you update the commit message to reflect that.

[hans, 2014-06-18 20:49:44]

The CQ bit was unchecked by hans@chromium.org

[Nico, 2014-06-18 20:49:53]

https://codereview.chromium.org/343763002/diff/1/Source/platform/mediastream/...
File Source/platform/mediastream/MediaStreamComponent.h (right):

https://codereview.chromium.org/343763002/diff/1/Source/platform/mediastream/...
Source/platform/mediastream/MediaStreamComponent.h:84: AudioSourceProviderImpl()
Isn't this a ctor?

[hans, 2014-06-18 20:54:06]

On 2014/06/18 20:49:53, Nico (away) wrote:
>
https://codereview.chromium.org/343763002/diff/1/Source/platform/mediastream/...
> File Source/platform/mediastream/MediaStreamComponent.h (right):
> 
>
https://codereview.chromium.org/343763002/diff/1/Source/platform/mediastream/...
> Source/platform/mediastream/MediaStreamComponent.h:84:
AudioSourceProviderImpl()
> Isn't this a ctor?

I think Reid is saying there is no inline ctor for MediaStreamComponent.. I
missed that. I guess it's the dtor then. I'll update the description.

[hans, 2014-06-18 20:59:07]

The CQ bit was checked by hans@chromium.org

[commit-bot: I haz the power, 2014-06-18 21:00:23]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/hans@chromium.org/343763002/1

[commit-bot: I haz the power, 2014-06-18 23:28:17]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-06-18 23:28:17]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/12592)

[hans, 2014-06-18 23:29:31]

The CQ bit was checked by hans@chromium.org

[commit-bot: I haz the power, 2014-06-18 23:29:41]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/hans@chromium.org/343763002/1

[commit-bot: I haz the power, 2014-06-19 00:45:37]

Change committed as 176468