Throw illegal exception when formatting with invalid template index.

Throw illegal exception when formatting with invalid template index.

R=arv@chromium.org
BUG=chromium:492526
LOG=N

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

[arv (Not doing code reviews), 2015-05-27 14:07:11]

LGTM

[Igor Sheludko, 2015-05-27 14:16:16]

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

[Igor Sheludko, 2015-05-27 14:16:17]

https://codereview.chromium.org/1146923004/diff/1/src/messages.cc
File src/messages.cc (right):

https://codereview.chromium.org/1146923004/diff/1/src/messages.cc#newcode328
src/messages.cc:328: break;
DBC: do we still need break here?

[arv (Not doing code reviews), 2015-05-27 14:16:55]

https://codereview.chromium.org/1146923004/diff/1/src/messages.cc
File src/messages.cc (right):

https://codereview.chromium.org/1146923004/diff/1/src/messages.cc#newcode328
src/messages.cc:328: break;
On 2015/05/27 14:16:17, Igor Sheludko wrote:
> DBC: do we still need break here?

Nope. This break should be removed.

[Yang, 2015-05-28 06:39:26]

The CQ bit was checked by yangguo@chromium.org

[Yang, 2015-05-28 06:39:26]

The patchset sent to the CQ was uploaded after l-g-t-m from arv@chromium.org
Link to the patchset: https://codereview.chromium.org/1146923004/#ps20001
(title: "addressed comment")

[commit-bot: I haz the power, 2015-05-28 06:39:39]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1146923004/20001

[commit-bot: I haz the power, 2015-05-28 07:05:07]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-28 07:05:22]

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