GitLab

HostSharedBitmapManager is shifted to components/display_compositor.
ChildSharedBitmapManager is shifted to services/ui/public/cpp/bitmap.

The Mojo messages used in allocating shared bitmap is decoupled from
render_message_filter.h and moved to cc/ipc/shared_bitmap_manager.mojom.

Note that the SharedBitmapManager is an associated mojo interface of
RenderMessageFilter so as to maintain the existing FIFO order of mojo
messages between the two SharedBitmap allocation functions and the
rest of RenderMessageFilter functions.

BUG=670162
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

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

Here we are implementing one IAccessibility method, accLocation(), in terms of
ui::AXPlatformNodeWin. The idea is that we can forward IAccessibility calls to
ui::AXPlatformNodeWin and then slowly trim down, and maybe remove,
BrowserAccessibilityWin.

This CL adds a small helper method that allows the cast conversation of
platform_node_, which is created in AXPlatformNode, from a AXPlatformNode to
AXPlatformNodeWin. If this result is nullptr, we will fall back to using the
BrowserAccessibilityWin implementation.

BUG=703369

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

Also renames PageInfoPopupViewBrowserTest to PageInfoBubbleViewBrowserTest.

BUG=708379

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

Design doc:
https://docs.google.com/document/d/1izV4uC-tiRJG3JLooqY3YRLU22tYOsLTNq0P_InPJeE

To test:
1) Install BobPay.apk from
https://drive.google.com/a/google.com/file/d/0B3ISiXgGE1MNc1FCNlZkN3FMMWs/view?usp=sharing
2) Enable chrome://flags/#android-payment-apps.
3) Navigate to https://rsolomakhin.github.io/pr/bob/.
4) Click "Buy" button on the webpage.

BUG=707246

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

This CL records the network data use for downloading favicons. Also the
size of non 200 responses for favicon requests are recorded.

BUG=698347

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

BUG=578344
R=danakj@chromium.org

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

https://pdfium.googlesource.com/pdfium.git/+log/1e8c39f18df6..6a5c20cd0874

$ git log 1e8c39f18..6a5c20cd0 --date=short --no-merges --format='%ad %ae %s'
2017-04-07 npm Remove the ICC transform where it is never initialized.

Created with:
  roll-dep src/third_party/pdfium

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

TBR=dsinclair@chromium.org

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

BUG=526463

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

In converting accLocation() in BrowserAccessibility to use AXPlatformNode, we
found that the simple model of using local coordinates plus a global offset just
doesn't work.

In Views we've been able to get away with this simple model because Views do not
have the same nested transformations that the web can like rotation and scaling.

This change also allows us to remove GetGlobalCoordinateOffset() as
the  x.() and y.() of the result of GetScreenBoundsRect() can be used instead.

BUG=703369

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

This CL add two assertions that abstracts the checks for JavaScript
traits.

 - should().ownProperty()
 - should().inheritFrom()

BUG=702708
TEST=webaudio/unit-tests/audit.html

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

Because of the implementation specific of the layout opportunity iterator
an empty opportunity can mean 2 things:
- search for layout opportunities is exhausted.
- opportunity has an infinite size. That's because CS is infinite.

This patch fixes DCHECK crashes when we return an infinite opportunity.

BUG=635619

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

This CL updates third_party/checkstyle to 7.6.1 so checkstyle would
work with Java 8 lambda features.

BUG=703238

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

BUG=661799

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

Added EscapeString() to convert certain characters into XML escape (&xxx;) sequences

BUG=708705

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

TBR=jbudorick@chromium.org
BUG=

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

https://chromium.googlesource.com/external/khronosgroup/webgl.git/+log/d0783b8..32cfddc

BUG=685232, 701060, 709351

TEST=bots

TBR=zmo@chromium.org

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

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

This CL updates WebApkInstaller so that the murmur2 hash of the best
badge icon would be calculated and passed to the WAM server in WebAPK
proto along with the url of the best badge icon.

BUG=649771

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

BUG=695306

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

1. Sticky keys
2. Tap dragging

Changes in this CL,

1. Add four functions in AccessibilityDelegate interface.
   This is used to show above two a11y features in system menu.

BUG=632107

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

This software isn't under our control, so suppress deadlock warnings we
can't fix.

BUG=691029

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

This is a logical thing to do by itself, improves performance and makes
it easier to reason about things.

It may also help to prevent crashes if we somehow get into inconsistent state
by not calling delegate methods at random times. Although not directly
fixing the linked crash, I expect it to go away.

BUG=704346
TBR=dtrainor@chromium.org

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

This CL is the second part of the conversion. It manually replaces:

 - RELEASE_ASSERT() with CHECK().
 - ASSERT(...args) with one or more DCHECK_op(arg).

With this CL the following script returns 0 result.

```
cd ${CHROME_SRC}/third_party/WebKit/Source/platform/audio
grep -rnw . -e 'ASSERT('
```

BUG=707655

