GitLab

Some of third_party/WebKit/Source/webdatabase uses sqlite withq
internals exposed and some uses it with internals not exposed. That
mismatch causes issues in jumbo builds since it compiles several files
together and need headers to say the same thing.

This changes webdatabase so that it all compile with the
CHROMIUM_SQLITE_INTERNALS define set.

R=michaeln@chromium.org

Bug: 740542
Change-Id: I0b5c66fde7d191afd7a7143ef32c9c6f1b21b9e2
Reviewed-on: https://chromium-review.googlesource.com/565502Reviewed-by: Michael Nordman <michaeln@chromium.org>
Commit-Queue: bratell at Opera <bratell@opera.com>
Cr-Commit-Position: refs/heads/master@{#485558}

Both Profile and Browser are used in code.

BUG=

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

Patch r483065 (codereview.chromium.org/2958103002/) used the wrong
function to convert the size of a root window to the desired units.
It used ConvertDIPToScreenInPixels, which won't always do what we want
depending on the relative position of the displays. Instead we should
just multiply by the device scale factor.

BUG=723147

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

The PNG fuzzer included all of PNGImageDecoder.cpp and effectively
compiled it a second time. Since the fuzzer already depends on the
png encoder it will just be linked together anyway.

This double compilation caused jumbo errors because symbols appeared
twice.

Change-Id: I0ef1502ba90247e34c33055056bdeb808e0dbb89
Reviewed-on: https://chromium-review.googlesource.com/565411Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: bratell at Opera <bratell@opera.com>
Cr-Commit-Position: refs/heads/master@{#485555}

This reverts commit 7fa78bb0.

Reason for revert: Crashing browser process on null ptr dereference.

Original change's description:
> Add memory UKMs for existing UMA metrics.
> 
> Bug: 730783, 728707
> Change-Id: I744c73d36ca76d97eff1e202fa20ba5d4a9adb7b
> Reviewed-on: https://chromium-review.googlesource.com/531804
> Commit-Queue: Jao-ke Chin-Lee <jchinlee@chromium.org>
> Reviewed-by: Robert Kaplow <rkaplow@chromium.org>
> Reviewed-by: Erik Chen <erikchen@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#485159}

TBR=rkaplow@chromium.org,erikchen@chromium.org,jchinlee@chromium.org

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

Bug: 730783, 728707
Change-Id: I0993305f4eb65343faf6678054de0d1c2d1fcddb
Reviewed-on: https://chromium-review.googlesource.com/566518Reviewed-by: Tomasz Mikolajewski <mtomasz@chromium.org>
Reviewed-by: Yusuke Sato <yusukes@chromium.org>
Commit-Queue: Tomasz Mikolajewski <mtomasz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485554}

image-rendering's camel-case keyword does not follow normal
convention because it comes from a different format.
https://developer.mozilla.org/en/docs/Web/CSS/image-rendering#Specifications
text-align's webkit keyword is a webkit prefix keywords, if we can decide
we should follow the convention of CSS property to be consistent.

Bug: 
Change-Id: I0fea055fe67ea521ad0456ef120ba91adf273267
Reviewed-on: https://chromium-review.googlesource.com/566142
Commit-Queue: Minh-Duc Nguyen <nmduc@google.com>
Reviewed-by: Darren Shen <shend@chromium.org>
Reviewed-by: nainar <nainar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485553}

This patch introduces a field trial for the ResourceLoadScheduler
and makes the feature configurable through chrome://flags and
field trial parameters.

Bug: 729953
Change-Id: Id8619ad8979669748e94a9988c0a46ef6a318726
Reviewed-on: https://chromium-review.googlesource.com/558902
Commit-Queue: Takashi Toyoshima <toyoshim@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485552}

DOMTreeMutationDetector missed to find unexpected DOM tree mutation if |parent|
node is adopted to another Document during a watched operation. This CL fixes it.

Bug: 740410
Change-Id: I7b4d2c4c525e10e0d6b19229c27b5d1ce077a271
Reviewed-on: https://chromium-review.googlesource.com/566267Reviewed-by: Hayato Ito <hayato@chromium.org>
Commit-Queue: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485551}

