GitLab

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/be1e6ee..842d065

TBR=jsbell@chromium.org,timvolodine@chromium.org

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

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

Improve quality of software rendering path by replacing call to SkCanvasVideoRenderer::ConvertVideoFrameToRGB in video_decoder_shim with a shader and render-to-texture.

Questions about this CL:

* gpu::gles2::GLESInterface
Would it be correct to send a base::WeakPtr<GLESInterface> to the YUVConv
class constructor to use for everything in this instance of the class?
ATM, it just sends a GLESInterface* to methods that need it (which is
probably wrong).  It was not clear to me whether the GLESInterface * could
change between the ::Initialize() method

* OpenGL state
YUVConv::convert() changes texture bindings and viewport.  I currently read
the old viewport so that I can restore it - is this necessary?  I don't save
former texture bindings either - is this necessary?

* OpenGL 'getters'
When compiled debug, why do all output variables to GL functions need to
contain either 0 or -1 before calling the function?  In my case GetIntegerv
and GetShaderiv.

* Errors
It is possible that shader compile could fail.  If it fails, an error string
can be retrieved.  What should I do in this case?
Same with opengl errors.  I thought it would be good to check for them in
debug builds, but what do I do if I catch one?

* Texture formats
I need a single-component texture format for uploading the Y,U,V components.
On desktop I would use GL_RED, but unextended GLES does not have GL_RED, you
have to use GL_LUMINANCE.  The current code uses GL_RED, and it works, but do
I need to check for GLES extension for GL_RED or can I assume it is there?

* Texture sizes
Even with the description in video_frame.h the distinction between coded_size
visible_size and natural_size was not entirely clear to me.  It seemed the
output texture size in VideoDecoderShim::OnOutputComplete followed "coded_size"
so that is what I used as well.  It was my intention to always decode the
entire image to the user's buffer, not some subregion - so is my code correct?

* FBO's
I see that DrawBuffers is DrawBuffersEXT, and that the default GLES
implementation does not support it.  Is it not necessary in this environment
to call DrawBuffers?

* ::SendPictures
If there is a format error yuv_converter will return false.  How can an error
be handled here?

* testing
I have tested with the video_decoder example program (using VPx) and our
application which uses h264.  I believe both use I420 format.  Do you have an
easy way to test other formats?  Is this class part of a unit test?

BUG=483183,477737
R=bbudge@chromium.org
TEST=Run pepper video_decoder example in ppapi/examples/video_decode

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

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

BUG=none
R=ganesh@chromium.org,pkasting@chromium.org

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

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

Because Android's API doesn't let us know when a user has
deleted a shortcut from the home screen, record that the
shortcut still exists whenever the shortcut is launched.
This applies to both regular shortcuts and web app launches.

BUG=460229

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

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

Useful for mojo developers.

BUG=486809

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

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

BUG=486916
TEST=No more NC_AUTO_SUBFRAME renderer kills.

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

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

"Recently closed" is always hidden and is first on the chopping block.

R=estade@chromium.org
BUG=329637

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

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

This is particularly important on ChromeOS with freon stack.
SwapBuffers ack is sent when the buffers actually swapped, not when the
call to swap them was made. This changed the scheduler behavior and delayed
beginning of the next frame, so to regain it we allowed multiple in flight
frames. The effect this has on latency info is that two frames arriving in
rapid succession would override incorrect latency info making it look like
things ultimately happened earlier than they did.

This patch associates latency info with a particular frame's callback
thus avoiding it being overridden.

BUG=485302

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

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

This CL saves ~50 minutes from the average Android Cycle times.

Note to perf sheriff, this may result a minor shift in the result for the android tests. It is unlikely since the hardware is typical.

R=sullivan@chromium.org
BUG=466101,471921

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

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

BUG=

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

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

Accidentally inverted a check during refactoring in
https://crrev.com/1137513002

Add back the '!' character that I deleted, and add unit tests to
guard against future regressions.

BUG=486675

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

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

