Fix wrong JS import linebreak

[futuarmo]

Summary

Fixes #52935
Also I could fix it by wrapping 2 if-statements in UnwrappedLineFormatter.cpp (lines 1191 and 1196) but I added more checks to ContinuationIndenter.cpp.

[futuarmo]

Issue was fixed by other patch

clang/lib/Format/ContinuationIndenter.cpp

if ((startsNextParameter(Current, Style) || Previous.is(tok::semi) ||
         // completely unnecessary line break after a template type that isn't
         // line-wrapped.
         (Previous.NestingLevel == 1 || Style.BinPackParameters)) ||
        (Style.BreakBeforeTernaryOperators && Current.is(TT_ConditionalExpr) &&
         Previous.isNot(tok::question)) ||
        (!Style.BreakBeforeTernaryOperators &&
         Previous.is(TT_ConditionalExpr))) &&
       State.Stack.back().BreakBeforeParameter && !Current.isTrailingComment() &&
-      !Current.isOneOf(tok::r_paren, tok::r_brace))
+      !Current.isOneOf(tok::r_paren, tok::r_brace) &&
+      ((State.Line->Type == LT_ImportStatement &&
+        Style.JavaScriptWrapImports) ||
+      State.Line->Type != LT_ImportStatement))
     return true;
   if (State.Stack.back().IsChainedConditional &&
       ((Style.BreakBeforeTernaryOperators && Current.is(TT_ConditionalExpr) &&
         Current.is(tok::colon)) ||
        (!Style.BreakBeforeTernaryOperators && Previous.is(TT_ConditionalExpr) &&
         Previous.is(tok::colon))))
     return true;
   if (((Previous.is(TT_DictLiteral) && Previous.is(tok::l_brace)) ||

clang/unittests/Format/FormatTestJS.cpp

verifyFormat("import {\n"
                "  A,\n"
                "} from\n"
                "    'some/path/longer/than/column/limit/module.js';",
                " import  {  \n"
                "    A,  \n"
                "  }    from\n"
                "      'some/path/longer/than/column/limit/module.js'  ; ",
                Style);
+  Style.ColumnLimit = 0;
+  Style.JavaScriptWrapImports = false;
+  verifyFormat("import {aaa, bbb, ccc} from 'abc';",
+               "import {aaa, bbb, ccc} from 'abc';", Style);
 }
 
 TEST_F(FormatTestJS, TemplateStrings) {
   // Keeps any whitespace/indentation within the template string.
   verifyFormat("var x = `hello\n"
                "     ${name}\n"
                "  !`;",
                "var x    =    `hello\n"