[DevTools] Support redirects in protocol handler generator.

[DevTools] Support redirects in protocol handler generator.

BUG=

Committed: https://crrev.com/2840ec3bd5e5e590323b6d2e53df4159951cacd0
Cr-Commit-Position: refs/heads/master@{#321175}

[vkuzkokov, 2015-03-18 16:43:09]

Patchset #1 (id:1) has been deleted

[vkuzkokov, 2015-03-18 17:13:27]

vkuzkokov@chromium.org changed reviewers:
+ dgozman@chromium.org

[vkuzkokov, 2015-03-18 17:17:48]

Dmitry, is it important that we always redirect to a command with the same name
within the domain?

[dgozman, 2015-03-18 17:30:12]

> Dmitry, is it important that we always redirect to a command with the same
name within the domain?
That's what we need today. If we decide to redirect to another name in the
future, we'll update both generators.

lgtm

https://codereview.chromium.org/1018983002/diff/40001/content/browser/devtool...
File content/browser/devtools/protocol/devtools_protocol_handler_generator.py
(right):

https://codereview.chromium.org/1018983002/diff/40001/content/browser/devtool...
content/browser/devtools/protocol/devtools_protocol_handler_generator.py:651:
command_map["TargetDomain"] = json_domain["domain"]
I'd rather do
command_map["TargetDomain"] = command_map["Domain"]

[vkuzkokov, 2015-03-18 17:34:10]

https://codereview.chromium.org/1018983002/diff/40001/content/browser/devtool...
File content/browser/devtools/protocol/devtools_protocol_handler_generator.py
(right):

https://codereview.chromium.org/1018983002/diff/40001/content/browser/devtool...
content/browser/devtools/protocol/devtools_protocol_handler_generator.py:651:
command_map["TargetDomain"] = json_domain["domain"]
On 2015/03/18 17:30:11, dgozman wrote:
> I'd rather do
> command_map["TargetDomain"] = command_map["Domain"]

Done.

[vkuzkokov, 2015-03-18 17:34:26]

The CQ bit was checked by vkuzkokov@chromium.org

[vkuzkokov, 2015-03-18 17:34:26]

The patchset sent to the CQ was uploaded after l-g-t-m from dgozman@chromium.org
Link to the patchset: https://codereview.chromium.org/1018983002/#ps60001
(title: " ")

[commit-bot: I haz the power, 2015-03-18 17:34:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1018983002/60001

[commit-bot: I haz the power, 2015-03-18 18:27:32]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2015-03-18 18:28:50]

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