Create a placeholder icon source for extensions that don't have an icon, so that
they don't just look like puzzle pieces in the toolbar.
To start, only use this in the toolbar, but the class is flexible enough that
it could also be used, e.g., in the chrome://extensions page.

BUG=486206

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

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

Update //third_party/usb_ids/usb.ids to version 2015.05.05, released
2015-05-05 at 20:34:11.

This file was fetched from http://www.linux-usb.org/usb.ids and
includes vendor and product IDs for new USB devices and updates to
existing entries.

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

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

Internet Explorer will, apparently, generate bookmark files without any charset
specified. Fall back to utf-8 in that case, so that import can still succeed.

BUG=460423
TEST=unit_tests
R=gab@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/d0384ff..be1e6ee

TBR=jsbell@chromium.org,timvolodine@chromium.org

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

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

Only include items that are relevant to component extensions in their context
menus. That is to say, don't show the "Remove", "Manage extensions", or "Inspect
popup" items, and only show the "Options" item if there is an options page.

BUG=486223

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

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

Now sublime users can use command-/ to comment/uncomment lines
in gn and gni files.

R=dpranke@chromium.org

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

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

Will rename CronetUploadDataStreamDelegate to
CronetUploadDataStreamAdapter in a followup CL.

BUG=481119

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/7ae68a8..86a17e7

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
TBR=robertphillips@google.com

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

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

This will help us analyze how often the Data Reduction Proxy bypass
logic is applied for proxies that don't match the currently configured
Data Reduction Proxies, but do have the DRP via header in the response.

BUG=487852

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

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

As any includes present in the related header do not need to be included again in the related cc (i.e., foo.cc can rely on foo.h's includes).

This removes all includes from $X.cc files where the $X.h has the same include to confirm to the style guide.

BUG=460988

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

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

This will make it simpler to add new webcam interface types.

BUG=None

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

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

This histogram tracks how frequently SDCH is advertised over a secure scheme.

BUG=488086

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

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

These methods are called when the observed SdchManager adds or removess
dictionaries.

BUG=

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

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

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

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

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

BUG=466848

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

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

Ensure that audio_unittests pass with --require-audio-hardware-for-testing.
 * Convert tests to STA COM threads, not MTA.
 * Do COM thread initialization in AudioManager.
 * Disable exclusive mode tests.
 * Convert a couple of tests to use ABORT_AUDIO_TEST_IF_NOT.

BUG=416594
TEST=audio_unittests --require-audio-hardware-for-testing

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

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

Any URLFetchers using one of a Profile's URLRequestContextGetters
will now return an error on Profile shutdown, as opposed to causing
crashes in AssertNoURLRequests.

BUG=419095

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

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

Introduce the user action
WindowSelector_ActiveWindowChanged to track
the number of overview mode sessions in which
the user selected a window that is different
from the previously-active window.

BUG=486900
TEST=WindowSelectorTest.ActiveWindowChangedUserAction*

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

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

Looks like the extra frame duration is enough extra leeway to match
a 1-frame cadence if it was previously on the edge.

BUG=439548
TEST=new unittest

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

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

https://chromium.googlesource.com/angle/angle/+log/b11e24..21ce9b

BUG=484963
BUG=483252

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

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

BUG=236848
TEST=NavigationController* tests pass in --site-per-process mode.

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

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

so we only take a loader-lock the first time.

BUG=485656, 477137

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

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

BUG=412250

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

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

This will let us know what percentage of requests are a
result of periodic polling vs. the initial requests when
Chrome starts up.

BUG=483057

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

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

After talking it over more, it's been decided that this should really just be
enabled by default, rather than Finch'd.

BUG=441377

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

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

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

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

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

BUG=486897

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

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

This more accurately simulates a user making changes through the Android
sync settings in an end-to-end way.

BUG=480604

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

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

To make things easier for future developers, the test script now parses
the options using argparse and provides a --reset-results flag to easily
update the baselines.

BUG=none

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

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