GitLab

And instead have each tree maintain the mapping between client-id and
ServerWindow.

One of the things the id() encoded was the creator. Numerous places
still need that, so I kept it.

BUG=none
TEST=covered by tests

Change-Id: Ie6fc4833ce20a7c05bbe6ee1139ef361320d24d6
Reviewed-on: https://chromium-review.googlesource.com/936426Reviewed-by: Ria Jiang <riajiang@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539434}

This is to enable experiment-controlled rollout of MediaFoundation
based video capture on platforms where it is supported.

Bug: 792640
Change-Id: I325b3112d25b07ac6eb012bd3e8c59aaea131ff1
Reviewed-on: https://chromium-review.googlesource.com/917284
Commit-Queue: Christian Fremerey <chfremer@chromium.org>
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Reviewed-by: Jesse Doherty <jwd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539433}

Bug: 561800
Change-Id: I4b851241568d86a10475585130780ad73ebcf4ce
Reviewed-on: https://chromium-review.googlesource.com/934361Reviewed-by: srirama chandra sekhar <srirama.m@samsung.com>
Reviewed-by: Robert Flack <flackr@chromium.org>
Commit-Queue: srirama chandra sekhar <srirama.m@samsung.com>
Cr-Commit-Position: refs/heads/master@{#539432}

This CL lands server change 185528812 by dahollings.

BUG=488484

Change-Id: I265c7d8354b5258fbbf7d39cb446d3dda4f8c25f
Reviewed-on: https://chromium-review.googlesource.com/937687Reviewed-by: Bence Béky <bnc@chromium.org>
Commit-Queue: Dianna Hu <diannahu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539431}

This CL lands server change 185561279 by assar.

BUG=488484

Change-Id: I19a1525a9bfb6ec6b7f01ac163b9044f98f4fbd9
Reviewed-on: https://chromium-review.googlesource.com/937690Reviewed-by: Bence Béky <bnc@chromium.org>
Commit-Queue: Dianna Hu <diannahu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539430}

TBR=zakerinasab@chromium.org

No-Try: True
Bug: 816914
Change-Id: Ib62c7af7241350e80c18412b2c898734fe7b9074
Reviewed-on: https://chromium-review.googlesource.com/939470Reviewed-by: Henrik Boström <hbos@chromium.org>
Commit-Queue: Henrik Boström <hbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539429}

ImageBitmapRenderingContext is supposed to be shipped in Chrome 55 as stated
in the Chrome Platform Status here:
https://www.chromestatus.com/feature/5709799995998208. However, the code itself
bars this feature from being accessible by users.


Bug: 658734
Change-Id: I7b54bbee58529507411109866885623ef5b7a999
Reviewed-on: https://chromium-review.googlesource.com/938461Reviewed-by: Justin Novosad <junov@chromium.org>
Commit-Queue: Olivia Lai <xlai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539428}

This CL may cause a small binary size increase, roughly proportional
to how long it's been since our last AFDO profile roll. For larger
increases (around or exceeding 100KB), please file a bug against
gbiv@chromium.org. Additional context: https://crbug.com/805539

The AutoRoll server is located here: https://afdo-chromium-roll.skia.org

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

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=gbiv@chromium.org

Change-Id: I98b7c3aef6dfc64ba9ddef02305208867ad2ab60
Reviewed-on: https://chromium-review.googlesource.com/938575Reviewed-by: afdo-chromium-autoroll <afdo-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: afdo-chromium-autoroll <afdo-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#539427}

Recently I made some fileds deprecated (crrev.com/c/917845),
in addition, we need to make them optional to remove these fields.

Test None (doesn't change any behavior)

Bug: None
Change-Id: Ic1afc986013fb3f395fd1f1e89f19a9c13852b7a
Reviewed-on: https://chromium-review.googlesource.com/936948Reviewed-by: Jorge Lucangeli Obes <jorgelo@chromium.org>
Commit-Queue: Yoshiki Iguchi <yoshiki@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539426}

Bug: 816435
Change-Id: Iae8fd393c8fae53387ecf02d5e5779a44dee3030
Reviewed-on: https://chromium-review.googlesource.com/936588
Commit-Queue: Nico Weber <thakis@chromium.org>
Reviewed-by: Scott Graham <scottmg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539425}

