[Extensions] Remove RequestSender::ScopedTabID

[Extensions] Remove RequestSender::ScopedTabID

Remove RequestSender::ScopedTabID.  This was used to set the tab id, and
then passed to the browser in the request params for the API call, but
a) Was only set in a response to ExtensionMsg_DeliverMessage (?) and
b) The tab id can (and should) be obtained from the ExtensionFrameHelper

Ideally, we'll also remove the tab id from the request params entirely,
but this is a decent start.

It also helps remove a blocker for extension bindings.

BUG=667583
BUG=653596
Committed: https://crrev.com/c97b791906cf18bb1d5232ba881287db9898cd84
Cr-Commit-Position: refs/heads/master@{#433948}

[Devlin, 2016-11-22 01:47:17]

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

[commit-bot: I haz the power, 2016-11-22 01:47:45]

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

[Devlin, 2016-11-22 02:41:51]

Description was changed from

==========
Remove scoped tabid
BUG=
==========

to

==========
[Extensions] Remove RequestSender::ScopedTabID

Remove RequestSender::ScopedTabID.  This was used to set the tab id, and
then passed to the browser in the request params for the API call, but
a) Was only set in a response to ExtensionMsg_DeliverMessage (?) and
b) The tab id can (and should) be obtained from the ExtensionFrameHelper

Ideally, we'll also remove the tab id from the request params entirely,
but this is a decent start.

It also helps remove a blocker for extension bindings.

BUG=667583
==========

[commit-bot: I haz the power, 2016-11-22 02:47:26]

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

[commit-bot: I haz the power, 2016-11-22 02:47:27]

Dry run: This issue passed the CQ dry run.

[Devlin, 2016-11-22 15:44:40]

Description was changed from

==========
[Extensions] Remove RequestSender::ScopedTabID

Remove RequestSender::ScopedTabID.  This was used to set the tab id, and
then passed to the browser in the request params for the API call, but
a) Was only set in a response to ExtensionMsg_DeliverMessage (?) and
b) The tab id can (and should) be obtained from the ExtensionFrameHelper

Ideally, we'll also remove the tab id from the request params entirely,
but this is a decent start.

It also helps remove a blocker for extension bindings.

BUG=667583
==========

to

==========
[Extensions] Remove RequestSender::ScopedTabID

Remove RequestSender::ScopedTabID.  This was used to set the tab id, and
then passed to the browser in the request params for the API call, but
a) Was only set in a response to ExtensionMsg_DeliverMessage (?) and
b) The tab id can (and should) be obtained from the ExtensionFrameHelper

Ideally, we'll also remove the tab id from the request params entirely,
but this is a decent start.

It also helps remove a blocker for extension bindings.

BUG=667583
BUG=653596
==========

[Devlin, 2016-11-22 15:44:52]

rdevlin.cronin@chromium.org changed reviewers:
+ lazyboy@chromium.org

[lazyboy, 2016-11-22 19:19:33]

lgtm

[Devlin, 2016-11-22 19:41:52]

The CQ bit was checked by rdevlin.cronin@chromium.org

[commit-bot: I haz the power, 2016-11-22 19:42:13]

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-11-22 19:50:29]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1479843712968290, "parent_rev":
"f61e348a4ac03c7ec563229b043b9a16216514fc", "commit_rev":
"b5a8cdc3afda5c71cdd9cb475d801ad875983432"}

[commit-bot: I haz the power, 2016-11-22 19:51:00]

Description was changed from

==========
[Extensions] Remove RequestSender::ScopedTabID

Remove RequestSender::ScopedTabID.  This was used to set the tab id, and
then passed to the browser in the request params for the API call, but
a) Was only set in a response to ExtensionMsg_DeliverMessage (?) and
b) The tab id can (and should) be obtained from the ExtensionFrameHelper

Ideally, we'll also remove the tab id from the request params entirely,
but this is a decent start.

It also helps remove a blocker for extension bindings.

BUG=667583
BUG=653596
==========

to

==========
[Extensions] Remove RequestSender::ScopedTabID

Remove RequestSender::ScopedTabID.  This was used to set the tab id, and
then passed to the browser in the request params for the API call, but
a) Was only set in a response to ExtensionMsg_DeliverMessage (?) and
b) The tab id can (and should) be obtained from the ExtensionFrameHelper

Ideally, we'll also remove the tab id from the request params entirely,
but this is a decent start.

It also helps remove a blocker for extension bindings.

BUG=667583
BUG=653596
==========

[commit-bot: I haz the power, 2016-11-22 19:51:01]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-22 19:52:45]

Description was changed from

==========
[Extensions] Remove RequestSender::ScopedTabID

Remove RequestSender::ScopedTabID.  This was used to set the tab id, and
then passed to the browser in the request params for the API call, but
a) Was only set in a response to ExtensionMsg_DeliverMessage (?) and
b) The tab id can (and should) be obtained from the ExtensionFrameHelper

Ideally, we'll also remove the tab id from the request params entirely,
but this is a decent start.

It also helps remove a blocker for extension bindings.

BUG=667583
BUG=653596
==========

to

==========
[Extensions] Remove RequestSender::ScopedTabID

Remove RequestSender::ScopedTabID.  This was used to set the tab id, and
then passed to the browser in the request params for the API call, but
a) Was only set in a response to ExtensionMsg_DeliverMessage (?) and
b) The tab id can (and should) be obtained from the ExtensionFrameHelper

Ideally, we'll also remove the tab id from the request params entirely,
but this is a decent start.

It also helps remove a blocker for extension bindings.

BUG=667583
BUG=653596

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

[commit-bot: I haz the power, 2016-11-22 19:52:47]

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