Fix build error in headless_web_contents_impl.cc

Fix build error in headless_web_contents_impl.cc

e2df1d707cc2a7ba4a5b49f5f1f6be249ff05baf adds a new parameter to
WebContentsDelegate::WebContentsCreated. This CL updates the
implementation in headless_web_contents_impl.cc accordingly.

BUG=649855
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

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

[fwang, 2016-10-04 10:14:05]

Description was changed from

==========
Fix build error in headless_web_contents_impl.cc

e2df1d707cc2a7ba4a5b49f5f1f6be249ff05baf adds a new parameter to
WebContentsDelegate::WebContentsCreated. This CL updates the
implementation in headless_web_contents_impl.cc accordingly.

BUG=649855
==========

to

==========
Fix build error in headless_web_contents_impl.cc

e2df1d707cc2a7ba4a5b49f5f1f6be249ff05baf adds a new parameter to
WebContentsDelegate::WebContentsCreated. This CL updates the
implementation in headless_web_contents_impl.cc accordingly.

BUG=649855
==========

[fwang, 2016-10-04 10:14:05]

fwang@igalia.com changed reviewers:
+ alexclarke@chromium.org

[fwang, 2016-10-04 10:31:09]

Description was changed from

==========
Fix build error in headless_web_contents_impl.cc

e2df1d707cc2a7ba4a5b49f5f1f6be249ff05baf adds a new parameter to
WebContentsDelegate::WebContentsCreated. This CL updates the
implementation in headless_web_contents_impl.cc accordingly.

BUG=649855
==========

to

==========
Fix build error in headless_web_contents_impl.cc

e2df1d707cc2a7ba4a5b49f5f1f6be249ff05baf adds a new parameter to
WebContentsDelegate::WebContentsCreated. This CL updates the
implementation in headless_web_contents_impl.cc accordingly.

BUG=649855
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[fwang, 2016-10-04 10:34:01]

Patchset #2 (id:20001) has been deleted

[alex clarke (OOO till 29th), 2016-10-04 10:37:15]

lgtm

[fwang, 2016-10-04 10:38:14]

The CQ bit was checked by fwang@igalia.com

[commit-bot: I haz the power, 2016-10-04 10:38:24]

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-10-04 10:46:30]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-04 10:49:30]

Description was changed from

==========
Fix build error in headless_web_contents_impl.cc

e2df1d707cc2a7ba4a5b49f5f1f6be249ff05baf adds a new parameter to
WebContentsDelegate::WebContentsCreated. This CL updates the
implementation in headless_web_contents_impl.cc accordingly.

BUG=649855
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Fix build error in headless_web_contents_impl.cc

e2df1d707cc2a7ba4a5b49f5f1f6be249ff05baf adds a new parameter to
WebContentsDelegate::WebContentsCreated. This CL updates the
implementation in headless_web_contents_impl.cc accordingly.

BUG=649855
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

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

[commit-bot: I haz the power, 2016-10-04 10:49:31]

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

[nasko, 2016-10-04 23:02:01]

nasko@chromium.org changed reviewers:
+ nasko@chromium.org

[nasko, 2016-10-04 23:02:01]

Is headless not compiled as part of any target? I didn't see any breakage, the
CQ allowed the CL to go through, and I don't think the tree was closed.

[fwang, 2016-10-05 06:28:15]

On 2016/10/04 23:02:01, nasko (slow) wrote:
> Is headless not compiled as part of any target? I didn't see any breakage, the
> CQ allowed the CL to go through, and I don't think the tree was closed.

I guess so. I was testing ChromeOS+Ozone+(Wayland/X11/headless) which I don't
think are tested by the bot.

[alex clarke (OOO till 29th), 2016-10-05 09:05:20]

On 2016/10/04 23:02:01, nasko (slow) wrote:
> Is headless not compiled as part of any target? I didn't see any breakage, the
> CQ allowed the CL to go through, and I don't think the tree was closed.

Headless will be on the waterfall before long:
https://bugs.chromium.org/p/chromium/issues/detail?id=632008&desc=2