Remove gfx::FrameTime for a single clock source

Remove gfx::FrameTime for a single clock source

A single clock source should be used through out the code.
This patch removes the gfx::FrameTime clock and uses the base::TimeTicks
overall.

BUG=447329
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/7215854d64eb300e36bc064f8a987028a650491b
Cr-Commit-Position: refs/heads/master@{#331327}

[Abhishek Kanike, 2015-05-15 12:56:25]

abhishek.ka@samsung.com changed reviewers:
+ behara.ms@samsung.com, r.kasibhatla@samsung.com, siva.gunturi@samsung.com

[r.kasibhatla, 2015-05-18 12:22:20]

r.kasibhatla@samsung.com changed reviewers:
- behara.ms@samsung.com

[r.kasibhatla, 2015-05-18 12:22:21]

https://codereview.chromium.org/1143433005/diff/1/android_webview/browser/har...
File android_webview/browser/hardware_renderer.cc (right):

https://codereview.chromium.org/1143433005/diff/1/android_webview/browser/har...
android_webview/browser/hardware_renderer.cc:28: 
Extra line?

[r.kasibhatla, 2015-05-18 12:23:17]

r.kasibhatla@samsung.com changed reviewers:
+ vivek.vg@samsung.com

[r.kasibhatla, 2015-05-18 12:23:17]

Broadly looks fine.
Just remove the extra line added in all files where frame_time.h header was
removed.

[Abhishek Kanike, 2015-05-22 14:03:11]

abhishek.ka@samsung.com changed reviewers:
+ aelias@chromium.org, ajuma@chromium.org, alexclarke@chromium.org,
asvitkine@chromium.org, boliu@chromium.org, brianderson@chromium.org,
ccameron@chromium.org, cpu@chromium.org, danakj@chromium.org, enne@chromium.org,
jbauman@chromium.org, mfomitchev@chromium.org, reveman@chromium.org
- r.kasibhatla@samsung.com, siva.gunturi@samsung.com, vivek.vg@samsung.com

[alex clarke (OOO till 29th), 2015-05-22 14:12:37]

lgtm

LGTM for components/scheduler/renderer/

[alex clarke (OOO till 29th), 2015-05-22 14:12:38]

lgtm

LGTM for components/scheduler/renderer/

[sivag, 2015-05-22 14:17:04]

siva.gunturi@samsung.com changed reviewers:
+ sievers@chromium.org
- aelias@chromium.org, ajuma@chromium.org, asvitkine@chromium.org,
boliu@chromium.org, brianderson@chromium.org, ccameron@chromium.org,
cpu@chromium.org, enne@chromium.org, jbauman@chromium.org,
mfomitchev@chromium.org, reveman@chromium.org

[sivag, 2015-05-22 14:18:06]

siva.gunturi@samsung.com changed reviewers:
+ boliu@chromium.org

[sivag, 2015-05-22 14:18:58]

siva.gunturi@samsung.com changed reviewers:
+ sadrul@chromium.org

[Abhishek Kanike, 2015-05-22 14:28:03]

Reviewers:
Danakj - For cc
Sadrul - For uievents
alexclarke - For components/scheduler
Bolui and Sievers - For content

[Abhishek Kanike, 2015-05-22 14:31:08]

Sorry for spelling your name wrong boliu.

On 2015/05/22 14:28:03, Abhishek Kanike wrote:
> Reviewers:
> Danakj - For cc
> Sadrul - For uievents
> alexclarke - For components/scheduler
> Bolui and Sievers - For content

[boliu, 2015-05-22 16:18:41]

boliu@chromium.org changed reviewers:
+ brianderson@chromium.org

[boliu, 2015-05-22 16:18:42]

+brianderson who filed the bug

[danakj, 2015-05-22 19:12:24]

danakj@chromium.org changed reviewers:
- danakj@chromium.org

[danakj, 2015-05-22 19:12:26]

brian can review cc

[brianderson, 2015-05-22 19:20:56]

cc lgtm

https://codereview.chromium.org/1143433005/diff/100001/cc/scheduler/begin_fra...
File cc/scheduler/begin_frame_source.cc (right):

https://codereview.chromium.org/1143433005/diff/100001/cc/scheduler/begin_fra...
cc/scheduler/begin_frame_source.cc:214: time_source =
DelayBasedTimeSourceHighRes::Create(initial_vsync_interval,
We can also get rid of this if block and remove the DelayBasedTimeSourceHighRes
class altogether now.

Feel free to remove it in this patch or open a followup bug.

[boliu, 2015-05-22 20:40:33]

android_webview and content/browser/android/in_process rubberstamp lgtm

[no sievers, 2015-05-22 22:14:22]

content lgtm

[sadrul, 2015-05-25 04:44:47]

lgtm

[Abhishek Kanike, 2015-05-25 14:40:39]

The CQ bit was checked by abhishek.ka@samsung.com

[commit-bot: I haz the power, 2015-05-25 14:40:48]

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

[Abhishek Kanike, 2015-05-25 16:02:14]

The CQ bit was unchecked by abhishek.ka@samsung.com

[Abhishek Kanike, 2015-05-26 06:08:38]

On 2015/05/22 19:20:56, brianderson wrote:
> cc lgtm
> 
>
https://codereview.chromium.org/1143433005/diff/100001/cc/scheduler/begin_fra...
> File cc/scheduler/begin_frame_source.cc (right):
> 
>
https://codereview.chromium.org/1143433005/diff/100001/cc/scheduler/begin_fra...
> cc/scheduler/begin_frame_source.cc:214: time_source =
> DelayBasedTimeSourceHighRes::Create(initial_vsync_interval,
> We can also get rid of this if block and remove the
DelayBasedTimeSourceHighRes
> class altogether now.
> 
> Feel free to remove it in this patch or open a followup bug.

Hi brianderson,
I have opened a followup issue
https://code.google.com/p/chromium/issues/detail?id=492033 to remove
DelayBasedTimeSourceHighRes.

[Abhishek Kanike, 2015-05-26 06:08:48]

The CQ bit was checked by abhishek.ka@samsung.com

[commit-bot: I haz the power, 2015-05-26 06:08:56]

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

[commit-bot: I haz the power, 2015-05-26 06:13:30]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2015-05-26 06:14:19]

Patchset 6 (id:??) landed as
https://crrev.com/7215854d64eb300e36bc064f8a987028a650491b
Cr-Commit-Position: refs/heads/master@{#331327}

[Abhishek Kanike, 2015-05-27 09:56:24]

Patchset #7 (id:120001) has been deleted