Otherwise, Mac Retina wants 256.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/aeb778c817766d5c3b17097556218b0bafdbd1d4 commit aeb778c817766d5c3b17097556218b0bafdbd1d4 Author: pfeldman <pfeldman@chromium.org> Date: Fri May 06 22:08:57 2016 DevTools: account for the display scale factor while validating manifest under emulation. BUG= 609580 TBR=sky Review-Url: https://codereview.chromium.org/1950133009 Cr-Commit-Position: refs/heads/master@{#392171} [add] https://crrev.com/aeb778c817766d5c3b17097556218b0bafdbd1d4/chrome/browser/banners/app_banner_manager_emulation.cc [add] https://crrev.com/aeb778c817766d5c3b17097556218b0bafdbd1d4/chrome/browser/banners/app_banner_manager_emulation.h [modify] https://crrev.com/aeb778c817766d5c3b17097556218b0bafdbd1d4/chrome/browser/chrome_service_worker_browsertest.cc [modify] https://crrev.com/aeb778c817766d5c3b17097556218b0bafdbd1d4/chrome/browser/ui/browser.cc [modify] https://crrev.com/aeb778c817766d5c3b17097556218b0bafdbd1d4/chrome/browser/ui/browser.h [modify] https://crrev.com/aeb778c817766d5c3b17097556218b0bafdbd1d4/chrome/chrome_browser.gypi
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/23ac1ebc5f3a65cc260e6d4e83015d04ff7a3441 commit 23ac1ebc5f3a65cc260e6d4e83015d04ff7a3441 Author: sky <sky@chromium.org> Date: Fri May 06 23:39:07 2016 Revert of DevTools: account for the display scale factor while validating manifest under emulation. (patchset #5 id:80001 of https://codereview.chromium.org/1950133009/ ) Reason for revert: For reasons outlined in latest comment. Original issue's description: > DevTools: account for the display scale factor while validating manifest under emulation. > > BUG= 609580 > TBR=sky > > Committed: https://crrev.com/aeb778c817766d5c3b17097556218b0bafdbd1d4 > Cr-Commit-Position: refs/heads/master@{#392171} TBR=dominickn@chromium.org,dfalcantara@chromium.org,pfeldman@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= 609580 Review-Url: https://codereview.chromium.org/1955183002 Cr-Commit-Position: refs/heads/master@{#392204} [delete] https://crrev.com/6e156cf96a391ee35e402da76455969a1b11f169/chrome/browser/banners/app_banner_manager_emulation.cc [delete] https://crrev.com/6e156cf96a391ee35e402da76455969a1b11f169/chrome/browser/banners/app_banner_manager_emulation.h [modify] https://crrev.com/23ac1ebc5f3a65cc260e6d4e83015d04ff7a3441/chrome/browser/chrome_service_worker_browsertest.cc [modify] https://crrev.com/23ac1ebc5f3a65cc260e6d4e83015d04ff7a3441/chrome/browser/ui/browser.cc [modify] https://crrev.com/23ac1ebc5f3a65cc260e6d4e83015d04ff7a3441/chrome/browser/ui/browser.h [modify] https://crrev.com/23ac1ebc5f3a65cc260e6d4e83015d04ff7a3441/chrome/chrome_browser.gypi
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/7a69caa094b972461ad889e1684697b7c171677d commit 7a69caa094b972461ad889e1684697b7c171677d Author: pfeldman <pfeldman@chromium.org> Date: Sat May 07 21:33:50 2016 DevTools: account for the display scale factor while validating manifest under emulation. BUG= 609580 Review-Url: https://codereview.chromium.org/1950133009 Cr-Commit-Position: refs/heads/master@{#392268} [add] https://crrev.com/7a69caa094b972461ad889e1684697b7c171677d/chrome/browser/banners/app_banner_manager_emulation.cc [add] https://crrev.com/7a69caa094b972461ad889e1684697b7c171677d/chrome/browser/banners/app_banner_manager_emulation.h [modify] https://crrev.com/7a69caa094b972461ad889e1684697b7c171677d/chrome/browser/chrome_service_worker_browsertest.cc [modify] https://crrev.com/7a69caa094b972461ad889e1684697b7c171677d/chrome/browser/ui/browser.cc [modify] https://crrev.com/7a69caa094b972461ad889e1684697b7c171677d/chrome/browser/ui/browser.h [modify] https://crrev.com/7a69caa094b972461ad889e1684697b7c171677d/chrome/chrome_browser.gypi
Comment 1 by bugdroid1@chromium.org
, May 6 2016