Make sure InspectorBaseAgent::disable is not renamed.

Make sure InspectorBaseAgent::disable is not renamed.

When we started blacklisting renaming of "disable" method (in r380773)
the virtual method was implemented by InspectorAgent class.  Since then
(in 396629) the virtual method was moved into InspectorBaseAgent class
and consequently blacklisting stopped working - this CL fixes this.

I've tested this CL by running the clang tool on
third_party/WebKit/Source/modules/storage/StorageNamespaceController.cpp
and verifying that "disable" method in core/inspector/InspectorBaseAgent.h
was not renamed after this CL (and was renamed before this CL).

BUG=593446
Review-Url: https://codereview.chromium.org/2646803002
Cr-Commit-Position: refs/heads/master@{#444815}
Committed: https://chromium.googlesource.com/chromium/src/+/535b89a267d880c41efed60526213483ae79e707

[Łukasz Anforowicz, 2017-01-19 17:58:57]

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

[commit-bot: I haz the power, 2017-01-19 17:59:19]

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

[Łukasz Anforowicz, 2017-01-19 17:59:22]

lukasza@chromium.org changed reviewers:
+ danakj@chromium.org

[Łukasz Anforowicz, 2017-01-19 17:59:22]

Dana, can you PTAL?

[Łukasz Anforowicz, 2017-01-19 18:00:34]

Description was changed from

==========
Make sure InspectorBaseAgent::disable is not renamed.

When we started blacklisting renaming of "disable" method (in r380773)
the virtual method was implemented by InspectorAgent class.  Since then
(in 396629) the virtual method was moved into InspectorBaseAgent class
and consequently blacklisting stopped working - this CL fixes this.

BUG=593446
==========

to

==========
Make sure InspectorBaseAgent::disable is not renamed.

When we started blacklisting renaming of "disable" method (in r380773)
the virtual method was implemented by InspectorAgent class.  Since then
(in 396629) the virtual method was moved into InspectorBaseAgent class
and consequently blacklisting stopped working - this CL fixes this.

I've tested this CL by running the clang tool on
third_party/WebKit/Source/modules/storage/StorageNamespaceController.cpp
and verifying that "disable" method in core/inspector/InspectorBaseAgent.h
was not renamed after this CL (and was renamed before this CL).

BUG=593446
==========

[commit-bot: I haz the power, 2017-01-19 18:08:54]

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

[commit-bot: I haz the power, 2017-01-19 18:08:55]

Dry run: This issue passed the CQ dry run.

[danakj, 2017-01-19 19:24:31]

LGTM

[danakj, 2017-01-19 19:24:35]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2017-01-19 19:25:07]

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, 2017-01-19 19:34:02]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1484853875765780, "parent_rev":
"1f9c06b2bc825d86db2fbc2b21dc35fd453f8b69", "commit_rev":
"535b89a267d880c41efed60526213483ae79e707"}

[commit-bot: I haz the power, 2017-01-19 19:34:33]

Description was changed from

==========
Make sure InspectorBaseAgent::disable is not renamed.

When we started blacklisting renaming of "disable" method (in r380773)
the virtual method was implemented by InspectorAgent class.  Since then
(in 396629) the virtual method was moved into InspectorBaseAgent class
and consequently blacklisting stopped working - this CL fixes this.

I've tested this CL by running the clang tool on
third_party/WebKit/Source/modules/storage/StorageNamespaceController.cpp
and verifying that "disable" method in core/inspector/InspectorBaseAgent.h
was not renamed after this CL (and was renamed before this CL).

BUG=593446
==========

to

==========
Make sure InspectorBaseAgent::disable is not renamed.

When we started blacklisting renaming of "disable" method (in r380773)
the virtual method was implemented by InspectorAgent class.  Since then
(in 396629) the virtual method was moved into InspectorBaseAgent class
and consequently blacklisting stopped working - this CL fixes this.

I've tested this CL by running the clang tool on
third_party/WebKit/Source/modules/storage/StorageNamespaceController.cpp
and verifying that "disable" method in core/inspector/InspectorBaseAgent.h
was not renamed after this CL (and was renamed before this CL).

BUG=593446

Review-Url: https://codereview.chromium.org/2646803002
Cr-Commit-Position: refs/heads/master@{#444815}
Committed:
https://chromium.googlesource.com/chromium/src/+/535b89a267d880c41efed6052621...
==========

[commit-bot: I haz the power, 2017-01-19 19:34:34]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/535b89a267d880c41efed6052621...