GitLab

Summary of changes available at:
https://chromium.googlesource.com/external/google-toolbox-for-mac.git/+log/a095262..17eee69

Notable changes include:
17eee69  Make GTMSMJobRemove() have the same behavior on OSX 10.10+ as it has
         on previous OSX versions.

BUG=390276

Review URL: https://codereview.chromium.org/858093004

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

d:\src\cr3\src\base\metrics\histogram.cc(498): warning C4457: declaration of 'count' hides function parameter
d:\src\cr3\src\base\metrics\histogram.cc(490): note: see declaration of 'count'

R=isherman@chromium.org
BUG=440500

Review URL: https://codereview.chromium.org/878223002

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

With swarming, the run command gets baked into the .isolate file instead
of going through runtest.py.  Usually, an invocation of test_env.py is
baked into the isolate file. In asan mode, this used to call
asan_symbolize.py, which on Windows doesn't work -- prepend it with
sys.executable to make this work on all platforms.

BUG=451741

Review URL: https://codereview.chromium.org/877183002

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

BUG=438287

Review URL: https://codereview.chromium.org/856393005

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

In old implementaion, when SourceState::Append failed, it will report
DEMUXER_ERROR_COULD_NOT_OPEN error in ChunkDemuxer::AppendData case INITIALIZING.
But in fact, appended data may include initialization segment and media segment both.
The error can happen in any point.
In MSE spec
( http://w3c.github.io/media-source/#byte-stream-formats )
The user agent must run the end of stream algorithm with the error parameter
set to "decode" when in some conditions. They all happen in parsing the appended data.
So, here when SourceState::Append fails, it should report PIPELINE_ERROR_DECODE.

Another side, when the segment parser loop successfully parses a complete
initialization segment, it should call ReportMetadata immediately. Otherwise,
when meeting PIPELINE_ERROR_DECODE but WebMediaPlayer's ready_state_ is still
ReadyStateHaveNothing, it will be treated as NetworkStateFormatError not
NetworkStateDecodeError.

This Cl will resolve above two problems.

BUG=None
TEST=LayoutTest: https://codereview.chromium.org/742653002

Review URL: https://codereview.chromium.org/710693003

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

SingleThreadProxy was previously not considering
DidFailToInitializeOutputSurface responses as an outstanding
RequestNewOutputSurface call. This would cause the embedder to start
servicing an output surface request, then composite, receive another,
and then have double output surface requests.

The fix is to consider this failure state as a request.

BUG=444277

Review URL: https://codereview.chromium.org/871743002

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

http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=RenderFrameHostImplBrowserTest.IsFocused_Change&testType=content_browsertests

BUG=452631
TBR=mlamouri@chromium.org

Review URL: https://codereview.chromium.org/865063003

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

BUG=451624
R=raymes@chromium.org

Review URL: https://codereview.chromium.org/874703003

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

Cherry-pick potential divide-by-zero fix
https://codereview.chromium.org/877783004

This also removes the last remnants of the AVX attempts:
https://codereview.chromium.org/873313005
It should have been a no-op since it only changes cpu detection. The functions
themselves have been disabled and are not compiled so this is the last remaining
related change.

CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_chromium_vista_rel_ng

R=tomfinegan@chromium.org
BUG=396572

Review URL: https://codereview.chromium.org/877123002

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

The test fails or times out.

NOTRY=True
TBR=phoglund@chromium.org
BUG=452623

Review URL: https://codereview.chromium.org/882713003

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

R=lcwu@chromium.org,servolk@chromium.org
BUG=None

Review URL: https://codereview.chromium.org/878843004

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

This adds a Finch trial that can enable and disable bubbles as needed.
The flags will still override the Finch trial.

BUG=364686
TEST=
Visit https://adrifelt.github.io/bubble-demo.html and check to see if
you get a permission bubbles or an infobar, depending on the setting:
1. Infobar: set chrome://flags/#enable-permissions-bubbles to 'Disabled'
2. Bubble: set chrome://flags/#enable-permissions-bubbles to 'Enabled'
3. Infobar: set chrome://flags/#enable-permissions-bubbles to 'Default'
4. Infobar: --force-fieldtrials=PermissionBubbleRollout/Disabled/
5. Bubble: --force-fieldtrials=PermissionBubbleRollout/Enabled/

Review URL: https://codereview.chromium.org/791033004

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

BUG=none

Review URL: https://codereview.chromium.org/873643002

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

This patch adds conversion rule for aria-grabbed attribute and
exposes it only when it's defined.

The related patch is
https://codereview.chromium.org/736943002/

BUG=98817

Review URL: https://codereview.chromium.org/736023002

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

R=mbarbella@chromium.org, tsepez@chromium.org
BUG=450268

Review URL: https://codereview.chromium.org/881883002

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

BUG=451173
R=robertshield@chromium.org

Review URL: https://codereview.chromium.org/880603002

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

Revert of [Chromoting] Enable AppRemoting sample apps by default in all.gyp. (patchset #4 id:60001 of https://codereview.chromium.org/878793003/)

Reason for revert:
Broke the official builders:
Traceback (most recent call last):
  File "../remoting/webapp/build-webapp.py", line 413, in <module>
    sys.exit(main())
  File "../remoting/webapp/build-webapp.py", line 409, in main
    service_environment)
  File "../remoting/webapp/build-webapp.py", line 198, in buildWebApp
    + buildtype + ': ' + service_environment)
