GitLab

This flag will prevent an on* property from being created on the view's element.

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

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

Although ultimately we want to remove validators from command buffer, but the
fastest way to allow an experimental WebGL 2 is actually appending the current
validators.

Appended the BufferTarget as an sample to make sure code generator works.

BUG=463599
TEST=gpu_unittests, webgl conformance tests
R=sievers@chromium.org

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

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

Add -fsanitize=cfi-vptr to list of flags disabled in AVX2 code.
https://codereview.chromium.org/982363003/

Update to b12f53
https://codereview.chromium.org/996503002

BUG=464797
R=tomfinegan@chromium.org,pcc@chromium.org

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

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

Previously Gallery.app truncates the file before a blob of new image is created.
This means that when some error happens during the creation of new image,
the file is overwritten with 0 bytes.
To avoid it, this CL creates the blob before truncates the file.

BUG=463837
TEST=out/Release/browser_tests --gtest_filter=GalleryJsTest.GalleryItemTest

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

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

We no longer use uncategorized mode.

BUG=451222

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

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

ServiceWorkerProviderHost will maintain associated
registration and all matching living registrations during
its lifetime. Get registration for "ready" property request
will be handled by fetching the longest matched one without
hitting the DB.

The MatchRegistration() method could also be used by
"claim" functionality later.

Blink CL: https://codereview.chromium.org/894983002/
Cleanup & Test: https://codereview.chromium.org/912443002/

BUG=454726,454250,462529

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

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

Chromecast has a utf8 locale, and all calls to setlocale with non-NULL
locale name fail.  Some of these issues were addressed previously in
https://codereview.chromium.org/790903003/
but we still had failures remaining in SysStrings unit tests, in
particular ScopedLocale was failing to set the locale.

BUG=internal b/19627216

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

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

BUG=464578
TEST=Launch the app, check the tooltip when fully backed up.

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

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

These are mostly unit test changes. There are a fair number of
occurances still, but they are not easy (or possible) to change due
to STL, third_party, or template requirements.

R=Nico
BUG=464816

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

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

Ran into infinite recursion while looking at skia benchmarked skpictures
in trace viewer.  BenchmarkingCanvas::onDrawPicture was calling
SkCanvas::drawPicture, which would call BenchmarkingCanvas::onDrawPicture.

BenchmarkingCanvas::onDrawPicture now calls SkCanvas::onDrawPicture,
which should break up pictures into multiple commands.

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

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

This is a re-do of https://codereview.chromium.org/927183002/ which
was reverted in CL https://codereview.chromium.org/981853002/ because
of (unsubstantiated) concerns that it caused a crash.

Adding StringPiece/StringPiece16 read/write support to pickle, and
update unit tests.

The IPC perf tests do a lot of allocations which can, with large block
sizes, harm their performance. The high allocation counts also make
their performance very dependent on the quirks of the Windows heap, as
it applies undocumented heuristics to decide when to decommit memory
and when to save it for later.

And, doing unnecessary allocations is generally always a bad thing.

So, this change adds StringPiece support to PickleIterator (reading)
and Pickle (writing). The StringPiece function now handles all strings
when writing, but must be requested explicitly when reading.

ipc_mojo_perftests does more allocations than ipc_perftests. This
removes one message-sized allocation from both tests.

The unit tests were updated so that WriteString and WriteString16 are
exercised with both string objects and char/char16 arrays (no
allocations required!). Reading into StringPiece and StringPiece16 is
also tested and the tests were verified with:
out\Release\base_unittests --gtest_filter=PickleTest.*

The main performance test command line used was:

out\Release\ipc_mojo_perftests --gtest_filter=MojoChannelPerfTest.ChannelPingPong

Typical test results on HP Z620 (Windows 8.1) with thread affinity and
high-performance power settings prior to this change:
IPC_Channel_Perf_50000x_12      1140.01 ms
IPC_Channel_Perf_50000x_144     1182.55 ms
IPC_Channel_Perf_50000x_1728    1266.42 ms
IPC_Channel_Perf_12000x_20736   1289.19 ms
IPC_Channel_Perf_1000x_248832   584.619 ms