There are multiple g_instance_for_testing globals after
https://chromium-review.googlesource.com/c/chromium/src/+/935841 - let's
rename one of them to avoid jumbo build breakage.

TBR=primiano@chromium.org

Bug: 815234,775547
Change-Id: I184578b694e59dad8b06643db199f32820fd5887
Reviewed-on: https://chromium-review.googlesource.com/938821
Commit-Queue: Mostyn Bramley-Moore <mostynb@vewd.com>
Reviewed-by: Mostyn Bramley-Moore <mostynb@vewd.com>
Cr-Commit-Position: refs/heads/master@{#539424}

It's also flaky on Android, so disable on all platforms

Bug: chromium:807638
No-Try: True
Change-Id: I0b7dc9cb14198c590538f3a8125e47636fc2c1d7
Reviewed-on: https://chromium-review.googlesource.com/939162
Commit-Queue: Oleh Prypin <oprypin@chromium.org>
Reviewed-by: Patrik Höglund <phoglund@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539423}

Bug: 816499
Change-Id: Ib3ecf6730908289be6f799c7df39d3667f1552d0
Reviewed-on: https://chromium-review.googlesource.com/937657Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Commit-Queue: Owen Min <zmin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539422}

This reverts commit 637c7485.

Reason for revert: Suspected culprit of consistent Linux MSan Tests failures of GpuImageDecodeCacheTest.NonLazyImageUploadDownscaled since
https://uberchromegw.corp.google.com/i/chromium.memory/builders/Linux%20MSan%20Tests/builds/8195

Original change's description:
> cc: Ensure correct scaling for non-lazy images in GPU cache.
> 
> When using a non-lazy (bitmap) image in the GPU cache, we assume the
> upload to be scaled at the mip-level required for draw, when we
> actually use the original image for the upload. Fix this by considering
> non-lazy scaled images as lazy generated. This also ensures we use
> lower gpu memory, by caching downscaled uploads.
> 
> R=​ericrk@chromium.org
> 
> Bug: 815045, 814219
> Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_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
> Change-Id: I4b7d3f4edbcf0e5ba5574512f662a3fdf53cabd6
> Reviewed-on: https://chromium-review.googlesource.com/935441
> Reviewed-by: Zhenyao Mo <zmo@chromium.org>
> Reviewed-by: Eric Karl <ericrk@chromium.org>
> Commit-Queue: Khushal <khushalsagar@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#539276}

TBR=zmo@chromium.org,khushalsagar@chromium.org,ericrk@chromium.org

Change-Id: I441305f1ebfa962b3a78e5fa53f2169996fae196
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 815045, 814219
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_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
Reviewed-on: https://chromium-review.googlesource.com/939361Reviewed-by: Henrik Boström <hbos@chromium.org>
Commit-Queue: Henrik Boström <hbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539421}

TBR=ericwilligers@chromium.org

No-Try: True
Bug: 816868
Change-Id: I0ccb2fa2ad89e2dfcbfb8e15e2dfb8fbc1e8e85b
Reviewed-on: https://chromium-review.googlesource.com/939176Reviewed-by: Henrik Boström <hbos@chromium.org>
Commit-Queue: Henrik Boström <hbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539420}

This reverts commit d690c638.

Reason for revert: Suspect for Linux MSAN bot consistent failures, first one being:
https://uberchromegw.corp.google.com/i/chromium.webkit/builders/WebKit%20Linux%20Trusty%20MSAN/builds/6338

Original change's description:
> [heap-profiler] Merge DOM node and its JS wrapper node in heap snapshot.
> 
> Each DOM node has a corresponding JS wrapper node. In heap snapshot
> they appear as duplicates.
> 
> Example retaining path with duplicates:
> - [1] in InternalNode @2041178784
> - [1] in HTMLDivElement @2041079168  // DOM node
> - [3] in HTMLDivElement @2231        // JS wrapper
> - retainer in Window / @2105
> 
> The heap snapshot generator now can merge an embedder node with a node
> returned by EmbedderGraph::Node::WrapperNode() function.
> 
> This patch implements the WrapperNode() function for each DOM node.
> 
> Bug: chromium:811925
> Change-Id: I9ea6bc7e45f8ab3d54828b9ac61f7230d04d8019
> Reviewed-on: https://chromium-review.googlesource.com/928503
> Reviewed-by: Michael Lippautz <mlippautz@chromium.org>
> Commit-Queue: Ulan Degenbaev <ulan@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#539132}

