Fix template angle bracket syntax in platform

Fix template angle bracket syntax in platform
    
In C++11 the parser is able to parse >> as the end of
the template argument list.

BUG=449681
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=188567

[rwlbuis, 2015-01-16 22:28:27]

rob.buis@samsung.com changed reviewers:
+ adamk@chromium.org

[rwlbuis, 2015-01-16 22:28:45]

PTAL

[adamk, 2015-01-16 22:34:32]

This seems fine, but I'm curious what the broader plan is.

Does this patch include everything under Source/platform? Do you have similar
patches for the rest of the codebase?

[rwlbuis, 2015-01-16 22:38:59]

On 2015/01/16 22:34:32, adamk wrote:
> This seems fine, but I'm curious what the broader plan is.
> 
> Does this patch include everything under Source/platform? Do you have similar
> patches for the rest of the codebase?

Yes this one is just platform.
Doing patches like this is just running a one liner perl script. So I think I'll
prepare more patches over next week(s). Right now it looks like 400 more file
need to be patched, most in core with the rest in modules. I'll try to create
reasonably sized patches.

[rwlbuis, 2015-01-16 22:39:31]

On 2015/01/16 22:38:59, rwlbuis wrote:
> On 2015/01/16 22:34:32, adamk wrote:
> > This seems fine, but I'm curious what the broader plan is.
> > 
> > Does this patch include everything under Source/platform? Do you have
similar
> > patches for the rest of the codebase?
> 
> Yes this one is just platform.
> Doing patches like this is just running a one liner perl script. So I think
I'll
> prepare more patches over next week(s). Right now it looks like 400 more file
> need to be patched, most in core with the rest in modules. I'll try to create
> reasonably sized patches.

Also, I can create a bug if you prefer.

[rwlbuis, 2015-01-16 22:39:33]

On 2015/01/16 22:38:59, rwlbuis wrote:
> On 2015/01/16 22:34:32, adamk wrote:
> > This seems fine, but I'm curious what the broader plan is.
> > 
> > Does this patch include everything under Source/platform? Do you have
similar
> > patches for the rest of the codebase?
> 
> Yes this one is just platform.
> Doing patches like this is just running a one liner perl script. So I think
I'll
> prepare more patches over next week(s). Right now it looks like 400 more file
> need to be patched, most in core with the rest in modules. I'll try to create
> reasonably sized patches.

Also, I can create a bug if you prefer.

[adamk, 2015-01-16 22:53:40]

On 2015/01/16 22:39:33, rwlbuis wrote:
> On 2015/01/16 22:38:59, rwlbuis wrote:
> > On 2015/01/16 22:34:32, adamk wrote:
> > > This seems fine, but I'm curious what the broader plan is.
> > > 
> > > Does this patch include everything under Source/platform? Do you have
> similar
> > > patches for the rest of the codebase?
> > 
> > Yes this one is just platform.
> > Doing patches like this is just running a one liner perl script. So I think
> I'll
> > prepare more patches over next week(s). Right now it looks like 400 more
file
> > need to be patched, most in core with the rest in modules. I'll try to
create
> > reasonably sized patches.
> 
> Also, I can create a bug if you prefer.

A bug would be good, I think.

I've looked at a random subset of these changes and they all look reasonable,
lgtm.

[rwlbuis, 2015-01-16 22:59:54]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2015-01-16 23:00:04]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/858663002/1

[commit-bot: I haz the power, 2015-01-16 23:00:30]

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

[commit-bot: I haz the power, 2015-01-16 23:00:32]

Failed to apply patch for Source/platform/graphics/GraphicsContextRecorder.cpp:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  can't find file to patch at input line 6
  Perhaps you used the wrong -p or --strip option?
  The text leading up to this was:
  --------------------------
  |Index: Source/platform/graphics/GraphicsContextRecorder.cpp
  |diff --git a/Source/platform/graphics/GraphicsContextRecorder.cpp
b/Source/platform/graphics/GraphicsContextRecorder.cpp
  |index
1e9160f9dad72a4a5dacbc1445e399618a8d2bdc..e0387e935a3703150df6b6eee3246f508392a4e5
100644
  |--- a/Source/platform/graphics/GraphicsContextRecorder.cpp
  |+++ b/Source/platform/graphics/GraphicsContextRecorder.cpp
  --------------------------
  No file to patch.  Skipping patch.
  3 out of 3 hunks ignored

Patch:       Source/platform/graphics/GraphicsContextRecorder.cpp
Index: Source/platform/graphics/GraphicsContextRecorder.cpp
diff --git a/Source/platform/graphics/GraphicsContextRecorder.cpp
b/Source/platform/graphics/GraphicsContextRecorder.cpp
index
1e9160f9dad72a4a5dacbc1445e399618a8d2bdc..e0387e935a3703150df6b6eee3246f508392a4e5
100644
--- a/Source/platform/graphics/GraphicsContextRecorder.cpp
+++ b/Source/platform/graphics/GraphicsContextRecorder.cpp
@@ -89,10 +89,10 @@ static bool decodeBitmap(const void* data, size_t length,
SkBitmap* result)
     return true;
 }
 
-PassRefPtr<GraphicsContextSnapshot> GraphicsContextSnapshot::load(const
Vector<RefPtr<TilePictureStream> >& tiles)
+PassRefPtr<GraphicsContextSnapshot> GraphicsContextSnapshot::load(const
Vector<RefPtr<TilePictureStream>>& tiles)
 {
     ASSERT(!tiles.isEmpty());
-    Vector<RefPtr<SkPicture> > pictures;
+    Vector<RefPtr<SkPicture>> pictures;
     pictures.reserveCapacity(tiles.size());
     FloatRect unionRect;
     for (const auto& tileStream : tiles) {
@@ -118,7 +118,7 @@ PassRefPtr<GraphicsContextSnapshot>
GraphicsContextSnapshot::load(const Vector<R
     return adoptRef(new
GraphicsContextSnapshot(adoptRef(recorder.endRecordingAsPicture())));
 }
 
-PassOwnPtr<Vector<char> > GraphicsContextSnapshot::replay(unsigned fromStep,
unsigned toStep, double scale) const
+PassOwnPtr<Vector<char>> GraphicsContextSnapshot::replay(unsigned fromStep,
unsigned toStep, double scale) const
 {
     const SkIRect bounds = m_picture->cullRect().roundOut();
     SkBitmap bitmap;
@@ -130,7 +130,7 @@ PassOwnPtr<Vector<char> >
GraphicsContextSnapshot::replay(unsigned fromStep, uns
         canvas.resetStepCount();
         m_picture->playback(&canvas, &canvas);
     }
-    OwnPtr<Vector<char> > base64Data = adoptPtr(new Vector<char>());
+    OwnPtr<Vector<char>> base64Data = adoptPtr(new Vector<char>());
     Vector<char> encodedImage;
     if (!PNGImageEncoder::encode(bitmap, reinterpret_cast<Vector<unsigned
char>*>(&encodedImage)))
         return nullptr;

[rwlbuis, 2015-01-16 23:28:02]

The CQ bit was checked by rob.buis@samsung.com

[commit-bot: I haz the power, 2015-01-16 23:28:30]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/858663002/20001

[commit-bot: I haz the power, 2015-01-17 00:49:31]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=188567