Typical test results with same settings but using base::StringPiece:
IPC_Channel_Perf_50000x_12      1123.33 ms
IPC_Channel_Perf_50000x_144     1164.53 ms
IPC_Channel_Perf_50000x_1728    1242.99 ms
IPC_Channel_Perf_12000x_20736   1186.84 ms
IPC_Channel_Perf_1000x_248832   496.469 ms

Modest improvement with small buffers, but significant speedup with large buffers.

Typical test results with large-blocks only prior to this change:
IPC_Channel_Perf_1000x_248832   1211.33 ms
IPC_Channel_Perf_1000x_248832   961.404 ms
IPC_Channel_Perf_1000x_248832   600.911 ms
IPC_Channel_Perf_1000x_248832   468.356 ms
IPC_Channel_Perf_1000x_248832   430.859 ms
IPC_Channel_Perf_1000x_248832   425.147 ms

Typical test results with large-blocks only (base::StringPiece):
IPC_Channel_Perf_1000x_248832   909.571 ms
IPC_Channel_Perf_1000x_248832   731.435 ms
IPC_Channel_Perf_1000x_248832   493.697 ms
IPC_Channel_Perf_1000x_248832   417.966 ms
IPC_Channel_Perf_1000x_248832   397.377 ms
IPC_Channel_Perf_1000x_248832   397.725 ms

Note that it takes a while for the Windows heap to 'realize' that it
should hang on to some of the large blocks which is why performance
increases over multiple runs.

Chrome will not immediately be affected because StringPiece reading has
to be explicitly selected. Note that the effect on ipc_perftests is
more variable due to the odd Windows heap heuristics.

Reliable results require setting the power plan to high-performance.
On Linux this is done with this command:
sudo cpupower frequency-set --governor performance

The ipc_perftests command-line is:
out/Release/ipc_perftests --gtest_filter=IPCChannelPerfTest.ChannelPingPong

Typical before/after Linux results for ipc_perftests are:
IPC_Channel_Perf_50000x_12      465.271 ms
IPC_Channel_Perf_50000x_144     480.224 ms
IPC_Channel_Perf_50000x_1728    510.871 ms
IPC_Channel_Perf_12000x_20736   318.016 ms
IPC_Channel_Perf_1000x_248832   309.325 ms

IPC_Channel_Perf_50000x_12      459.245 ms
IPC_Channel_Perf_50000x_144     479.347 ms
IPC_Channel_Perf_50000x_1728    506.57  ms
IPC_Channel_Perf_12000x_20736   289.583 ms
IPC_Channel_Perf_1000x_248832   255.083 ms

Before after Linux results for ipc_mojo_perftests:
IPC_Channel_Perf_50000x_12      670.727 ms
IPC_Channel_Perf_50000x_144     713.6   ms
IPC_Channel_Perf_50000x_1728    808.157 ms
IPC_Channel_Perf_12000x_20736   464.221 ms
IPC_Channel_Perf_1000x_248832   365.258 ms

IPC_Channel_Perf_50000x_12      653.12  ms
IPC_Channel_Perf_50000x_144     697.418 ms
IPC_Channel_Perf_50000x_1728    772.575 ms
IPC_Channel_Perf_12000x_20736   446.315 ms
IPC_Channel_Perf_1000x_248832   348.38  ms

So, consistent improvements on Linux.

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

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

