Reorder protobuf compiler command line arguments.

Reorder protobuf compiler command line arguments.

Attempt to chase non-repoducible bug. CL crrev.com/2239383002 has
changed a command line for protoc and there are rare occasional
crashes on Win32. It could be because of some bug in protoc's parser.
This change restores a command line like before the suspecting CL
only for the most common case with only one proto file in a library.

BUG=644525
Committed: https://crrev.com/e4dfc3b51a21c5c771efe6108d0e75c4239706ee
Cr-Commit-Position: refs/heads/master@{#417544}

[kraynov, 2016-09-08 16:36:48]

Description was changed from

==========
Reorder protobuf compiler command line arguments.

Attempt to chase non-repoducible bug. CL crrev.com/2239383002 has
changed a command line for protoc and there are rare occasional
crashes on Win32. It could be because of some bug in protoc's parser.
This change doesn't affect normal build in any way.

BUG=644525
==========

to

==========
Reorder protobuf compiler command line arguments.

Attempt to chase non-repoducible bug. CL crrev.com/2239383002 has
changed a command line for protoc and there are rare occasional
crashes on Win32. It could be because of some bug in protoc's parser.
This change restores a command line like before the suspecting CL
only for the most common case with only one proto file in a library.

BUG=644525
==========

[kraynov, 2016-09-08 16:37:34]

kraynov@chromium.org changed reviewers:
+ primiano@chromium.org, scottmg@chromium.org

[kraynov, 2016-09-08 16:37:35]

PTAL

[kraynov, 2016-09-08 16:37:58]

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

[commit-bot: I haz the power, 2016-09-08 16:38:25]

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

[Primiano Tucci (use gerrit), 2016-09-08 16:46:42]

thanks, non-owner LGTM.
I can't think of any other reason why the same invocation of the same binary
started flakily crashing

[commit-bot: I haz the power, 2016-09-08 18:06:25]

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

[commit-bot: I haz the power, 2016-09-08 18:06:27]

Dry run: This issue passed the CQ dry run.

[scottmg, 2016-09-09 04:14:20]

Sorry, for the delay was out today.

lgtm to try if you want, but as it hasn't happened again in the last 48h I
probably wouldn't bother landing for now as we're not really going to know if it
fixes anything.

I think we'd have to find some way to repro to have any chance of fixing. Did we
try syncing to the specific rev that one of the failures was building to see if
maybe goma had a corrupt cache or something? Maybe that'd be flushed by now
anyway.

[kraynov, 2016-09-09 09:42:54]

The CQ bit was checked by kraynov@chromium.org

[commit-bot: I haz the power, 2016-09-09 09:43:05]

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-09-09 09:46:48]

Description was changed from

==========
Reorder protobuf compiler command line arguments.

Attempt to chase non-repoducible bug. CL crrev.com/2239383002 has
changed a command line for protoc and there are rare occasional
crashes on Win32. It could be because of some bug in protoc's parser.
This change restores a command line like before the suspecting CL
only for the most common case with only one proto file in a library.

BUG=644525
==========

to

==========
Reorder protobuf compiler command line arguments.

Attempt to chase non-repoducible bug. CL crrev.com/2239383002 has
changed a command line for protoc and there are rare occasional
crashes on Win32. It could be because of some bug in protoc's parser.
This change restores a command line like before the suspecting CL
only for the most common case with only one proto file in a library.

BUG=644525
==========

[commit-bot: I haz the power, 2016-09-09 09:46:49]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-09 09:48:23]

Description was changed from

==========
Reorder protobuf compiler command line arguments.

Attempt to chase non-repoducible bug. CL crrev.com/2239383002 has
changed a command line for protoc and there are rare occasional
crashes on Win32. It could be because of some bug in protoc's parser.
This change restores a command line like before the suspecting CL
only for the most common case with only one proto file in a library.

BUG=644525
==========

to

==========
Reorder protobuf compiler command line arguments.

Attempt to chase non-repoducible bug. CL crrev.com/2239383002 has
changed a command line for protoc and there are rare occasional
crashes on Win32. It could be because of some bug in protoc's parser.
This change restores a command line like before the suspecting CL
only for the most common case with only one proto file in a library.

BUG=644525

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

[commit-bot: I haz the power, 2016-09-09 09:48:24]

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