This CL adds a CreateVideoEncodeAccelerator() methods to the gpu.mojom and
gpu_service.mojom files (renderer-to-browser interface and browser-to-gpu
interfaces, resp.), and wires the new mojom::VideoEncodeAcceleratorRequest
pass through.

This is step 0 of VEA mojification, which is currently IPC-based. More
info can be found in the DD https://goo.gl/PbTKnt, concretely this
CL can be better understood in the context of the first diagram in the
document, also here, for conveniency: https://goo.gl/HkwYvA.

A playground with most of the changes pre-review can be found in 
https://crrev.com/c/558846.

Bug: 736517
Change-Id: Id1f6d60d487d97ce3f304bead73738b2ba531408
Reviewed-on: https://chromium-review.googlesource.com/558833
Commit-Queue: Miguel Casas <mcasas@chromium.org>
Reviewed-by: Antoine Labour <piman@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Dan Sanders <sandersd@chromium.org>
Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485550}

BUG=None

Change-Id: I8c2963866273a92e2b5b6e7ea80ca176d761a1b6
Reviewed-on: https://chromium-review.googlesource.com/566214Reviewed-by: Timothy Loh <timloh@chromium.org>
Commit-Queue: Dominick Ng <dominickn@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485549}

This macro requires an undefined-but-common behavior of C preprocessor,
and is not used widely.

We can simply replace USE(FOO) with defined(WTF_USE_FOO).
WTF_USE_FOO macros are usually defined in BUILD.gn.

As for WTF_USE_APPLE_SYSTEM_LOG, we don't need this macro because
we support only macOS 10.9+.

Bug: 667238
Change-Id: I55c6b952bc9d77744ccd3e9b0a241e3093b10c46
Reviewed-on: https://chromium-review.googlesource.com/566343Reviewed-by: Yuta Kitamura <yutak@chromium.org>
Commit-Queue: Yuta Kitamura <yutak@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485548}

This CL augments the existing ServiceManager embedding on iOS with
support for per-user embedded services. A "per-user" service is one
that has one instance per user, rather than a shared instance for all
users. Concretely, each BrowserState has a unique UserID, and there is
one instance of a per-user service per UserID. Essentially, one can
think of per-user services as the equivalent of profile-keyed services
in the services world.

The support added in this CL closely follows that of //content:

- Public APIs are identical
- Implementation in BrowserState follows that of BrowserContext with minor
  changes:
  - iOS doesn't use File Service at this time
  - No client of //ios/web needs to override how a user ID is generated

This CL adds an Earl Grey test of web_shell that a per-user service can be
embedded and if connected to from a given BrowserState, has its user ID
properly associated with that of the BrowserState.

Finally, this added support requires that all BrowserState subclasses call
BrowserState::Initialize(). This is also done in this CL.

Bug: 731588
Change-Id: Ie3bb79d4f754aecddd9c35c72576262dfc87cb55
Reviewed-on: https://chromium-review.googlesource.com/559529Reviewed-by: Ken Rockot <rockot@chromium.org>
Reviewed-by: Eugene But <eugenebut@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Commit-Queue: Colin Blundell <blundell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485547}

This malformed summary description causes continuous presubmit
warnings for changes touching tools/metrics.

TBR=isherman@chromium.org

Change-Id: I3b745fb55f108b3a741046ac8809b6f421d15a0f
Reviewed-on: https://chromium-review.googlesource.com/566165Reviewed-by: Takashi Toyoshima <toyoshim@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Commit-Queue: Takashi Toyoshima <toyoshim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485546}

This is preparation to remove MockCryptohomeClient,
as prepration to replace Callback by OnceCallback in chromeos/dbus.

BUG=739622
TEST=Ran trybot.

Change-Id: I51397108a365d8b79b8de8a48263d7b62ef0182b
Reviewed-on: https://chromium-review.googlesource.com/563153Reviewed-by: Ryo Hashimoto <hashimoto@chromium.org>
Reviewed-by: Bartosz Fabianowski <bartfab@chromium.org>
Commit-Queue: Hidehiko Abe <hidehiko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485545}

