Use Local<> instead of Handle<> in public/*

Use Local<> instead of Handle<> in public/*

Replace Handle<> with Local<>

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

[deepak.s, 2015-04-29 11:00:29]

deepak.s@samsung.com changed reviewers:
+ haraken@chromium.org

[deepak.s, 2015-04-29 11:00:29]

PTAL?
Thanks!

[haraken, 2015-04-29 11:28:45]

Don't we need to update the chromium source code that overrides these functions
as well? For example, you'll need to update
src/extensions/renderer/dom_activity_logger.h.

I don't quite understand why this CL compiles without fixing the Chromium side,
but if it compiles for some reason, I'm fine with landing this CL and fixing the
Chromium side in a separate CL.

[deepak.s, 2015-04-29 11:30:52]

On 2015/04/29 11:28:45, haraken wrote:
> Don't we need to update the chromium source code that overrides these
functions
> as well? For example, you'll need to update
> src/extensions/renderer/dom_activity_logger.h.
> 
> I don't quite understand why this CL compiles without fixing the Chromium
side,
> but if it compiles for some reason, I'm fine with landing this CL and fixing
the
> Chromium side in a separate CL.

Thats why I ran try bots on this. Let all the try bots run its course and then
we'll see.
Thanks!

[haraken, 2015-04-29 11:52:25]

On 2015/04/29 11:30:52, deepak.s wrote:
> On 2015/04/29 11:28:45, haraken wrote:
> > Don't we need to update the chromium source code that overrides these
> functions
> > as well? For example, you'll need to update
> > src/extensions/renderer/dom_activity_logger.h.
> > 
> > I don't quite understand why this CL compiles without fixing the Chromium
> side,
> > but if it compiles for some reason, I'm fine with landing this CL and fixing
> the
> > Chromium side in a separate CL.
> 
> Thats why I ran try bots on this. Let all the try bots run its course and then
> we'll see.
> Thanks!

I noticed that Handle is now just an alias of Local. So this change should be
safe. Let's update the Chromium side code in a reasonable timeline.

https://code.google.com/p/chromium/codesearch#chromium/src/v8/include/v8.h&q=...

LGTM.

[deepak.s, 2015-04-29 11:53:44]

The CQ bit was checked by deepak.s@samsung.com

[commit-bot: I haz the power, 2015-04-29 11:53:53]

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

[commit-bot: I haz the power, 2015-04-29 11:59:43]

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

[commit-bot: I haz the power, 2015-04-29 11:59:43]

Try jobs failed on following builders:
  blink_presubmit on tryserver.blink (JOB_FAILED,
http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/3...)

[deepak.s, 2015-04-29 12:03:21]

Suggested owner for public/* ?

[haraken, 2015-04-29 12:03:57]

haraken@chromium.org changed reviewers:
+ tkent@chromium.org

[haraken, 2015-04-29 12:03:57]

+tkent for an owner review.

[tkent, 2015-04-29 22:45:21]

The CQ bit was checked by tkent@chromium.org

[tkent, 2015-04-29 22:45:21]

lgtm

[commit-bot: I haz the power, 2015-04-29 22:46:08]

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

[commit-bot: I haz the power, 2015-04-30 00:21:47]

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