|
|
Created:
6 years, 8 months ago by kenneth.r.christiansen Modified:
6 years, 8 months ago CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam Base URL:
svn://svn.chromium.org/chrome/trunk/src Visibility:
Public. |
DescriptionEnable shrinksViewportContentToFit for Android
This behavior is now a setting, so enabled it for Android.
Follow up to Blink side: 219223003.
BUG=359047
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=261369
Patch Set 1 #
Messages
Total messages: 18 (0 generated)
lgtm
The CQ bit was checked by kenneth.r.christiansen@intel.com
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/kenneth.r.christiansen@intel.com/21934...
The CQ bit was unchecked by commit-bot@chromium.org
Retried try job too often on android_dbg for step(s) slave_steps http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_db...
The CQ bit was checked by kenneth.r.christiansen@intel.com
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/kenneth.r.christiansen@intel.com/21934...
Moving this line outside the ifdef compiles fine locally. Could it be that it somehow is not fetching the latest blink? ../../content/renderer/web_preferences.cc:350:13:error: no member named 'setShrinksViewportContentToFit' in 'blink::WebSettings' settings->setShrinksViewportContentToFit(true);
The CQ bit was unchecked by commit-bot@chromium.org
Retried try job too often on android_clang_dbg for step(s) slave_steps http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android_cl...
The CQ bit was checked by kenneth.r.christiansen@intel.com
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/kenneth.r.christiansen@intel.com/21934...
The CQ bit was unchecked by commit-bot@chromium.org
Commit queue rejected this change because the description was changed between the time the change entered the commit queue and the time it was ready to commit. You can safely check the commit box again.
The CQ bit was checked by kenneth.r.christiansen@intel.com
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/kenneth.r.christiansen@intel.com/21934...
Message was sent while issue was closed.
Change committed as 261369 |