Prefix Did{Start|Stop}Loading methods.

Prefix Did{Start|Stop}Loading methods.

This CL prefixes the DidStartLoading and DidStopLoading methods in
RenderFrameImpl, so they don't conflict with the WebFrameClient versions
which currently start with lowercase letter. As part of the Blink rewrite
to match Chromium style, those will be conflicting.

BUG=582312
Committed: https://crrev.com/2d67b3c0fcb6e5ed693d90d8403e324033b7f804
Cr-Commit-Position: refs/heads/master@{#441372}

[nasko, 2017-01-04 00:31:02]

nasko@chromium.org changed reviewers:
+ dcheng@chromium.org, raymes@chromium.org

[nasko, 2017-01-04 00:31:03]

Hey Raymes and Daniel,
Can you review this CL for me? It adds a prefix to some methods to avoid
collisions after we rewrite Blink to match Chromium style.
Thanks in advance!
Nasko

[nasko, 2017-01-04 00:33:36]

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

[commit-bot: I haz the power, 2017-01-04 00:33:57]

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

[nasko, 2017-01-04 01:23:53]

nasko@chromium.org changed reviewers:
+ jochen@chromium.org
- raymes@chromium.org

[nasko, 2017-01-04 01:23:53]

s/raymes/jochen/

jochen@, can you do an OWNERS review for components/pdf?

[commit-bot: I haz the power, 2017-01-04 01:53:45]

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

[commit-bot: I haz the power, 2017-01-04 01:53:46]

Dry run: This issue passed the CQ dry run.

[jochen (gone - plz use gerrit), 2017-01-04 06:56:28]

lgtm

[nasko, 2017-01-04 14:45:02]

The CQ bit was checked by nasko@chromium.org

[commit-bot: I haz the power, 2017-01-04 14:45:18]

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-04 14:48:22]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1483541102865370, "parent_rev":
"64e65485d38adf2e66062f1214b4bdf6ad60183a", "commit_rev":
"f3ab3e2bfe491179cf09f42d99aaff2c1460cba4"}

[commit-bot: I haz the power, 2017-01-04 14:48:50]

Description was changed from

==========
Prefix Did{Start|Stop}Loading methods.

This CL prefixes the DidStartLoading and DidStopLoading methods in
RenderFrameImpl, so they don't conflict with the WebFrameClient versions
which currently start with lowercase letter. As part of the Blink rewrite
to match Chromium style, those will be conflicting.

BUG=582312
==========

to

==========
Prefix Did{Start|Stop}Loading methods.

This CL prefixes the DidStartLoading and DidStopLoading methods in
RenderFrameImpl, so they don't conflict with the WebFrameClient versions
which currently start with lowercase letter. As part of the Blink rewrite
to match Chromium style, those will be conflicting.

BUG=582312

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

[commit-bot: I haz the power, 2017-01-04 14:48:51]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2017-01-04 14:51:44]

Description was changed from

==========
Prefix Did{Start|Stop}Loading methods.

This CL prefixes the DidStartLoading and DidStopLoading methods in
RenderFrameImpl, so they don't conflict with the WebFrameClient versions
which currently start with lowercase letter. As part of the Blink rewrite
to match Chromium style, those will be conflicting.

BUG=582312

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

to

==========
Prefix Did{Start|Stop}Loading methods.

This CL prefixes the DidStartLoading and DidStopLoading methods in
RenderFrameImpl, so they don't conflict with the WebFrameClient versions
which currently start with lowercase letter. As part of the Blink rewrite
to match Chromium style, those will be conflicting.

BUG=582312

Committed: https://crrev.com/2d67b3c0fcb6e5ed693d90d8403e324033b7f804
Cr-Commit-Position: refs/heads/master@{#441372}
==========

[commit-bot: I haz the power, 2017-01-04 14:51:45]

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