Also fixes the known issues with enabling it:

- crbug/665787: The tab background view was added in front of the content and
  blocked interaction with the top of the scroll bar when the toolbar was
  hidden. It's now inserted behind the content, like before.

- crbug/655112: FramedBrowserWindowTest.DoesHideTitle took screenshots of the
  frame view, which gets drawn using a different (layer-backed?) path when
  NSFullSizeContentViewWindowMask is on. The test now calls -[CATransaction
  flush] before taking each screenshot, and uses a different API to screenshot
  the whole window instead of trying to grab just the frame view by accessing
  the content view's superview.

Bug: 605219, 655112, 665787, 730679, 734713, 737206
Change-Id: Id78c21aff6748e02fd4b3e461c77ef9f7454d744
Reviewed-on: https://chromium-review.googlesource.com/566030
Commit-Queue: Sidney San Martin <sdy@chromium.org>
Reviewed-by: Trent Apted <tapted@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485544}

Context reporting records the URL, title, and selection to Icing.
If the context is the same as last time, skip it to avoid useless
API calls to Icing.

This CL lowers the reporting per navigation to ~1.

Bug: 739570
Change-Id: Id8d538c02f08e51ebdc729190a683856832b3a2e
Reviewed-on: https://chromium-review.googlesource.com/562546
Commit-Queue: Wei-Yin Chen (陳威尹) <wychen@chromium.org>
Reviewed-by: Ted Choc <tedchoc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485543}

The typedef was added back in 2014 in commit 38b7b40a ("Improve the
declared IDL type for postMessage()'s transferables"), when the HTML Living
Standard defined a Transferable typedef like this:

    typedef (ArrayBuffer or CanvasProxy or MessagePort) Transferable;

since the bindings layer did not support unions at the time and all uses of
Transferable in our IDLs amounted to MessagePort objects, the current
"typedef MessagePort Transferable" was introduced for compliance.

https://github.com/whatwg/html/commit/bfb960c938580c95e77365e614218b952f96375b ("Write
structured clone algorithm in terms of ECMAScript") got rid of the typedef
in the spec altogether though, so we were again not compliant in our IDLs.

This CL makes the switch by using object instead of Transferable in the
IDLs.

In practice, this does not make any difference, as |transfer| is always
parsed via SerializedScriptValue::ExtractTransferables(), which does all the
argument parsing on its own.

While here, also make a few other IDL updates that do not change the
generated bindings code, in some cases due to the fact that the
postMessage() bindings code is more customized than usual:
- Add a "= []" to the "optional sequence<object>" postMessage() arguments.
- Change |message|'s type from "SerializedScriptValue" to "any" in some
  postMessage() declarations to match the spec(s).
- Update the spec link in ServiceWorker.idl.

Bug: 701482
Change-Id: I0a5c0dc7fed8f3498ccd072b4422fb269e5def97
Reviewed-on: https://chromium-review.googlesource.com/561459Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Reviewed-by: Joshua Bell <jsbell@chromium.org>
Reviewed-by: Hiroki Nakagawa <nhiroki@chromium.org>
Commit-Queue: Raphael Kubo da Costa (rakuco) <raphael.kubo.da.costa@intel.com>
Cr-Commit-Position: refs/heads/master@{#485542}

Since OutlineValue has been deleted, whether an outline is auto or not
is now stored in a separate field. This means we no longer need
EBorderStyle to store it. This patch removes the unused auto value
in EBorderStyle.

Change-Id: I3d698a65f6b0a8a1e2b4bc7cb2e1279e410a7461
Reviewed-on: https://chromium-review.googlesource.com/560807
Commit-Queue: Darren Shen <shend@chromium.org>
Reviewed-by: nainar <nainar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485541}

This patch moves SVG-related tests from the top level of the
LayoutTests/animations directory to a dedicated subdirectory. Existing
SVG-related subdirectories are untouched.

BUG=623434

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

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/10d68afb..0cd46302

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: Ia0ed92cbb5c506b3cad318cc266d1df4312038bd
Reviewed-on: https://chromium-review.googlesource.com/566331Reviewed-by: v8 autoroll <v8-autoroll@chromium.org>
Commit-Queue: v8 autoroll <v8-autoroll@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485539}

