GitLab

We previously moved the two classes from stack to heap in order to make
TextNodeHandler also work with LayoutNG by polymorphism. However, the
on-going prototype (*) doesn't work in this way. Furthermore, we might
need a TextNodeHandler that works with both legacy and NG in order to
handle mixed layout tree.

Therefore, this patch makes the two classes stack allocated again.

(*) https://codereview.chromium.org/2968803002

Bug: 721957
Change-Id: I84ed1bff2acd92685fd53555e7b8f8ba62dbd4c3
Reviewed-on: https://chromium-review.googlesource.com/578715Reviewed-by: Yoshifumi Inoue <yosin@chromium.org>
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488102}

The KeyedServiceDependencyManager marks contexts as live in order to
detect when a context is used after shutdown is initiated. Ensure that
Profile contexts are marked as live by calling
BrowserContextDependencyManager::GetInstance()->MarkBrowserContextLive(this);
from the Profile constructor.

Bug: 745729

Change-Id: Icaa7950d77c1e33cbc098da8282ba3ff7f6977f3
Reviewed-on: https://chromium-review.googlesource.com/576339Reviewed-by: Elliot Glaysher <erg@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: Devlin <rdevlin.cronin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488101}

BUG=none

Change-Id: I0e5f8febd45ef6580ae829a10b7660d78095461a
Reviewed-on: https://chromium-review.googlesource.com/577908
Commit-Queue: Sadrul Chowdhury <sadrul@chromium.org>
Reviewed-by: danakj <danakj@chromium.org>
Reviewed-by: Fady Samuel <fsamuel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488100}

This CL rewrites usages of leveldb::DB::Open() to leveldb_env::OpenDB(),
exposing memory usage of leveldb databases to Chrome's tracing.

The only API difference of leveldb_env::OpenDB() is related to how it returns
its result:

1. It returns (via output parameter) std::unique_ptr<leveldb::DB>, while
   leveldb::DB::Open() returns a raw leveldb::DB pointer.

2. It doesn't modify output parameter on failure, while leveldb::DB::Open()
   returns (writes) nullptr.

Both of these changes are consistent with Chrome's guidelines.

Under the hood leveldb_env::OpenDB() uses leveldb_env::DBTracker to open and
expose databases to Chrome's tracing. See crrev.com/2855953002 for details.

Finally, this CL does two more things:

1. Adds a presubmit check to enforce usage of leveldb_env::OpenDB().

2. Fixes existing memory dump providers that already dump some leveldb
   databases to properly attribute their memory usage. Further work on
   those MDPs is tracked by crbug.com/735269.

BUG=711518

Review-Url: https://codereview.chromium.org/2953473002
Cr-Commit-Position: refs/heads/master@{#488099}

When the Play Store app search feature is enabled, we have
separators between items in the SearchResultTileItemListView.
We have to skip those separators while indexing items.

BUG=739410
BUG=739841

Review-Url: https://codereview.chromium.org/2972243002
Cr-Commit-Position: refs/heads/master@{#488098}

- Use current_cpu instead of target_cpu.
- Explicitly wrap the Android case with |is_android|.

Change-Id: I849c34ccafc3cdd89d164b01bf00ffb0292476f4
Reviewed-on: https://chromium-review.googlesource.com/569319Reviewed-by: Brett Wilson <brettw@chromium.org>
Commit-Queue: Wez <wez@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488097}

This patch does two things broadly...

1) If an empty block clears a float, we set the previous inflow
   position to the empty block *minus* the margin strut.

   This matches edges behaviour. I couldn't determine a sane behaviour
   for the other user agents.

2) If we resolve our BFC offset as a result of a child potentially
   clearing floats, *and* we have unpositioned direct float children,
   we set the previous inflow position to the top border edge, and
   reset the margin strut.

   I think this broadly matches other UAs behaviour, there is a note
   which I added about only forcing our BFC to resolve if
   unpositioned_floats_ contains the right type of float, but this
   can be looked at later.

Bug: 635619
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: I80ad01e12ad91f302b927103050055cffc6121f2
Reviewed-on: https://chromium-review.googlesource.com/577267Reviewed-by: Christian Biesinger <cbiesinger@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Commit-Queue: Emil A Eklund <eae@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488096}

Since integers returned by SelectionStartEnd() don't relate 
each other except start LayoutObject == end LayoutObject.
To emphasize that for readability, we should split it.

