From c0f90a8f93546f4e9a708c44ac610f36b5e9b1f8 Mon Sep 17 00:00:00 2001 From: Steve Block Date: Wed, 29 Sep 2010 11:16:49 +0100 Subject: Trim empty lines when comparing text results in DumpRenderTree2 Change-Id: I94c9de8e6131f6b60456700e9d5a3b27c8c9d1c3 --- .../com/android/dumprendertree2/TextResult.java | 30 ++++++++++++++-------- 1 file changed, 20 insertions(+), 10 deletions(-) (limited to 'tests/DumpRenderTree2/src') diff --git a/tests/DumpRenderTree2/src/com/android/dumprendertree2/TextResult.java b/tests/DumpRenderTree2/src/com/android/dumprendertree2/TextResult.java index f835b6a..1460178 100644 --- a/tests/DumpRenderTree2/src/com/android/dumprendertree2/TextResult.java +++ b/tests/DumpRenderTree2/src/com/android/dumprendertree2/TextResult.java @@ -87,20 +87,30 @@ public class TextResult extends AbstractResult { @Override public ResultCode getResultCode() { - if (mResultCode != null) { - return mResultCode; - } - - if (mExpectedResult == null) { - mResultCode = AbstractResult.ResultCode.NO_EXPECTED_RESULT; - } else if (!mExpectedResult.equals(mActualResult)) { - mResultCode = AbstractResult.ResultCode.RESULTS_DIFFER; - } else { - mResultCode = AbstractResult.ResultCode.RESULTS_MATCH; + if (mResultCode == null) { + if (mExpectedResult == null) { + mResultCode = AbstractResult.ResultCode.NO_EXPECTED_RESULT; + } else { + mResultCode = resultsMatch() ? AbstractResult.ResultCode.RESULTS_MATCH + : AbstractResult.ResultCode.RESULTS_DIFFER; + } } return mResultCode; } + private boolean resultsMatch() { + assert mExpectedResult != null; + assert mActualResult != null; + // Trim leading and trailing empty lines, as other WebKit platforms do. + String leadingEmptyLines = "^\\n+"; + String trailingEmptyLines = "\\n+$"; + String trimmedExpectedResult = mExpectedResult.replaceFirst(leadingEmptyLines, "") + .replaceFirst(trailingEmptyLines, ""); + String trimmedActualResult = mActualResult.replaceFirst(leadingEmptyLines, "") + .replaceFirst(trailingEmptyLines, ""); + return trimmedExpectedResult.equals(trimmedActualResult); + } + @Override public boolean didCrash() { return false; -- cgit v1.1