This patch converts |TextGranularity| to enum class as a preparation of
moving "TextGranularity.h" from header file to cpp file for reducing build
time.

Change-Id: I489b573e6717c44de55367e528f492fe52e11e85
Reviewed-on: https://chromium-review.googlesource.com/563138Reviewed-by: Kent Tamura <tkent@chromium.org>
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485538}

Unbind and rebind if onServiceConnection does not come after a timeout,
currently set to 10s. Add a unit test, but do not enable this behavior
in production yet.

The current plan is then to add some UMA metrics and evaluate result
to decide whether it's worth turning this on or not.

BUG=736066

Change-Id: Id06b11005c781932810b32e11592727f580ef38c
Reviewed-on: https://chromium-review.googlesource.com/562653
Commit-Queue: Bo Liu <boliu@chromium.org>
Reviewed-by: Jay Civelli <jcivelli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485537}

This patch makes a number of small rearrangements in the
LayoutTests/animations directory:
- Enables a disabled but passing test (font-size-using-ems)
- Deletes the under-used script-tests directory (inlining the only use,
  animation-css-rule-types)
- Moves animate-shorthand-var to the custom-properties test subdirectory
- Moves four tests to a new hit-testing subdirectory
- Renames a test that refers to 'players' instead of 'animations'
  (get-css-players -> getAnimations-css)

BUG=623434

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

Change-Id: I5455b94c44f8a896879723a3a7630386edbee807
Reviewed-on: https://chromium-review.googlesource.com/565258Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: Colin Blundell <blundell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485535}

MTP related fixes caused regression on the behavior when multiple files
selected are double-clicked.
These MTP related fixes were reverted in:
* https://codereview.chromium.org/2973003002/
* https://codereview.chromium.org/2966163005/

This CL restores these reverted CLs while fixes double click regression.

BUG=462989,712121,731483,738803
TEST=
unit: out/Debug/browser_tests '--gtest_filter=FileManagerJsTest.TaskController'
* Verified it fails with the original patches.
manual:
* Transfer files on MTP while selecting many source files.
* Transfer files on MTP while opening context menu.
* Double click multiple selected files.

Change-Id: If61e5f306f821d0bf096c068b01866ce1cfd38c9
Reviewed-on: https://chromium-review.googlesource.com/562912Reviewed-by: Naoki Fukino <fukino@chromium.org>
Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485534}

Also:
- Make the crash tests for CSSTransformValue parameterised instead
of repeating the same code for each test.
- Make ToCSSValue return nullptr if the CSSPerspective contains a
negative length to avoid later DCHECK, and add a test for this.

BUG=545318

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

This will initialize loading the keyboard extension when the RootWindowController calls ActivateKeyboard.

Bug: 724908
Change-Id: I3af07338dd3e619f9ae6d85e0f191890d0cd5a67
Reviewed-on: https://chromium-review.googlesource.com/547921
Commit-Queue: Blake O'Hare <blakeo@chromium.org>
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Reviewed-by: Yuichiro Hanada <yhanada@chromium.org>
Reviewed-by: Keigo Oka <oka@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485532}

This test has been stable on FYI, and having it on the CQ will
help catch regressions sooner.

Bug: 638141
Change-Id: Ie318f62993402a0f1b89f4aff47c821a0310722f
Reviewed-on: https://chromium-review.googlesource.com/557861Reviewed-by: Eugene But <eugenebut@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Mike Baxley <baxley@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485531}

This CL does the following:

1. Move SharedBitmap to components/viz/common/quads
2. Move SharedBitmapManager to components/viz/common/resources

All other changes are mechanical updates to includes, namespace, etc.

TBR=junov@chromium.org

Bug: 722935
Change-Id: Iefcc37c250275b577153efa56f0cf6e78314ac94
Reviewed-on: https://chromium-review.googlesource.com/565841
Commit-Queue: Fady Samuel <fsamuel@chromium.org>
Reviewed-by: Fady Samuel <fsamuel@chromium.org>
Reviewed-by: Antoine Labour <piman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485530}

