Move FrameHostMsg_GetCookies to mojom

Move FrameHostMsg_GetCookies to mojom

BUG=612500
R=jam@chromium.org

Committed: https://crrev.com/6a2d66a519fe40cf73c3b976431440234b4ba124
Cr-Commit-Position: refs/heads/master@{#409677}

[Ken Rockot(use gerrit already), 2016-08-02 20:39:09]

Description was changed from

==========
Move FrameHostMsg_GetCookies to mojom

BUG=612500
R=jam@chromium.org
==========

to

==========
Move FrameHostMsg_GetCookies to mojom

BUG=612500
R=jam@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[Ken Rockot(use gerrit already), 2016-08-02 20:39:14]

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

[commit-bot: I haz the power, 2016-08-02 20:39:43]

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

[Ken Rockot(use gerrit already), 2016-08-02 20:42:37]

https://codereview.chromium.org/2202723005/diff/1/content/browser/frame_host/...
File content/browser/frame_host/render_frame_message_filter.cc (left):

https://codereview.chromium.org/2202723005/diff/1/content/browser/frame_host/...
content/browser/frame_host/render_frame_message_filter.cc:325: //
http://crbug.com/99242
Note: I removed this chunk of code because the bug is archived and it doesn't
look like there have been any relevant crash reports for a very long time.
Likely fixed.

[commit-bot: I haz the power, 2016-08-02 21:54:59]

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

[commit-bot: I haz the power, 2016-08-02 21:55:02]

Dry run: This issue passed the CQ dry run.

[jam, 2016-08-03 17:48:47]

lgtm

[Ken Rockot(use gerrit already), 2016-08-03 18:54:34]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2016-08-03 18:55:16]

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-08-03 19:03:08]

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

[commit-bot: I haz the power, 2016-08-03 19:03:10]

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...)

[Ken Rockot(use gerrit already), 2016-08-03 19:09:49]

rockot@chromium.org changed reviewers:
+ tsepez@chromium.org

[Ken Rockot(use gerrit already), 2016-08-03 19:09:50]

+tsepez for mojom and frame_messages.h

[Tom Sepez, 2016-08-03 20:50:43]

LGTM.  Do we still need the associated interface stuff we added in the first CL
now that both are over mojo?

[Ken Rockot(use gerrit already), 2016-08-03 20:54:43]

On 2016/08/03 at 20:50:43, tsepez wrote:
> LGTM.  Do we still need the associated interface stuff we added in the first
CL now that both are over mojo?

Nah, I don't think we want to do that yet. It's unclear what ordering
dependencies exist among the many different frame messages. It's conceivable
that some other frame messages expect to be ordered with respect to these ones.

My current plan is to conservatively keep all converted frame and view messages
associated with the Channel, so we can isolate the mojom conversion work from
any work which might change ordering.

[Ken Rockot(use gerrit already), 2016-08-03 20:55:17]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2016-08-03 20:55:57]

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-08-03 23:44:36]

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

[commit-bot: I haz the power, 2016-08-03 23:44:38]

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

[Ken Rockot(use gerrit already), 2016-08-04 00:09:37]

Description was changed from

==========
Move FrameHostMsg_GetCookies to mojom

BUG=612500
R=jam@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Move FrameHostMsg_GetCookies to mojom

BUG=612500
R=jam@chromium.org
==========

[Ken Rockot(use gerrit already), 2016-08-04 00:09:42]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2016-08-04 00:10:13]

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-08-04 00:16:00]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-04 00:18:00]

Description was changed from

==========
Move FrameHostMsg_GetCookies to mojom

BUG=612500
R=jam@chromium.org
==========

to

==========
Move FrameHostMsg_GetCookies to mojom

BUG=612500
R=jam@chromium.org

Committed: https://crrev.com/6a2d66a519fe40cf73c3b976431440234b4ba124
Cr-Commit-Position: refs/heads/master@{#409677}
==========

[commit-bot: I haz the power, 2016-08-04 00:18:02]

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