Bug: 739062
Change-Id: I81011fcbdbff0f17c3ff8917e1dca1101b89712e
Reviewed-on: https://chromium-review.googlesource.com/577432Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Yoichi Osato <yoichio@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488095}

Bug: 746402
Change-Id: I61685489f1a96ea5fc89b2a553462d470d53b28b
Reviewed-on: https://chromium-review.googlesource.com/578552Reviewed-by: Robert Kaplow <rkaplow@chromium.org>
Reviewed-by: Steven Holte <holte@chromium.org>
Commit-Queue: Robert Kaplow <rkaplow@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488094}

Also fix some typos.

Change-Id: I8dac5e543e118865726c81c399d27a258c602307
Reviewed-on: https://chromium-review.googlesource.com/578007Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: Wei-Yin Chen (陳威尹) <wychen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488093}

This CL corrects a typo in the aria-describedby for lockScreenSecondary.

Bug: None
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I5ab0d3ca4c684dfbec3601d29d1dfcb0f33ec343
Reviewed-on: https://chromium-review.googlesource.com/578430Reviewed-by: Hector Carmona <hcarmona@chromium.org>
Commit-Queue: Dave Schuyler <dschuyler@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488092}

R=eae@chromium.org

Change-Id: Ib267b515d4620ffa31de2ade2cdd4739c99027b9
Reviewed-on: https://chromium-review.googlesource.com/578391
Commit-Queue: Christian Biesinger <cbiesinger@chromium.org>
Commit-Queue: Emil A Eklund <eae@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488091}

Bug: 741980
Change-Id: I0f6d8c4b7a076a6ccaf8e3de692b3ad657e94cfc
Reviewed-on: https://chromium-review.googlesource.com/578688Reviewed-by: Joshua Pawlicki <waffles@chromium.org>
Commit-Queue: Sorin Jianu <sorin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488090}

Revert of Disable COM Initialization on SchedulerWorkers When COM_INIT_CHECK_HOOK_ENABLED() (patchset #1 id:1 of https://codereview.chromium.org/2978253002/ )

Reason for revert:
Causes

base/task_scheduler/scheduler_worker.h(199,40):  error: private field 'backward_compatibility_' is not used [-Werror,-Wunused-private-field]
  const SchedulerBackwardCompatibility backward_compatibility_;

Original issue's description:
> Disable COM Initialization on SchedulerWorkers When COM_INIT_CHECK_HOOK_ENABLED()
>
> This change will allow for the detection of tasks running in
> TaskScheduler that fail to post their task in a COM STA Task Runner
> instead.
>
> BUG=706512
> TBR=gab@chromium.org
> Previously Reviewed at https://chromium-review.googlesource.com/c/549223/
>
> Review-Url: https://codereview.chromium.org/2978253002
> Cr-Commit-Position: refs/heads/master@{#487625}
> Committed: https://chromium.googlesource.com/chromium/src/+/122a24f59d90a1b225e00b3ab6c9fd37fd365b8f

TBR=gab@chromium.org,rnk@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=706512

Review-Url: https://codereview.chromium.org/2983143002
Cr-Commit-Position: refs/heads/master@{#488089}

https://chromium.googlesource.com/native_client/src/native_client.git/+log/015356a98e44..01a28a6069a9

$ git log 015356a98..01a28a606 --date=short --no-merges --format='%ad %ae %s'
2017-07-19 thomasanderson Add missing dependency on exe_and_shlib_deps

Created with:
  roll-dep src/native_client
BUG=745773


Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls


CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_dbg_32_ng,linux_nacl_sdk_build
TBR=mseaborn@chromium.org

Change-Id: Iead54c72a7c511750ff4a4c16526e85505fd708c
Reviewed-on: https://chromium-review.googlesource.com/578714
Reviewed-by: <nacl-deps-roller@chromium.org>
Commit-Queue: <nacl-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488088}

Now we migrated ArcService classes into KeyedService,
so these are destructed on Profile destruction.
With this CL, the lifetime of ArcServiceManager, ArcSessionManager
and KeyedService classes becomes nested, i.e., now
while KeyedService is alive, ArcServiceManager and ArcSessionManager
are alive.

BUG=672829
TEST=Ran try, and run on DUT.

Change-Id: Iadeea98b19ea6eb6fb93cdb91f9bd1cf4b973794
Reviewed-on: https://chromium-review.googlesource.com/574986
Commit-Queue: Hidehiko Abe <hidehiko@chromium.org>
Reviewed-by: Yusuke Sato (in China Mon-Thurs, may be offline) <yusukes@chromium.org>
Reviewed-by: Luis Hector Chavez <lhchavez@chromium.org>
Reviewed-by: Dan Erat <derat@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488087}

The Site Details page currently only shows two options in the drop-down menu
for each permission, 'Allow' and 'Block'. This change adds a third option to the
list, which shows the default option, similar to the Page Info bubble.

The "Site Details" page that opens should show matching permissions to those
seen in the Page Info bubble, including the defaults - e.g. Javascript should be
"Allow (default)", Popups should be "Block (default)", and Camera should be
"Ask (default)". Navigating to chrome:settings/content/camera and toggling the
"Ask before accessing (recommended)" option should then make the Site Details
page for for https://permission.site show "Block (default)" next to the Camera
permission.

Bug: 656758
Change-Id: I74cd332c44d8fcb1ab5c700e4fa394ce74fe973b
Test: With the #enable-site-details flag turned on, navigate to
https: //permission.site, open the Page Info bubble, and click "Site settings".
Reviewed-on: https://chromium-review.googlesource.com/558541
Commit-Queue: Patti <patricialor@chromium.org>
Reviewed-by: Dave Schuyler <dschuyler@chromium.org>
Reviewed-by: calamity <calamity@chromium.org>
Reviewed-by: Raymes Khoury <raymes@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488086}

