Issue metadata
Sign in to add a comment
|
"Show page size on resize" option in render settings not persistent.
Reported by
valentin...@gmail.com,
Mar 8 2016
|
||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/50.0.2661.11 Safari/537.36 Steps to reproduce the problem: 1. Enable "Show page size on resize". 2. Close chrome. 3. Open chrome, "Show page size on resize" option is un-checked. What is the expected behavior? The selected option to persist between chrome sessions. What went wrong? The page size fixed in https://bugs.chromium.org/p/chromium/issues/detail?id=582421 is not persistent. Did this work before? Yes Chrome 48 (non dev) Chrome version: 50.0.2661.11 Channel: dev OS Version: 10.0 Flash Version: Shockwave Flash 21.0 r0
,
Mar 9 2016
,
Mar 9 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/5e9b5ed287ad4e8a91e7e99e6b426a2ae9897f5a commit 5e9b5ed287ad4e8a91e7e99e6b426a2ae9897f5a Author: dgozman <dgozman@chromium.org> Date: Wed Mar 09 21:05:59 2016 [DevTools] Show viewport size on resize by default, disable in device mode. BUG= 592847 Review URL: https://codereview.chromium.org/1771853004 Cr-Commit-Position: refs/heads/master@{#380212} [modify] https://crrev.com/5e9b5ed287ad4e8a91e7e99e6b426a2ae9897f5a/third_party/WebKit/Source/devtools/front_end/emulation/DeviceModeModel.js [modify] https://crrev.com/5e9b5ed287ad4e8a91e7e99e6b426a2ae9897f5a/third_party/WebKit/Source/devtools/front_end/main/Main.js [modify] https://crrev.com/5e9b5ed287ad4e8a91e7e99e6b426a2ae9897f5a/third_party/WebKit/Source/devtools/front_end/main/RenderingOptions.js
,
Mar 11 2016
Verified on mac canary 51.0.2673.0.
,
Mar 11 2016
Your change meets the bar and is auto-approved for M50 (branch: 2661)
,
Mar 11 2016
Please try to merge your change to M50 branch 2661 ASAP if you think it is a safe merge as we're close to M50 Beta candidate cut for next week. Thank you.
,
Mar 15 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/81c1f124a8a6258f075b51b474abaf3bd9ce4d09 commit 81c1f124a8a6258f075b51b474abaf3bd9ce4d09 Author: Dmitry Gozman <dgozman@chromium.org> Date: Tue Mar 15 18:22:01 2016 Merge to 2661 "[DevTools] Show viewport size on resize by default, disable in device mode." > [DevTools] Show viewport size on resize by default, disable in device mode. > > BUG= 592847 > > Review URL: https://codereview.chromium.org/1771853004 > > Cr-Commit-Position: refs/heads/master@{#380212} (cherry picked from commit 5e9b5ed287ad4e8a91e7e99e6b426a2ae9897f5a) TBR=pfeldman Review URL: https://codereview.chromium.org/1807573002 . Cr-Commit-Position: refs/branch-heads/2661@{#241} Cr-Branched-From: ef6f6ae5e4c96622286b563658d5cd62a6cf1197-refs/heads/master@{#378081} [modify] https://crrev.com/81c1f124a8a6258f075b51b474abaf3bd9ce4d09/third_party/WebKit/Source/devtools/front_end/emulation/DeviceModeModel.js [modify] https://crrev.com/81c1f124a8a6258f075b51b474abaf3bd9ce4d09/third_party/WebKit/Source/devtools/front_end/main/Main.js [modify] https://crrev.com/81c1f124a8a6258f075b51b474abaf3bd9ce4d09/third_party/WebKit/Source/devtools/front_end/main/RenderingOptions.js
,
Mar 15 2016
,
Mar 16 2016
@valentino.belardi: Could you please let us know where exactly we can see the option "Show page size on resize" in the Rendering Settings in DevTools. Could you please provide us the screen-recording for better understanding. Please find the attached screen-recording. Moreover, the UI is now appearing as reported in the issue beneath. https://bugs.chromium.org/p/chromium/issues/detail?id=582421 Thank you.
,
Mar 16 2016
Sure, this option is available in version 50.0.2661.18 under Render Settings. When I submitted this case, it was listed under 50.0.2661.11, which still had this option available.
,
Mar 16 2016
@valentino.belardi: Do we need to enable any flag to see this option? Since I was not able to see it in Chrome Beta Version - 50.0.2661.37
,
Mar 16 2016
I am sure the option has been removed in the newer builds as it has been enabled by default when not in device mode, as mentioned in the above fix.
,
Mar 16 2016
Verified the fix on Windows 7, MAC (10.11.3) & Ubuntu Trusty (14.04) for Google Chrome Beta Version - 50.0.2661.37 Screen-recording & Screen-shot are attached. TE-Verified labels are added.
,
Mar 16 2016
Thanks for the confirmation, I have upgraded the dev version to 51.0.2679.0 and all is working as expected.
,
Mar 18 2016
Issue 596004 has been merged into this issue. |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by alph@chromium.org
, Mar 8 2016Status: Assigned (was: Unconfirmed)