Switch some fields in omnibox.mojom to non-nullable.

Switch some fields in omnibox.mojom to non-nullable.

BUG=406580
TEST=None

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

[yzshen1, 2014-08-25 20:25:33]

Hi, Scott.

Would you please take a look? Thanks!

https://codereview.chromium.org/501143003/diff/1/chrome/browser/ui/webui/omni...
File chrome/browser/ui/webui/omnibox/omnibox.mojom (right):

https://codereview.chromium.org/501143003/diff/1/chrome/browser/ui/webui/omni...
chrome/browser/ui/webui/omnibox/omnibox.mojom:52: // TODO(yzshen): Conceptually
|input_string| could be non-nullable. However,
I have reported this bug to the author.

I am fine if you would like me to hold off this patch till the bug is resolved
(so that I don't need to send you a follow-up patch to remove the nullable flag
and the TODO).

Please let me know. Thanks!

[sky, 2014-08-25 20:48:26]

LGTM

[yzshen1, 2014-08-25 20:49:34]

The CQ bit was checked by yzshen@chromium.org

[commit-bot: I haz the power, 2014-08-25 20:51:13]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/yzshen@chromium.org/501143003/1

[commit-bot: I haz the power, 2014-08-25 21:59:48]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  ios_rel_device_ninja on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)

[commit-bot: I haz the power, 2014-08-25 22:34:49]

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

[commit-bot: I haz the power, 2014-08-25 22:34:51]

Try jobs failed on following builders:
  ios_rel_device_ninja on tryserver.chromium.mac
(http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)

[yzshen1, 2014-08-25 22:40:08]

The CQ bit was checked by yzshen@chromium.org

[commit-bot: I haz the power, 2014-08-25 22:43:03]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/yzshen@chromium.org/501143003/1

[commit-bot: I haz the power, 2014-08-25 23:01:54]

Committed patchset #1 (1) as 6e7be00af52049c3945862c0fec30d12c49c6521

[commit-bot: I haz the power, 2014-09-10 02:38:10]

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