After migrating last ones, description will become mandatory.

BUG=734762

Change-Id: I2b839c5255841fd98a6e0799af928d03f4f0a84d
Reviewed-on: https://chromium-review.googlesource.com/578570
Commit-Queue: Dmitry Gozman <dgozman@chromium.org>
Commit-Queue: Andrey Lushnikov <lushnikov@chromium.org>
Reviewed-by: Andrey Lushnikov <lushnikov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488085}

It is now possible to command-click to select multiple items, and to use
command+up/down/space to navigate through the bookmark list. The existing
shortcut for opening bookmarks with command-down has been removed to
allow for this.

Bug: 745040
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Ic37d271e04625aac540af778cf84528131b1787c
Reviewed-on: https://chromium-review.googlesource.com/575792Reviewed-by: calamity <calamity@chromium.org>
Commit-Queue: Tim Sergeant <tsergeant@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488084}

Bug: 687231
Change-Id: Ic2beee11fd9002b48e1c87265790cedca5e4190d
Reviewed-on: https://chromium-review.googlesource.com/578690Reviewed-by: Joshua Pawlicki <waffles@chromium.org>
Commit-Queue: Sorin Jianu <sorin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488083}

This is a follow-up to https://crrev.com/c/558200,
which replaces corresponding_pull_request_for_commit with
WPTGitHub.pr_for_chromium_commit, so that change ID is used first,
and position is only used as a backup.

This CL also changes MockWPTGitHub and MockChromiumCommit, updates the
tests, and adds an ommitted `return` in pr_for_chromium_commit.

Change-Id: Iae2c3eaef91d6848deee7091d39ac8298e588c66
Reviewed-on: https://chromium-review.googlesource.com/565923
Commit-Queue: Quinten Yearsley <qyearsley@chromium.org>
Reviewed-by: Jeff Carpenter <jeffcarp@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488082}

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/7de63c29970b..c7c5420fbc58

$ git log 7de63c299..c7c5420fb --date=short --no-merges --format='%ad %ae %s'
2017-07-19 simonhatch Dashboard - Add /add_histograms to dispatcher

Created with:
  roll-dep src/third_party/catapult


Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls


CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel
TBR=sullivan@chromium.org

Change-Id: Id7294de5e4ba17dd1b7640f83cd8770b5f13e9f6
Reviewed-on: https://chromium-review.googlesource.com/578709
Reviewed-by: <catapult-deps-roller@chromium.org>
Commit-Queue: <catapult-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488081}

Saves 8904 bytes in resources.arsc

BUG=703134

Change-Id: Id82597710ab1055a083bb354cf0feeeea6f40be6
Reviewed-on: https://chromium-review.googlesource.com/578389Reviewed-by: Ted Choc <tedchoc@chromium.org>
Commit-Queue: Andrew Grieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488080}

