Fix for #US#US issue in inprocess PDF case.

Fix for #US#US issue in inprocess PDF case.

Now old '#' is searched and replaced by the new #XXX string.
This is fix for 445184 issue for In process case.

BUG=445184
Committed: https://crrev.com/55b67c0f2d850283e08b89e80751719ecb42b0f6
Cr-Commit-Position: refs/heads/master@{#310496}

[Deepak, 2015-01-08 04:52:20]

deepak.m1@samsung.com changed reviewers:
+ thestig@chromium.org

[Deepak, 2015-01-08 04:52:20]

This is #US#US issue fix for in process case.
PTAL.

[Lei Zhang, 2015-01-08 04:55:54]

lgtm

https://codereview.chromium.org/838103002/diff/1/pdf/instance.cc
File pdf/instance.cc (right):

https://codereview.chromium.org/838103002/diff/1/pdf/instance.cc#newcode1229
pdf/instance.cc:1229: // if '#' is already present in |url_| then remove old tag
and add new
The proper name for 'tag' is fragment BTW.
http://en.wikipedia.org/wiki/Fragment_identifier

[Deepak, 2015-01-08 05:00:57]

On 2015/01/08 04:55:54, Lei Zhang wrote:
> lgtm
> 
> https://codereview.chromium.org/838103002/diff/1/pdf/instance.cc
> File pdf/instance.cc (right):
> 
> https://codereview.chromium.org/838103002/diff/1/pdf/instance.cc#newcode1229
> pdf/instance.cc:1229: // if '#' is already present in |url_| then remove old
tag
> and add new
> The proper name for 'tag' is fragment BTW.
> http://en.wikipedia.org/wiki/Fragment_identifier

Thanks, I will change 'tag' to fragment.

[Deepak, 2015-01-08 05:01:02]

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

[commit-bot: I haz the power, 2015-01-08 05:01:33]

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

[commit-bot: I haz the power, 2015-01-08 07:02:57]

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

[commit-bot: I haz the power, 2015-01-08 07:03:01]

Exceeded time limit waiting for builds to trigger.

[Deepak, 2015-01-08 07:22:06]

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

[commit-bot: I haz the power, 2015-01-08 07:23:11]

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

[commit-bot: I haz the power, 2015-01-08 09:24:23]

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

[commit-bot: I haz the power, 2015-01-08 09:24:23]

Exceeded time limit waiting for builds to trigger.

[Deepak, 2015-01-08 13:42:13]

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

[commit-bot: I haz the power, 2015-01-08 13:43:10]

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

[commit-bot: I haz the power, 2015-01-08 14:08:09]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-01-08 14:08:58]

Patchset 2 (id:??) landed as
https://crrev.com/55b67c0f2d850283e08b89e80751719ecb42b0f6
Cr-Commit-Position: refs/heads/master@{#310496}