Review-Url: https://codereview.chromium.org/2803073002
Cr-Original-Commit-Position: refs/heads/master@{#462904}
Committed: https://chromium.googlesource.com/chromium/src/+/f90cae07269c724f98ca3c196ea37ce7f6a801d5
Review-Url: https://codereview.chromium.org/2803073002
Cr-Commit-Position: refs/heads/master@{#462958}

BUG=693650

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

The v8 bindings layer does the finite check for us so we don't have to
check again.

BUG=709061
TEST=covered by existing tests

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

These are the only two (important) calls that do not have a return
value, so can be made oneway. "oneway" is roughly equivalent to being
asynchronous, since binder calls are synchronous by default.

This is like an early test to make sure oneway doesn't cause issues.

BUG=689758

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

Flips the polarity of the --enable-native-cups flag to --disable-native-cups.

BUG=583155
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

This CL applies a pull request submitted to Polymer to add RTL support
for paper-sliders.

BUG=696437
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

BUG=708808

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

Screenshot: http://imgur.com/a/7WXgf

BUG=602666

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

https://skia.googlesource.com/skia.git/+log/7d3d87233190..4e3abc1ad5f0

$ git log 7d3d87233..4e3abc1ad --date=short --no-merges --format='%ad %ae %s'
2017-04-07 reed cacherator upscales colortables to unify caching
2017-04-03 halcanary SkMultiPictureDocument : clean up MultiPictureDocumentReader interface
2017-04-07 robertphillips Make GrCopySurfaceOp friendlier to GrSurfaceProxy (take 2)
2017-04-07 scroggo Simplify subset computation in SkGifImageReader
2017-04-06 mtklein stop disabling threadsafe statics
2017-04-07 borenet Fix missing valgrind.supp in recipe-bundled tasks
2017-04-07 kjlubick Add ChromeOS Test/Perf jobs and recipe changes
2017-04-06 halcanary SkTypeface_win_dw: populate_glyph_to_unicode uses lowest unichar
2017-04-06 mtklein simplify SkRRect serialization
2017-04-06 mtklein Try disabling MSAN workarounds in SkJpegCodec
2017-04-07 borenet Reland: Use bundled recipes for all test/perf tasks
2017-04-06 mtklein jumper, gather_f16

Created with:
  roll-dep src/third_party/skia
BUG=538133


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=jvanverth@chromium.org

Change-Id: I31295c32b1fde854b28bc400f41c82c3c9bfe307
Reviewed-on: https://chromium-review.googlesource.com/471049Reviewed-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@{#462950}

We have a few users of it, which are better to use underlying models.
We also have a log of SDKObject's like classes which do not inherit from SDKObject -
that means this abstraction doesn't really work.

As a bonus, we only inherit from Common.Object when necessary, reducing the overhead.

BUG=none

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

Content-initiated main-frame navigations to data URLs are being blocked
in crbug.com/594215. After crrev.com/2702503002 lands, these test will
stop working because they navigate to data URLs. The fix is to change the
data URLs in these tests to HTTP urls, but doing that causes the tests to
fail more often -- they start to fail on Windows. I investigated the
failures and can't see a good way of fixing them
(https://crbug.com/703727#c8) so I'm disabling them in the meanwhile.

BUG=594215

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

Test Network traffic annotation tag is added to network requests of unittest
in chrome/browser/*.

BUG=656607

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

This is a follow-up CL for:
https://codereview.chromium.org/2785383002/diff/80001/device/bluetooth/bluetooth_remote_gatt_service_unittest.cc

This CL added some test characteristic and descriptor UUIDs to
bluetooth_test.h and updated device bluetooth unit tests to use them.

BUG=None

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

In Jan 20015, this test case on a ASAN run caused timeouts. An exception was
added and life moved on. Re-running this tests locally does not appear to cause
any timeout, and so we're going to re-enable the exception to see if there still
exists a problem.

BUG=446661

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

This removes the forced Google URL re-check on a profile reset.  The old behavior was added to correct cases where users said "no" to a server request to use a new TLD.  Nowadays we don't prompt about this, so users can't be stuck on "the wrong" server.

BUG=244291
TEST=none

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

Reporting is a spec for delivering out-of-band reports from various
other parts of the browser. See http://wicg.github.io/reporting/ for
the spec, or https://goo.gl/pygX5I for details of the planned
implementation in Chromium.

This wraps the context in a service that keeps the implementation
details of Reporting private while implementing and exposing the
high-level operations that other components require from it.

BUG=704259

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

After discussing in http://crbug.com/702251, we decide to enable browser
shortcuts when the tab-strip / toolbar is visible on Mac OS. There is no
comparable behavior on other platforms.

This change adds a BrowserWindow::IsToolbarShowing() function to indicate
whether the toolbar is showing up on the screen. BrowserWindowCocoa needs to
override this function, which has not been done yet in this change.

BUG=680809, 702251

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

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/c5350347..ef8c92ef

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,vogelheim@chromium.org

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

As a first step of ASSERT removal from modules/webaudio, this CL
removes all DCHECK_IS_ON() condition check from isGraphOwner() and put
the check inside of the method.

The method is only used inside of DCHECK() so it must be no-op in the
release build. That is why it returns false, so we can catch the
anomaly when it happens.

BUG=707643

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