Previously, I'd failed to account for an existing time offset when
reversing an animation. This caused errors when reversing an animation
twice.

Bug: None
Change-Id: I0758dff00627897e076687e8125bb3a06de9f74a
Reviewed-on: https://chromium-review.googlesource.com/577923
Commit-Queue: Ian Vollick <vollick@chromium.org>
Reviewed-by: Yash Malik <ymalik@chromium.org>
Reviewed-by: Tibor Goldschwendt <tiborg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488079}

Bug: 740070
Change-Id: I26fa14774e0bc697346033af805dd8bf68e14d0c
Reviewed-on: https://chromium-review.googlesource.com/578730Reviewed-by: Emil A Eklund <eae@chromium.org>
Commit-Queue: Emil A Eklund <eae@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488078}

This CL makes the app build with i18n resources and replaces some
hardcoded texts with the localized strings.

Follow up CLs will be needed to fully localize the app.

Bug: 
Change-Id: I78476ed413e0283b56d7053418ec47138d4d72f5
Reviewed-on: https://chromium-review.googlesource.com/578187Reviewed-by: Scott Nichols <nicholss@chromium.org>
Commit-Queue: Yuwei Huang <yuweih@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488077}

A follow-up to https://codereview.chromium.org/2479333002.

Bug: 641334
Change-Id: If0e92b89422b3f9406d5f8d10fec2d90f83f0e06
Reviewed-on: https://chromium-review.googlesource.com/573500
Commit-Queue: Emil A Eklund <eae@chromium.org>
Reviewed-by: Emil A Eklund <eae@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488076}

This is to address feedback from UX in regard to how app window is
being hidden when closed on the lock screen.

BUG=737067

Change-Id: Iefe768166b5d014bd5e095d7008da04d45547b43
Reviewed-on: https://chromium-review.googlesource.com/578719
Commit-Queue: Toni Barzic <tbarzic@chromium.org>
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488075}

ppapi_cdm_adapter now passes RequestStorageId() calls through the
pp::PlatformVerification API. Also adds a test that ensures the call
responds.

BUG=478960
TEST=new test passes

Change-Id: Ibbdc526fcb31d753f387f0628e96ca889cbb7dc7
Reviewed-on: https://chromium-review.googlesource.com/571339
Commit-Queue: John Rummell <jrummell@chromium.org>
Reviewed-by: Xiaohan Wang <xhwang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488074}

The offset param for File::Write() is int64_t.
However, pwrite() takes a signed integer for offset.
As a result, File::Write() will not work for offset larger than 2GB.
Change to pwrite64() so it will work on 32bit legacy systems.
This issue is not present on Linux because __USE_FILE_OFFSET64 is set.

BUG=739587

Change-Id: I357488b95cd6c501a9f9547ff220f86ef069dc3b
Reviewed-on: https://chromium-review.googlesource.com/578290
Commit-Queue: Min Qin <qinmin@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488073}

This CL avoids a nbsp that was holding space for a paper-input label.
The no-label-float was used to tell paper-input not to reserve the space.

Bug: 746207
Change-Id: Id05d33c74ca9f68b54b26e4afe66a3874161ce93
Reviewed-on: https://chromium-review.googlesource.com/578134Reviewed-by: Hector Carmona <hcarmona@chromium.org>
Commit-Queue: Dave Schuyler <dschuyler@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488072}

This should cause animations when entering and exiting
fullscreen mode.

Bug: None
Change-Id: I348e6283b3060abd98162878c5cb4b566e81cf13
Reviewed-on: https://chromium-review.googlesource.com/575879
Commit-Queue: Ian Vollick <vollick@chromium.org>
Reviewed-by: Christopher Grant <cjgrant@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488071}

Expose local state prefs from the browser using a
InProcessPrefServiceFactory in BrowserProcessImpl and register it as a
embedded service factory in ChromeContentBrowserClient.

Add plumbing of a PrefValueStore::Delegate to PrefServiceFactory to
match PrefServiceSyncableFactory to capture the PrefStores to be
exposed via the local state pref service.

Duplicate the existing pref service manifest as "local_state" to avoid
colliding with the per-profile pref service; each service may only be
packaged in a single service so a distinct service name is required.

