Improve inline breakpoint discovery when expression is multiline. Fix #521 (#522)

* Improve inline breakpoint discovery when expression is multiline. Fix #521

Author: gayanper

com.microsoft.java.debug.plugin/src/main/java/com/microsoft/java/debug/BreakpointLocationLocator.java

@@ -22,8 +22,8 @@ public class BreakpointLocationLocator
 
     public BreakpointLocationLocator(CompilationUnit compilationUnit, int lineNumber,
             boolean bindingsResolved,
-            boolean bestMatch) {
-        super(compilationUnit, lineNumber, bindingsResolved, bestMatch);
+            boolean bestMatch, int offset, int end) {
+        super(compilationUnit, lineNumber, bindingsResolved, bestMatch, offset, end);
     }
 
     @Override

com.microsoft.java.debug.plugin/src/main/java/com/microsoft/java/debug/plugin/internal/JdtSourceLookUpProvider.java

@@ -72,8 +72,8 @@
 import com.microsoft.java.debug.core.Configuration;
 import com.microsoft.java.debug.core.DebugException;
 import com.microsoft.java.debug.core.DebugSettings;
-import com.microsoft.java.debug.core.JavaBreakpointLocation;
 import com.microsoft.java.debug.core.DebugSettings.Switch;
+import com.microsoft.java.debug.core.JavaBreakpointLocation;
 import com.microsoft.java.debug.core.adapter.AdapterUtils;
 import com.microsoft.java.debug.core.adapter.Constants;
 import com.microsoft.java.debug.core.adapter.IDebugAdapterContext;
@@ -210,8 +210,11 @@ public JavaBreakpointLocation[] getBreakpointLocations(String sourceUri, SourceB
                 // mark it as "unverified".
                 // In future, we could consider supporting to update the breakpoint to a valid
                 // location.
+
+                // passing the offset to the constructor, it can recognize the multiline lambda
+                // expression well
                 BreakpointLocationLocator locator = new BreakpointLocationLocator(astUnit,
-                        sourceLine, true, true);
+                        sourceLine, true, true, astUnit.getPosition(sourceLine, 0), 0);
                 astUnit.accept(locator);
                 // When the final valid line location is same as the original line, that
                 // represents it's a valid breakpoint.