Rewrap comments to 80 columns in Source/platform/graphics/.

Rewrap comments to 80 columns in Source/platform/graphics/.

This also makes some minor modifications (rewrite for clarity, etc.).

BUG=563793
TEST=none

Committed: https://crrev.com/5502b5a19ee85fb3d150dd20ce386f1699d45c01
Cr-Commit-Position: refs/heads/master@{#423280}

[Peter Kasting, 2016-10-04 22:57:32]

pkasting@chromium.org changed reviewers:
+ senorblanco@chromium.org

[Peter Kasting, 2016-10-04 22:57:43]

The CQ bit was checked by pkasting@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-04 22:58:00]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-04 23:02:46]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-04 23:02:47]

Dry run: Try jobs failed on following builders:
  android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)
  chromeos_x86-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_x86-ge...)
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_clobber_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)
  win_clang on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_clang/builds/...)

[Peter Kasting, 2016-10-04 23:06:56]

The CQ bit was checked by pkasting@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-04 23:07:31]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-05 00:46:43]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-05 00:46:45]

Dry run: This issue passed the CQ dry run.

[Stephen White, 2016-10-05 14:45:38]

LGTM.

Nits are at your discretion.

https://codereview.chromium.org/2398453002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/AcceleratedStaticBitmapImage.h
(right):

https://codereview.chromium.org/2398453002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/AcceleratedStaticBitmapImage.h:88:
// Thread that m_image belongs to. Set to null when |m_image| belongs to the
Nit: the first instance of m_image on this line doesn't have ||s, while the
second now does. I couldn't find anything in the style guide on this, though.

https://codereview.chromium.org/2398453002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/Gradient.cpp (right):

https://codereview.chromium.org/2398453002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/Gradient.cpp:102: // m_gradient has
been set.
Nit: |m_gradient|?

https://codereview.chromium.org/2398453002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/GraphicsContext.cpp (right):

https://codereview.chromium.org/2398453002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/GraphicsContext.cpp:1049: // back to
drawDRRect instead.
Nit: drawDRRect() (not new to this patch)

https://codereview.chromium.org/2398453002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/GraphicsLayer.h (right):

https://codereview.chromium.org/2398453002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/GraphicsLayer.h:129: // The double
version is only used in |updateScrollingLayerGeometry()| for
Nit: not new to this patch, but this seems to be one of the few places with |s
on a function name.

[Peter Kasting, 2016-10-05 18:46:02]

https://codereview.chromium.org/2398453002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/AcceleratedStaticBitmapImage.h
(right):

https://codereview.chromium.org/2398453002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/AcceleratedStaticBitmapImage.h:88:
// Thread that m_image belongs to. Set to null when |m_image| belongs to the
On 2016/10/05 14:45:37, Stephen White wrote:
> Nit: the first instance of m_image on this line doesn't have ||s, while the
> second now does. I couldn't find anything in the style guide on this, though.

Yeah, there's no style rule, and existing code is all over -- some of it uses
||, some doesn't.  We should certainly be consistent within a comment.  I added
|| to the other instance.

https://codereview.chromium.org/2398453002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/Gradient.cpp (right):

https://codereview.chromium.org/2398453002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/Gradient.cpp:102: // m_gradient has
been set.
On 2016/10/05 14:45:37, Stephen White wrote:
> Nit: |m_gradient|?

Done.

https://codereview.chromium.org/2398453002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/GraphicsContext.cpp (right):

https://codereview.chromium.org/2398453002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/GraphicsContext.cpp:1049: // back to
drawDRRect instead.
On 2016/10/05 14:45:37, Stephen White wrote:
> Nit: drawDRRect() (not new to this patch)

Done.

https://codereview.chromium.org/2398453002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/graphics/GraphicsLayer.h (right):

https://codereview.chromium.org/2398453002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/graphics/GraphicsLayer.h:129: // The double
version is only used in |updateScrollingLayerGeometry()| for
On 2016/10/05 14:45:37, Stephen White wrote:
> Nit: not new to this patch, but this seems to be one of the few places with |s
> on a function name.

Yeah, those shouldn't be there.  Removed.

[Peter Kasting, 2016-10-05 18:46:48]

The CQ bit was checked by pkasting@chromium.org

[Peter Kasting, 2016-10-05 18:46:49]

The patchset sent to the CQ was uploaded after l-g-t-m from
senorblanco@chromium.org
Link to the patchset: https://codereview.chromium.org/2398453002/#ps40001
(title: "Review feedback")

[commit-bot: I haz the power, 2016-10-05 18:47:20]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-05 20:46:38]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-05 20:48:28]

Description was changed from

==========
Rewrap comments to 80 columns in Source/platform/graphics/.

This also makes some minor modifications (rewrite for clarity, etc.).

BUG=563793
TEST=none
==========

to

==========
Rewrap comments to 80 columns in Source/platform/graphics/.

This also makes some minor modifications (rewrite for clarity, etc.).

BUG=563793
TEST=none

Committed: https://crrev.com/5502b5a19ee85fb3d150dd20ce386f1699d45c01
Cr-Commit-Position: refs/heads/master@{#423280}
==========

[commit-bot: I haz the power, 2016-10-05 20:48:30]

Patchset 3 (id:??) landed as
https://crrev.com/5502b5a19ee85fb3d150dd20ce386f1699d45c01
Cr-Commit-Position: refs/heads/master@{#423280}