Remove _URLOnStartLoading from crw_web_controller

Remove _URLOnStartLoading from crw_web_controller

Remove _URLOnStartLoading from crw_web_controller as its not read anywhere.


BUG=

Review-Url: https://codereview.chromium.org/2919473004
Cr-Commit-Position: refs/heads/master@{#476451}
Committed: https://chromium.googlesource.com/chromium/src/+/ae0d11211bae4d973474c10a26c40dc54cd97073

[mrefaat, 2017-06-01 19:55:08]

Description was changed from

==========
Remove _URLOnStartLoading from crw_web_controller

Remove _URLOnStartLoading from crw_web_controller as its not read anywhere.
Also add more debugging information in GetDescription of navigationItemImpl


BUG=
==========

to

==========
Remove _URLOnStartLoading from crw_web_controller

Remove _URLOnStartLoading from crw_web_controller as its not read anywhere.
Also add more debugging information in GetDescription of navigationItemImpl


BUG=
==========

[mrefaat, 2017-06-01 19:55:08]

mrefaat@chromium.org changed reviewers:
+ eugenebut@chromium.org, kkhorimoto@chromium.org

[mrefaat, 2017-06-01 19:55:17]

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

[commit-bot: I haz the power, 2017-06-01 19:56:22]

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, 2017-06-01 20:22:47]

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

[commit-bot: I haz the power, 2017-06-01 20:22:48]

Dry run: Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[Eugene But (OOO till 7-30), 2017-06-01 20:44:17]

You also want to update the test which is failing. Do you want to split this CL
into 2 parts (WC and GetDescription)?

[mrefaat, 2017-06-01 21:05:02]

On 2017/06/01 20:44:17, Eugene But wrote:
> You also want to update the test which is failing. Do you want to split this
CL
> into 2 parts (WC and GetDescription)?

Done

[mrefaat, 2017-06-01 21:05:10]

Description was changed from

==========
Remove _URLOnStartLoading from crw_web_controller

Remove _URLOnStartLoading from crw_web_controller as its not read anywhere.
Also add more debugging information in GetDescription of navigationItemImpl


BUG=
==========

to

==========
Remove _URLOnStartLoading from crw_web_controller

Remove _URLOnStartLoading from crw_web_controller as its not read anywhere.


BUG=
==========

[kkhorimoto, 2017-06-01 21:09:26]

lgtm

[mrefaat, 2017-06-01 21:11:28]

The CQ bit was checked by mrefaat@chromium.org

[commit-bot: I haz the power, 2017-06-01 21:12:24]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Eugene But (OOO till 7-30), 2017-06-01 21:12:57]

lgtm, thanks for cleanup!

[commit-bot: I haz the power, 2017-06-01 22:12:13]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1496351488195330,
"parent_rev": "a58bf8c9220bff80d68a92f3d6b3f53fb76c4173", "commit_rev":
"ae0d11211bae4d973474c10a26c40dc54cd97073"}

[commit-bot: I haz the power, 2017-06-01 22:12:21]

Description was changed from

==========
Remove _URLOnStartLoading from crw_web_controller

Remove _URLOnStartLoading from crw_web_controller as its not read anywhere.


BUG=
==========

to

==========
Remove _URLOnStartLoading from crw_web_controller

Remove _URLOnStartLoading from crw_web_controller as its not read anywhere.


BUG=

Review-Url: https://codereview.chromium.org/2919473004
Cr-Commit-Position: refs/heads/master@{#476451}
Committed:
https://chromium.googlesource.com/chromium/src/+/ae0d11211bae4d973474c10a26c4...
==========

[commit-bot: I haz the power, 2017-06-01 22:12:22]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/ae0d11211bae4d973474c10a26c4...