Expose the MediaStream interface

Expose the MediaStream interface

Intent to Ship:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/9WIVyDz1T34/5Nmj46UXAwAJ

BUG=649331
Committed: https://crrev.com/febb0c97b0e7a59fbd1e3ca9bbf00759e684d4b0
Cr-Commit-Position: refs/heads/master@{#421502}

[foolip, 2016-09-23 13:02:46]

The CQ bit was checked by foolip@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-23 13:02:52]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-23 13:13:54]

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

[commit-bot: I haz the power, 2016-09-23 13:13:55]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[foolip, 2016-09-23 13:39:15]

Description was changed from

==========
Expose the MediaStream interface

BUG=649331
==========

to

==========
Expose the MediaStream interface

Intent to Ship:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/9WIVyDz1T34/5Nmj46UX...

BUG=649331
==========

[foolip, 2016-09-23 15:08:58]

The CQ bit was checked by foolip@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-23 15:09:15]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-23 15:29:26]

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

[commit-bot: I haz the power, 2016-09-23 15:29:27]

Dry run: Try jobs failed on following builders:
  win_clang on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_clang/builds/...)

[foolip, 2016-09-23 21:28:17]

The CQ bit was checked by foolip@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-23 21:28:56]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-23 22:30:08]

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

[commit-bot: I haz the power, 2016-09-23 22:30:09]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[foolip, 2016-09-27 08:51:04]

foolip@chromium.org changed reviewers:
+ bbudge@chromium.org, guidou@chromium.org, raymes@chromium.org,
tommi@chromium.org

[foolip, 2016-09-27 08:51:04]

PTAL

bbudge@/raymes@: ppapi/

guidou@/tommi@: everything else

[Guido Urdaneta, 2016-09-27 08:54:26]

lgtm

[tommi (sloooow) - chröme, 2016-09-27 08:58:41]

lgtm

[Srirama, 2016-09-27 09:32:17]

srirama.m@samsung.com changed reviewers:
+ srirama.m@samsung.com

[Srirama, 2016-09-27 09:32:18]

Do we need to update here also?
https://cs.chromium.org/chromium/src/android_webview/tools/system_webview_she...

[foolip, 2016-09-27 09:46:40]

On 2016/09/27 09:32:18, Srirama_OnBizTrip_till_26OCT wrote:
> Do we need to update here also?
>
https://cs.chromium.org/chromium/src/android_webview/tools/system_webview_she...

From `git log --stat --
third_party/WebKit/LayoutTests/webexposed/global-interface-listing-expected.txt
--
android_webview/tools/system_webview_shell/test/data/webexposed/global-interface-listing-expected.txt`
it looks like most CLs update only the one in third_party/WebKit. Any time I've
updated the one in android_webview I've done it manually, or is there a bot one
can run?

[Srirama, 2016-09-27 10:03:15]

On 2016/09/27 09:46:40, foolip wrote:
> On 2016/09/27 09:32:18, Srirama_OnBizTrip_till_26OCT wrote:
> > Do we need to update here also?
> >
>
https://cs.chromium.org/chromium/src/android_webview/tools/system_webview_she...
> 
> From `git log --stat --
>
third_party/WebKit/LayoutTests/webexposed/global-interface-listing-expected.txt
> --
>
android_webview/tools/system_webview_shell/test/data/webexposed/global-interface-listing-expected.txt`
> it looks like most CLs update only the one in third_party/WebKit. Any time
I've
> updated the one in android_webview I've done it manually, or is there a bot
one
> can run?

I think it is fine, if you are updating it later manually. I am not sure if
there is a bot for this.
Looking at the git log of
https://chromium.googlesource.com/chromium/src/+log/master/android_webview/to...
some of them are updated along with third_party/WebKit, though most of them are
only removals.

[foolip, 2016-09-27 10:29:48]

On 2016/09/27 10:03:15, Srirama_OnBizTrip_till_26OCT wrote:
> On 2016/09/27 09:46:40, foolip wrote:
> > On 2016/09/27 09:32:18, Srirama_OnBizTrip_till_26OCT wrote:
> > > Do we need to update here also?
> > >
> >
>
https://cs.chromium.org/chromium/src/android_webview/tools/system_webview_she...
> > 
> > From `git log --stat --
> >
>
third_party/WebKit/LayoutTests/webexposed/global-interface-listing-expected.txt
> > --
> >
>
android_webview/tools/system_webview_shell/test/data/webexposed/global-interface-listing-expected.txt`
> > it looks like most CLs update only the one in third_party/WebKit. Any time
> I've
> > updated the one in android_webview I've done it manually, or is there a bot
> one
> > can run?
> 
> I think it is fine, if you are updating it later manually. I am not sure if
> there is a bot for this.
> Looking at the git log of
>
https://chromium.googlesource.com/chromium/src/+log/master/android_webview/to...
> some of them are updated along with third_party/WebKit, though most of them
are
> only removals.

I plan to just leave it alone, since apparently it's not actually in use and
updates to it are ad-hoc.

[bbudge, 2016-09-27 17:37:14]

ppapi lgtm

[foolip, 2016-09-28 08:44:38]

The CQ bit was checked by foolip@chromium.org

[commit-bot: I haz the power, 2016-09-28 08:44:56]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-28 12:31:36]

Description was changed from

==========
Expose the MediaStream interface

Intent to Ship:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/9WIVyDz1T34/5Nmj46UX...

BUG=649331
==========

to

==========
Expose the MediaStream interface

Intent to Ship:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/9WIVyDz1T34/5Nmj46UX...

BUG=649331
==========

[commit-bot: I haz the power, 2016-09-28 12:31:37]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-09-28 12:33:58]

Description was changed from

==========
Expose the MediaStream interface

Intent to Ship:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/9WIVyDz1T34/5Nmj46UX...

BUG=649331
==========

to

==========
Expose the MediaStream interface

Intent to Ship:
https://groups.google.com/a/chromium.org/d/msg/blink-dev/9WIVyDz1T34/5Nmj46UX...

BUG=649331

Committed: https://crrev.com/febb0c97b0e7a59fbd1e3ca9bbf00759e684d4b0
Cr-Commit-Position: refs/heads/master@{#421502}
==========

[commit-bot: I haz the power, 2016-09-28 12:33:59]

Patchset 4 (id:??) landed as
https://crrev.com/febb0c97b0e7a59fbd1e3ca9bbf00759e684d4b0
Cr-Commit-Position: refs/heads/master@{#421502}