GitLab

Android AAPT 21+ will automatically move attributes to a version
of the XML file with -vXX resource qualifier. So we no longer need
our generate_v14_compatible_resources.py script.

But AAPT only drops the attributes, so if we don't use our script,
we need to put left/right in addition to start/end in our layout xmls.

The plan is, disabling our script for Android support library and
keep using for ourselves, until we can drop the script entirely
when we discontinue support for JB.

TBR=cjhopman@chromium.org
BUG=487391

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

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

This changes implementation of Directory::NextId() to no longer depend on
"next_id" counter stored in Sync DB. Even though there was a special precaution
to avoid potential collision of local IDs by fast forwarding next_id by a large
number on every save there are still a couple scenarios leading the the collision
between a local ID generated on a client and originator_client_item_id coming
from the server (which is basically a local ID of an item previously committed
to the server.

The new implementation of Directory::NextId() is based on base::GenerateGUID.
The following changes accompany the transition to the new ID format:
1) The next_id field is no longer needed in Directory::PersistedKernelInfo
2) The code that persists next_id is no longer needed.
3) Forcing the directory to save by marking it with KERNEL_SHARE_INFO_DIRTY
    flag right after loading it is no longer necessary. That was done primarily
    to increment next_id by a large value as a way to attempt this problem.
4) Fixing #3 revealed a bug in Directory::IncrementTransactionVersion that
    doesn't mark the directory with KERNEL_SHARE_INFO_DIRTY when incrementing
    a datatype's transaction version. That caused a number of bookmark tests
    to fail due to the transaction version check. Fixed it.
5) Changing local IDs to GUIDs had a subtle effect on some tests. For the
    types that don't support ordering, sorting of entries in ParentChildIndex
    defaults to ID comparison. Since generation of local IDs base on next_id
    followed a predictable pattern, some tests relied on order of items returned
    by ParentChildIndex. Switching to GUIDs disrupted the expected order
    (of entries that are unordered in their nature). To preserve the existing
    ordering pattern for unordered entries, I changed ParentChildIndex to
    default to META_HANDLE comparison.
6) Another test that failed due to the GUID transition was
    SyncableDirectoryTest.BookmarkTagTest. The test relied on a predefined,
    repeatable local ID which was no longer the case with the GUID generation.
    I found the test to be redundant. There is another tests that verifies
    the same functionality - unique bookmark tag generation algorithm.

For now the change leaves next_id in the database even though the field is
no longer used. I should probably file another bug to track its removal.

BUG=362467

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

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

The use_pinch_zoom_scrollbars flag is dead.

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

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

Add a test to confirm that data type selections are set when SCF is
closed rather than immediately when the selection is made.

BUG=480604

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

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

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}