Exception: Invalid service_environment targeted for Official: dev
ninja: build stopped: subcommand failed.

http://build.chromium.org/p/chromium.chrome/builders/Google%20Chrome%20Linux%20x64/builds/55334/steps/compile/logs/stdio#error1

Original issue's description:
> [Chromoting] Enable AppRemoting sample apps by default in all.gyp.
>
> Where "by default" = whenever we build Chromoting.
>
> BUG=
>
> Committed: https://crrev.com/e0bf65f1e2fcd3515b92c1b49f04aa30fd0d47e6
> Cr-Commit-Position: refs/heads/master@{#313324}

TBR=scottmg@chromium.org,garykac@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=

Review URL: https://codereview.chromium.org/880083002

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

Revert of Plumb allow_overlay flag for video path into cc (patchset #3 id:80001 of https://codereview.chromium.org/806413004/)

Reason for revert:
This breaks the Chrome OS daisy (32-bit ARM) builder:

http://build.chromium.org/p/chromiumos.chromium/builders/Daisy%20%28chromium%29/builds/4364/steps/BuildPackages/logs/stdio

It just looks like a missing callsite.

Original issue's description:
> Plumb allow_overlay flag for video path into cc
>
> This will allow hardware video overlays to be enabled in the compositor
>
> Committed: https://crrev.com/32c0609baa6c3eea2400c6fe899a0ff60a1358dd
> Cr-Commit-Position: refs/heads/master@{#313310}

TBR=jschuh@chromium.org,piman@chromium.org,alexst@chromium.org,dalecurtis@chromium.org,posciak@chromium.org,achaulk@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Review URL: https://codereview.chromium.org/881893002

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

There is a small API change in a recently-added pdfium API exposed by
this roll, so pdfium_engine.cc is simultaneously updated to match.

This pulls in:
eef0050 Kill scattered extern _PDF_CharType declarations.
a039323 Fix infinite recursion in CPDF_Parser::ParseIndirectObjectAt().
fa370ac Fix test naming in previous commit.
e0bbe4a Fix null crash in CheckTrailer.
af9be4f Add pdfium_embeddertests to BUILD.gn
36faa4a Fix build of pdfium_embeddertest under V8_USE_EXTERNAL_STARTUP_DATA.
219e462 Use IPDF_DataAvail interface to hide CPDF_DataAvail.
83f96a0 Add API tests: NamedDestsByName, DestGetPageIndex.
9dcd7b8 Add tests for GetNamedDests() API.
dc43b32 Use signed long for FPDF_GetNamedDest buffer length.
0185408 Simplify UTF16LE_Encode and add unittest.

Review URL: https://codereview.chromium.org/881013002

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

d:\src\cr3\src\base\strings\safe_sprintf.cc(513): error C2220: warning treated as error - no 'object' file generated
d:\src\cr3\src\base\strings\safe_sprintf.cc(513): warning C4456: declaration of 'ch' hides previous local declaration
d:\src\cr3\src\base\strings\safe_sprintf.cc(450): note: see declaration of 'ch'

R=thestig@chromium.org
BUG=440500

Review URL: https://codereview.chromium.org/868023005

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

d:\src\cr3\src\base\threading\thread_local_storage.cc(143): error C2220: warning treated as error - no 'object' file generated
d:\src\cr3\src\base\threading\thread_local_storage.cc(143): warning C4457: declaration of 'value' hides function parameter
d:\src\cr3\src\base\threading\thread_local_storage.cc(110): note: see declaration of 'value'

R=thakis@chromium.org
BUG=440500

Review URL: https://codereview.chromium.org/880023002

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

BUG=447277

Review URL: https://codereview.chromium.org/878673004

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

Review URL: https://codereview.chromium.org/875593002

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/f77365f..52b64b4

CQ_EXTRA_TRYBOTS=tryserver.blink:linux_blink_rel,linux_blink_dbg
TBR=fmalita@google.com

Review URL: https://codereview.chromium.org/883693003

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

- Make all Tab#initialize(0 methods take a WebContents instead of a native WebContents pointer or a ContentViewCore.
- Make ContentViewCore#initialize() take a java WebContents
- Update methods in ContentViewUtil to no longer expose native WebContents.
- Add WebContents#destroy() to handle deleting a native WebContents and Java WebContents object.
- Remove many other references that cascade from these changes.

Review URL: https://codereview.chromium.org/831523005

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

This is just a mechanical rename of tracing includes.
See crrev.com/837303004 and the related bug for motivations.

BUG=451032
TBR=sky@chromium.org

Review URL: https://codereview.chromium.org/866893002

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

Constrain the mouse cursor to the painted region of the screen when overscan
insets are set.

BUG=427619
TEST=Manual, see bug

Review URL: https://codereview.chromium.org/873563002

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

Yet another cluster-fuzz crash happens due to invalid
render_frame_message_source_. As I see the code, this field is
always reset to NULL at the every IPC message but not initialized
at all, therefore the crash may still happen if the message is
the very first message to the object.

BUG=451755
R=jam@chromium.org

Review URL: https://codereview.chromium.org/869053004

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

The CL [1] tightened the error check too much and revealed
an existing inconsistency and resulted a production crash.
This CL makes a workaround to that crash, turning a PCHECK()
to DPCHECK().

This is the second attempt. The first one was reverted at [2]

[1] https://crrev.com/ce44fef5fd60dd2be5c587d4b084bdcd36adcee4
[2] https://crrev.com/b83c03c7b691914a054abaab13fb9131b4e0152b

BUG=449233,448245
R=agl@chromium.org,jam@chromium.org

Committed: https://crrev.com/cf3eae19b456536c5ff8dacf8800ec09f9af4bca
Cr-Commit-Position: refs/heads/master@{#313188}

Review URL: https://codereview.chromium.org/882543002

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

Damien mentioned an odd issue when adding this about it not compiling
correctly at the time. It compiles fine for me now.

R=lcwu@chromium.org
BUG=None

Review URL: https://codereview.chromium.org/881043002

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

Modifies media/filters/skcanvas_video_renderer to expose its utility
method to convert YUV VideoFrames to RGB.

BUG=450205

Review URL: https://codereview.chromium.org/864113002

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

* Gives the two classes a common base class called
  TabPersister to extend.

* Moves some functions for dealing with TabStates into
  the TabState class.

* Starts combining logic for saving Tabs between the
  StorageDelegate and the TabPersistentStore.

BUG=443773,451280

Review URL: https://codereview.chromium.org/868323003

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

If the tile is solid, don't create a resource.  In tile manager,
check for null resources before attempting to release.

BUG=445919

Review URL: https://codereview.chromium.org/884453003

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

Since we don't swap tilings anymore, we only need to push properties
when a property, or a tiling, or the raster source (invalidation) has
changed.

R=vmpstr
BUG=303943

Review URL: https://codereview.chromium.org/874613003

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

BUG=452279

Review URL: https://codereview.chromium.org/878223003

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

The recent change to add jscompile annotations include a change that
breaks that default OK/Cancel button of the loading window (because
of the number -> string conversion).

This cl removes the conversion that was added and and updates the
type annotation for sendReply_ to address the jscompile warning.

NOTRY=True
BUG=

Review URL: https://codereview.chromium.org/883813002

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

This will include the recent fix to fancy_urllib when running Python 2.7.9

BUG=452137
R=sbc@chromium.org

Review URL: https://codereview.chromium.org/880963003

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

This is just a mechanical rename of tracing includes.
See crrev.com/837303004 and the related bug for motivations.

BUG=451032
TBR=jochen@chromium.org

Review URL: https://codereview.chromium.org/864333003

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

Where "by default" = whenever we build Chromoting.

BUG=

Review URL: https://codereview.chromium.org/878793003

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

NOTRY=True
BUG=

Review URL: https://codereview.chromium.org/868393003

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