TBR=ulan@chromium.org,haraken@chromium.org,mlippautz@chromium.org

Change-Id: If05b23d13bb68f0dc870cc047b399ed43c4472f6
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: chromium:811925
Reviewed-on: https://chromium-review.googlesource.com/939341Reviewed-by: Henrik Boström <hbos@chromium.org>
Commit-Queue: Henrik Boström <hbos@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539419}

Bug: 734095
Change-Id: Ia29c55079a1fa0be50ef2053636b63288c0d92d6
Reviewed-on: https://chromium-review.googlesource.com/938656Reviewed-by: Finnur Thorarinsson <finnur@chromium.org>
Commit-Queue: Finnur Thorarinsson <finnur@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539418}

https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/6f83031f3d0e..9b4d0c77152a

$ git log 6f83031f3..9b4d0c771 --date=short --no-merges --format='%ad %ae %s'
2018-02-26 machenbach Gerrit: Enable overriding step_test_data

Created with:
  roll-dep src/third_party/depot_tools


The AutoRoll server is located here: https://depot-tools-chromium-roll.skia.org

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

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=agable@chromium.org

Change-Id: I382596c11e2071152e364f82e7a9be49f1c410d5
Reviewed-on: https://chromium-review.googlesource.com/938975Reviewed-by: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#539417}

This CL adjusts the accounts submenu positioning
relative to the anchor and sets a fixed size
according to the mocks under:
https://gallery.googleplex.com/projects/MCHbtQVoQ2HCZWnUS9sdsb0Q/files/MCHtA7U1iMGr6-WzjO3kn_RsRHBgUr2ucD8

Screenshots:
https://drive.google.com/file/d/1ID5ydvPqe2Ivkyg8nxJz3Ckb5nQQtKJD/view?usp=sharing
https://drive.google.com/file/d/1-4d6qWLvfJ1-_mFZZEjO6FZBXzizY-p_/view?usp=sharing
Screenshot in RTL mode:
https://drive.google.com/file/d/1yrQdgq9ANVFV1K-AfdzSpe4b90esRJGx/view?usp=sharing

Bug: 814809
Change-Id: Id36f2443b6a7cc68a026006273daf96c659ebe71
Reviewed-on: https://chromium-review.googlesource.com/934247Reviewed-by: Trent Apted <tapted@chromium.org>
Commit-Queue: Thomas Tangl <tangltom@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539416}

Some recent breaking changes have caused the icall bot to be red for
extended periods of time, make it follow ToT so it gets fixes more
quickly.

Change-Id: Ie927fddaf8fb2ac25f60fdf6c59c02a47be4852c
Reviewed-on: https://chromium-review.googlesource.com/938484Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Peter Collingbourne <pcc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539415}

R=nednguyen@google.com

Bug: 792495, 784025
Change-Id: I05345c81f31b30418eb0bf76da1e2c52f0dc0f54
Reviewed-on: https://chromium-review.googlesource.com/937519Reviewed-by: Ned Nguyen <nednguyen@google.com>
Commit-Queue: Michael Hablich <hablich@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539414}

Introduce a new protocol BrowsingDataCommands that can be
implemented by the dispatcher. Modify code using the old
command ClearBrowsingDataCommand to instead use this new
protocol as a dispatcher.

Bug: 738881
Change-Id: I76b8650c6c9d4311a611980dc26c8850c4960d5b
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Reviewed-on: https://chromium-review.googlesource.com/934501
Commit-Queue: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: Olivier Robin <olivierrobin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539413}

This cl removes the deprecated CR_DEFINE_STATIC_LOCAL in Keyboard
Shortcut Viewer (KSV). It is now using NoDestructor.

Bug: 768932
Test: Passed all unittests.
Change-Id: I4872fe4cb6994592e93ed907d621194f99ad12a8
Reviewed-on: https://chromium-review.googlesource.com/938664Reviewed-by: Ahmed Fakhry <afakhry@chromium.org>
Commit-Queue: Tao Wu <wutao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539412}

This CL removes additional wait after timeout exception in CT leak detector.

