Convert SetIsIncognitoProcess to use mojo.

chrome/common/renderer.mojom

+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+module chrome.mojom;
+
+struct InitialConfigurationParams {
+  // Indicates whether this process is running in incognito mode.
+  bool is_incognito_process;
+
+  // TODO(nigeltao): port the ChromeViewMsg_SetContentSettingRules legacy IPC
+  // message to use Mojo.
+};
+
+// The primordial Channel-associated interface implemented by a render process.
+// This should be used for implementing browser-to-renderer control messages
+// which need to retain FIFO with respect to legacy IPC messages.
+//
+// See also content.mojom.Renderer defined in content/common/renderer.mojom.
+interface Renderer {
+  // Configures the renderer, queued to send when the render process will
+  // launch.
+  InitialConfiguration(InitialConfigurationParams params);

[Ken Rockot(use gerrit already), 2016/12/01 18:46:30]

Please just pass a bool argument. You need need a Params struct for this.

[Ken Rockot(use gerrit already), 2016/12/01 18:49:45]

Err... that should be "don't need"

[nigeltao1, 2016/12/01 21:57:20]

It could be just a bool in this CL, but as per the TODO above, I intend to add
more fields to the params struct when I port the
ChromeViewMsg_SetContentSettingRules legacy IPC message to Mojo. It's sent in
ChromeContentBrowserClient::RenderProcessWillLaunch in
chrome/browser/chrome_content_browser_client.cc just next to the
ChromeViewMsg_SetIsIncognitoProcess legacy IPC message that I replaced in this
CL.

[Ken Rockot(use gerrit already), 2016/12/01 22:02:59]

I would still then do:

SetInitialConfiguration(bool is_incognito, ContentSettingRules rules)

Also nit while I'm thinking about it again: Please try to use imperative verb
phrases for message names. SetInitialConfiguration instead of
InitialConfiguration.

[nigeltao1, 2016/12/02 05:36:53]

Done.

+};