Remove unused routing ID param from Did(Show|Destroy)FullscreenWidget.

Remove unused routing ID param from Did(Show|Destroy)FullscreenWidget.

Looks like this was introduced in
https://chromiumcodereview.appspot.com/12026029, where the routing IDs
were used in web_contents_video_capture_device.cc coupled with
the process ID from WebContents (i.e., main frame's process), but
this has since been removed, and the routing IDs aren't currently used
in any observers.  With out-of-process iframes, this routing ID should
really be coupled with process ID to be meaningful, but since nobody
is using it, just remove the parameter.

BUG=593522
Committed: https://crrev.com/899e06f316b1e9353b59a28e88f80796ad1b3d3b
Cr-Commit-Position: refs/heads/master@{#393677}

[alexmos, 2016-05-13 17:41:04]

Description was changed from

==========
Remove unused routing ID param from Did(Show|Destroy)FullscreenWidget.

BUG=593522
==========

to

==========
Remove unused routing ID param from Did(Show|Destroy)FullscreenWidget.

Looks like this was introduced in
https://chromiumcodereview.appspot.com/12026029, where the routing IDs
were used in web_contents_video_capture_device.cc coupled with
the process ID from WebContents (i.e., main frame's process), but
this has since been removed, and the routing IDs aren't currently used
in any observers.  With out-of-process iframes, this routing ID should
really be coupled with process ID to be meaningful, but since nobody
is using it, just remove the parameter.

BUG=593522
==========

[alexmos, 2016-05-13 18:01:33]

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

[alexmos, 2016-05-13 18:01:33]

Yuri, can you please take a look at whether this makes sense?

[miu, 2016-05-13 22:44:33]

lgtm

[alexmos, 2016-05-13 22:57:43]

alexmos@chromium.org changed reviewers:
+ nick@chromium.org

[alexmos, 2016-05-13 22:57:44]

Nick, can you please review content/ for OWNERS?

[ncarter (slow), 2016-05-13 23:00:32]

lgtm thanks for the cleanup!

[alexmos, 2016-05-13 23:03:01]

The CQ bit was checked by alexmos@chromium.org

[commit-bot: I haz the power, 2016-05-13 23:03:38]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1980633002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1980633002/1

[commit-bot: I haz the power, 2016-05-13 23:08:23]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-13 23:10:20]

Description was changed from

==========
Remove unused routing ID param from Did(Show|Destroy)FullscreenWidget.

Looks like this was introduced in
https://chromiumcodereview.appspot.com/12026029, where the routing IDs
were used in web_contents_video_capture_device.cc coupled with
the process ID from WebContents (i.e., main frame's process), but
this has since been removed, and the routing IDs aren't currently used
in any observers.  With out-of-process iframes, this routing ID should
really be coupled with process ID to be meaningful, but since nobody
is using it, just remove the parameter.

BUG=593522
==========

to

==========
Remove unused routing ID param from Did(Show|Destroy)FullscreenWidget.

Looks like this was introduced in
https://chromiumcodereview.appspot.com/12026029, where the routing IDs
were used in web_contents_video_capture_device.cc coupled with
the process ID from WebContents (i.e., main frame's process), but
this has since been removed, and the routing IDs aren't currently used
in any observers.  With out-of-process iframes, this routing ID should
really be coupled with process ID to be meaningful, but since nobody
is using it, just remove the parameter.

BUG=593522

Committed: https://crrev.com/899e06f316b1e9353b59a28e88f80796ad1b3d3b
Cr-Commit-Position: refs/heads/master@{#393677}
==========

[commit-bot: I haz the power, 2016-05-13 23:10:22]

Patchset 1 (id:??) landed as
https://crrev.com/899e06f316b1e9353b59a28e88f80796ad1b3d3b
Cr-Commit-Position: refs/heads/master@{#393677}

[Charlie Reis, 2016-05-13 23:31:48]

Description was changed from

==========
Remove unused routing ID param from Did(Show|Destroy)FullscreenWidget.

Looks like this was introduced in
https://chromiumcodereview.appspot.com/12026029, where the routing IDs
were used in web_contents_video_capture_device.cc coupled with
the process ID from WebContents (i.e., main frame's process), but
this has since been removed, and the routing IDs aren't currently used
in any observers.  With out-of-process iframes, this routing ID should
really be coupled with process ID to be meaningful, but since nobody
is using it, just remove the parameter.

BUG=593522

Committed: https://crrev.com/899e06f316b1e9353b59a28e88f80796ad1b3d3b
Cr-Commit-Position: refs/heads/master@{#393677}
==========

to

==========
Remove unused routing ID param from Did(Show|Destroy)FullscreenWidget.

Looks like this was introduced in
https://chromiumcodereview.appspot.com/12026029, where the routing IDs
were used in web_contents_video_capture_device.cc coupled with
the process ID from WebContents (i.e., main frame's process), but
this has since been removed, and the routing IDs aren't currently used
in any observers.  With out-of-process iframes, this routing ID should
really be coupled with process ID to be meaningful, but since nobody
is using it, just remove the parameter.

BUG=593522

Committed: https://crrev.com/899e06f316b1e9353b59a28e88f80796ad1b3d3b
Cr-Commit-Position: refs/heads/master@{#393677}
==========