GitLab

Revert of arc: Fix default app availability reporting. (patchset #1 id:1 of https://codereview.chromium.org/2468973006/ )

Reason for revert:
It's breaking all ARC CTS autotests b/32660818,
and I highly suspect it is also affecting the Chrome OS CQ crbug.com/662451.

BUG=b/32660818

Original issue's description:
> arc: Fix default app availability reporting.
>
> This solves the case when App list model is created before
> ArcAppListPrefs and notifications about defualt apps are
> not dispatched.
>
> BUG=641535
> TEST=Locally on device.
>
> Committed: https://crrev.com/fdae64e689c5c1424d5da8f827e9d1d10d905c1c
> Cr-Commit-Position: refs/heads/master@{#429408}

TBR=xiyuan@chromium.org,hidehiko@chromium.org,khmel@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=641535

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

This is preparation for [1], fixing test failure of
virtual/rootlayerscrolls/fast/history/scroll-restoration/scroll-restoration-fragment-navigation-crossdoc.html
with [1].

Before [1], the test is passing because, in FrameLoader::finishedParsing(),
restoreScrollPositionAndViewState() is called (in checkCompleted()) before
processFragment(), and thus processFragment() doesn't cause scroll
because |shouldScrollToFragment| is false because
|didRestoreFromHistory| is set to true by restoreScrollPositionAndViewState().

However, after [1], restoreScrollPositionAndViewState() is NOT called in
checkCompleted() because [1] causes the document onload to be called after
finishedParsing(), and thus processFragment() causes scrolling unexpectedly,
not reflecting history.scrollRestoration = 'manual'.

This CL makes processFragment() to check scrollRestorationType explicitly,
not depending on previous restoreScrollPositionAndViewState() call.

[1] https://codereview.chromium.org/2269043002/.

BUG=624697, 417782
TEST=virtual/rootlayerscrolls/fast/history/scroll-restoration/scroll-restoration-fragment-navigation-crossdoc.html

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

Reland of xo: Add drm support to motion_events. (patchset #1 id:1 of https://codereview.chromium.org/2481713002/ )

Reason for revert:
This cl was not to blame.

Seems like the real problem is due to daylight saving time on the bots.

Original issue's description:
> Revert of exo: Add drm support to motion_events. (patchset #11 id:240001 of https://codereview.chromium.org/2477043002/ )
>
> Reason for revert:
> Seems to break PrecacheFetcherTest.DailyQuot
> componnentest on Mac 10.9 and 10.10
>
> https://build.chromium.org/p/chromium.mac/builders/Mac10.9%20Tests/builds/31355
>
> Original issue's description:
> > exo: Add drm support to motion_events.
> >
> > Add support to render using dmabufs to motion_events wayland
> > test client.
> >
> > BUG=661010
> > TEST=XDG_RUNTIME_DIR=/var/run/chrome /tmp/wayland_motion_events --linux_dmabuf_v1 on kevin.
> >
> > Committed: https://crrev.com/67808baf1012d79550204d4cb25c4e3a26caa416
> > Cr-Commit-Position: refs/heads/master@{#430224}
>
> TBR=reveman@chromium.org,dcastagna@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=661010
>
> Committed: https://crrev.com/33d7e2f3e00011aa6e02ac99896c1622d9aa7b44
> Cr-Commit-Position: refs/heads/master@{#430230}

TBR=reveman@chromium.org,dcastagna@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=661010

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

User agent CSS such as html.css is processed per renderer start, and blocks time-to-first-meaningful-paint.
This CL introduces a simple CSS minimizer which processes html.css before bundling it into resource pak file.

html.css:
original 23451 bytes
minimized 18168 bytes
22% reduction

BUG=472927

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

This aims to reduce Web MIDI logics in content/renderer so to make
coming mojofication change as much simple as possible.

BUG=582327

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

PowerUsageMonitor is currently only used by Chrome and has significant //content
dependencies, to decouple it we just move it to //chrome/browser at the current
time.
The power_usage_monitor.h is deleted as there is no need to export the
StartPowerUsageMonitor interface from chrome any more. We add the
PowerUsageMonitor as a member into the ChromeBrowserMain.
Unittest are also changed accordingly.

BUG=647247

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

This is follow up CL for https://codereview.chromium.org/2466823002/

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

This is much easier now that we have the AXAction enum!

BUG=660227

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

Revert of exo: Add drm support to motion_events. (patchset #11 id:240001 of https://codereview.chromium.org/2477043002/ )

Reason for revert:
Seems to break PrecacheFetcherTest.DailyQuot
componnentest on Mac 10.9 and 10.10

https://build.chromium.org/p/chromium.mac/builders/Mac10.9%20Tests/builds/31355

Original issue's description:
> exo: Add drm support to motion_events.
>
> Add support to render using dmabufs to motion_events wayland
> test client.
>
> BUG=661010
> TEST=XDG_RUNTIME_DIR=/var/run/chrome /tmp/wayland_motion_events --linux_dmabuf_v1 on kevin.
>
> Committed: https://crrev.com/67808baf1012d79550204d4cb25c4e3a26caa416
> Cr-Commit-Position: refs/heads/master@{#430224}

TBR=reveman@chromium.org,dcastagna@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=661010

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

On a mac, when showing list of available low energy devices, low energy devices which are connected are not shown.

To get the list of devices, -[CBCentralManager scanForPeripheralsWithServices:options:] is used. This method reports only advertising devices.

For devices that are connected but doesn't advertise, -[CBCentralManager retrieveConnectedPeripheralsWithServices:] needs to be called when starting a discovery session.

BUG=630581

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

This patch converts the layout test with spellcheck_test as a
preparation for implementing idle time spellchecker.

BUG=517298

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

This patch converts the layout test with spellcheck_test as a
preparation for implementing idle time spellchecker.

BUG=517298

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

Reland of Use the serviceworker with mojo by default (patchset #1 id:1 of https://codereview.chromium.org/2443983002/ )

Reason for revert:
Hopefully fixed the issue on https://crbug.com/658702.

It was a flaky bug, so let's keep our eyes open for the flakiness dashboard for several days.

Original issue's description:
> Revert of Use the serviceworker with mojo by default (patchset #2 id:20001 of https://codereview.chromium.org/2436223004/ )
>
> Reason for revert:
> I think this is causing InstallableManagerBrowserTest.CheckManifest404 to crash, see http://crbug.com/658702
>
> Original issue's description:
> > Use the serviceworker with mojo by default
> >
> > This patch changes --mojo-service-worker to --disable-mojo-service-worker.
> > Let's try mojo for the service worker.
> >
> > BUG=629701
> >
> > Committed: https://crrev.com/391fa63bcefd25b35addf678356828dded5ae71e
> > Cr-Commit-Position: refs/heads/master@{#427013}
>
> TBR=horo@chromium.org,avi@chromium.org,shimazu@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=629701
>
> Committed: https://crrev.com/cd13e5fd2bc139909075578e71483e0069246eab
> Cr-Commit-Position: refs/heads/master@{#427088}

TBR=horo@chromium.org,avi@chromium.org,dmazzoni@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=629701

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

- Added kPurgeAndSuspend to content feature list.
- If kPurgeAndSuspend is enabled, purge memory and suspend.
  Otherwise, do nothing except recording UMA.

- intent-to-implement-and-ship of background renderer's purge +
  suspend is
  https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/DK189tnM8l4
- one of the documents attached in the above intent is
  https://docs.google.com/document/d/1EgLimgxWK5DGhptnNVbEGSvVn6Q609ZJaBkLjEPRJvI/edit?usp=sharing
- The feature is not enabled by default because
  purge-and-suspend-time is 0.

BUG=635419

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

Add support to render using dmabufs to motion_events wayland
test client.

BUG=661010
TEST=XDG_RUNTIME_DIR=/var/run/chrome /tmp/wayland_motion_events --linux_dmabuf_v1 on kevin.

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

This is used to test crbug.com/641600.
VEA driver should not return an output buffer without any input.

BUG=648861
TEST=test on device.

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

https://skia.googlesource.com/skia.git/+log/f98b730af08d..ad512797c940

$ git log f98b730af..ad512797c --date=short --no-merges --format='%ad %ae %s'
2016-11-06 herb Use perfect forwarding in createT of SkSmallAllocator.

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_precise_blink_rel
TBR=scroggo@google.com

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

It takes 30+ seconds to finish each of them with Release build.

BUG=490939
TBR=kojii@chromium.org
NOTRY=true

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

BUG=none
R=chrishtr@chromium.org, wangxianzhu@chromium.org

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

These fail as well as timeout on Debug bots.

BUG=626703, 662765
TBR=qyearsley@chromium.org
NOTRY=true

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

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/ae24592f..771b75bc

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

TBR=hablich@chromium.org,machenbach@chromium.org,littledan@chromium.org,vogelheim@chromium.org

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

Just adding interfaces in blink side and implementing them in follow-up CL.
The interfaces are behind a new runtime flag.

Intent to implement:
  https://groups.google.com/a/chromium.org/forum/m/#!topic/blink-dev/2ojnMk_T9_c

BUG=661608

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

This CL makes URLLoader and URLLoaderClient associated for mojo-loading in
order to deal with the IPC ordering issue with the exising IPC messages.

BUG=603396

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

https://chromium.googlesource.com/native_client/src/native_client.git/+log/63d17e18381b..d564ff3bb656

$ git log 63d17e183..d564ff3bb --date=short --no-merges --format='%ad %ae %s'
2016-11-06 petarj PNaCl: Update llvm revision in pnacl/COMPONENT_REVISIONS

BUG=pnacl-llc aborts on run_vector_extension_test for MIPS

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

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

Adds support for sticky header rows to tray icon scroll views. When
marked as sticky the headers will not scroll above the top of the
visible viewport unless there is another sticky header row just below.
Multiple rows can be indicated as header rows by setting View::id_.

BUG=631831

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

Cr-Commit-Position: refs/heads/master@{#430213}

TBR=qyearsley@chromium.org

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

This patch converts the layout test with spellcheck_test as a
preparation for implementing idle time spellchecker.

This patch also contains a performance optimization to spellcheck_test
in handling CharacterData nodes with long text.

BUG=517298

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

BUG=662693
TBR=bartfab@chromium.org, kainino@chromium.org

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

Native Mac NSAlert dialogs show the dialog buttons on the TouchBar, we
want similar behaviour for Chrome dialogs.

If a Widget's delegate implements views::DialogDelegate, add buttons to
the touchbar for the "OK" and "Cancel" buttons. Note the labels might
not necessarily be labeled "OK" and "Cancel".

Nearly all dialogs and bubbles work correctly with the TouchBar this
way. One exception is the Add Bookmark bubble -
LocationBarBubbleDelegateView::GetDialogButtons() returns
ui::DIALOG_BUTTON_NONE. That makes sense for the ZoomBubble.
SaveCardBubbleViews overrides again to return both buttons and works.
The translate bubble has "TODO: This should be using GetDialogButtons."

Screenshot: http://crbug.com/660126#c13

BUG=661581

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

As observation in https://crbug.com/658702, I found the mojo error handler
(EWInstance::Detach in this case) is called on the UI thread when the message
loop on the UI thread gets destructed though EWInstance is on the IO
thread. This patch is to check the running thread before the error handling to
avoid unexpected threading issues.

BUG=658702,604762

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

Results in multiple calls to getCharacteristic(s) to return the same object.

BUG=495270

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

Remove services from attribute instance map when a device disconnects or when disconnect is called.
Check that the service is in the attribute instance map and reject if it isn't.

Small refactor of disconnect() and dispatchGattServerDisconnected() to more closely
match the spec.

BUG=654950

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

BUG=626703,662765
TBR=qyearsley@chromium.org
NOTRY=true

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

effect-reference-tile.html flaky on Win
Passed: https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win7/builds/47565
Failed: https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win7/builds/47566

margin_left.html NeedsManualRebaseline (missed in the batch added in r430198)

BUG=662722
NOTRY=true
TBR=fmalita@chromium.org

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

These failed in
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty/builds/19591
but not in
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty/builds/19590
after being rebaselined in
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty/builds/19588

They seem to rely on hover effects and have a history of flakiness in TestExpectations.

BUG=662722
NOTRY=true
TBR=fmalita@chromium.org

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

Failing since
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win10/builds/16850 (initial landing)
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win10/builds/16855 (autorebaseline)

BUG=662722
NOTRY=true
TBR=fmalita@chromium.org

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

Crashes flakily. Couldn't find a culprit.

BUG=662688
TBR=tzik@chromium.org,boliu@chromium.org

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

- Add imported/wpt/dom/nodes/Document-createElement*.html which are currently
  failing on the bots.

- Merge tests with crbug.com/619746 into the section of crbug.com/508730.

- Triage some tests associated to crbug.com/508734

BUG=619746, 508730, 508734
TBR=qyearsley@chromium.org
NOTRY=true

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

Blink has some loose heuristics to ignore images, even ones that are important for interaction.

Specifically, web authors do use horizontal images (i.e. 1d images) to convey information to screen readers that is not purely presentational.

At the very least, it should be up to each client to make an intelligent determination.

This cl also adds missing graphics predicates and key bindings to ChromeVox.

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

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