Change-Id: I1c7ce71f96803860da9dbe92ac09c8e94098fd89
Reviewed-on: https://chromium-review.googlesource.com/936926
Commit-Queue: Yuzu Saijo <yuzus@chromium.org>
Reviewed-by: Keishi Hattori <keishi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539411}

This is part of efforts to replace base::ThreadTaskRunnerHandle::Get()
and SequencedTaskRunnerHandle::Get() with other appropriate task runners
in the renderer.

This avoids the SequencedTaskRunnerHandle::Get() call when the constructor's
default parameter is used. We are trying to ban SequencedTaskRunnerHandle::Get()
in content/renderer because it usually is not the right task runner to use,
as a per-frame runner should be used instead. In this case, it doesn't
really matter since frames are not involved, but we still want to remove the
calls for the bigger benefit of banning it in content/renderer.

Bug: 780785, 786332
Change-Id: I307cd4923912809b0577074585ea9fd953a84ef3
Reviewed-on: https://chromium-review.googlesource.com/937104
Commit-Queue: Hajime Hoshi <hajimehoshi@chromium.org>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Alexander Timin <altimin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539410}

TBR=chrome-os-gardeners@google.com

Change-Id: I1e9d00de8034299cfade38e3ab1ce4848c62000c
Reviewed-on: https://chromium-review.googlesource.com/938777Reviewed-by: ChromeOS Commit Bot <chromeos-commit-bot@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539409}

This CL removes reddit from leak detection url set because it has been flakily failing because of time-outs.

Change-Id: If845aecb5414200fcc0aae75be9a523c65e42680
Reviewed-on: https://chromium-review.googlesource.com/936826Reviewed-by: Keishi Hattori <keishi@chromium.org>
Commit-Queue: Yuzu Saijo <yuzus@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539408}

Makes callback interface's operations support "callback this value".

Also supports |InvokeAndReportException| member function as same as
callback function does.  The member function is defined only when
there is a single operation (without any overload) and its return
type is |void| (otherwise, the call sites must check the return value).

Bug: 778580
Change-Id: I9e9b1e6eaf53d45356d7374963c14c321378af00
Reviewed-on: https://chromium-review.googlesource.com/936930Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Hitoshi Yoshida <peria@chromium.org>
Commit-Queue: Yuki Shiino <yukishiino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539407}

To clarify what the requestor origin in Blink is, let me add some
comments in FrameFetchContext.

Change-Id: I32e8b45dc5189e773a02466960cce7e9c45292b9
NOTRY: true
Reviewed-on: https://chromium-review.googlesource.com/937042Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Commit-Queue: Takashi Toyoshima <toyoshim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539406}

https://pdfium.googlesource.com/pdfium.git/+log/0bc02c152bd0..821c7236cea9

$ git log 0bc02c152..821c7236c --date=short --no-merges --format='%ad %ae %s'
2018-02-26 tsepez Shrink some tables and kill dead code in xfa.
2018-02-26 tsepez Get rid of some static variables.

Created with:
  roll-dep src/third_party/pdfium


The AutoRoll server is located here: https://pdfium-roll.skia.org

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

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=dsinclair@chromium.org

Change-Id: Iaf3a18e1bc49d87c7c53a77a62345b6beae05af0
Reviewed-on: https://chromium-review.googlesource.com/938630Reviewed-by: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#539405}

Bug: 933701
Change-Id: I584d363a3f3fe88dcaf68291105bf73bd8d02de9
Reviewed-on: https://chromium-review.googlesource.com/934630Reviewed-by: Yaron Friedman <yfriedman@chromium.org>
Reviewed-by: Nick Carter <nick@chromium.org>
Commit-Queue: Yash Malik <ymalik@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539404}

Following bot results are included.
3943 3944 3947 3948 3953 3955 3958 3961 3964

1 lines were removed and 26 lines were deflaked by consecutive results
since 3901.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: I25713f3ae0713ad284c455eb10b67e808ef4e1e7
Reviewed-on: https://chromium-review.googlesource.com/936885
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539403}

The Blob system can allocate a lot of memory, so we should integrate
with MemoryDumpProvider infra to account for this usage.

