No line break when dollar sign follows minus

No line break when dollar sign follows minus

Spec: http://unicode.org/reports/tr14/#PR.
"Many currency signs can appear on both sides, or even the middle, of a numeric expression.
Therefore the line breaking algorithm, by default, does not break between PR and numbers or
letters on either side."
Here PR refers to:
002B    PLUS SIGN
005C    REVERSE SOLIDUS
00B1    PLUS-MINUS
2116    NUMERO SIGN
2212    MINUS SIGN
2213    MINUS-OR-PLUS-SIGN

BUG=663542
Committed: https://crrev.com/bebd7123bbffb4d99b2e4b74c96dc394df92fa70
Cr-Commit-Position: refs/heads/master@{#431426}

[xing.xu, 2016-11-10 08:49:34]

The CQ bit was checked by xing.xu@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2016-11-10 08:49:49]

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

[xing.xu, 2016-11-10 08:50:46]

Description was changed from

==========
No line break when dollar sign follows minus

Spec: http://unicode.org/reports/tr14/#PR.
"Many currency signs can appear on both sides, or even the middle, of a numeric
expression.
Therefore the line breaking algorithm, by default, does not break between PR and
numbers or
letters on either side."
Here PR refers to:
002B    PLUS SIGN
005C    REVERSE SOLIDUS
00B1    PLUS-MINUS
2116    NUMERO SIGN
2212    MINUS SIGN
2213    MINUS-OR-PLUS-SIGN

BUG=663542
==========

to

==========
No line break when dollar sign follows minus

Spec: http://unicode.org/reports/tr14/#PR.
"Many currency signs can appear on both sides, or even the middle, of a numeric
expression.
Therefore the line breaking algorithm, by default, does not break between PR and
numbers or
letters on either side."
Here PR refers to:
002B    PLUS SIGN
005C    REVERSE SOLIDUS
00B1    PLUS-MINUS
2116    NUMERO SIGN
2212    MINUS SIGN
2213    MINUS-OR-PLUS-SIGN

BUG=663542
==========

[xing.xu, 2016-11-10 08:50:46]

xing.xu@intel.com changed reviewers:
+ dtapuska@chromium.org, npc@google.com, pdr@chromium.org

[commit-bot: I haz the power, 2016-11-10 10:02:42]

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

[commit-bot: I haz the power, 2016-11-10 10:02:42]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[xing.xu, 2016-11-10 12:36:26]

The CQ bit was checked by xing.xu@intel.com to run a CQ dry run

[commit-bot: I haz the power, 2016-11-10 12:36:37]

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

[xing.xu, 2016-11-10 14:08:34]

xing.xu@intel.com changed reviewers:
+ pdr@google.com
- pdr@chromium.org

[xing.xu, 2016-11-10 14:09:27]

xing.xu@intel.com changed reviewers:
+ pdr@chromium.org
- pdr@google.com

[xing.xu, 2016-11-10 14:47:54]

PTAL, thanks

[dtapuska, 2016-11-10 15:02:17]

On 2016/11/10 14:47:54, xing.xu wrote:
> PTAL, thanks

sorry I have no context in this area; I just triaged the issue will have to
leave review to OWNERS.

[commit-bot: I haz the power, 2016-11-10 15:07:10]

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

[commit-bot: I haz the power, 2016-11-10 15:07:11]

Dry run: This issue passed the CQ dry run.

[pdr., 2016-11-10 19:13:39]

pdr@chromium.org changed reviewers:
+ drott@chromium.org, eae@chromium.org
- dtapuska@chromium.org, npc@google.com, pdr@chromium.org

[pdr., 2016-11-10 19:14:10]

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

[pdr., 2016-11-10 19:14:11]

I'm not familiar with this area. I've added eae@chromium.org and
drott@chromium.org as reviewers and removed others.

[eae, 2016-11-10 23:38:00]

OK, LGTM

[xing.xu, 2016-11-11 00:32:01]

The CQ bit was checked by xing.xu@intel.com

[xing.xu, 2016-11-11 00:32:59]

The CQ bit was checked by xing.xu@intel.com

[xing.xu, 2016-11-11 00:32:59]

The CQ bit was unchecked by xing.xu@intel.com

[commit-bot: I haz the power, 2016-11-11 00:33:01]

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

[xing.xu, 2016-11-11 00:39:47]

The CQ bit was checked by xing.xu@intel.com

[commit-bot: I haz the power, 2016-11-11 00:40:33]

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-11-11 00:47:14]

Description was changed from

==========
No line break when dollar sign follows minus

Spec: http://unicode.org/reports/tr14/#PR.
"Many currency signs can appear on both sides, or even the middle, of a numeric
expression.
Therefore the line breaking algorithm, by default, does not break between PR and
numbers or
letters on either side."
Here PR refers to:
002B    PLUS SIGN
005C    REVERSE SOLIDUS
00B1    PLUS-MINUS
2116    NUMERO SIGN
2212    MINUS SIGN
2213    MINUS-OR-PLUS-SIGN

BUG=663542
==========

to

==========
No line break when dollar sign follows minus

Spec: http://unicode.org/reports/tr14/#PR.
"Many currency signs can appear on both sides, or even the middle, of a numeric
expression.
Therefore the line breaking algorithm, by default, does not break between PR and
numbers or
letters on either side."
Here PR refers to:
002B    PLUS SIGN
005C    REVERSE SOLIDUS
00B1    PLUS-MINUS
2116    NUMERO SIGN
2212    MINUS SIGN
2213    MINUS-OR-PLUS-SIGN

BUG=663542
==========

[commit-bot: I haz the power, 2016-11-11 00:47:15]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-11-11 00:48:52]

Description was changed from

==========
No line break when dollar sign follows minus

Spec: http://unicode.org/reports/tr14/#PR.
"Many currency signs can appear on both sides, or even the middle, of a numeric
expression.
Therefore the line breaking algorithm, by default, does not break between PR and
numbers or
letters on either side."
Here PR refers to:
002B    PLUS SIGN
005C    REVERSE SOLIDUS
00B1    PLUS-MINUS
2116    NUMERO SIGN
2212    MINUS SIGN
2213    MINUS-OR-PLUS-SIGN

BUG=663542
==========

to

==========
No line break when dollar sign follows minus

Spec: http://unicode.org/reports/tr14/#PR.
"Many currency signs can appear on both sides, or even the middle, of a numeric
expression.
Therefore the line breaking algorithm, by default, does not break between PR and
numbers or
letters on either side."
Here PR refers to:
002B    PLUS SIGN
005C    REVERSE SOLIDUS
00B1    PLUS-MINUS
2116    NUMERO SIGN
2212    MINUS SIGN
2213    MINUS-OR-PLUS-SIGN

BUG=663542

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

[commit-bot: I haz the power, 2016-11-11 00:48:53]

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