Create a browser test for ManualTest printing-resize-starts-transition.html |
||
Issue descriptionI am currently unable to create a Browser test required to land the fix here: https://codereview.chromium.org/2089373002 I am landing this fix for a regression with a Manual Test - and will work on the Browser test in the meanwhile. This bug is to track my progress on this.
,
Sep 21 2016
Just uploaded a patch here: https://codereview.chromium.org/2355303002
,
Sep 21 2016
Great. When can we expect it to be in Chrome then?
,
Sep 21 2016
This should hopefully land in the next few days and be in ToT immediately after. You can take a look at the code for the test at the link above (https://codereview.chromium.org/2355303002) if needed. However the fix landed some time ago - on the day I filed this bug in fact.
,
Sep 21 2016
Awesome. Can you provide a comment here when it has been merged to Chrome?
,
Sep 21 2016
The bot will automatically comment here and I will mark it as Fixed when it does. So that should definitely give you the relevant heads up :)
,
Sep 22 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/273e2988ee4e07931eb3c8f61068826c138e645e commit 273e2988ee4e07931eb3c8f61068826c138e645e Author: nainar <nainar@chromium.org> Date: Thu Sep 22 08:59:28 2016 Replace Manual test with a LayoutTest BUG= 634565 Review-Url: https://codereview.chromium.org/2355303002 Cr-Commit-Position: refs/heads/master@{#420286} [add] https://crrev.com/273e2988ee4e07931eb3c8f61068826c138e645e/third_party/WebKit/LayoutTests/printing/printing-resize-starts-transition.html [delete] https://crrev.com/bb539825927adfa6f082056d63a9a0c47be8a3b4/third_party/WebKit/ManualTests/printing-resize-starts-transition.html
,
Sep 22 2016
Fixed as per above bugdroid comment |
||
►
Sign in to add a comment |
||
Comment 1 by as...@asnoba.dk
, Sep 20 2016