Introduce gfx::XScopedPtr<> as a convenience wrapper around scoped_ptr<>
for managing X11 objects. The return types of the X11 deleter functions are
somewhat inconsistent (e.g. XFree returns int, whereas XIFreeDeviceInfo
doesn't return anything), which makes this a bit tricky. So introduce a
custom deleter gfx::XObjectDeleter to use with XScopedPtr<>. Using this
XScopedPtr<> allows for cleaner code in a few places, and should generally
make it easier to manage X11 objects.

Additional clean-ups:
. content/common/gpu/x_util.cc is unused. So remove that.
. ui::XScopedString is unused. So remove that.
. DeviceListCacheX11 has support for maintaining the device-list cache for
  multiple X11 connections, but chrome doesn't have support for more than
  one. So replace the map with a single cache. (necessitated by the use of
  XScopedPtr<> in DeviceList.)

BUG=none
R=piman@chromium.org, sky@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/86e31a1..167eb17

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

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

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

Revert of [Sync] Add size and crc32c to AttachmentId. (patchset #8 id:140001 of https://codereview.chromium.org/982883002/)

Reason for revert:
Broke build on iOS_Device

http://build.chromium.org/p/chromium.mac/builders/iOS_Device/builds/26684/steps/compile/logs/stdio

Undefined symbols for architecture armv7:
  "syncer::Attachment::CreateFromParts(syncer::AttachmentId const&, scoped_refptr<base::RefCountedMemory> const&, unsigned int)", referenced from:
      syncer::(anonymous namespace)::MockAttachmentStore::RespondToRead(std::__1::set<syncer::AttachmentId, std::__1::less<syncer::AttachmentId>, std::__1::allocator<syncer::AttachmentId> > const&) in attachment_service_impl_unittest.o
      syncer::(anonymous namespace)::MockAttachmentDownloader::RespondToDownload(syncer::AttachmentId const&, syncer::AttachmentDownloader::DownloadResult const&) in attachment_service_impl_unittest.o
      syncer::gtest_case_AttachmentStoreTest_::Write_NoOverwriteNoError<syncer::InMemoryAttachmentStoreFactory>::TestBody() in in_memory_attachment_store_unittest.o
      syncer::OnDiskAttachmentStoreSpecificTest_MismatchedCrc_Test::TestBody() in on_disk_attachment_store_unittest.o
      syncer::gtest_case_AttachmentStoreTest_::Write_NoOverwriteNoError<syncer::OnDiskAttachmentStoreFactory>::TestBody() in on_disk_attachment_store_unittest.o
  "syncer::CreateAttachmentIdProto()", referenced from:
      syncer::AddAttachment(sync_pb::AttachmentMetadata*, bool) in directory_commit_contribution_unittest.o
      syncer::DirectoryUpdateHandlerProcessUpdateTest_ProcessAndApplyUpdatesWithAttachments_Test::TestBody() in directory_update_handler_unittest.o
      syncer::DirectoryUpdateHandlerApplyUpdateTest_SimpleConflictDifferentAttachmentMetadata_Test::TestBody() in directory_update_handler_unittest.o
      syncer::DirectoryUpdateHandlerApplyUpdateTest_SimpleConflictSameAttachmentMetadataDifferentOrder_Test::TestBody() in directory_update_handler_unittest.o
      syncer::AttachmentIdProtoTest_UniqueIdFormat_Test::TestBody() in attachment_id_proto_unittest.o
      syncer::AttachmentIdProtoTest_CreateAttachmentMetadata_NonEmpty_Test::TestBody() in attachment_id_proto_unittest.o
      syncer::SyncManagerChangeProcessingTest_AttachmentMetadataOnlyChanges_Test::TestBody() in sync_manager_impl_unittest.o
      ...
  "syncer::AttachmentId::Create()", referenced from:
      syncer::(anonymous namespace)::SyncDataTest_CreateLocalDataWithAttachments_Test::TestBody() in sync_data_unittest.o
      syncer::AttachmentDownloaderImplTest_HappyCase_Test::TestBody() in attachment_downloader_impl_unittest.o
      syncer::AttachmentDownloaderImplTest_SameIdMultipleDownloads_Test::TestBody() in attachment_downloader_impl_unittest.o
      syncer::AttachmentDownloaderImplTest_RequestAccessTokenFails_Test::TestBody() in attachment_downloader_impl_unittest.o
      syncer::AttachmentDownloaderImplTest_URLFetcher_BadToken_Test::TestBody() in attachment_downloader_impl_unittest.o
      syncer::AttachmentDownloaderImplTest_URLFetcher_ServiceUnavailable_Test::TestBody() in attachment_downloader_impl_unittest.o
      syncer::AttachmentDownloaderImplTest_NoHash_Test::TestBody() in attachment_downloader_impl_unittest.o
      ...
ld: symbol(s) not found for architecture armv7
clang:error: linker command failed with exit code 1 (use -v to see invocation)

Original issue's description:
> [Sync] Add size and crc32c to AttachmentId.
>
> The purpose of this change is to ensure that if a client knows about an
> attachment (i.e. has an AttachmentId or AttachmentIdProto) it will know
> the attachment's size even if the attachment has never been available on
> the local device.
>
> The idea is that by storing size locally, we can simplify remote storage
> management.
>
> Move crc32c out of Attachment now that it's part of AttachmentId.
>
> BUG=464431
>
> Committed: https://crrev.com/23ae3128db0d84a6b1ffa640568a5ec90cfc8808
> Cr-Commit-Position: refs/heads/master@{#319794}

TBR=pavely@chromium.org,maniscalco@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=464431

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

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

BUG=464610
TEST=base_unittests

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

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

Roll up all the changes for issue 443863 and issue 401971 into google.patch.
google.patch is not code, so no new tests or try runs needed.

BUG=443863,401971
TEST=none needed

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

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

As the comment in net/spdy/spdy_frame_builder.h by hkhalil@,
rename WriteString to WriteStringPiece16.

BUG=458880
R=bnc@chromium.org

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

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

BUG=465458

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

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

This is an extra check in FindLocalIdToUpdate which should
help to narrow down the cause of the crash in syncer::syncable::ChangeEntryIDAndUpdateChildren and gather
more information on the state of the directory leading to the crash.

BUG=362467

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

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

This is a temporary line-saving solution for ease of merging. A proper
separation of concerns will be done in a future CL.

BUG=442831

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/796d284..baae7d7

TBR=jbroman@chromium.org,dstockwell@chromium.org

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

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

BUG=451771

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

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

Block platform verification and file IO in the CDM adapter if the CDM configuration disallows access to them.

Based on the configured value of distinctiveIdentifier and persistentState, we compute permission bits and pass them through PPAPI to the CDM adapter. This enables us to enforce 'not-allowed' configs even when user permission has been granted.

BUG=487452

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

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

Add unittest for Spdy::GetNumberRequiredContinuationFrames, including the test
case provided in the bug report.

BUG=464748

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

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

Adding support for disabling the parity symbol.
Improving reliability for long tokens.

See cl/87989795 for the NaCl changes.

BUG=464967
R=rkc@chromium.org

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

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

The operator& is dangerous as it makes code unclear what is happening.

For ScopedPropVariant there is already a get() method returning a
const& of the internal type. So doing &foo.get() will get a pointer to
the internal type.

For ScopedVariant, there is only operator&, so replace it with an
operator*() that returns a const&. This changes callsites from doing
something like V_FOO(&scoper) to V_FOO(&*scoper) which makes it clear
that it is getting the address of the thing inside scoper, not the
scoper itself.

R=Nico
BUG=464816

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

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

The purpose of this change is to ensure that if a client knows about an
attachment (i.e. has an AttachmentId or AttachmentIdProto) it will know
the attachment's size even if the attachment has never been available on
the local device.

The idea is that by storing size locally, we can simplify remote storage
management.

Move crc32c out of Attachment now that it's part of AttachmentId.

BUG=464431

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

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

Header guards should be defined based on the path and file name of the
header file. Some headers have it wrong, some are missing the guards,
and some just have the matching comment wrong.

R=Nico
BUG=464816

Committed: https://crrev.com/301b392761fd8f66f3a701ab1dd011c6e7a55e19
Cr-Commit-Position: refs/heads/master@{#319722}

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

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

for store locally checkbox

BUG=464640

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

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

The earlier attempt https://codereview.chromium.org/973123003/ was incorrect. Reverted parts that patch
which changed the WebMouseEventBuilder::Build function to convert from DIP to pixels and back. The initial
conversion from DIP to pixels was not needed as the values are picked up from the Windows message which gives
us these coordinates in pixels.

Changes in this patch are as below:-
1. The WebMouseEventBuilder::Build function has been changed to convert the WebMouseEvent::globalX and globalY values
   to DIP. The other changes in the earlier patch as mentioned above have been reverted.

2. The RenderWidgetHostViewAura::UpdateMouseLockRegion function which executes on Windows was clipping the windows cursor
   to a rectangle in DIPs. We need to convert this rectangle to pixels before calling the ClipCursor API. This was the
   main reason for the lock operation not working correctly.

3. I was seeing a DCHECK in the renderer process in the WebViewImpl::pointerLockMouseEvent function as we were sending
   the mouse leave message. Added code in the CanRendererHandleEvent function in the render_widget_host_view_aura.cc file
   to not send this message if we are in the mouse locked state.

BUG=411634
TEST= The main test case in bug 411634

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

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

r319208 added the Low Box token to PreSpawnTarget. For some reason, this doesn't play well with Windows 10 preview builds, so this CL selectively enables the feature on Windows 8 & 8.1

BUG=464779,455496

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

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

We currently combine clips in the target space of the clip's transform
node. This is incorrect. We want to combine clips in the space of the
transform node's _content_ target space.

BUG=386810

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

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

The two formats GL_LUMINANCE and GL_RED_EXT are now been tested in addition to GL_RGBA.
GL_LUMINANCE and GL_RED_EXT are used for yuv resources that are uploaded by VideoLayerImpl.

BUG=423481

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

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

Note that the "Change" button for selecting the downloads path doesn't do
anything meaningful yet. We still need to figure out how to call into the
C++ layer to display the appropriate picker.

BUG=464785

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

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

BUG=465344
TEST=linux_chromium_clobber_rel_ng does not block new ash_unittests from landing

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

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

Revert of Roll PDFium to 1e580a10fad09833552deb797128e00c50d74bce (patchset #1 id:1 of https://codereview.chromium.org/989213003/)

Reason for revert:
Broke at least Win8 GN

http://build.chromium.org/p/chromium.win/builders/Win8%20GN/builds/5379/steps/compile/logs/stdio

FAILED: ninja -t msvc -e environment.x86 -- E:\b\build\goma/gomacc.exe "E:\b\depot_tools\win_toolchain\vs2013_files\VC\bin\amd64_x86/cl.exe" /nologo /showIncludes /FC @obj/third_party/pdfium/samples/pdfium_diff.image_diff_png.obj.rsp /c ../../third_party/pdfium/samples/image_diff_png.cc /Foobj/third_party/pdfium/samples/pdfium_diff.image_diff_png.obj /Fdobj/third_party/pdfium/samples/pdfium_diff_cc.pdb
e:\b\build\slave\win8_gn\build\src\third_party\pdfium\samples\image_diff_png.cc(246) :error C2220: warning treated as error - no 'object' file generated
e:\b\build\slave\win8_gn\build\src\third_party\pdfium\samples\image_diff_png.cc(246) : warning C4702: unreachable code
e:\b\build\slave\win8_gn\build\src\third_party\pdfium\samples\image_diff_png.cc(574) : warning C4702: unreachable code
FAILED: ninja -t msvc -e environment.x86 -- E:\b\build\goma/gomacc.exe "E:\b\depot_tools\win_toolchain\vs2013_files\VC\bin\amd64_x86/cl.exe" /nologo /showIncludes /FC @obj/third_party/pdfium/samples/pdfium_test.image_diff_png.obj.rsp /c ../../third_party/pdfium/samples/image_diff_png.cc /Foobj/third_party/pdfium/samples/pdfium_test.image_diff_png.obj /Fdobj/third_party/pdfium/samples/pdfium_test_cc.pdb
e:\b\build\slave\win8_gn\build\src\third_party\pdfium\samples\image_diff_png.cc(246) :error C2220: warning treated as error - no 'object' file generated
e:\b\build\slave\win8_gn\build\src\third_party\pdfium\samples\image_diff_png.cc(246) : warning C4702: unreachable code
e:\b\build\slave\win8_gn\build\src\third_party\pdfium\samples\image_diff_png.cc(574) : warning C4702: unreachable code
ninja: build stopped: subcommand failed.

Original issue's description:
> Roll PDFium to 1e580a10fad09833552deb797128e00c50d74bce
>
> This brings in:
> 1e580a1 Fix multiply defined macro in zlib.h on linux_chromium_clobber_rel_ng builder.
> 0700106 Make conversions for CPDF_Link explicit.
> 8842c62 Fix the pdfium component build.
> 5a370d7 Make conversion between CPDF_Dest and its object explicit.
> 7348fae Add gmock to .gitignore
> 624b6c1 Make conversion between CPDF_Action and its dictionary explicit.
> 61c25ee Merge to Master: Fix a compiling error in fxjs_v8.h
> e1671bd Kill off JS_ErrorString type.
> b720d0a Return error information from pdfium to JS.
> 944ccad Do hit tests against Annots in reverse order.
> 5b47dbf Remove unused IDS_ constants from pdfium.
> 3d25502 Pull in gmock for standalone pdfium builds.
> ec61a85 Upgrade openjpeg to revision 2997.
> 42a8ada Implement a delegate for EmbedderTests.
> 57e4c86 Kill off unused IFXJS interface methods.
> 4351ac2 Fix compilation error on Windows.
> f9cfdda Add small flate decode unit test.
> e796478 Kill multi-byte comments in fxet_list.h
> 23f9288 Convert pdfium expected test results to PNG format.
> 0fd53cf Fix GN build after PNG support backport
> ef1e85c Backport PNG output format to origin/master branch.
>
> BUG=445408,457493,PDFIUM:123
>
> Committed: https://crrev.com/608e59f9ceecd5b60ae5e50c4f0da4bc6a7f405e
> Cr-Commit-Position: refs/heads/master@{#319781}

TBR=brettw@chromium.org,thestig@chromium.org,tsepez@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=445408,457493,PDFIUM:123

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

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

Improve SpdyHttpUtils::GetUrlFromHeaderBlock() implementation with early
returns.  Get rid of a number of temporary variables that were declared too far
from where they were first used.

Also remove SPDY2-specific code, as SPDY2 is no longer supported in Chromium.

Acknowledgements to Daniel Bratell for suggesting to use append instead of
inline operator+ that would create unnecessary temporary strings, see
https://crrev.com/965773002.

BUG=

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

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

This change is chromium part of
https://codereview.chromium.org/921053003/

And out-of-date TODO is removed as well.

TEST=
BUG=458409

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

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

BUG=none

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

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

This brings in:
1e580a1 Fix multiply defined macro in zlib.h on linux_chromium_clobber_rel_ng builder.
0700106 Make conversions for CPDF_Link explicit.
8842c62 Fix the pdfium component build.
5a370d7 Make conversion between CPDF_Dest and its object explicit.
7348fae Add gmock to .gitignore
624b6c1 Make conversion between CPDF_Action and its dictionary explicit.
61c25ee Merge to Master: Fix a compiling error in fxjs_v8.h
e1671bd Kill off JS_ErrorString type.
b720d0a Return error information from pdfium to JS.
944ccad Do hit tests against Annots in reverse order.
5b47dbf Remove unused IDS_ constants from pdfium.
3d25502 Pull in gmock for standalone pdfium builds.
ec61a85 Upgrade openjpeg to revision 2997.
42a8ada Implement a delegate for EmbedderTests.
57e4c86 Kill off unused IFXJS interface methods.
4351ac2 Fix compilation error on Windows.
f9cfdda Add small flate decode unit test.
e796478 Kill multi-byte comments in fxet_list.h
23f9288 Convert pdfium expected test results to PNG format.
0fd53cf Fix GN build after PNG support backport
ef1e85c Backport PNG output format to origin/master branch.

BUG=445408,457493,PDFIUM:123

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

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