[Extensions] Remove the source_tab_id property on API request messages

[Extensions] Remove the source_tab_id property on API request messages

ExtensionHostMsg_Request_Params includes a "source_tab_id", but this
shouldn't be necessary. The browser can determine the tab id from which
RFH sent the request. Remove the property.

BUG=667584
Committed: https://crrev.com/5eb66dfa480bed01684c4ae1e44599e613e9d5a6
Cr-Commit-Position: refs/heads/master@{#434688}

[Devlin, 2016-11-23 18:28:28]

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

[commit-bot: I haz the power, 2016-11-23 18:28: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-11-23 18:44:05]

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

[commit-bot: I haz the power, 2016-11-23 18:44:05]

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

[Devlin, 2016-11-23 20:05:23]

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

[commit-bot: I haz the power, 2016-11-23 20:05:46]

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-11-23 21:58:16]

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

[commit-bot: I haz the power, 2016-11-23 21:58:17]

Dry run: This issue passed the CQ dry run.

[Devlin, 2016-11-23 22:07:53]

Description was changed from

==========
kill source tab id
BUG=
==========

to

==========
[Extensions] Remove the source_tab_id property on API request messages

ExtensionHostMsg_Request_Params includes a "source_tab_id", but this
shouldn't be necessary. The browser can determine the tab id from which
RFH sent the request. Remove the property.

BUG=667584
==========

[Devlin, 2016-11-23 22:08:37]

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

[Devlin, 2016-11-23 22:08:37]

lazyboy@ (extensions) and dcheng@ (extension_messages.h), mind taking a look?

[Devlin, 2016-11-23 22:18:42]

lazyboy@ (extensions) and dcheng@ (extension_messages.h), mind taking a look?

[lazyboy, 2016-11-24 00:02:14]

extensions lgtm

[dcheng, 2016-11-24 00:04:22]

lgtm

[Devlin, 2016-11-24 00:08:37]

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

[commit-bot: I haz the power, 2016-11-24 00:09:20]

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-24 02:10:44]

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

[commit-bot: I haz the power, 2016-11-24 02:10:45]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)

[Devlin, 2016-11-24 06:26:20]

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

[commit-bot: I haz the power, 2016-11-24 06:26:49]

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-24 08:27:56]

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

[commit-bot: I haz the power, 2016-11-24 08:27:57]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_TIMED_OUT, no build
URL)

[Devlin, 2016-11-28 17:06:58]

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

[commit-bot: I haz the power, 2016-11-28 17:07:24]

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-28 18:04:59]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1480352818891310,
"parent_rev": "d77139c5a878f7b9ff62ba95530f25340ad32a28", "commit_rev":
"7b2e6494fcf0d6a7a12ee75967626b3915effa0c"}

[commit-bot: I haz the power, 2016-11-28 18:05:24]

Description was changed from

==========
[Extensions] Remove the source_tab_id property on API request messages

ExtensionHostMsg_Request_Params includes a "source_tab_id", but this
shouldn't be necessary. The browser can determine the tab id from which
RFH sent the request. Remove the property.

BUG=667584
==========

to

==========
[Extensions] Remove the source_tab_id property on API request messages

ExtensionHostMsg_Request_Params includes a "source_tab_id", but this
shouldn't be necessary. The browser can determine the tab id from which
RFH sent the request. Remove the property.

BUG=667584
==========

[commit-bot: I haz the power, 2016-11-28 18:05:26]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-28 18:08:37]

Description was changed from

==========
[Extensions] Remove the source_tab_id property on API request messages

ExtensionHostMsg_Request_Params includes a "source_tab_id", but this
shouldn't be necessary. The browser can determine the tab id from which
RFH sent the request. Remove the property.

BUG=667584
==========

to

==========
[Extensions] Remove the source_tab_id property on API request messages

ExtensionHostMsg_Request_Params includes a "source_tab_id", but this
shouldn't be necessary. The browser can determine the tab id from which
RFH sent the request. Remove the property.

BUG=667584

Committed: https://crrev.com/5eb66dfa480bed01684c4ae1e44599e613e9d5a6
Cr-Commit-Position: refs/heads/master@{#434688}
==========

[commit-bot: I haz the power, 2016-11-28 18:08:39]

Patchset 3 (id:??) landed as
https://crrev.com/5eb66dfa480bed01684c4ae1e44599e613e9d5a6
Cr-Commit-Position: refs/heads/master@{#434688}