Bug: 714855
Change-Id: I31131be6b5c720fa28e83a81febeea9c524c3cfe
Reviewed-on: https://chromium-review.googlesource.com/557580Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Noel Gordon <noel@chromium.org>
Reviewed-by: Bernhard Bauer <bauerb@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Commit-Queue: Sam McNally <sammc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488070}

Disables search by image based on the need for showing the search engine
promo.

BUG= 746695

Change-Id: Ie9732f6bae55605e79a71018812b64a0a973a449
Reviewed-on: https://chromium-review.googlesource.com/578589Reviewed-by: Ted Choc <tedchoc@chromium.org>
Commit-Queue: Yusuf Ozuysal <yusufo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488069}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/b55b2e87..5d7e07ff

Please follow these instructions for assigning/CC'ing issues:
https://github.com/v8/v8/wiki/Triaging%20issues

Please close rolling in case of a roll revert:
https://v8-roll.appspot.com/
This only works with a Google account.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel

TBR=hablich@chromium.org,machenbach@chromium.org,kozyatinskiy@chromium.org

Change-Id: I6cc505b8dea202b6eca9108ce953e47909154f65
Reviewed-on: https://chromium-review.googlesource.com/578495Reviewed-by: v8 autoroll <v8-autoroll@chromium.org>
Commit-Queue: v8 autoroll <v8-autoroll@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488068}

There is currently a WebView test that makes sure WebVR is not available
and that trying to use it does not cause any crashes, etc. However, this
is currently only run on bots that are not configured for VR. While there
shouldn't be a difference in this case, we want to be sure that running
on a VR-configured device yields the same results.

Bug: 732557
Change-Id: Ifdb4626b46550254b378849c79d3945135d996e8
Reviewed-on: https://chromium-review.googlesource.com/575727Reviewed-by: Marc-Antoine Ruel <maruel@chromium.org>
Reviewed-by: Brandon Jones <bajones@chromium.org>
Reviewed-by: Richard Coles <torne@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Commit-Queue: Brian Sheedy <bsheedy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488067}

TBR=achuith@chromium.org

Bug: 746723
Change-Id: I7ffb3d4f519639a5d769a7607fa6f82038367c7c
Reviewed-on: https://chromium-review.googlesource.com/578718
Commit-Queue: Scott Little <sclittle@chromium.org>
Reviewed-by: Scott Little <sclittle@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488066}

https://skia.googlesource.com/skia.git/+log/edfe4a5ca018..7e872caaf618

$ git log edfe4a5ca..7e872caaf --date=short --no-merges --format='%ad %ae %s'
2017-07-19 halcanary SkPDF: SkPDFMakeShader takes the paint color.
2017-07-19 reed removed dead colortable code
2017-07-19 bsalomon Make GrPipeline non-refcounted
2017-07-18 mtklein Tweak HWCAP_... names to avoid clash with hwcap.h
2017-07-19 halcanary Add Alpha Image GM
2017-07-19 egdaniel Fixes from sample count change
2017-07-19 egdaniel Update gpu caps for valid sample counts.
2017-07-19 halcanary SkPDF: more paths as rects

Created with:
  roll-dep src/third_party/skia


Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls


CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
TBR=caryclark@chromium.org

Change-Id: Ib02142fa326b636e4b332720031820e33e30c9e5
Reviewed-on: https://chromium-review.googlesource.com/578138Reviewed-by: Skia Deps Roller <skia-deps-roller@chromium.org>
Commit-Queue: Skia Deps Roller <skia-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488065}

TBR=yfriedman

Bug: 683256
Change-Id: Ieb2a761814449dc4c7e0f374846e676d925d20c6
Reviewed-on: https://chromium-review.googlesource.com/577951
Commit-Queue: Yipeng Wang <yipengw@chromium.org>
Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488064}

The test passes fine on the release build but shows flakiness on
the debug build whose trybot result I'm trying to stablize. It
takes a bit longer to get the test page fully loaded on the debug
build, therefore the test often timed out before proceeding to
the next step. This CL ensures the scale factor gets matched
by waiting for the tab to be created before the test is performed. 
Local tests shows 100% pass, compared to about 50% failures before
the patch.

BUG=737910, 746476

Change-Id: If270b2c5dba122c899cba0f050ca808878773048
Reviewed-on: https://chromium-review.googlesource.com/559330Reviewed-by: Tommy Nyquist <nyquist@chromium.org>
Reviewed-by: Changwan Ryu <changwan@chromium.org>
Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488063}