|XcodeWriter::CreateProductsProject| in xcode_writer.cc has evolved
into a giant function, so this CL makes it smaller by refactoring the
logic that adds source files to the project for indexing into a
separate function.

Bug: 709289
Change-Id: I19fab218e1af87a87cacf8f65f19c2acf8091713
Reviewed-on: https://chromium-review.googlesource.com/563650Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Commit-Queue: Yuke Liao <liaoyuke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485529}

BUG=740801
TBR=bauerb@chromium.org

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

This patch moves animations tests that are specific to the Web
Animations API from LayoutTests/animations to a dedicated subdirectory.

BUG=623434

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

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/b708a0d15934..fc9479ad5e3b

$ git log b708a0d15..fc9479ad5 --date=short --no-merges --format='%ad %ae %s'
2017-07-10 benjhayden Add related names GenericSet diagnostic.
2017-07-10 martiniss tracing: Fix subtle python bug in job.py
2017-07-10 simonhatch Dashboard - Allow logged in users for oauth.Authorize decorator.
2017-07-10 eakuefner [Tracing] Fix bug with Python dependency handling

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: Ie1a0d743fd67849a2c479ce68b92ce3600fe6e00
Reviewed-on: https://chromium-review.googlesource.com/566190
Reviewed-by: <catapult-deps-roller@chromium.org>
Commit-Queue: <catapult-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485526}

This patch:
- Added CSSPrpertyTextDecorationLineUtils and moved common function
  ConsumeTextDecorationLine from CSSPropertyParser into it.
- Implemented CSSPropertyAPITextDecorationLine::parseSingleValue,
  moving logic for properties CSSPropertyTextDecorationLine and
  CSSPropertyWebkitTextDecorationsInEffect from
  CSSPropertyParser::ParseSingleValue legacy switch statement
- Added details of API implementation for these properties to
  CSSProperties.json5 for generated files.

Diff for generated files: https://gist.github.com/BugsNash/1f4cae843ee5843e43b46ed3eebf0de6/revisions

Bug: 668012
Change-Id: I5f1d5f6f38d0fee61a6e3116f88e8ea78c7e2b3d
Reviewed-on: https://chromium-review.googlesource.com/565225Reviewed-by: Darren Shen <shend@chromium.org>
Reviewed-by: meade_UTC10 <meade@chromium.org>
Commit-Queue: Bugs Nash <bugsnash@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485525}

This patch moves tests related to animation timing (e.g. linear, ease,
steps) from LayoutTests/animations to a dedicated subdirectory.

BUG=623434

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

This patch Introduces |EphemeralRange| version of |TextIterator::RangeLegnth()|
to simplify call sites which pass start and end positions from |EphemeralRange|
for improving readaiblity.


Change-Id: I7c685c16e8f5abb8a1875aea2dc4cfc494a70acf
Reviewed-on: https://chromium-review.googlesource.com/564911
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Reviewed-by: Xiaocheng Hu <xiaochengh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485523}

This patch moves tests related to direction (e.g. reverse, alternate)
and fill (e.g. fill forwards, fill backwards) from
LayoutTests/animations to a dedicated subdirectory.

BUG=623434

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

This patch moves event-related tests from LayoutTests/animations to a
dedicated subdirectory.

BUG=623434

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

On Android, pages may be shown from a downloaded offline cache. In this
case, the security chip text and icon change. Reflect this in
ToolbarModel.

With this new ability, render the Offline security information in VR.

BUG=735770

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

Currently Devtools uses widget to highlight elements, i.e.,
windows, widgets, and views. We instead can ui::layer for highlighting
and to support future drawing/animation.

Bug: 730828
Change-Id: Ic84a79c382e2e3a2bcf0b9cba421d2cb9a8e72bc
Reviewed-on: https://chromium-review.googlesource.com/564036
Commit-Queue: Sadrul Chowdhury <sadrul@chromium.org>
Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485519}