[DevTools] Remove old generator in content.

[DevTools] Remove old generator in content.

BUG=664683
TBR=pfeldman

Committed: https://crrev.com/c99acd65573b59a1d18a4715c189c8eb76d1b3d0
Cr-Commit-Position: refs/heads/master@{#439377}

[dgozman, 2016-12-15 07:09:56]

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

[dgozman, 2016-12-15 07:09:57]

Could you please take a look?

[caseq, 2016-12-16 00:10:28]

lgtm

https://codereview.chromium.org/2573423002/diff/1/content/browser/devtools/de...
File content/browser/devtools/devtools_agent_host_impl.cc (right):

https://codereview.chromium.org/2573423002/diff/1/content/browser/devtools/de...
content/browser/devtools/devtools_agent_host_impl.cc:62:
base::StringToInt(tokens[0], &major) && major == 1 &&
StringToUInt() here and below :)

[dgozman, 2016-12-18 16:59:54]

https://codereview.chromium.org/2573423002/diff/1/content/browser/devtools/de...
File content/browser/devtools/devtools_agent_host_impl.cc (right):

https://codereview.chromium.org/2573423002/diff/1/content/browser/devtools/de...
content/browser/devtools/devtools_agent_host_impl.cc:62:
base::StringToInt(tokens[0], &major) && major == 1 &&
On 2016/12/16 00:10:28, caseq wrote:
> StringToUInt() here and below :)

Rewritten.

[dgozman, 2016-12-18 16:59:59]

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

[commit-bot: I haz the power, 2016-12-18 17:00:05]

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-12-18 17:54:44]

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

[commit-bot: I haz the power, 2016-12-18 17:54:45]

Dry run: This issue passed the CQ dry run.

[dgozman, 2016-12-18 18:48:45]

The CQ bit was checked by dgozman@chromium.org

[dgozman, 2016-12-18 18:48:45]

The patchset sent to the CQ was uploaded after l-g-t-m from caseq@chromium.org
Link to the patchset: https://codereview.chromium.org/2573423002/#ps20001
(title: "fixed review comment")

[commit-bot: I haz the power, 2016-12-18 18:48:56]

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-12-18 18:54:44]

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

[commit-bot: I haz the power, 2016-12-18 18:54:45]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[dgozman, 2016-12-18 20:40:22]

Description was changed from

==========
[DevTools] Remove old generator in content.

BUG=664683
==========

to

==========
[DevTools] Remove old generator in content.

BUG=664683
TBR=pfeldman
==========

[dgozman, 2016-12-18 20:40:26]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2016-12-18 20:40:33]

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-12-18 20:44:01]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1482093626465450,
"parent_rev": "d698a7adc677fea3eb102c868df85ecce44c5b88", "commit_rev":
"7dea2c4c06d9c1ad2e3536e753183b05e6466aa0"}

[commit-bot: I haz the power, 2016-12-18 20:44:27]

Description was changed from

==========
[DevTools] Remove old generator in content.

BUG=664683
TBR=pfeldman
==========

to

==========
[DevTools] Remove old generator in content.

BUG=664683
TBR=pfeldman

Review-Url: https://codereview.chromium.org/2573423002
==========

[commit-bot: I haz the power, 2016-12-18 20:44:28]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-12-18 20:46:58]

Description was changed from

==========
[DevTools] Remove old generator in content.

BUG=664683
TBR=pfeldman

Review-Url: https://codereview.chromium.org/2573423002
==========

to

==========
[DevTools] Remove old generator in content.

BUG=664683
TBR=pfeldman

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

[commit-bot: I haz the power, 2016-12-18 20:46:59]

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