Bug: 813046
Change-Id: Ifabb469c36afb68df3031fe6660a76a97274950b
Reviewed-on: https://chromium-review.googlesource.com/938761Reviewed-by: Xing Liu <xingliu@chromium.org>
Reviewed-by: Erik Chen <erikchen@chromium.org>
Commit-Queue: Marijn Kruisselbrink <mek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539402}

This reverts commit 0a9a5c31.

Reason for revert: We decided to delay the launch of SlimmingPaintV175
to M67, so that we have more time to fix the regressions.

The revert CL is slightly modified so that later enabling/disabling of
SlimmingPaintV175 by default can just change one line in
content/public/common/content_features.cc (learnt from
RootLayerScrolling crrev/c/929662).

Original change's description:
> [SPv175] Enable SlimmingPaintV175 by default
>
> SlimmingPaintV175 enables raster invalidation and paint chunks based
> on paint properties. See crbug.com/771643.
>
> Bug: 771643
> Change-Id: I5fdbcabc7ca0f2fa8e69ae0fbf5a0077bba46cfe
> Reviewed-on: https://chromium-review.googlesource.com/923572
> Reviewed-by: Antoine Labour <piman@chromium.org>
> Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#537454}

TBR=dgozman@chromium.org,wangxianzhu@chromium.org,chrishtr@chromium.org,piman@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 771643
Change-Id: I04177ada5d6b133fe47c0e0dbff42b6989f099ba
Reviewed-on: https://chromium-review.googlesource.com/938562Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539401}

This is revert of https://crrev.com/c/892187 and change config to use no_symbols explicitly for linux_chromium_rel_ng builder.

Bug: 794423
Change-Id: I557b114292fd79e959140913f3077fd735dbdb5f
Reviewed-on: https://chromium-review.googlesource.com/938883Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Commit-Queue: Takuto Ikuta <tikuta@google.com>
Cr-Commit-Position: refs/heads/master@{#539400}

    icon size to 20x20, align text

Bug: 794275
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Ib7bdaf43dce5074c4b758d27ae47afb54b1d2750
Reviewed-on: https://chromium-review.googlesource.com/937961Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Commit-Queue: Esmael El-Moslimany <aee@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539399}

# This is an automated release commit.
# Do not revert without consulting chrome-pmo@google.com.
NOAUTOREVERT=true
TBR=govind@chromium.org

Change-Id: I72195ce2a1caad79fa2e06ec80c0b5a025939301
Reviewed-on: https://chromium-review.googlesource.com/938774Reviewed-by: chrome-release-bot@chromium.org <chrome-release-bot@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539398}

Overlay content view height should have taken the toolbar
into account to show the content. Incorrect height caused
the reported bug not showing the bottom part of the content.
Note that physical backing size doesn't need this but
should always be set to the full content view size.

Bug: 815052
Change-Id: Ia5cd8f24f72e507c0537c586b7b1c8244ca51a3c
Reviewed-on: https://chromium-review.googlesource.com/936823Reviewed-by: Matthew Jones <mdjones@chromium.org>
Reviewed-by: Donn Denman <donnd@chromium.org>
Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539397}

Update ash::TabletModeController to report a new histogram
containing the computed lid angle once per hour. (Note that
the computed angle is often inaccurate by tens of degrees.)

Also change "tabletmode" in a few members to "tablet_mode"
(this looks like an earlier search-and-replace bug) and
initialize POD members in the header.

Bug: 814363
Change-Id: I5066fb6e859a34343506f88453845156186b81b3
Reviewed-on: https://chromium-review.googlesource.com/933732Reviewed-by: Mark Pearson <mpearson@chromium.org>
Reviewed-by: Qiang Xu <warx@google.com>
Commit-Queue: Dan Erat <derat@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539396}

This CL changes the ownership of base::Clock and base::TickClock from
injectee-owned to injecter-owned. Before this CL, these instances are
owned by the owner of the injectee or one of the injectees themselves.
That makes the ownership handling complex.

After this CL, the injectee of clocks never own the clock. Instead,
injecters owns a clock for testing, and a shared clock is used on the
production code.

Bug: 789079
Change-Id: I992b13352a6fa1028bd8eeb2223056cc2cecf610
Reviewed-on: https://chromium-review.googlesource.com/900882Reviewed-by: Varun Khaneja <vakh@chromium.org>
Commit-Queue: Taiju Tsuiki <tzik@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539395}