commit 5f4cd74d2693e23b27237b935f28a87611e25e6b
author Henrique Nakashima <hnakashima@chromium.org> Thu Aug 16 15:45:09 2018 +0000
committer Chromium commit bot <commit-bot@chromium.org> Thu Aug 16 15:45:09 2018 +0000
tree 039954f1d14f2395a6c727da928fd7e995f53b10
parent 3e360453cded5f2f435195923ede0935f6847194

Do not set barcode module width and height in unittest.

Setting these to anything outside of [1, 10] is non-op. Not sure why
the test did that, but these lines have no effect.

Bug: pdfium:1135
Change-Id: I175d07137674ab1b6b9c86b3bc68b4a128601b9f
Reviewed-on: https://pdfium-review.googlesource.com/40250
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Ryan Harrison <rharrison@chromium.org>
Commit-Queue: Henrique Nakashima <hnakashima@chromium.org>

xfa/fwl/cfx_barcode_unittest.cpp

diff --git a/xfa/fwl/cfx_barcode_unittest.cpp b/xfa/fwl/cfx_barcode_unittest.cpp
index 585d92f..0c8a05f 100644
--- a/xfa/fwl/cfx_barcode_unittest.cpp
+++ b/xfa/fwl/cfx_barcode_unittest.cpp
@@ -44,8 +44,6 @@
     if (!barcode_)
       return false;
 
-    barcode_->SetModuleHeight(300);
-    barcode_->SetModuleWidth(420);
     barcode_->SetHeight(298);
     barcode_->SetWidth(418);
     return true;