Remove |remote| and |readonly| members of MediaStreamTrack

Remove |remote| and |readonly| members of MediaStreamTrack.

Spec reference:
http://w3c.github.io/mediacapture-main/getusermedia.html#attributes-1

removal was in the February 22, 2016'.
[#321] Remove track attributes "remote" and "readonly" :
https://github.com/w3c/mediacapture-main/pull/321

Intent to Deprecate and Remove :
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/d20ECb2sWzI

BUG=598704
Review-Url: https://codereview.chromium.org/2425703002
Cr-Commit-Position: refs/heads/master@{#452851}
Committed: https://chromium.googlesource.com/chromium/src/+/a72385a6addeb6194d3568beda3f352e07c97260

[Yeol Park, 2016-10-17 13:51:15]

Description was changed from

==========
Remove |remote| and |readonly| members of MediaStreamTrack

Spec reference:
http://w3c.github.io/mediacapture-main/getusermedia.html#attributes-1

- removal was in the February 22, 2016'.
[#321] Remove track attributes "remote" and "readonly"

BUG=598704
==========

to

==========
Remove |remote| and |readonly| members of MediaStreamTrack

Spec reference:
http://w3c.github.io/mediacapture-main/getusermedia.html#attributes-1

- removal was in the February 22, 2016'.
[#321] Remove track attributes "remote" and "readonly"

BUG=598704
==========

[Yeol Park, 2016-10-17 13:51:15]

peary2@gmail.com changed reviewers:
+ aboxhall@chromium.org, bbudge@chromium.org, drott@chromium.org,
esprehn@chromium.org, hongchan@chromium.org, hta@chromium.org,
mcasas@chromium.org, peter@chromium.org, scottmg@chromium.org

[Yeol Park, 2016-10-17 13:52:01]

PTAL :)

aboxhall@ for chrome/renderer/extensions/cast_streaming_native_handler.cc
		scottmg@ for content/public/renderer/*
		mcasas@ for content/renderer/media/*
		bbudge@ for content/renderer/pepper/pepper_media_stream_video_track_host.cc
		esprehn@ for content/renderer/renderer_blink_platform_impl.cc
		peter@ for  third_party/WebKit/Source/modules/mediastream/*
		            content/shell/renderer/layout_test/blink_test_runner.cc
		hongchan@ for
third_party/WebKit/Source/modules/webaudio/MediaStreamAudioDestinationNode.cpp
		drott@ for third_party/WebKit/Source/platform/*

[Yeol Park, 2016-10-17 13:53:09]

PTAL :)

aboxhall@ for chrome/renderer/extensions/cast_streaming_native_handler.cc
scottmg@ for content/public/renderer/*
mcasas@ for content/renderer/media/*
bbudge@ for content/renderer/pepper/pepper_media_stream_video_track_host.cc
esprehn@ for content/renderer/renderer_blink_platform_impl.cc
peter@ for  third_party/WebKit/Source/modules/mediastream/*
            content/shell/renderer/layout_test/blink_test_runner.cc
hongchan@ for
third_party/WebKit/Source/modules/webaudio/MediaStreamAudioDestinationNode.cpp
drott@ for third_party/WebKit/Source/platform/*

[drott, 2016-10-17 14:22:54]

> drott@ for third_party/WebKit/Source/platform/*

third_party/WebKit/Source/platform/* LGTM

[jochen (gone - plz use gerrit), 2016-10-17 14:23:43]

jochen@chromium.org changed reviewers:
+ jochen@chromium.org

[jochen (gone - plz use gerrit), 2016-10-17 14:23:43]

can you please link to the intent to remove thread in the CL description

[bbudge, 2016-10-17 17:38:33]

content/renderer/pepper lgtm

[mcasas, 2016-10-17 18:08:25]

lgtm content/renderer/media with a few comments
and assuming bots are happy.

https://codereview.chromium.org/2425703002/diff/1/content/renderer/media/exte...
File content/renderer/media/external_media_stream_audio_source.cc (right):

https://codereview.chromium.org/2425703002/diff/1/content/renderer/media/exte...
content/renderer/media/external_media_stream_audio_source.cc:14: :
MediaStreamAudioSource(true),
I would say that this parameter ripples through to
MediaStreamAudioSource::is_local_source_ [1], which
is probably made superfluous by this CL, and should
be removed. If that's not done in this CL, please add
a TODO with indication of the bug in that class.

(Note that MediaStreamAudioSource::is_local_source_ 
is  only used for a DVLOG, so probably is safe to 
remove right away).

https://cs.chromium.org/chromium/src/content/renderer/media/media_stream_audi...

https://codereview.chromium.org/2425703002/diff/1/content/renderer/media/webm...
File content/renderer/media/webmediaplayer_ms_compositor.cc (right):

https://codereview.chromium.org/2425703002/diff/1/content/renderer/media/webm...
content/renderer/media/webmediaplayer_ms_compositor.cc:147: if (remote_video &&
l.145-146 in the LHS tested video_tracks.size() as a 
way to avoid crashing when accessing video_tracks[0];
with that and since now we're assuming |remote| is 
always false, I think we can just remove |remote_video| 
and the whole pack of lines l.147-155 in the RHS, and 
simplify l. 160, wdyt?

https://codereview.chromium.org/2425703002/diff/1/third_party/WebKit/Source/p...
File third_party/WebKit/Source/platform/mediastream/MediaStreamSource.cpp
(right):

https://codereview.chromium.org/2425703002/diff/1/third_party/WebKit/Source/p...
third_party/WebKit/Source/platform/mediastream/MediaStreamSource.cpp:34:
MediaStreamSource* MediaStreamSource::create(const String& id,
Bring back l.34 from the LHS.

https://codereview.chromium.org/2425703002/diff/1/third_party/WebKit/public/p...
File third_party/WebKit/public/platform/WebMediaStreamSource.h (right):

https://codereview.chromium.org/2425703002/diff/1/third_party/WebKit/public/p...
third_party/WebKit/public/platform/WebMediaStreamSource.h:90: const WebString&
name);  // DEPRECATED
I guess this initialize() is not DEPRECATED anymore ?

[hta - Chromium, 2016-10-17 18:28:37]

The usage isn't huge (low enough to just bounce up and down on
https://www.chromestatus.com/metrics/feature/timeline/popularity/1306) - but I
think we should follow the process:

- Intent to deprecate and remove
- Change the "MeasureAs" in the idl file to "DeprecateAs"
- Wait a byte
- Delete

[hta - Chromium, 2016-10-17 18:34:19]

Note: The |readonly| member was removed some time ago. Please update the title
and description accordingly.

(And thanks for pushing on this!)

[Peter Beverloo, 2016-10-18 15:50:43]

//content/shell and //third_party/WebKit/Source/modules/mediastream/ lgtm

This property saw such incredible minor usage over the past seven days
that I'd almost argue it's just a cleanup. Even so, I agree that it'd be
nice to send an e-mail to blink-dev@, but would err towards a PSA.

Different from method removals, using the property won't throw.
`track.remote` will now evaluate to undefined, which is falsy. AFAICT
that's the common case already (Please confirm!).

[hongchan, 2016-10-18 15:55:59]

WebAudio lgtm

(Source/modules/webaudio/MediaStreamAudioDestinationNode.cpp)

[Yeol Park, 2016-10-19 07:28:29]

On 2016/10/17 18:34:19, hta - Chromium wrote:
> Note: The |readonly| member was removed some time ago. Please update the title
> and description accordingly.
> 
> (And thanks for pushing on this!)

yes, |readonly| member was removed in 1915793002.
but This patch removed remnants of the |readonly|.
e.g) cast_streaming_native_handler.cc, media_stream_utils.cc,
media_stream_utils.h, ...)

[Yeol Park, 2016-10-19 08:07:27]

Patchset #2 (id:20001) has been deleted

[Yeol Park, 2016-10-19 08:41:58]

Patchset #2 (id:40001) has been deleted

[Yeol Park, 2016-10-19 08:49:49]

Description was changed from

==========
Remove |remote| and |readonly| members of MediaStreamTrack

Spec reference:
http://w3c.github.io/mediacapture-main/getusermedia.html#attributes-1

- removal was in the February 22, 2016'.
[#321] Remove track attributes "remote" and "readonly"

BUG=598704
==========

to

==========
Remove |remote| and |readonly| members of MediaStreamTrack

Spec reference:
http://w3c.github.io/mediacapture-main/getusermedia.html#attributes-1

- removal was in the February 22, 2016'.
[#321] Remove track attributes "remote" and "readonly"
https://github.com/w3c/mediacapture-main/pull/321

BUG=598704
==========

[Yeol Park, 2016-10-19 08:51:03]

On 2016/10/17 14:23:43, jochen wrote:
> can you please link to the intent to remove thread in the CL description

I have added in the description :)
https://github.com/w3c/mediacapture-main/pull/321

[jochen (gone - plz use gerrit), 2016-10-19 09:14:00]

On 2016/10/19 at 08:51:03, peary2 wrote:
> On 2016/10/17 14:23:43, jochen wrote:
> > can you please link to the intent to remove thread in the CL description
> 
> I have added in the description :)
> https://github.com/w3c/mediacapture-main/pull/321

Sorry for being unclear. You'll need to follow this process:
https://www.chromium.org/blink#TOC-Launch-Process:-Deprecation

[aboxhall, 2016-10-19 21:33:51]

On 2016/10/17 13:52:01, Park Yeol wrote:
> PTAL :)
> 
> aboxhall@ for chrome/renderer/extensions/cast_streaming_native_handler.cc
> 		scottmg@ for content/public/renderer/*
> 		mcasas@ for content/renderer/media/*
> 		bbudge@ for content/renderer/pepper/pepper_media_stream_video_track_host.cc
> 		esprehn@ for content/renderer/renderer_blink_platform_impl.cc
> 		peter@ for  third_party/WebKit/Source/modules/mediastream/*
> 		            content/shell/renderer/layout_test/blink_test_runner.cc
> 		hongchan@ for
> third_party/WebKit/Source/modules/webaudio/MediaStreamAudioDestinationNode.cpp
> 		drott@ for third_party/WebKit/Source/platform/*

Sorry, I'm only a per-file reviewer for chrome/renderer/extensions! Try one of
these fine folks: https://cs.chromium.org/chromium/src/extensions/OWNERS

[miu, 2016-10-23 00:51:29]

miu@chromium.org changed reviewers:
+ miu@chromium.org

[miu, 2016-10-23 00:51:30]

lgtm % a couple concerns.

BTW--When you go to remove the is_local_source_ from
content/renderer/media/media_stream_audio_source.h, which will unravel a number
of files in content/renderer/media, feel free to use me as a code reviewer since
I'm very familiar with that code.

https://codereview.chromium.org/2425703002/diff/60001/content/renderer/media/...
File content/renderer/media/external_media_stream_audio_source.cc (right):

https://codereview.chromium.org/2425703002/diff/60001/content/renderer/media/...
content/renderer/media/external_media_stream_audio_source.cc:14: :
MediaStreamAudioSource(true),
The value for |is_remote| was true here (from the
cast_streaming_native_handler.cc), which would call the base class ctor with
false. This will mess-up the UMA histograms in content::TrackAudioRenderer
(e.g.,
https://cs.chromium.org/chromium/src/content/renderer/media/track_audio_rende...).

Is that acceptable? If you haven't already, please contact the owner of those
UMAs.

https://codereview.chromium.org/2425703002/diff/60001/content/renderer/media/...
File content/renderer/media/webmediaplayer_ms_compositor.cc (right):

https://codereview.chromium.org/2425703002/diff/60001/content/renderer/media/...
content/renderer/media/webmediaplayer_ms_compositor.cc:158: serial_ =
(hash_value << 1) | ((video_tracks.size() > 0) ? 1 : 0);
Looks like this should now be:

  serial_ = base::Hash(stream_id);

...since there is no longer a way to differentiate local vs. remote.

[esprehn, 2016-10-25 00:05:37]

third_party/Webkit lgtm

[hta - Chromium, 2016-12-14 09:51:12]

lgtm

Note that my previous comments on the subject line and CL description have not
been addressed, and I can't imagine that it doesn't need a rebase, but it looks
OK.

[aboxhall, 2017-01-11 00:25:06]

aboxhall@chromium.org changed reviewers:
- aboxhall@chromium.org

[Yeol Park, 2017-01-20 02:55:31]

Description was changed from

==========
Remove |remote| and |readonly| members of MediaStreamTrack

Spec reference:
http://w3c.github.io/mediacapture-main/getusermedia.html#attributes-1

- removal was in the February 22, 2016'.
[#321] Remove track attributes "remote" and "readonly"
https://github.com/w3c/mediacapture-main/pull/321

BUG=598704
==========

to

==========
Remove |remote| and |readonly| members of MediaStreamTrack.

Spec reference:
http://w3c.github.io/mediacapture-main/getusermedia.html#attributes-1

- removal was in the February 22, 2016'.
[#321] Remove track attributes "remote" and "readonly"
https://github.com/w3c/mediacapture-main/pull/321

Intent to Deprecate and Remove : 
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/d20ECb2sWzI

BUG=598704
==========

[Yeol Park, 2017-01-20 02:56:11]

Description was changed from

==========
Remove |remote| and |readonly| members of MediaStreamTrack.

Spec reference:
http://w3c.github.io/mediacapture-main/getusermedia.html#attributes-1

- removal was in the February 22, 2016'.
[#321] Remove track attributes "remote" and "readonly"
https://github.com/w3c/mediacapture-main/pull/321

Intent to Deprecate and Remove : 
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/d20ECb2sWzI

BUG=598704
==========

to

==========
Remove |remote| and |readonly| members of MediaStreamTrack.

Spec reference:
http://w3c.github.io/mediacapture-main/getusermedia.html#attributes-1

removal was in the February 22, 2016'.
[#321] Remove track attributes "remote" and "readonly" :
https://github.com/w3c/mediacapture-main/pull/321

Intent to Deprecate and Remove :
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/d20ECb2sWzI

BUG=598704
==========

[Yeol Park, 2017-01-20 04:04:25]

On 2016/12/14 09:51:12, hta - Chromium wrote:
> lgtm
> 
> Note that my previous comments on the subject line and CL description have not
> been addressed, and I can't imagine that it doesn't need a rebase, but it
looks
> OK.

I was sent email to blink-dev, and add 'Intent to Deprecate' URL in Description.
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/d20ECb2sWzI

If there is no problem, we will proceed to rebase :)

[jochen (gone - plz use gerrit), 2017-01-20 07:31:21]

thank you. I'll approve this CL once the intent has its required three
approvals!

[Yeol Park, 2017-01-25 09:10:48]

I was rebase in Patch set 4 for latest source.
Could you run CQ bot? :)

On 2017/01/20 07:31:21, jochen wrote:
> thank you. I'll approve this CL once the intent has its required three
> approvals!

[jochen (gone - plz use gerrit), 2017-01-25 12:24:16]

lgtm

[jochen (gone - plz use gerrit), 2017-01-25 12:24:24]

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

[commit-bot: I haz the power, 2017-01-25 12:24:35]

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, 2017-01-25 12:39:34]

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

[commit-bot: I haz the power, 2017-01-25 12:39:36]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Yeol Park, 2017-01-26 06:04:29]

I was rebase in Patch set 4 for unittests
Could you run CQ bot? :)

[hta - Chromium, 2017-01-26 08:03:35]

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

[commit-bot: I haz the power, 2017-01-26 08:03:49]

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, 2017-01-26 09:29:51]

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

[commit-bot: I haz the power, 2017-01-26 09:29:53]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[hta - Chromium, 2017-01-26 09:45:24]

On 2017/01/26 06:04:29, Park Yeol wrote:
> I was rebase in Patch set 4 for unittests
> Could you run CQ bot? :)

The CQ run indicates global-interface-listings need updating + 1 test.

My script (shell function) for getting all the global-interface-listing tests:

# Test the "global interface listing" error
interfacetest() {
  CMDSTEM="third_party/WebKit/Tools/Scripts/run-webkit-tests -f
--no-retry-failures $TEST_DEBUG_FLAG"
  TESTS=$(cd $DIR/src/third_party/WebKit/LayoutTests && echo
webexposed/global-interface-listing*.html)
  CMD="$CMDSTEM $TESTS"
  echo $CMD
  $CMD
  for TEST in $TESTS; do
    echo $CMDSTEM virtual/stable/$TEST
    $CMDSTEM virtual/stable/$TEST
  done
}

[miu, 2017-01-30 21:54:35]

Park: Friendly reminder, please check on my comments from #22 before committing.

[Yeol Park, 2017-02-07 11:47:10]

The CQ bit was checked by peary2@gmail.com

[Yeol Park, 2017-02-07 11:47:11]

The patchset sent to the CQ was uploaded after l-g-t-m from drott@chromium.org,
bbudge@chromium.org, peter@chromium.org, hongchan@chromium.org,
mcasas@chromium.org, esprehn@chromium.org, miu@chromium.org, hta@chromium.org,
jochen@chromium.org
Link to the patchset: https://codereview.chromium.org/2425703002/#ps120001
(title: "fix build error")

[Yeol Park, 2017-02-07 11:47:21]

The CQ bit was unchecked by peary2@gmail.com

[commit-bot: I haz the power, 2017-02-07 11:47:25]

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

[Yeol Park, 2017-02-07 11:47:47]

The CQ bit was checked by peary2@gmail.com to run a CQ dry run

[commit-bot: I haz the power, 2017-02-07 11:47:56]

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

[Yeol Park, 2017-02-07 11:49:09]

The CQ bit was unchecked by peary2@gmail.com

[Yeol Park, 2017-02-07 11:56:01]

PTAL :)
miu@ for your comment #22
../content/renderer/media/external_media_stream_audio_source.cc
../content/renderer/media/webmediaplayer_ms_compositor.cc

Thank you for your information.

On 2017/01/30 21:54:35, miu wrote:
> Park: Friendly reminder, please check on my comments from #22 before
committing.

[Yeol Park, 2017-02-22 08:57:57]

The CQ bit was checked by peary2@gmail.com

[Yeol Park, 2017-02-22 08:57:58]

The patchset sent to the CQ was uploaded after l-g-t-m from drott@chromium.org,
bbudge@chromium.org, jochen@chromium.org, peter@chromium.org,
esprehn@chromium.org, hongchan@chromium.org, mcasas@chromium.org,
miu@chromium.org, hta@chromium.org
Link to the patchset: https://codereview.chromium.org/2425703002/#ps140001
(title: "rebase")

[Yeol Park, 2017-02-22 08:58:16]

The CQ bit was unchecked by peary2@gmail.com

[Yeol Park, 2017-02-22 08:59:00]

The CQ bit was checked by peary2@gmail.com to run a CQ dry run

[commit-bot: I haz the power, 2017-02-22 08:59:42]

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, 2017-02-22 09:14:29]

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

[commit-bot: I haz the power, 2017-02-22 09:14:31]

Dry run: Try jobs failed on following builders:
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)

[Yeol Park, 2017-02-22 09:24:53]

The CQ bit was checked by peary2@gmail.com to run a CQ dry run

[commit-bot: I haz the power, 2017-02-22 09:26:09]

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, 2017-02-22 10:53:33]

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

[commit-bot: I haz the power, 2017-02-22 10:53:34]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Yeol Park, 2017-02-23 13:17:26]

The CQ bit was checked by peary2@gmail.com to run a CQ dry run

[commit-bot: I haz the power, 2017-02-23 13:17:47]

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, 2017-02-23 13:20:22]

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

[commit-bot: I haz the power, 2017-02-23 13:20:24]

Dry run: Try jobs failed on following builders:
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[Yeol Park, 2017-02-23 18:05:07]

The CQ bit was checked by peary2@gmail.com to run a CQ dry run

[commit-bot: I haz the power, 2017-02-23 18:05:27]

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, 2017-02-23 18:08:25]

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

[commit-bot: I haz the power, 2017-02-23 18:08:26]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[Yeol Park, 2017-02-24 04:12:32]

The CQ bit was checked by peary2@gmail.com to run a CQ dry run

[commit-bot: I haz the power, 2017-02-24 04:12:53]

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, 2017-02-24 06:07:21]

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

[commit-bot: I haz the power, 2017-02-24 06:07:22]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Yeol Park, 2017-02-24 07:49:51]

The CQ bit was checked by peary2@gmail.com to run a CQ dry run

[commit-bot: I haz the power, 2017-02-24 07:50:04]

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, 2017-02-24 08:16:38]

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

[commit-bot: I haz the power, 2017-02-24 08:16:40]

Dry run: Try jobs failed on following builders:
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)

[Yeol Park, 2017-02-24 10:15:52]

The CQ bit was checked by peary2@gmail.com to run a CQ dry run

[commit-bot: I haz the power, 2017-02-24 10:16:02]

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, 2017-02-24 13:29:55]

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

[commit-bot: I haz the power, 2017-02-24 13:29:56]

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

[Yeol Park, 2017-02-24 14:37:05]

The CQ bit was checked by peary2@gmail.com to run a CQ dry run

[commit-bot: I haz the power, 2017-02-24 14:37:16]

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, 2017-02-24 16:24:42]

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

[commit-bot: I haz the power, 2017-02-24 16:24:45]

Dry run: This issue passed the CQ dry run.

[Yeol Park, 2017-02-24 16:47:40]

The CQ bit was checked by peary2@gmail.com

[Yeol Park, 2017-02-24 16:47:41]

The patchset sent to the CQ was uploaded after l-g-t-m from drott@chromium.org,
bbudge@chromium.org, jochen@chromium.org, peter@chromium.org,
esprehn@chromium.org, hongchan@chromium.org, mcasas@chromium.org,
miu@chromium.org, hta@chromium.org
Link to the patchset: https://codereview.chromium.org/2425703002/#ps200001
(title: "Remove |remote| and |readonly| members of MediaStreamTrack.")

[commit-bot: I haz the power, 2017-02-24 16:48:07]

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, 2017-02-24 16:53:05]

CQ is committing da patch.

Bot data: {"patchset_id": 200001, "attempt_start_ts": 1487954860759220,
"parent_rev": "7ad84acdb570f8879cab458545b75f603fa1e884", "commit_rev":
"a72385a6addeb6194d3568beda3f352e07c97260"}

[commit-bot: I haz the power, 2017-02-24 16:53:45]

Description was changed from

==========
Remove |remote| and |readonly| members of MediaStreamTrack.

Spec reference:
http://w3c.github.io/mediacapture-main/getusermedia.html#attributes-1

removal was in the February 22, 2016'.
[#321] Remove track attributes "remote" and "readonly" :
https://github.com/w3c/mediacapture-main/pull/321

Intent to Deprecate and Remove :
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/d20ECb2sWzI

BUG=598704
==========

to

==========
Remove |remote| and |readonly| members of MediaStreamTrack.

Spec reference:
http://w3c.github.io/mediacapture-main/getusermedia.html#attributes-1

removal was in the February 22, 2016'.
[#321] Remove track attributes "remote" and "readonly" :
https://github.com/w3c/mediacapture-main/pull/321

Intent to Deprecate and Remove :
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/d20ECb2sWzI

BUG=598704

Review-Url: https://codereview.chromium.org/2425703002
Cr-Commit-Position: refs/heads/master@{#452851}
Committed:
https://chromium.googlesource.com/chromium/src/+/a72385a6addeb6194d3568beda3f...
==========

[commit-bot: I haz the power, 2017-02-24 16:53:49]

Committed patchset #10 (id:200001) as
https://chromium.googlesource.com/chromium/src/+/a72385a6addeb6194d3568beda3f...

[Mathieu, 2017-02-24 17:44:06]

A revert of this CL (patchset #10 id:200001) has been created in
https://codereview.chromium.org/2710213005/ by mathp@chromium.org.

The reason for reverting is: Failure on webkit_tests:

https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty...

external/wpt/mediacapture-streams/MediaStreamTrack-init.https.html.